ポインタ参照が難しくて上手く関数化できない

Question

下記のコードで冗長な処理を関数化したいのですが、ポインタ参照やら配列やらで関数化に手こずってます。

処理内のコメントに記載している
0. 商品毎の売り上げ降順表示、商品リストをループ
0. 商品リストの作成
この2つを関数化したらどのような記載になりますか？
完成前後で見比べて考えたいので、見本で関数化して頂きたいです。
よろしくお願いします。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define MACHINENAME 5
#define PRODUCT 7
#define LINEDATAMAX 21
#define FILEMAX 3
#define FILENAMEMAX 30
#define FILEDATAMAX 4

typedef struct{
	char name[MACHINENAME];
	char kind[PRODUCT];
	int price;
	int sales;
}DATAin;

typedef struct{
	char nameOut[MACHINENAME];
	char dammy[3];
	int total;
}DATAout;

void sortName(DATAin *pp,int line);
void sortPrice(DATAout *po, int line);
DATAin* readFile(char filename[], DATAin **pl, int *cntF);
void productOut(DATAin *ppIn, DATAin *pl, DATAout po);
void calcPrice(DATAin *pp, char kind[PRODUCT], DATAout *po, int *cntKind);
void reduceMemory(DATAin *mIn, DATAin *mL, int cntF);

int main(void){
	char fileName[FILEMAX][FILENAMEMAX] = {
			{"vending-machine-a.txt"},
			{"vending-machine-b.txt"},
			{"vending-machine-c.txt"}
	};
	int loop1 = 0;
	int loop2 = 0;
	int cntFile = 0;
	int cntKind = 0;

	DATAout output[FILEMAX] = {
			{"\0", "\0", 0},
			{"\0", "\0", 0},
			{"\0", "\0", 0}
	};

	DATAin *ppIn[FILEMAX] = {NULL, NULL, NULL};
	DATAin *ppL = NULL;
	DATAout *ppO = NULL;
	ppO = output;

	/*各ファイル読み取り、商品リストを作成*/
	/*inputfileAから情報を抽出*/
	for(loop1 = 0; loop1 < FILEMAX; loop1++){
		ppIn[loop1] = readFile(fileName[loop1], &ppL, &cntFile);
		if (ppIn[loop1] == NULL){
			printf("a");
			return 0;
		}
	}
	/*商品毎の売り上げ降順表示、商品リストをループ*/
//	productOut(ppIn, ppL, ppO);
	for(loop1 = 0; strncmp((ppL + loop1)->kind , "\0", 1) != 0; loop1++){
		printf("\n-------------\n %s\n-------------\n",(ppL + loop1)->kind);
		cntKind = 0;  /*商品名カウンター*/
		for(loop2 = 0; loop2 < FILEMAX; loop2++){
			calcPrice(ppIn[loop2], (ppL + loop1)->kind, ppO, &cntKind);
		}
		/*商品毎の合計値を金額降順でソート*/
		sortPrice(ppO, cntKind);
		/*商品毎の合計出力*/
		for(loop2 = 0; loop2 < cntKind; loop2++){
			printf(" %s : %5d\n",(ppO + loop2)->nameOut,(ppO + loop2)->total);
		}
	}
	/*自販機名の再初期化*/
	for(loop1 = 0; loop1 < FILEMAX; loop1++){
		strncpy((ppO + loop1)->nameOut, (ppIn[loop1])->name, MACHINENAME);
	}
	/*合計値の再初期化*/
	for(loop1 = 0; loop1 < cntFile; loop1++){
		(ppO + loop1)->total = 0;
	}
	/*合計値代入*/
	for(loop1 = 0; loop1 < FILEMAX; loop1++){
		for (loop2 = 0; strncmp((ppIn[loop1] + loop2)->kind, "\0", 1) != 0; loop2++) {
			(ppO + loop1)->total = (ppO + loop1)->total + ((ppIn[loop1] + loop2)->price * (ppIn[loop1] + loop2)->sales);
		}
	}
	/*メモリ開放*/
	reduceMemory(ppIn[0], ppL, cntFile);
	/*自販機毎の合計値を金額降順でソート*/
	sortPrice(ppO, cntFile);
	/*Total出力*/
	printf("\n-------------\n Total\n-------------\n");
	/*自販機毎の合計出力*/
	for(loop1 = 0; loop1 < cntFile ; loop1++){
		printf(" %s : %5d\n",(ppO + loop1)->nameOut,(ppO + loop1)->total);
	}
	return 0;
}

