ポインタ勉強中です。しかも実行するとおかしいです。

質問

＜本に載ってたソース＞
#include<stdio.h>
#include<string.h>
int main()
{
char msg[20];
char *str=NULL;
int i;
int cnt;

str=&msg[0];
printf("文字を入力してください");
scanf("%s",&str);
cnt=strlen(msg);
str=msg+cnt;
for(i=cnt;i>=0;i--){
printf("%c",*(str--));
}
printf("\n");
return 0;
}
char *str=NULL;は、ポインタstrを空にするということでしょうか？

いつもながらstrlenとsizeofが混じります。
sizeofがバイトの大きさで、strlenが、文字数でしたっけ？

回答No.3

hegemon · ベストアンサー

＞実行するとおかしいです。

すでに指摘されてますが、どうおかしいのか明記していただかないと回答のしようがありません。
コードを見る限り、入力された文字列を逆順で表示するもののようです。
「ABC」と入力すれば、「CBA」と表示されるでしょう。
ただし、マルチバイト文字を考慮していないので、漢字などを入力すると何が起こるかわかりません。

＞char *str=NULL;は、ポインタstrを空にするということでしょうか？

ヌルポインタは「どこも指していない」という意味なので、「空」というのは不正確ですね。

＞sizeofがバイトの大きさで、strlenが、文字数でしたっけ？

sizeof()は指定した変数が占めるメモリ領域のバイトサイズを返します。
strlen()は文字列の文字数を返します。
ちなみに、sizeof()は関数ではなく演算子です。

ただし、C/C++では文字列の終端をあらわす'\0'が必ず必要なので、文字列を格納する際には文字数+１の領域が必要です。
今回の例だと、
＞　　char msg[20];
となっているので、実際に入力できるのは19文字以下です。
その意味で、何文字だろうと受け入れてしまうscanf()は、非常に危険な関数だといえます。

お礼

確かにscanfは危険だと思いました。
sizeofとstrlenは理解できました。

回答No.2

php504

scanf("%s",&str);
は間違いですね
scanf("%s", str);
が正しいです

回答No.1

asuncion

タイトルで書かれている「実行するとおかしい」の
具体的な内容が、本文に書いてありません。
実行すると、どのようにおかしいのでしょうか？

＞char *str=NULL;は、ポインタstrを空にするということでしょうか？

ポインタ変数の値をNULLにするということは、
そのポインタ変数がどこも指していない状態にする、ということです。

＞sizeofがバイトの大きさで、strlenが、文字数でしたっけ？

char msg[20];
と定義した場合、sizeof(msg)は20です。
また、msgの内容が例えば"abcde"ならば、strlen(msg)は5です。
特殊な場合として、msgの内容が"abc\0xyz123"なら、strlen(msg)は3です。
strlenは、'\0'の直前までの長さを求めます。

お礼

ありがとうございます。
ポインタの勉強まだまだひつようだなと思いました。
sizeofとstrlenの違い理解できました。
今後使っていこうと思います。