C言語でファイルを読み込み、一部の要素を取り出すコードのエラー

Question

###前提・実現したいこと
c言語でテキストファイルを読み込み、行としてではなく、各要素を読み込んで３列目と4列目だけ出力しようとしています。以下がコード作成のために参考にしているサイトです。
参考サイト

実現したいこと
以下のように全ての行でデータが存在する３列目と4列目だけを出力するためにコードをどのように変更すれば良いかアドバイスをいただけますと幸いです。

%./trial trial2.txt
3,100
4,400

###エラー
各要素にNULLがないtrial.txtの場合はコードが実行されて、実現したい解が求められます。

trial.txt

2018/01/15,Jan,3,100
2018/01/16,Jan,4,400

現状では全て読み込んで出力しているので、エラーが表示されています。
trial2.txt(NULLデータを含む）

2018/01/15,Jan,3,100
2018/01/16,,4,400

エラー

2018/01/15,Jan,3,100
2018/01/15,Jan,3,100
2018/01/15,Jan,3,100
・
・
・

###該当のソースコード

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

int main(int argc, char *argv[]){
  FILE* fp; // file pointer

  char *fname = argv[1];
  fp = fopen(fname, "r"); // open a file

  char date[11], month[4];
  int num1, num2;

  while(fscanf(fp, "%[^,],%[^,],%d,%d\n", date, month, &num1, &num2) != EOF) {
        printf("%s,%s,%d,%d\n", date, month, num1, num2);
  }

  fclose(fp);
  return 0;
}

###実行環境
ターミナル　バージョン2.8
MaxOS High Sierra

Answer 1

超手抜きサンプルです。
1行読み込んで、カンマを2個飛ばし、次の文字からsscanfで分解しています。

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

int main(int argc, char *argv[]){
	FILE* fp; // file pointer

	char *fname = argv[1];
	fp = fopen(fname, "r"); // open a file

	char line[128];
	int num1, num2;
	char *p;
	int c;

	while(fgets(line, sizeof(line), fp) != NULL) {
		p = line;
		c = 0;
		while(p){
			if(*p == ',') c++;
			p++;
			if(c == 2) break;
		}
		sscanf(p, "%d,%d\n", &num1, &num2);
		printf("%d,%d\n", num1, num2);
	}

	fclose(fp);
	return 0;
}

Comments

[退会済みユーザー]

ご回答いただきましてありがとうございます。
質問でお聞きした２行のテキストファイル（trial2.txt）では動作しましたが、900行近くあるテキストファイルでは char line[128];をchar line[256]としても
3,0
4,0
もしくは
3,4195808
4,4195808
というように、４列目の値が取れないという問題が起こりました。

[ttyp03]

解決したのでしょうか？
ちなみにline変数は1行分のバッファなので、増やしても意味ありませんよ。
もっとも1行の長さが128バイトを超える行があるなら別ですが。

[退会済みユーザー]

900行近くあるテキストファイルでは 解決できていません。

[ttyp03]

おそらく900行中のどこかで1行4要素のフォーマットではない行があるのではと思います。

[episteme]

...なんだかなー。トラブルの原因を突き止めるのは僕らじゃないぞ?
900行近くあることが問題なのか否かくらいは調べておくれよ。

[ttyp03]

ですよねぇ

[退会済みユーザー]

お手数おかけして申し訳ございません。

Answer 2

ヒントになるかもしれないおためしコードを書いてみた。

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

void tokenize(char* str, char delim, void (*callback)(const char*,void*), void* arg) {
  const char* first = str;
  char*       last  = str;
  while ( *last != '\0' ) {
    if ( *last == delim ) {
      *last = '\0';
      callback(first, arg);
      first = last+1;
    }
    ++last;
  }
  callback(first, arg);
}

void print(const char* str, void* arg) {
  printf("[%s] ", str);
  if ( *str != '\0' ) ++*((int*)arg);
}

int main() {

  int count;
  char str[32];
  
  count = 0;
  strcpy(str, "abc,def,ghi,jkl");
  tokenize(str, ',', print, &count);
  printf(" : %d tokens read\n", count);
  
  count = 0;
  strcpy(str, "abc,,ghi,,jkl");
  tokenize(str, ',', print, &count);
  printf(" : %d tokens read\n", count);

  return 0;
}

/* 実行結果
[abc] [def] [ghi] [jkl]  : 4 tokens read
[abc] [] [ghi] [] [jkl]  : 3 tokens read
*/

Comments

[退会済みユーザー]

ご回答いただきましてありがとうございます。

Answer 3

fgets()で一行まるごと読み込み、出現する複数の','が隣接していなければ存在しないカラムはないでしょう。
それが判定できたならまるごと読み込んだ一行をsscanfなりなんなりで各カラムを取り出せばいいんちゃいます?