toupperがうまく機能しない

Question

プログラムを実行するとコアダンプが発生してしまいます。ポインタがうまくいっていないのが原因だと思いますが、どのように
すればよいかわかりませんもしよろしければ教えて下さい。

include <stdio.h>
include<string.h>
include<ctype.h>

void main(){
int i,j;
char *p=NULL;
char *q;
char str[]="hello world";
char *suit[]={"Hearts","Diamonds","Clubs","Spades"};
printf("%p\n","abc");
p="ABC";
printf("%p\n",p);
printf("%c\n",p[0]);
p+=1;
printf("%s\n",p);
p-=1;
printf("%s\n",p);
while(*p!=0){
printf("%c\n",*p);
p+=1;
}
for(i=0;i<strlen(str);i++){
if(str[i]>=97 && str[i]<=122){
str[i]-=32;
}
}
printf("%s\n",str);
for(i=0;i<strlen(suit[i]);i++){
char *s=suit[i];
for(j=0;j<strlen(suit[i]);j++){
s[j]=toupper(s[j]);
}
printf("%s\n",s);
}
}

Answer 1

-	char *suit[] = { "Hearts","Diamonds","Clubs","Spades" };
+	char suit[][9] = { "Hearts","Diamonds","Clubs","Spades" };

文字列リテラルは書き換え禁止です。

-	for (i = 0; i < strlen(suit[i]); i++) {
+	for (i = 0; i< sizeof(suit) / sizeof(suit[0]); i++){

要素数でループ。

---

suitの書き換えがなぜそのようにするとうまくいくのか

char *suit[] = { "Hearts","Diamonds","Clubs","Spades" };
とした場合、suit[]は書き換え禁止領域に置かれた文字列リテラルへのポインタの配列となります。
よって内容を書き換えようとするとセグメンテーション違反/アクセス違反となります。

char suit[][9] = { "Hearts","Diamonds","Clubs","Spades" };
とした場合、charの二次元配列suit[4][9]に文字列リテラルの内容がコピーされます。
この配列はローカル変数なので書き換え可能です。

Comments

[linkinpark]

ありがとうございました！suitの書き換えがなぜそのようにするとうまくいくのか教えていただけないでしょうが？

[linkinpark]

とてもわかりやすい説明ほんとうにありがとうございました！！

Answer 2

変えたのですがコアダンプとでてしまいました

Answer 3

for(i=0;i<strlen(suit[i]);i++){

これを

for(i=0;i<sizeof(suit) / sizeof(suit[0]);i++){

こうする。

Comments

[linkinpark]

変えたのですがコアダンプがでてしまいました

[SHOMI]

文字列リテラルは書き換え禁止です