入力した文字列を1文字ずつ後ろから消して表示したい

Question

前提・実現したいこと

キーボードから文字列を入力し、入力された文字列を後ろから1文字ずつ消して表示したいです。
表示例）
入力文字：ABCDE
実現したい表示内容↓
1行目　ABCDE
2行目　ABCD
3行目　ABC
4行目　AB
5行目　A

発生している問題・エラーメッセージ

コンパイルはできていますが、実行すると以下の内容で表示されてしまいます。
abcdeaaaaaaaaaaaaaaaaaaaaa

該当のソースコード

#include <stdio.h>
void del(char x[]);

int main(void){
    
    char a[20] = {0};
    
    printf("文字を入力->");
    fgets(a, 20, stdin);
    
    del(a);
}

void del(char x[]){
    
    int i, j;
    printf("%s", x);
    
    for(i=0; i<=20; i++){
        for(j=0; j<=20-i; i++){
            printf("%c", x[j]);
        }
        puts("");
    }
}

試したこと

for(j=0; j<=20-i; i++)→for(j=0; j<=5-i; i++)
上記のとおりfor文を変更しましたが表示内容がabcdeaaaaaaになっただけで、変化は見られませんでした。

for(j=0; j<=x[20]-i; i++)→for(j=0; j<=19-i; j++)
文末がi++となっていたのでj＋＋に修正しました。
abcdeが15行文余計に表示された後、下記のようになり希望の表示内容になりませんでした。
abcde
abcd
abc
ab

補足情報（FW/ツールのバージョンなど）

パイザを使っています。

Answer 1

int main(void)
{
    int n=0;
    char a[20] = { 0 };
    printf( "input str : " );
    fgets( a, 20, stdin );
    
    for( n=0; (a[n]!='\n' && a[n]!='\0'); ++n ){}  //まともな文字の個数nを数える
    for( ; n>0; --n ){  printf( "%.*s\n", n, a );  }  //nを減らしながら表示する
    
    return 0;
}

Comments

[fana]

（ "%.*s" とかいう知らない書式指定方法を見つけたので使ってみた的な）

[bambino]

丁寧にソースコードを記入いただきありがとうございます。
私も(%.*s)というのは初めて見ました。
改めて調べてみたいと思います。

Answer 2

