C言語 文字列の操作に関する問題 配列の並びの反転

Question

問題
文字列sの文字の並びを反転する関数を作成せよ。
void rev_string(char s[]){/・・・/}
たとえば、sに"SEC"を受け取ったら、その配列を"CES"に更新すること。

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

#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>

void rev_string(char s[])
{
	int i = 0;
	int len = 0;

	while (s[i]) {
		len++;
	}

	for (i = 0; i < len; i++) {
		s[i] = s[len -1 - i];
	}
	printf("%s", s);

	return 0;
}

int main(void)
{
	char str[128];

	printf("文字列を入力してください：\n");
	scanf("%s", str);

	rev_string(str);

	return 0;
}

このプログラムを実行すると入力しただけでプログラムが止まってしまいます。
なぜ、そのようになるのか理由が知りたいです。
よろしくお願いします。

下記は解決後のコードです。

改訂1

#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>

void rev_string(char s[])
{
	char temp[128];
	int i = 0;
	int len = 0;

	len = strlen(s);

	for (i = 0; i <(len / 2); i++) {
		temp[i] = s[i];
		s[i] = s[len - 1 - i];
	}

	if ((len % 2) == 0) {
		for (i = (len / 2); i < len; i++) {
			s[i] = temp[len - 1 - i];
		}
	}
	else {
		for (i = (len / 2) + 1; i < len; i++) {
			s[i] = temp[len - 1 - i];
		}
	}

}

int main(void)
{
	char str[128];

	printf("文字列を入力してください：\n");
	scanf("%s", str);

	rev_string(str);
	printf("%s", str);

	return 0;
}

改訂2(改訂1をシンプルにしました)

#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>

void rev_string(char s[])
{
	int i = 0;
	int len = 0;
	char temp;

	len = strlen(s);

	for (i = 0; i <(len / 2); i++) {
		temp = s[i];
		s[i] = s[len - 1 - i];
		s[len - 1 - i] = temp;
	}

}

int main(void)
{
	char str[128];

	printf("文字列を入力してください：\n");
	scanf("%s", str);

	rev_string(str);
	printf("%s", str);

	return 0;
}

Answer 1

解決済みなら、できたコードを質問に追加して、見せてもらえませんか？

「文字列sの文字の並びを反転して、それを表示するする関数を作成せよ。」
とはなっていないので、rev_string の中で表示をしてはいけないと思います。
また、void rev_string ですから、return 0; を付けてはいけません。

参考にならないコード

#include <stdio.h>

int revstr(char *s, int i)
{
	char c = s[i];
	return c ? i = revstr(s, i + 1), s[i] = c, i + 1 : 0;
}

void rev_string(char s[]) { revstr(s, 0); }

int main(void)
{
    char str[128];
    puts("文字列を入力してください：");
    scanf("%127s", str);
    rev_string(str);
	puts(str);
}

別解

#include <stdio.h>   // puts, scanf
#include <string.h>  // strlen

void rev_string(char s[])
{
	for (int i = 0, j = strlen(s); i < --j; i++) {
		char t = s[i];
		s[i] = s[j];
		s[j] = t;
	}
}

int main(void)
{
    char str[128];
    puts("文字列を入力してください：");
    scanf("%127s", str);
    rev_string(str);
	puts(str);
}

Comments

[aya0]

ご指摘ありがとうございます！
kazuma-sさんがご指摘頂いた箇所も訂正して、質問に訂正後のコードを追加しました。

[aya0]

別解のコードありがとうございます！
とても、シンプルで見やすいコードですね！
参考にします！

Answer 2

すこし文字列が長くても反転する様にしてみました

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

void rev_string(char s[])
{
    int i = 0;
    int len = 0;
    char w;

    len = strlen(s);

    for (i = 0; i < (len / 2); i++) {
        w=s[i];
        s[i] = s[len -1 - i];
        s[len -1 - i] = w;
    }
    printf("%s\n", s);

}

int main(void)
{
    char str[128];
    memset(str,'\0',128);
    printf("文字列を入力してください：\n");
    scanf("%s", str);

    rev_string(str);

    return 0;
}

tempに反転セットするにしてみました

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

void rev_string(char s[])
{
    char temp[128];
    int i = 0;
    int len = 0;

    len = strlen(s);

    memset(temp,'\0',128);
    for (i = 0; i < len; i++) {
        temp[len - i - 1] = s[i];
    }
    strcpy(s, temp);

}

int main(void)
{
    char str[128];

    printf("文字列を入力してください：\n");
    scanf("%s", str);

    rev_string(str);
    printf("%s", str);

    return 0;
}

Comments

[aya0]

コードありがとうございます！
私はif文を使って処理してたのですが、
w=s[i];
s[i] = s[len -1 - i];
s[len -1 - i] = w;
こちらの方がコードがシンプルでいいですね！
参考にします！

[aya0]

別解のコードありがとうございます！
そのようなやり方もあったのですね！
参考にします！

Answer 3

while(s[i])のループブロックが無限ループします。
⇒iを加算し忘れていますよ。
そもそも文字列の長さを取得するなら、
len = strlen(s);
で良いです。

scanfで取得する文字列長が127を超えるとバッファオーバーフローが発生し得ます。

また、文字列の入れ替えですが、このロジックでは配列の中央から先は既に移動済の文字が代入されてしまいます。

Comments

[maisumakun]

（そこまで求められることはないとは思いますが）1バイト文字への対応だけでいいのかという仕様も気になります。

[aya0]

ありがとうございます！
すごく助かりました！

Answer 4

まずは気づいたところだけですが、

    while (s[i]) {
        len++;
    }

このループでlenを変えてもs[i]は変化しないので、無限ループとなってしまっています。

Comments

[aya0]

ありがとうございます！
すごく助かりました！