入力した文字列が反転しない

Question

入力した文字列を、反転させたいんですが、うまくいきません。
なぜなんでしょうか？
メモリは確保入力した文字列+1にしています。

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

// 入力した文字列の反転を表示するプログラム　引数：なし　戻り値：0 メモリの確保失敗-1
int main()
{
	int i;
	char input[16];
	char *inputHantai;
	printf("文字列を入力してください：");
	scanf("%s", input);
	// 確保したメモリ(入力した文字列の長さ+1)をcharポインタにキャスト
	inputHantai = (char*)malloc(sizeof(char)*(strlen(input) + 1));	

	// メモリの確保に失敗したとき
	if(inputHantai == NULL)
	{
		puts("メモリの確保に失敗しました");
		return -1;
	}

	printf("文字列反転：");

	// 入力文字列の数だけ
	for(i = strlen(input) - 1; i >= 0; i--)
	{
		int j = 0;
		// 入力文字列反転を代入
		inputHantai[j] = input[i];
		j++;
	}

	// 結果出力
	printf("%s", inputHantai);

	// メモリ開放
	free(inputHantai);
	return 0;
}

Comments

[y_waiwai]

うまくいかないとはどうなるんでしょうか

[Luke02561]

実行結果を見せてください

[cateye]

char input[16];としておきながら、あえてmalloc()を使っているのは何故でしょう？
char inputHantai[16];ではいけないのでしょうか？
また、１６文字以上の入力に関してガードしていないのは？（スタック壊しますが？）

Answer 1

    // 入力文字列の数だけ
    int j = 0; /* for-loop内から移動 */
    for (i = strlen(input) - 1; i >= 0; i--)
    {
        // 入力文字列反転を代入
        inputHantai[j] = input[i];
        j++;
    }
    inputHantai[j] = '\0'; /* 追加 */

Answer 2

    // 入力文字列の数だけ
    for(i = strlen(input) - 1; i >= 0; i--)
    {
        int j = 0;
        // 入力文字列反転を代入
        inputHantai[j] = input[i];
        j++;
    }

いくらループしても j が増えないのではないでしょうか.

Answer 3

for文の閉じカッコのうしろに、
inputHantai[j]='\0';
をいれておきましょう

Comments

[L4zy]

なぜヌル文字を入れる必要があるのでしょうか？

[y_waiwai]

まずやってみてどうなるかみてみれば？

[jimbe]

> なぜヌル文字を入れる必要があるのでしょうか？
"入力した文字列の長さ+1" の領域を確保した "+1" とは, 何のためとお考えなのでしょう.