fgets(x, 20, stdin); のところで、ABCDE Enter と入力すると、
x = { 'A', 'B', 'C', 'D, 'E', '\n', '\0', ... } となります。
配列の添え字と値の関係はこうです。

|[0]|[1]|[2]|[3]|[4]|[5]|[6]|
|:--|:--:|--:|
|'A'|'B'|'C'|'D'|'E'|'\n'|'\0'|

次に、これを表示するために、添え字（インデックス）を書き出してみます。
右側は表示される文字列です。

[0] [1] [2] [3] [4] => ABCDE
[0] [1] [2] [3] => ABCD
[0] [1] [2] => ABC
[0] [1] => AB
[0] => A

つまり
１回目は[0]から[4]まで表示
２回目は[0]から[3]まで表示
３回目は[0]から[2]まで表示
４回目は[0]から[1]まで表示
５回目は[0]から[0]まで表示

おそらくループを学んで日が浅いのでしょう。「二重のforループでできそうだ」という見通しは間違いではありませんが、私に言わせればあてずっぽうで書いているように見えます。

• 外側のループが何をするもので、いくつからいくつまで変化させればよいか
• 内側のループが何をするもので、いくつからいくつまで変化させればよいか

プログラムにさせることを一度、上のように書き出してみて、考えを整理して、それからプログラムに書き変えてみてください。

Comments

[bambino]

丁寧に解説いただきありがとうございます。
改めて書きかえてみます。

Answer 3

fgets(x, 20, stdin); のところで、ABCDE Enter と入力すると、
x = { 'A', 'B', 'C', 'D, 'E', '\n', '\0', ... } となります。

x の先頭から順に見ていき '\n' を見つけます。x[5] ですね。
そこに '\0' を入れて '\n' を削除して、x を表示すると ABCDE と出ます。

次に x[4] に '\0' を入れて 'E' を削除して、x を表示すると ABCD と出ます。
次に x[3] に '\0' を入れて 'D' を削除して、x を表示すると ABC と出ます。

このようにして、x[i] の i が 0 になったら終了です。

追記
質問に、「1文字ずつ後ろから消して」とあるので x[i] に '\0' を入れましたが、
そうではなく、「x[0]～x[i-1] を1文字ずつ表示する」でも構いません。

Comments

[bambino]

fgetsでの文字入力について、解説いただきありがとうございます。
解説の内容を考慮して、改めて考えてみます。

Answer 4

for(j=0; j<=20-i; i++){

よく見ましょう。 最後が、i++ です。
配列サイズのサイズが　i<=20 --> i<20 とかもありますが、、、。

また、上記の修正だけでは、多分、期待する結果に足りないと思いますが、直してから、考えましょう。

Comments

[bambino]

的確なご指摘ありがとうございます。
修正したところ、表示内容が少し理想に近づきました。
また、お時間があるときにアドバイスいただけますと助かります。

Answer 5

char a[20]

として定義したら、とり得るインデックスは、
a[0]からa[19]までです。
a[20]ではアクセス違反となります

でもひとつ、C言語における文字列、というのは、後端に'\0'が入っている文字配列です。
なので、２０バイトの配列に入れることのできる文字列は１９文字分まで、となります

---

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

void heruprint(char*s)
{
    for(int i=strlen(s)-1;i>=0;i--){
        puts(s);
        s[i]='\0';
    }
 }


int main(void){
    char s[]="123456789";
    heruprint(s);
   
}

Comments

[bambino]

分かりやすく回答いただきありがとうございます。
ご指摘いただいた内容に修正しましたが、表示内容に変化はありませんでした。
お時間があるときに改めてアドバイスいただけますと幸いです。

Answer 6

#include <stdio.h>

void del(char x[], int size);
int len_a(char x[], int size);
int main(void) {
  char a[20] = {'\0'};

  printf("文字を入力->");
  fgets(a, sizeof(a), stdin);
  del(a, sizeof(a));
}

int len_array(char s[], int size) {
  for (int i = 0; i < size; i++) {
    if ((s[i] == '\0') || (s[i] == '\n') || (s[i] == '\r')) {
      return i;
    }
  }
  return size - 1 ;
}

void del(char s[], int size) {
  for (int i = len_array(s, size); i > 0; i--) {
    for (int j = 0; j < i; j++) {
      printf("%c", s[j]);
    }
    printf("\n");
  }
}

実行例

Answer 7

アドバイスいただいた内容を一緒に勉強している仲間と見て、完成させることができました。
アドバイスいただいた方々にお礼申し上げます。
一応、下記に完成させたソースコードを載せておきます、皆様からしたら少し回りくどい方法かもしれませんが興味がありましたらご覧ください。

#include <stdio.h>

void kesu(char s[], int a);

int main(void){
    
    char moji[100];
    int i;
    
    printf("文字を入力してください：\n");
    scanf("%s", &moji[0]);
    
   　　for(i=0; moji[i]!='\0'; i++);
    
    kesu(moji, i);
}

void kesu(char s[], int a){
    
    int b, f;
     
    for(b=a; b>0; b--){
        for(f=0; f<=b-1; f++){
            printf("%c", s[f]);
        }
        printf("\n");
    }
}

Answer 8

(特にビギナのうちは)一つの関数でなんもかんもやろうと思わないのが肝心かも。

#include <stdio.h>

// [1] 文字列 x の長さを返す
int length(char x[]) {
  int i;
  for ( i = 0; x[i] != '\0'; ++i );
  return i;
}

// [2] 文字列 x を n 文字分書いて改行
void print_n(char x[], int n) {
  for ( int i = 0; i < n; ++i ) {
    printf("%c", x[i]);
  }
  puts("");
}

// [1]と[2]を作っておけばあとは楽勝
void del(char x[]) {
  for ( int len = length(x); len > 0; --len ) {
    print_n(x, len);
  }
}

int main(void){
  char str[] = "abracadabra";
  del(str);
  return 0;
}