C言語 文字列の操作の問題

Question

問題
文字列の配列を読み込む関数を作成する。
"$$$$$"を読み込んだ時点で読み込みを中断・終了する。
"$$$$$"より前に入力された全文字列を表示する。

下記は私が考えた、この問題に対するプログラムです。

#define _CRT_SECURE_NO_WARNINGS
#define N 128
#define A 128
#include <stdio.h>
#include <string.h>
#include <ctype.h>


void put_strary2(const char s[][A], int n)
{
	int i, j, l, m;
	/*-入力-*/
	for (i = 0; i < n; i++) {

		int j = 0;
		int k = 0;

		while (s[i][j]) {
			do {
				scanf("%c", &s[i][j]);
				if (s[i][j] == '$') {
					k++;
				}
				else
					break;
				j++;
			} while (k <  5);
		}

		if (k == 5)
			break;

	 }
	/*-出力-*/
	printf("%d", i);
	/*上の行はiに代入されてる値を調べるために追加しました。なぜi=0となってるのか分からないです*/
	for (l = 0; l < i; l++) {
		for (m = 0; s[l][m] != '\0'; m++) {
			printf("%c", s[l][m]);
		}
		printf("\n");
	}
}


int main(void)
{
	char cs[N][A] = { "0" };
	printf("文字列を入力\n");
	put_strary2(cs, N);
	
}

"$$$$$"を読み込んだ時点で読み込みを中断・終了するところまでは、実行できたのですが、"$$$$$"より前に入力された全文字列を表示することができませんでした。
そこで試しに、入力が終了したところでiの値を確認したところ0となっていました。
なぜi=0になってしまうのか理由が知りたいです。
宜しくお願いします。

下記は解決後のプログラムです。

#define _CRT_SECURE_NO_WARNINGS
#define N 128
#define A 128
#include <stdio.h>
#include <string.h>
#include <ctype.h>


void put_strary2(char s[][A], int n)
{
	int i, j, l, m;
	/*-入力-*/
	for (i = 0; i < n; i++) {

		int j = 0;
		int k = 0;

		scanf("%s", s[i]);
		
		while (k < 5) {
			if (s[i][j] == '$') {
				k++;
				j++;
			}
			else
				break;
		}

		if (k == 5)
			break;

	}
	/*-出力-*/
	printf("%d\n", i);
	/*上の行はiに代入されてる値を調べるために追加しました。*/
	for (l = 0; l < i; l++) {
		for (m = 0; s[l][m] != '\0'; m++) {
			printf("%c", s[l][m]);
		}
		putchar('\n');
	}
}


int main(void)
{
	char cs[N][A] = { "0" };
	printf("文字列を入力\n");
	put_strary2(cs, N);

}

Comments

[cateye]

char cs[][A] = { "0" };は
char cs[N][A] = { "0" };と違いますか？

[aya0]

ご指摘ありがとうございます！
修正しました。

Answer 1

#define _CRT_SECURE_NO_WARNINGS があることから
コンパイラは Visual C++ を使っているんだと思いますが、
コンパイル時に警告メッセージが出ませんか？

put_strary2 の引数 s は const char の配列(へのポインタ) です。
const char は書き換え不可能な char という型です。
なのに scanf("%s", s[i]); で配列の中の文字を書き換えていて、その警告が出るはずです。
const を付けてはいけません。

また、$ が 5個以上あるかどうかは strncmp を使えば簡単に分かります。

#define _CRT_SECURE_NO_WARNINGS
#define N 128
#define A 128
#include <stdio.h>   // scanf, puts, printf
#include <string.h>  // strncmp

void put_strary2(char s[][A], int n)
{
    int i, j;
    for (i = 0; i < n; i++) {
        scanf("%s", s[i]);
		if (strncmp(s[i], "$$$$$", 5) == 0) break;
    }
    printf("%d\n", i);
    for (j = 0; j < i; j++)
		puts(s[j]);
}

int main(void)
{
    char cs[N][A];
    puts("文字列を入力");
    put_strary2(cs, N);
}

Comments

[aya0]

ご指摘ありがとうございます！
const使うと書き換えできないのに使ってしまっていたことに気づけて良かったです！
strncmp使うとコードがすっきりしていいですね！
参考にします！

Answer 2

ここの部分が無限ループになっていて k = 5 にならないと抜けれない状態です。

while (s[i][j]) {
            do {
                scanf("%c", &s[i][j]);
                if (s[i][j] == '$') {
                    k++;
                }
                else
                    break;
                j++;
            } while (k <  5);
        }

でこの二つのループを抜けると　k のチェックが待っていて確定で for loop も抜けるので i はインクリメントされないので 0 のままです。

if (k == 5)
            break;

また j のインクリメントもされないので s[0][0] に無限に代入を続けている形です。最後の出力される s は一番最後に保存した1行のみです。自分はc言語はあまり分からないので何とも言えませんがconst char に文字列を入れるのはありなのでしょうか？

何かの助けになれば幸いです。

Comments

[aya0]

回答ありがとうございます！
mipoponさんのおかげで、for文の中のwhile文でk>5にならない限り、無限にループしてしまってiが増えてないことに気づけました！
プログラムも修正して正常に動きました！
すごく助かります！

Answer 3

for (i = 0; i < n; i++)の中にあるwhileのループを抜けるときにkは5になります。抜けたところでif(k==5)でforを抜けるのでiは0です。

Comments

[aya0]

回答ありがとうございます。
for文を抜けた後、iの値が0になるのか別のコードで試したのですが、以下のコードの場合、i = ４になりました。
なぜ、for文を抜けた後、i=(初期値）になる場合とi=(最終値）になる場合があるのか知りたいです。
宜しくお願いします。

```
int main(void)
{
int i, j;
j = 0;

for (i = 0; i < 10; i++) {
if (j == 4)
break;
else
j++;
}

printf("%d",i);
}
```

[mipopon]

常にi=(最終値）になります。ただ @aya0 さんのコードだと for loop が1周しかしないので i = 0 と言う事です。