C言語のファイルの入出力について

質問

現在、下記の課題に取り組んでいるのですが、qsortの所で詰まってしまいました。
どなたか修正点を教えて頂けないでしょうか？

習問題000　～構造化プログラミングの基礎3　ファイル入出力～

　電卓アプリケーションの作成
　　以下の機能を満たすアプリケーションを作成してください。
　　　※画面の表示やログの形式は原則として例示されている内容に従ってください。

　　　1) 四則演算が出来ること
　　　2) コンソールから計算対象となる数値と演算子を受け取る
　　　　 例） 5 + 6
　　　3) 結果をコンソール上に表示する
　　　　 例） 11
　　　4) 演算結果と実行した日、時間、秒をログファイルとして保存する
　　　　 ログの上限は1000行とする（上限を超過するケースはひとまず考慮しなくて良い）
　　　　 例） log.txt に 以下の内容を記録
　　　　　　　2015/04/27 14:30:51, 5 + 6, 11
　　　5) コンソールからパラメータを受け取り、ログを実行時間の昇順/降順に並べ替えて
　　　　 コンソールに表示する
　　　　 例） ASC　を入力
　　　　　　　2015/04/26 10:20:00, 5 + 6, 11
　　　　　　　2015/04/27 14:30:51, 7 - 6, 1
　　　　　　　2015/04/27 15:30:00, 7 + 8, 15
　　　　　　　2015/04/28 14:30:51, 8 + 9, 17
　　　　 例） DESC　を入力
　　　　　　　2015/04/28 14:30:51, 8 + 9, 17
　　　　　　　2015/04/27 15:30:00, 7 + 8, 15
　　　　　　　2015/04/27 14:30:51, 7 - 6, 1
　　　　　　　2015/04/26 10:20:00, 5 + 6, 11

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

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

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

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

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

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

while (1) {
r = scanf("%d%c%d", &num1, &op, &num2);
if (r != 3) {
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 + 1900, 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 + 1900, 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\n", &c);
if (strcmp(c, "no") == 0) {
break ;
}
}

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

if (strcmp(ad, "ASC") == 0) {
qsort(sin, 9, sizeof(char), cmp_u);
}
else {
qsort(sin, 9, sizeof(char), cmp_d);
}

for (i = 0; i < 9; i++) {
printf("%d\n", sin[i]);
}

fclose(fp);

return 0;
}

回答No.2

AsarKingChang · ベストアンサー

多分ですが、qsortを使おうとしたことが、そもそもの原因でしょう。
この命令だと、
2015/04/26 10:20:00, 5 + 6, 「11」←最後のカッコ
2015/04/27 14:30:51, 7 - 6, 「1」←最後のカッコ
だけをソートする物なので、それだと、日付や計算のログがソートの対象になりません。
なので、qsortをやめるのが先でしょうね。

私ならですが、
struct _tag_log {
// 今回日付は、後から変化しない想定なので、char datetime[入りきるサイズ];で代入時点で確定でもOK
int year; // 代入段階で1900加算
int month; // 代入段階で1加算
int day;
int hour;
int min;
int sec;
// num1+op+num2も後から変化しないので文字列化してしまってもいい
int num1;
int op; // char op;をintにキャストするかunion化
int num2;
// 大事なのはソート対象となるこの変数のみ。
int answer;
}
これを1セットとして、必要個数配置。
それをもとに
base[n].answer <> base[n+1].answerの比較を
ASC/DESCでn-1までループ
ただし、そのままやると、構造体そのものを入れ替える
オーバーヘッドが大きいので、
index[n]=n; を最大数までやったテーブルを作り
その中だけを入れ替えるでしょうね。
または、log自体をポインターとして、ポインタのみをソートするか

などが私がやると思われるやり方でしょうね。
（別に速度チューニングは不要でしょうから、動けば何でもいいかとは、
　思いますけどね）

回答No.1

AsarKingChang

>qsortの所で詰まってしまいました。
どういう、状態なのかも書いてもらえたらな気がするものの。
answer = (float)num1 / num2;
この計算の結果を「どこにも書いてない」のが気になります。
char sin[9];
こちらに代入している部屋がないのです。
また、floatにしているわけですが、ソート対象となる変数は
signed char幅なので、-128～+127でしかないので、
この変数に入れるつもりなら、floatにした意味が全くありません。
（小数点を入れられないので結局切り捨てられる）

なので、うまくいかない？のではなく、
そもそも、代入してない？が原因だと思います。