ポインタを使って文字列を逆順に表示させたいです。

Question

ポインタを使ってキーボード入力した文字列を逆順に表示させたいのですが、うまくいきません。参考書を読んだり、試行錯誤したのですがわからずじまいでした。
きっと無茶苦茶なコードだと思うのですが、どこをどう直せばいか教えていただけると嬉しいです。

コード
#include <stdio.h>

int str_length(const char *s)
{
    int len = 0;
    while(*s++)
        len++;
        return len;
}
int main(void)
{
    printf("文字列を入力してください:");
	char t[100];
	char str[100];
	char *p1 = t;
    char *p2;
    scanf("%c",t);
	int i;

	p1 = &t[0];
	p2 = &str[0];

   int  str_length = k;
	for(i=0;i<k;i++){
		*(p2+k-i) = *(p1+i);
	}

	*(p2+k) = '\0';

	printf("str2の文字列は %s です．¥n", str);
}

実行結果

コード
ensyuu5a.c: In function ‘main’:
ensyuu5a.c:23:22: error: ‘k’ undeclared (first use in this function)
   23 |    int  str_length = k;
      |                      ^
ensyuu5a.c:23:22: note: each undeclared identifier is reported only once for each function it appears in

Answer 1

間違っているのは次のところです。

-コード
+//コード

-    scanf("%c",t);
+    scanf("%s",t);
 
-   int  str_length = k;
+    int k = str_length(t);

-        *(p2+k-i) = *(p1+i);
+        *(p2+k-i-1) = *(p1+i);
 
-    printf("str2の文字列は %s です．\n", str);
+    printf("str の文字列は %s です．\n", str);

しかし、*(p1+i) という書き方は p1[i] と同じで、
i を添字とする配列のアクセスと変わりません。
ポインタを使ったプログラムというのは、ポインタを動かして配列の要素を
参照するプログラムを要求されているのだと思います。

例えば、次のようなものです。

#include <stdio.h>

int str_length(const char *s)
{
	const char *p = s;
    while (*p) p++;
    return p - s;
}

int main(void)
{
    char t[100];
    char str[100];
    printf("文字列を入力してください:");
    scanf("%s", t);
    char *p1 = t + str_length(t);
	char *p2 = str;
	while (p1 > t) *p2++ = *--p1;
	*p2 = '\0';
    printf("str の文字列は %s です．\n", str);
}

Answer 2

#include <stdio.h>

int str_length(const char *s) {
    int len = 0;
    while(*s++)
        len++;
    return len;
}

int main(void) {
    char t[100];
    char str[100];

    printf("文字列を入力してください:");
    scanf("%s",t); // %c -> %s

    int  k = str_length(t); // k = tの長さ
    for ( int i = 0; i < k; i++ ) {
        *(str+i) = *(t+k-i-1);
    }
    *(str+k) = '\0';

    printf("str2の文字列は %s です．\n", str);
}

Answer 3

まず、エラーメッセージを読んで対応しましょう。文法エラーが出ているならそれはプログラムの要件を満たしていない、「プログラム未満の何か」でしかないということですから、まずプログラムとして成立させましょう。

int str_length = k;
これは、str_lengthという変数を宣言し、初期値として(この時点で)変数kが持っている値を与えるという指示です。しかし、変数kというのはプログラムにそれまで登場していませんから、なんじゃそりゃ初めて見たぞそんなの知らない、ということでエラーになっています。

基本的な文法は教科書等を確認してください。ポインタ以前(別に前とか後ろがあるわけじゃないですけれど)がわからないで新しい話に手を出したら混乱が増すばかりかと思います。

関数の呼び出しは
関数名( 引数リスト )
です。
関数で得た値を変数に代入したいのなら代入の演算子=を使って
変数 = 関数名(引数リスト)
で「右辺の値を左辺に」代入してください。

さて。別に悪いことではないのですがstr_length()だけを関数に切り出したのはどういう理由でしょう。なんだか唐突な印象があります。
関数に切り出すのなら、例えば、「入力した文字列を逆順に表示させたい」というお題に則ってやることを考えて

• リスト入力を保存する変数を用意

• 入力を得る関数呼び出し

• 反転した結果を保存する変数を用意

• 反転する関数呼び出し

• 反転した結果を表示する関数呼び出し

という作業をすればよさそうで、いきなり「文字数を数える関数が要るなぁ」とは考えないのではないでしょうか。(こういうのを考えて、できれば紙に書き出してからプログラムコードを打ち始める、というのは結構大事なことだと思います)

そうすると、

int main(void){
  //入力する
  char in[100];
  scanf("%99s",in);

  //反転する
  char rev[100];
  reverse(in,rev);

  //表示する
  printf("%s\n",rev);

  return 0;
}

ここまではポインタもへったくれもないですから、reverse関数にポインタ操作を盛り込んで

void reverse( char* in, char* out){
  int len=str_length(in);
  in+=len-1; //ポインタが入力の一番最後の文字を指すようにする
  for(int i=0;i<len;i++){
    *out++=in--;
  }
  *out='\0';
}

とでもすればそれっぽい、という話になるでしょう。

もっとも、お題が何の制限もなく「ポインタを使ってキーボード入力した文字列を逆順に表示」だけなら、途中他の文字列を介する必要もなく

int main(void){
  char in[100];
  scanf("%99s",in);
  int len=str_length(in);
  char* t=in+len;
  while(t!=in){
    putchar(*--t);//printf("%c", *--t);でも同じ結果
  }
  return 0;
}

あたりで十分じゃないでしょうか? いろいろ使って複雑にするのならそれなりの理由が必要と思います。

付け加えると、これも間違いではないのですが、変数の宣言と使う場所が微妙に離れているのも「なんだかいきあたりばったり/あちこちから集めた情報の切り貼りで作ってるなぁ」という印象を醸し出しているような気がします。

余談。
できるだけ「いろいろ使わない」とこんなのもできるげど...ポインタも使ってないや。

int main(void){
  char c=getchar();
  // cへの値取り込みはscanf("%c",&c);を使ってもよい
  if(c!='\n'){
    main();
    putchar(c);
  }
  return 0;
}

Comments

[sawayakaunagi]

void reverse( char* in, char* out){
int len=str_length(in);
in+=len-1; //ポインタが入力の一番最後の文字を指すようにする
for(int i=0;i<len;i++){
*out++=in--;
}
この２行目のstr_length(in);の(in)はどのような役割をしているのでしょうか？