C言語 文字列を表示する簡単な関数（初心者）

Question

初心者です。「明解C言語 入門編」という本の「演習11-1」がうまくできません。

###やりたいこと「演習11-1」

下記に示した関数put_stringと同じ動作をする関数を、添字演算子を使わずに作成せよ。
void put_string(const char str[])
{
unsigned i = 0;

while (str[i])
	putchar(str[i++]);

}

###発生している問題・エラーメッセージ
自分でコードを考えて書いてみたのですが、うまく表示できないうえ、フリーズします。

###該当のソースコード

#include <stdio.h>

void put_string(const char *s)
{
	while (*s++) {
		putchar(*s);
	};

}

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

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

	put_string(str);

	return 0;
}

###試したこと
まだ初心者のため、ポインタや配列など分からないことだらけなので、皆さんの力を貸してほしいです。

###補足情報(言語/FW/ツール等のバージョンなど)
使用しているソフトはvisual stdioです。

Answer 1

scanf_s()での文字列読み込みはサイズ指定が必要です。
scanf_s("%s", str, 100);としましょうd^^; ソースを見る限りsizeof(str)でもいけると思うんですが参考になるサイトが見つからない＾＾；
参考：scanf_sの落とし穴

Comments

[roun]

できました！サイズ指定なんて全く知りませんでした。
ありがとうございました。

Answer 2

お題のプログラムと ++ を使うタイミングが変わってますね．それが原因。
putchatrに渡している＊ｓは whileでチェックしたsの一つ先ですよ。＋＋してますからね。

Comments

[roun]

void put_string(const char *s)
{
while (*s) {
putchar(*s++);
};

}
にするということでしょうか？

[a_saitoh]

それでいけるはず。
・・・ごめんなさい。putc()がマクロであるような処理系が今でもあるのを忘れてました。
putchar(*s);
*s++;
にしてください。

あと、｝のあとの；は不要。あっても空文があることになるだけなのでコンパイルは通りますが。書かない癖をつけておいたほうがいいです。

[roun]

void put_string(const char *s)
{
while (*s) {
putchar(*s);
*s++;
}

}
にしても結果は同じでした。何をどうすればよいのでしょうか？

[a_saitoh]

put_string()はこれでいいです（手元のコンパイラで動作確認しました）。メインルーチン側にも問題がありましたね。別の人が回答済み。

[rubato6809]

重箱の隅のようではありますが、　*s++;　は　s++;　とすべきです。
「*」をつける必要はありません。動作はするけど * を付ければ間違いの部類に入るでしょう。

ここでやりたいことは、ポインタ変数 s が次の文字を指すように、 s をインクリメントすることだから。言い変えると、ここの一行で操作したいのはポインタ変数 s の値であって、s が指している文字ではありません。

[a_saitoh]

あ、コピペしたときに削り忘れました。