/*商品毎のデータ算出*/
void calcPrice(DATAin *pp, char kind[PRODUCT], DATAout *po, int *cntKind){
	int loop1 = 0;
	int sameProbuct = 0;  /*重複商品の確認*/

	for (loop1 = 0; strncmp((pp + loop1)->kind, "\0", PRODUCT) != 0; loop1++) {
		if(strncmp(kind, (pp + loop1)->kind, PRODUCT) == 0) {
			if(sameProbuct < 1){
				(po + *cntKind)->total = (pp + loop1)->price * (pp + loop1)->sales;
				strncpy((po + *cntKind)->nameOut, (pp + loop1)->name, MACHINENAME);
				*cntKind = *cntKind + 1;
				sameProbuct = 1;
			}else{
				/*2つ目以降の重複商品は1つ目の商品に合算する*/
				(po + *cntKind - sameProbuct)->total = (po + *cntKind \
				- sameProbuct)->total + ((pp + loop1)->price * (pp + loop1)->sales);
			}
		}
	}
}

///*商品毎の売り上げ降順表示、商品リストをループ*/
//関数化途中
//void productOut(DATAin *ppIn, DATAin *pl, DATAout po){
//	int loop1 = 0;
//	int loop2 = 0;
//	int cntKind = 0;
//
//	for(loop1 = 0; strncmp((pl + loop1)->kind , "\0", 1) != 0; loop1++){
//		printf("\n-------------\n %s\n-------------\n",(pl + loop1)->kind);
//		cntKind = 0;  /*商品名カウンター*/
//		for(loop2 = 0; loop2 < FILEMAX; loop2++){
//			calcPrice(ppIn[loop2], (pl + loop1)->kind, po, &cntKind);
//		}
//		/*商品毎の合計値を金額降順でソート*/
//		sortPrice(po, cntKind);
//		/*商品毎の合計出力*/
//		for(loop2 = 0; loop2 < cntKind; loop2++){
//			printf(" %s : %5d\n",(po + loop2)->nameOut,(po + loop2)->total);
//		}
//	}
//}
/*商品名降順ソート関数*/
void sortName(DATAin *pp, int line){
	DATAin bfData = {"\0", "\0", 0, 0};
	int loop1 = 0;
	int loop2 = 0;

	for(loop1 = 0; loop1 < line; loop1++){
		for(loop2 = loop1 + 1; loop2 < line; loop2++){
			if(strncmp((pp + loop1)->kind,(pp + loop2)->kind, PRODUCT) > 0){
				bfData = *(pp + loop1 );
				*(pp + loop1) = *(pp + loop2);
				*(pp + loop2)= bfData;
			}
		}
	}
}

/*金額降順ソート関数*/
void sortPrice(DATAout *po, int line){
	DATAout bfData = {"\0", "\0", 0};
	int loop1 = 0;
	int loop2 = 0;
	for (loop1 = 0; loop1 < line; loop1++) {
		for (loop2 = loop1 + 1; loop2 < line; loop2++) {
			if((po + loop1)->total < (po + loop2)->total) {
				bfData = *(po + loop1);
				*(po + loop1) = *(po + loop2);
				*(po + loop2) = bfData;
			}
		}
	}
}

