C言語の、戻り値/値渡し/アドレス渡しのついて

質問

【実装したコードに、戻り値/値渡し/アドレス渡しを用いたサブの関数を作成せよ。】

上記の課題に取り組んでいるのですが、何となく概念は分かったのですが、ソースコードに反映させようとすると詰まってしまって…
どなたか教えて頂けないでしょうか？

『ソースコード』

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

#define CALC (3)
#define FROM_YEAR (1900)
#define MAX_LINE (1000)

int cmp_u(const void* a, const void* d) {
return strcmp((char*)a, (char*)d);
}

int cmp_d(const void* a, const void* d) {
return strcmp((char*)d, (char*)a);
}

int main() {
int num1, num2;
char op;
float answer;
int r,i;
FILE* fp;
char e[11];
char sin[1000][1000];
char ad[8];

fp = fopen("log.txt", "a+");

if (fp == NULL) {
printf("ファイルオープン失敗\n");
return -1;
}

while (1) {
r = scanf("%d%c%d", &num1,&op, &num2);
if (r != CALC) {
puts("input error");
return 1;
}

if (op == '+') {
answer = num1 + num2;
}
else if (op == '-') {
answer = num1 - num2;
}
else if (op == '*') {
answer = num1 * num2;
}
else if (op == '/') {
answer = (float)num1 / num2;
}

time_t t = time(NULL);
struct tm* tm = localtime(&t);
printf("%d/%02d/%02d ", tm->tm_year + FROM_YEAR, tm->tm_mon + 1, tm->tm_mday);
printf("%02d:%02d:%02d ", tm->tm_hour, tm->tm_min, tm->tm_sec);
printf("%d%c%d,%f\n", num1, op, num2, answer);
fprintf(fp, "%d/%02d/%02d ", tm->tm_year + FROM_YEAR, tm->tm_mon + 1, tm->tm_mday);
fprintf(fp, "%02d:%02d:%02d ", tm->tm_hour, tm->tm_min, tm->tm_sec);
fprintf(fp, "%d%c%d,%f\n", num1, op, num2, answer);

printf("計算を続けますか？");
scanf("%s", e);
if (strcmp(e, "no") == 0) {
break ;
}
}

fclose(fp);

fp = fopen("log.txt", "r");

int cnt = 0;

for (i = 0;i < MAX_LINE;i = i + 1) {
if (fgets(sin[i], sizeof(sin[0]), fp))
++cnt;
else
break;
}

fclose(fp);

printf("ASC or DESC: ");
scanf("%s", ad);

if (strcmp(ad, "ASC") == 0) {
qsort(sin, cnt, sizeof(sin[0]), cmp_u);
}
else {
qsort(sin, cnt, sizeof(sin[0]), cmp_d);
}

for (i = 0;i < cnt;i = i + 1) {
printf("%s", sin[i]);
}

return 0;
}

回答No.2

superside0 · ベストアンサー

いまは、qsort( ) に渡すcmp_u( )とcmp_d( )という関数しか作ってなくて
それ以外の処理は、main( ) 内にベタに書いてあるけど、
たとえば、 四則演算するところだけを関数にして
・戻り値は　演算子が +-*/ 以外のときのエラー判定値
・値渡しは、num1,num2,op
・アドレス渡しは、answer（関数内で代入された結果を親側で使う)
としてしまうという方法はありそうですね。

関数化の別案としては、main( )　内の前半と後半の
・入力された文字を計算して　ログファイルとコンソールに出力
・ログファイルから読み込んで並び替えてコンソールの出力
の処理をそれぞれ関数にして、
main( )から呼び出すということはできそうですが
この２つの処理は独立性が高く、値渡しや
アドレス渡しにして結果を戻すようなものがあまりないので、
課題の狙いとは違うような気がします。

お礼

ありがとうございます。
凄い分かりやすかったです。

回答No.1

AsarKingChang

一度頭をリフレッシュしたほうがいいかも！
#define MAX_LINE (1000)
せっかくこれを作ったのに
char sin[1000][1000];
これでは意味がなく
for (i = 0;i < MAX_LINE;i = i + 1) {
if (fgets(sin[i], sizeof(sin[0]), fp))
++cnt;
else
break;
}
これだと、例えばMAX_LINEを実ワークより大きい値をセットしたら1発で
メモリ破壊しますよ。
char sin[MAX_LINE][1000];
の用が、よかったでしょうね。
ついでに、iの値と、cntの値って、常に同じな事にも気が付いてもらえたら
int cnt = 0;
for (i = 0;i < MAX_LINE;i = i + 1) {
このように始まっているので、
for (i=0,cnt=0;i < MAX_LINE;i++,cnt++) {
ロジック上、cntが加算されない時はbreakなので、
値は常に同じになってます。
つまり、i を使わなくても、直接 i を cntとするか、
ループ後に、int cnt=i;でもいいわけです。

次の問題。
【実装したコードに、戻り値/値渡し/アドレス渡しを用いたサブの関数を作成せよ。】
これと、書かれているソースに関連性がなく、別のソースと思われます。
上に書いている物だけを言うなら、
int func1(int a) {
return (a);
}
int func2(int *a) {
return (*a);
}

int abcd=1234;
int ret;
ret=func1(1234); 値渡し
ret=func1(abcd); 変数渡し（ただし関数側から見ればimmediate）
ret=func2(&abcd);ポインタ渡し
返り値はすべて、1234
だけで、要件定義は満たしていると思います。
もう一つ、構造体渡しってのもあるので、一応・・・
typedef struct {
int abcd;
}AA;

int func3(AA *a) {
return (a->abcd);
}

AA aa;
aa.abcd=1234;
ret=func3(&aa);
なんてのもありますが（要するに連続するメモリに名前を付けてオフセット移動してるだけ）

ともあれ、質問的には、単純でシンプルに考えていい気がする。