Cプログラミング - 文字列配列要素の変更文字

Question

可能性の重複：私はC.内の別の文字に文字列の文字を変更しようとしているので、
Problem with processing individual strings stored in an array of pointers to multiple strings in C

[OK]をつまり、各文字列は1D配列の要素なので、文字列自体が文字配列なので、本質的にすべて一緒に2次元配列になります。とにかく私はこれを行うコードを作成する問題があります。これも可能ですか？どんな助けもありがとうございます。ここで

はコードです：

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

int main() 
{ 
int i, size; 
char **a; 

a=(char**)malloc(sizeof(char*)); 

printf("Enter the size of the array:"); 
scanf("%d", &size); 

for(i=0;i<size;i++){ 
a[i]=(char*)malloc(sizeof(char)*8); 
} 

a[3]="Read"; 


while(*(a[3])!='\0'){ 
if(*(a[3]) == 'e'){ 
    *(a[3]) = 'r'; 
} 
} 

printf("%s\n", a[3]); 

system("pause"); 
return 0; 

} 

Answer 1

あなたはaのための十分なスペースを割り当てることができませんでした。代わりに

a=(char**)malloc(sizeof(char*)); 

のあなたは

a=(char**)malloc(sizeof(char*)*size); 

と明らかにこれを必要とsizeが読み込まれた後であることを移動しなければなりません。

あなた一度ソートそれはむしろ平凡な問題アウトは根本的な問題はここにある：

a[3]="Read"; 

これを変更することはできませんリテラルでポインタa[3]ポイントになります。代わりに、そのリテラルの内容をa[3]にコピーする必要があります。このように：

strcpy(a[3], "Read"); 

あなたはa[3]=...だけのポインタa[3]を割り当てへa[3]ポイントの文字列を変更しないことを理解しなければなりません。 sizeその後a[3]以来4未満の場合さて、あなたのコードは明らかにエラーになります

は範囲外になりますが、私はあなたがこれをデバッグしながら、a[3]はただ一時的であると思います。

あなたのwhileループはすべて間違っています。あなたはこのような何かをしたいあなたのコメントから判断：

char *p = a[3]; 
while (*p != '\0') 
{ 
    if (*p == 'e') 
     *p = 'r'; 
    p++; 
} 

Cでmallocの戻り値をキャストするので、キャストを削除する必要はありません。 sizeof(char)は常に1に等しいので、削除することもできます。

Answer 2

この：

a=(char**)malloc(sizeof(char*)); 

は、1つの文字列のためのスペースを割り当てます。 a、例えば、a[3]の要素を逆参照することができます、この時点で

char **a = NULL; 
size_t number_of_strings = 8; /* for argument's sake */ 

a = malloc(number_of_strings * sizeof(char*)); 
if (!a) 
    return NOT_ENOUGH_MEMORY_ERROR; 

：あなたはおそらくつもりはのようなものです。あなたはまだあまりにもそれらの人のためのスペースを割り当てることになるでしょう。それと

char *staticStr = "Read"; 
a[3] = malloc(strlen(staticStr) + 1); 
strncpy (a[3], staticStr, strlen(staticStr) + 1); 

スタートし、メモリを割り当てるしている方法を再考すると、あなたの他のバグを修正するのに役立ちますかどうかを確認します。

いくつかの注意：

• あなたは常に1
• である、あなたがメモリを割り当てるためsizeof(char)を使用する必要はありません
• C
に mallocの結果をキャストする必要がある必要はありません。私はanoth参照して、メモリリークを

Answer 3

を防止するために、各a[i]およびa自体の対応free()を使用することえー問題、あなたがのためにメモリを割り当てるとき：

a = (char **) malloc(sizeof(char *)); 

あなただけの1の位置にメモリを割り当てているが、あなたはsize位置を使用しています。あなたのコードは：

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

int main() 
{ 
int i, size; 
char **a; 
char *ptr; 



printf("Enter the size of the array:"); 
scanf("%d", &size); 

a=(char**)malloc(size * sizeof(char*)); 

for(i=0;i<size;i++){ 
a[i]=(char*)malloc(sizeof(char)*8); 
} 

strcpy(a[3], "Read"); 

ptr=a[3]; 
while(*ptr!='\0'){ 
if(*ptr == 'e'){ 
    *ptr = 'r'; 
} 
ptr++; 
} 

printf("%s\n", a[3]); 

system("pause"); 
return 0; 

} 

となります。もちろん、割り当てられたメモリを解放する必要があります。

'e'を 'r'に変更しようとすると、いつも同じcharを指しています。配列をスローするには、新しいポインタが必要です。

Answer 4

a=(char**)malloc(sizeof(char*)); 

printf("Enter the size of the array:"); 
scanf("%d", &size); 

for(i=0;i<size;i++){ 
a[i]=(char*)malloc(sizeof(char)*8); 
} 

nope。 1 char*を割り当てました。次に、それはsize要素のように扱います。 size * sizeof(char*)バイトを割り当てる必要があります。 （この乗算もオーバーフローする可能性があります。）

a[3]="Read"; 

悪い回。文字列リテラルの場所が"Read"の場合は、a[3]（以前は8文字の割り当てを指していました）を上書きしています。これにより、以前の割り当てがリークされ、変更不可能な文字列がa[3]に格納されます。あなたはstrncpyなどを見てください。このため。

Answer 5

あなたのwhileループは何もしません。

while(*(a[3])!='\0'){ 
if(*(a[3]) == 'e'){ 
    *(a[3]) = 'r'; 
} 
} 

ポインタを前進させず、最初の位置に保持します。

より適切っぽいな方法は、一時的なポインタを作成することや、文字列を歩くためにそれを使用することになり

char *temp = a[3]; 
while (*temp != '\0') { 
    if (*temp == 'e') *temp = 'r'; 
    temp++; 
}