レベルがあわない

へにっくす · 投稿記事 by **へにっくす** » 10年前

以下の投稿。

http://dixq.net/forum/viewtopic.php?f=3&t=16463

サンプルを活用できてないね。
（strcatをうまく使えていないあたり、レベルが違いすぎる印象）
質問者が指定していないSTLをバリバリに使ってるのがすごい嫌味な感じです。

どうなんでしょう・・・

投稿記事 by **みけCAT** » 10年前

へにっくすさんが書きました:質問者が指定していないSTLをバリバリに使ってるのがすごい嫌味な感じです。

どうなんでしょう・・・

C++を使っているうえ、STLを使うなという指定も無いので、STLを使ってもいいと思いました。
入力の条件が指定されていない以上、(メモリの許す限り)任意の長さに対応するべきだと思いました。

試しにSTLを使わずに入力処理を書いてみました。
連想配列の作成はサボったので、
・勝手に名前順にソートされることはありません
・同じ名前の項目がある場合、両方のデータが登録されます

► スポイラーを表示

CODE:

#include 

struct FileData {
	char *data;
	FileData *next;
};

struct FileEntry {
	char *name;
	FileData *dataList;
	FileEntry *next;
};

struct StringData {
	char c;
	StringData *next;
};

// 文字のリストから文字列に変換する
char *buildString(StringData *str) {
	size_t count = 0;
	StringData *p = str;
	while (p != NULL) {
		count++;
		p = p->next;
	}
	char *result = new char[count + 1];
	char *out = result;
	for (p = str; p != NULL; p = p->next) {
		*(out++) = p->c;
	}
	*out = '\0';
	return result;
}

// 文字のリストを開放する
void freeString(StringData *str) {
	while (str != NULL) {
		StringData *next = str->next;
		delete str;
		str = next;
	}
}

// データリストを開放する
void freeFileData(FileData *fd) {
	while (fd != NULL) {
		FileData *next = fd->next;
		delete[] fd->data;
		delete fd;
		fd = next;
	}
}

// ファイルデータリストを開放する
void freeFileEntry(FileEntry *fe) {
	while (fe != NULL) {
		FileEntry *next = fe->next;
		delete[] fe->name;
		freeFileData(fe->dataList);
		delete fe;
		fe = next;
	}
}