/*ファイル操作関数*/
DATAin* readFile(char filename[], DATAin **pl, int *cntF){
	FILE *fp = NULL;
	DATAin *pp = NULL;
	int cntData = 0;
	int loop1 = 0;
	int cntSame = 0;
	int ret = 0;
	static int listCount = 0;
	char str[LINEDATAMAX] = "\n";

	if(listCount == 0){
		*pl = (DATAin*)malloc(sizeof(DATAin));
		if (*pl == NULL){
			printf("malloc error「pl」\n");
			return NULL;
		}
	}

	pp = (DATAin*)malloc(sizeof(DATAin));
	if (pp == NULL){
		free(*pl);
		printf("malloc error「pp」\n");
		return NULL;
	}

	fp=fopen(filename, "r");
	if(fp == NULL){
		free(*pl);
		free(pp);
		printf("%s file not open\n",filename);
		return NULL;
	}else{
		*cntF = *cntF + 1;
	}

	/*inputfileから情報を抽出*/
	while(fgets(str, LINEDATAMAX, fp) != NULL){
		ret = sscanf(str, "%[^,],%[^,],%d,%d", (pp + cntData)->name,\
			(pp + cntData)->kind, &(pp + cntData)->price, &(pp + cntData)->sales);
		if( ret == EOF ) {
			free(*pl);
			free(pp);
			printf( "sscanf error\n" );
			fclose(fp);
			return NULL;
		}
		/*商品名リスト作成*/
//		if(listCount == 0){
//			strncpy((*pl)->kind, (pp + cntData)->kind, PRODUCT);
//			listCount = listCount + 1;
//			*pl = realloc(*pl,sizeof(DATAin) * (listCount + 1));
//		}else{
//		cntSame = 0;
//			for (loop1 = 0; loop1 < listCount; loop1++) {
//				if(strncmp((*pl + loop1)->kind, (pp + cntData)->kind, PRODUCT) == 0) {
//					/* 重複している*/
//					cntSame = 1;
//				}
//			}
//			if(cntSame == 0){
//
//				strncpy((*pl + listCount)->kind, (pp + cntData)->kind, PRODUCT);
//				listCount = listCount + 1;
//				*pl = realloc(*pl,sizeof(DATAin) * (listCount + 1));
//			}
//		}

		cntData = cntData + 1;
		pp = realloc(pp,sizeof(DATAin) * (cntData + 1));
	}
	/*配列の終端に\0を代入(PRODUCTNAME分)*/
	strncpy((*pl + listCount)->kind, "\0\0\0\0\0\0\0", PRODUCT);
	strncpy((pp + cntData)->kind, "\0\0\0\0\0\0\0", PRODUCT);

	fclose(fp);
	return pp;
}

/*メモリ開放関数*/
void reduceMemory(DATAin *mIn, DATAin *mL, int cntF){
	int loop1 = 0;

	if(mL != NULL){
		free(mL);
	}
//	for(loop1 = 0; loop1 < FILEMAX; loop1++){
//		if(&mIn[loop1] != NULL){
//			free(&mIn[loop1]);
//		}
//	}
	free(&mIn[0]);
}

Comments

[gentaro]

> 完成前後で見比べて考えたいので、見本で関数化して頂きたいです。

そういうのを「丸投げ」という。
https://teratail.com/help/avoid-asking

こういうの見ると知的生産活動で食ってる人間全員をナメてんのか？と思う。
誰かに完成形のコードを書いて欲しいなら正当な報酬を用意してクラウドソーシングで依頼しよう。

[退会済みユーザー]

> ポインタ参照が難しくて上手く関数化できない
原因がはっきりしているなら、ポインタについて勉強すれば解決しますね。基礎をすっ飛ばして結果だけ得ようとすると、今後も同じようなケースで同じような質問を繰り返す事になりますので、きちんと勉強しましょう。ポインタについてならここの短い回答欄での答えより、C言語の書籍やWebサイトを検索して調べた方が多くの情報が集まります。

[dodox86]

更に加えれば、先の質問を片付けてから（理解できないなら、理解できるよう勉強してから）取り組みましょう。
[アドレスの一般的な使い方とは] https://teratail.com/questions/317984

回答やコメントをいただいているのにないがしろにすると、回答者はどんどん離れていきます。

Answer 1

たとえば

(ppL + loop1)->kind

を
ppL[loop1].kind
のように、

(ポインタ+数)->メンバ　を ポインタ[数].メンバ に置き換えるとこから始めてはいかがか。