どこがちがうのでしょうか？

質問

以下の二つのプログラムはユーザーが文字を入力し、80文字以下なら
ピリオドを追加して表示するというものです。上はうまくいきますが、下はうまくいきません。なぜでしょうか？

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

int main()
{
char str[80];
int i;

printf("文字列を入力してください。\n");
gets(str);

if(strlen(str)<80)
{

for(i=strlen(str);i<79;i++)
strcat(str,".");
}

printf("%s",str);

}

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

int main()
{
char str[80];
int i;

printf("文字列を入力してください。\n");
gets(str);

if(strlen(str)<80)
{

for(i=strlen(str)+1;i<79;i++)
str[i] = ".";
}

printf("%s",str);

}

回答No.1

massano1 · ベストアンサー

str[]はchar型なので、
for(i=strlen(str)+1;i<79;i++)
str[i] = '.';
です。

お礼

解答ありがとうございます。

int main()
{
char str[81];
int i;

printf("文字列を入力してください。\n");
gets(str);

if(strlen(str)<80)
{

for(i=strlen(str)+1;i<79;i++)
str[i] = '.';
}

printf("%s",str);

}

でもうまくいきませんでした。.がでてきません。

回答No.4

chie65535

＞上はうまくいきますが、下はうまくいきません。

どちらも「失敗」しています。上は「失敗しているのが、表面化してないだけ」です。

どちらも
char str[80];
である為、strに入れられる文字列の最長は「７９文字」です。

Ｃ言語の文字列には「文字列の終端記号」と言うのが付加されますから、charが８０個の配列に入れられる最長の文字列は「８０－１＝７９」で「７９文字」です。

上も下も
char str[81];
と書かないと、８０文字の文字列を入れる事はできません。

もし「無理に入れる」と「メモリの何処かを書き替えて、何かを壊す」事になります。

予期しないメモリに書き込んだとしても「運良く、壊れても影響が出ない場所」だったり「メモリに書き込んでは居るが、運良く、書き込む前と同じ値を書き込んで、メモリが変化しなかった」だったりすると、今回の「上の方」のように「問題が表面化しないで終る」ので、バグに気付く事が出来ません。

＞str[i] = ".";

他の回答の通り「charに、文字列へのポインタを代入している」ので「それ以前の問題」です。

ここは
str[i] = '.';
にしなければなりません。

また、最後に
str[80] = '\0';
として「文字列の終端記号」を書き込まないと、おかしな事になります。

お礼

解答ありがとうございます。

回答No.3

php504

最後に0を
for(i=strlen(str)+1;i<79;i++)
str[i] = '.';
}
str[i] = '\0';

お礼

解答ありがとうございます。

回答No.2

redfox63

strが80個しか確保していないので

if ( strlen( str ) < 80 )
で判断して strcatで連結すると文字列の終端をあらわす NULLが予定外の部分を壊してしまう可能性があります

char str[81];
といった具合に確保して80文字分の領域とNULL文字格納分をとりましょう

お礼

解答ありがとうございます。