C言語 ポインタを用いた文字列の反転

Question

C言語のポインタを使って文字列(Winter)を逆から表示させたいです。
以下のプログラムで実行すると、エラーは出ませんが出力結果が
retnir UX}?1??k?? UX}?
のような、変な物が表示されます。retniまでは正しく表示されているのになぜWだけがおかしくなってしまうのか分かりません。

int main(void){
  char  s[7], *p_s;
  strcpy(s, "Winter");
  p_s = s;
  while(*p_s != '\0'){
   //最後の文字になるまで繰り返す（文字列の最後を知るため）
   *s = *p_s;
   p_s++;
   }

  while(*p_s != '0'){
   //最後の文字から順に表示していく
   printf("%c",*p_s);
   p_s--;
   }

   printf("\n");
   return 0;
}

Answer 1

このコードには3つ問題があって、ひとつは、文字列末尾を探すループの中で、
*s = *p_s;
としていること。つまり、文字列末尾へ向かって動くp_sにある文字を、文字列先頭を指すsの内容に代入している。文字列先頭の文字はループ一回ごとに書き換えられ、最終的には文字列の末尾(ヌル文字の直前)にある'r'に書き換えられる。対策は、この行を丸ごと削除。

そもそも、文字列の末尾の位置を探すなら、p_sの初期値を設定している部分で、
p_s = s + strlen(s) - 1;
と書けば済む話だが、逆順表示の先頭がヌル文字になる問題を修正した後で治すことにする。

さて、次は逆順に表示していく部分だが、2つめの問題は、この時点でp_sが文字列終端のヌル文字を指していることだ。このため、先頭に「何か」が画面に表示される。対策は逆順表示前にp_sの値を1減らす。

3つめの問題は、逆順表示のwhileループの継続条件が間違っていることだ。当然のことだが、ヌル文字は文字列の終端にだけ付いており、文字列の先頭には付いていない。対策は、文字列の先頭を示すsとp_sの値を比較する式に変更。

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

int main(void) {
  char s[7], *p_s;
  strcpy(s, "Winter");
  p_s = s + strlen(s) - 1;
  
  while (p_s >= s) {
    printf("%c", *p_s);
    p_s--;
  }

  printf("\n");
  return 0;
}

Comments

[keeen]

細かい説明ありがとうございます。*s = *p_sを消して2つ目のwhileの条件をp_s >= s にすることでretniWと表示することができました。

Answer 2

こんな感じでどうでしょうか。

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

int main(void){
   char *s = "Winter";
   for (char *p = s + strlen(s) - 1; p >= s; --p) {
      printf("%c", *p);
   }
   printf("\n");
   return 0;
}

もともとのソースコードが'\0'でなく'0'になっていたことも問題なのですが、文字列の最後には'\0'という番兵がつくのですが文字列の最初には付かないのですよね。だから文字列の開始アドレスと比較してやる必要があります。

[2020.6.17 追記]
ソースコード間違ってました。Daregadaさんのソースコードを参考にして直しました。ありがとうございます。

Comments

[keeen]

forを使うことでコードが短くて見やすくなりますね。
異なった考え方を教えてくださってありがとうございます。勉強になりました。

[Daregada]

紛らわしい名前ですまないが、微妙に綴りが違っている(笑)

[anndonut]

Daregadaさん、お名前を間違えてしまい申し訳ありませんでしたっ

Answer 3

while(*p_s != '0'){

このwhileの条件文が意味不明です。
その文字列の中には'0'の文字は存在してません

なぜWだけがおかしくなってしまうのか分かりません。

コードをよく見ましょう。
あなたのコードの中で、そこの文字を書き換えてしまっています

Comments

[keeen]

1番目の文字は配列の番号で0なので条件も0にすれば良いと思い込んでいました。確かに' 'は文字を表す物なので0というのはおかしいですよね。おかげで無事解決できました。ありがとうございます！