// テキストファイルを読み込む
FileEntry *readText(const char *fileName) {
	FILE *fp;
	fp = fopen(fileName, "r");
	if (fp == NULL) return NULL;

	FileEntry *result = NULL;
	FileEntry **tail = &result;
	FileData *dataList = NULL;
	FileData **dataListTail = &dataList;
	StringData *name = NULL; // 項目名
	StringData *data = NULL; // データ
	StringData *pendingInput = NULL; // 無視されるかもしれない入力
	StringData **nameTail = &name;
	StringData **dataTail = &data;
	StringData **pendingTail = &pendingInput;
	bool isReadingName = true;
	bool isReadingValidData = false;
	for (;;) {
		int input = getc(fp);
		if (input == EOF) break;
		if (isReadingName) {
			// 項目名またはコメントの入力中
			if (input == '\r' || input == '\n') {
				// =が現れずに項目名(仮)の入力が終わったので、コメントだった
				freeString(name);
				name = NULL;
				nameTail = &name;
				freeString(pendingInput);
				pendingInput = NULL;
				pendingTail = &pendingInput;
			} else if (input == ' ') {
				// 空白は=の前の無視するべきものかもしれないので、保留する
				*pendingTail = new StringData;
				(*pendingTail)->c = input;
				(*pendingTail)->next = NULL;
				pendingTail = &(*pendingTail)->next;
			} else if (input == '=') {
				// =が現れたので、データの入力に移行する
				dataList = NULL;
				dataListTail = &dataList;
				data = NULL;
				dataTail = &data;
				isReadingName = false;
				isReadingValidData = false;
				freeString(pendingInput);
				pendingInput = NULL;
				pendingTail = &pendingInput;
			} else {
				// 入力を名前に反映させる
				if (pendingInput != NULL) {
					*nameTail = pendingInput;
					nameTail = pendingTail;
				}
				*nameTail = new StringData;
				(*nameTail)->c = input;
				(*nameTail)->next = NULL;
				nameTail = &(*nameTail)->next;
				pendingInput = NULL;
				pendingTail = &pendingInput;
			}
		} else {
			// データの入力中
			if (input == '\r' || input == '\n') {
				// データの入力を終了して保存し、次の項目名(仮)の入力に移行する
				*dataListTail = new FileData;
				(*dataListTail)->data = buildString(data);
				(*dataListTail)->next = NULL;
				freeString(data);
				data = NULL;
				dataTail = &data;
				*tail = new FileEntry;
				(*tail)->name = buildString(name);
				(*tail)->dataList = dataList;
				(*tail)->next = NULL;
				tail = &(*tail)->next;
				dataList = NULL;
				dataListTail = &dataList;
				freeString(name);
				name = NULL;
				nameTail = &name;
				freeString(pendingInput);
				pendingInput = NULL;
				pendingTail = &pendingInput;
				isReadingName = true;
			} else if (input == ' ') {
				// 空白なので、=の後のものは無視する
				if (isReadingValidData) {
					*dataTail = new StringData;
					(*dataTail)->c = input;
					(*dataTail)->next = NULL;
					dataTail = &(*dataTail)->next;
				}
			} else if (input == ',') {
				// データの区切り
				*dataListTail = new FileData;
				(*dataListTail)->data = buildString(data);
				(*dataListTail)->next = NULL;
				dataListTail = &(*dataListTail)->next;
				freeString(data);
				data = NULL;
				dataTail = &data;
				isReadingValidData = true;
			} else {
				// 入力をデータに反映させる
				*dataTail = new StringData;
				(*dataTail)->c = input;
				(*dataTail)->next = NULL;
				dataTail = &(*dataTail)->next;
				isReadingValidData = true;
			}
		}
	}
	// 最終行のデータを保存する
	if (!isReadingName) {
		*dataListTail = new FileData;
		(*dataListTail)->data = buildString(data);
		(*dataListTail)->next = NULL;
		*tail = new FileEntry;
		(*tail)->name = buildString(name);
		(*tail)->dataList = dataList;
		(*tail)->next = NULL;
	}
	freeString(name);
	freeString(data);
	freeString(pendingInput);

	fclose(fp);
	return result;
}

int isSameString(const char *a, const char *b) {
	if (a == NULL) return b == NULL;
	if (b == NULL) return 0; // 上でチェックしているので、aはNULLではない
	while (*a != '\0' || *b != '\0') {
		if (*a != *b) return 0;
		a++;
		b++;
	}
	return 1;
}

int main(void) {
	// ファイルを読み込む
	FileEntry *data = readText("input.txt");

	// 変数に保存する
	char *hensu1 = NULL, *hensu2 = NULL;
	for (FileEntry *itr = data; itr != NULL; itr = itr->next) {
		if (isSameString(itr->name, "aaa")) {
			FileData *d1 = itr->dataList;
			if (d1 != NULL) {
				hensu1 = d1->data;
				FileData *d2 = d1->next;
				if (d2 != NULL) hensu2 = d2->data;
			}
		}
	}

	printf("hensu1 = %s\n", hensu1);
	printf("hensu2 = %s\n", hensu2);
	putchar('\n');

	// 読み込んだデータを出力する
	for (FileEntry *itr = data; itr != NULL; itr = itr->next) {
		char *name = itr->name;
		int count = 0;
		for (FileData *d = itr->dataList; d != NULL; d = d->next, count++) {
			printf("%s[%d] = %s\n", name, count, d->data);
		}
	}
	return 0;
}

入力(input.txt)

CODE:

hoge = 346
fuga = ayaneru,riesyon,mareitaso
aaa = 31,14
kugyu = kugimiya,rie
asumi kana = asumisu

出力

CODE:

hensu1 = 31
hensu2 = 14

