文字列sをn回連続して表示する関数について

Question

}```C

文字列sをn回だけ連続して表示する関数
void put_stringn(const char s[], int n)
{

int i;

for (i = 1; i <= n; i++)

{

　for (i = 0; s[i] == '\0'; i++)

{

　　　printf("%s", s[i]);

}

}

どこがいけないのかわからないです(-_-;)

Answer 1

原因（１）：
内側のForループの継続条件が間違えている。

質問のコードの内側のforループの部分は、以下のようになっています。

for (i = 0; s[i] == '\0'; i++)
{
　　　printf("%s", s[i]);
}

ここで、s[]は文字列、すなわち'\0'を終端とするCharの配列です。
普通、空文字列('\0'だけの配列）を表示しようとはしません(表示する文字がありませんからね)。
だから、s[0]は'\0'ではありません。

そうすると、forループの最初(i=0の時）で、s[0] == '\0'　が成り立ちませんから、"printf("%s", s[i]);"が実行されないまま、forループが終わってしまっているのです。（当然、何も表示されません）

文字列（'\0'を終端とするCharの配列）を表示したいのであれば、最初から順に１文字ずつ表示していくというのを'\0'が現れるまで続ければ良いですよね。
そうすると、forループを続ける条件は、s[i] == '\0'ではなく、s[i] != '\0'にしないといけないという事に気づくはずです。

また、１文字表示するのですから、printfのフォーマットは、"%s"ではなく"%c"です。
コードを修正すると、

for (i = 0; s[i] != '\0'; i++)
{
　　　printf("%c", s[i]);
}

これが、文字列を一回表示するコードです。　
（ 上記コードを「printf("%s",s);」と一行で済ませることもできますが、それではプログラムの練習にならないですよね）

原因（２）：
２つのforループで、iがループ変数として使われている。

　質問のコードには二重になったforループがありますが、どちらのforループもループ変数としてiを使っているのは問題です。
内側のforループは、上で説明したように i=0でループが終わります。外側のforループで、i++　とiを増やしても、内側のforループで i=0に戻ってしまいますから、外側のforループは終わることがありません。（質問のコードを実行したら、いつまで経っても終わらないです）

　この問題は、forループごとに、違うループ変数を使うことで解決できます。

以上の修正をすると、コードは以下のようになります。

/* 文字列sをn回だけ連続して表示する関数  */
void put_stringn(const char s[], int n) {
int i; // 外側のforループのループ変数
int j; // 内側のforループのループ変数 (追加しました)

for (i = 1; i <= n; i++)
{
　for (j = 0; s[j] != '\0'; j++) // jをループ関数として使います
   {
　　　printf("%c", s[j]);
   }
} 

Answer 2

なぜ二重ループにしてるんです？

for(int i = 0; i < n; ++i) {
    printf("%s\n", s);
}

---

逆に二重ループを使いたいなら、ループカウンタは分けないといけません。

for(int i = 0; i < n; ++i) {
    for(int j = 0; str[j] != '\0'; ++j) {
        printf("%c", str[j]);
    }
    printf("\n");
}

それ以外にもおかしな点を修正しています。

• 内側のループの継続条件
• 出力する際の書式指定子

Comments

[ssspwp]

詳しい説明ありがとうございます！

Answer 3

一応、

　　printf("%s", s[i]);

文字列を出力しようとしていますが、 ("%s")
s[i] は、文字です。 (違いが分かりますか?)

Comments

[ssspwp]

素早い回答ありがとうございます！