hoge[0] = 346
fuga[0] = ayaneru
fuga[1] = riesyon
fuga[2] = mareitaso
aaa[0] = 31
aaa[1] = 14
kugyu[0] = kugimiya
kugyu[1] = rie
asumi kana[0] = asumisu

自分の技術力が低いだけかもしれないですが、かなり長く、エンバグのリスクが高いコードになってしまいました。

どうなんでしょう・・・

usao · 投稿記事 by **usao** » 10年前

どうでもいいけど
>デスクトップアクセサリの様な物
に勝手に音楽鳴らされるのはなんか嫌だなぁ…

ISLe · 投稿記事 by **ISLe** » 10年前

質問者そっちのけはいまにはじまったことではないですけども。
関係ないと思いますが、学生はとっくに春休みなのにずいぶん投稿が少ないですよね。

Ketty · 投稿記事 by **Ketty** » 10年前

んー。

へにっくすさんが懸念されておられる通り、質問者さんは、
たぶん現時点であがってるレスに対しては、およそ理解に及びづらいように、私にも見えます。

この質問者さんの過去の質問に答えた経験からですが、
この人は、安易にコピペで済ませちゃおう系なヘンな人ではなさそうだ、という期待が、
私にはあるので、なるべく大事に見守ってあげたいところです(＾▽＾)

というわけで、なるべくみんなで助けてあげましょうぞな(＾ｖ＾)

へにっくす · 投稿記事 by **へにっくす** » 10年前

>> みけCATさん
> 任意の長さに対応するべき
これはokだと思いますが、
> STLを使うなという指定もない
これは無くてもいいですよね？
指定がないのは質問者が想定していない、ということだと思います。
みんながあなたと同じレベルとは限らないことを念頭に置きましょう。
仕事でそんなことすると、やり直しさせられますよ。
まあここでは別にそこまで気にする必要はないですが。
私が気にしすぎなだけなので。

> どうなんでしょう…

(^^;
いいけどさ。
なんで一文字ずつ読むことにこだわってんのかな。
一行ずつ読み込んで、そこから解析する方が良くない？
難しく考えすぎ…

>> usaoさん
私も同意。
でも質問者さんはそうしたいのでしょうからとやかく言えないですね (^^;

>> ISLeさん
> そっちのけ
まーねー
回答者は答えたいからというのがよく分かります　笑
> 春休み
今頃は花見で盛り上がってんでしょうねえ
（3/29が東京では満開らしい。あとは散るだけだな）
そういう問題でもないか・・・
たまたまっしょ

>> Kettyさん
> なるべくみんなで
みんなその意識でいると思います。
質問者の意向にそえているのかはともかく　笑

usao · 投稿記事 by **usao** » 10年前

音楽じゃなくて，不定期に「にゃー」とか言ってくれるとかなら有りかも…

> 春休み

夏や冬とちがって
・宿題的なものが出にくい
・日数が少なくて何か作るのにも比較的向かない
的な感じなのかな．

ISLe · 投稿記事 by **ISLe** » 10年前

> 春休み
過去の一覧をざっと眺めただけですが
3月末の1週間くらいで立ったトピックは昨年も一昨年も軽く2ページ分(60トピック)は超えている感じなんですけどね。
今年はページ半分もいってないみたいなので、減りすぎじゃないかな、と。

へにっくす · 投稿記事 by **へにっくす** » 10年前

> usaoさん

どこからともなくニャーの鳴き声
本物が来ても気づかないかも…笑

> ISLeさん

あーなるほど
確かに少ないね。
だからって何をするわけじゃないだろうけども。

		11月 2025
日	月	火	水	木	金	土
						1
2	3	4	5	6	7	8
9	10	11	12	13	14	15
16	17	18	19	20	21	22
23	24	25	26	27	28	29
30

レベルがあわない

レベルがあわない

RE: レベルがあわない

Re: レベルがあわない

Re: レベルがあわない

Re: レベルがあわない

Re: レベルがあわない

Re: レベルがあわない

Re: レベルがあわない

Re: レベルがあわない