Bir karakter [] dizgesel değişmez kötü uygulama ile başlatılıyor mu?

Ben başlıklı bir konu okuyordu CodeGuru üzerinde "sizeof vs strlen" ve cevaplar biri devletler "o zaten initialie [sic] a [sic] kötü uygulama olduğunu charbir dize ile dizide."

Bu doğru mu, yoksa sadece ("seçkin bir üye" de olsa) görüşü mü?

İşte asıl soru:

#include <stdio.h>
#include<string.h>
main()
{
    char string[] = "october";
    strcpy(string, "september");

    printf("the size of %s is %d and the length is %d\n\n", string, sizeof(string), strlen(string));
    return 0;
}

sağ. boy uzunluğu artı 1 evet olmalıdır?

bu çıktı

the size of september is 8 and the length is 9

boyutu kesinlikle 10 olmalıdır. onun strcpy ile değiştirilmeden önceki dizgenin boyutunu hesaplamak gibi bir şey.

Sözdizimde yanlış olan bir şey mi var?

İşte cevap :

Yine de, bir dizgenin değişmezi ile bir karakter dizisi başlatmak kötü bir uygulamadır. Dolayısıyla daima aşağıdakilerden birini yapın:

const char string1[] = "october";
char string2[20]; strcpy(string2, "september");

Yine de, bir dizgenin değişmezi ile bir karakter dizisi başlatmak kötü bir uygulamadır.

Bu yorumun yazarı hiçbir zaman haklı çıkarmaz ve ifadeyi şaşırtıcı buluyorum.

C'de (ve bunu C olarak etiketlediniz), dizge değeri olan bir diziyi başlatmanın tek yolu bu char(başlatma atamadan farklıdır). Ya yazabilirsiniz

char string[] = "october";

veya

char string[8] = "october";

veya

char string[MAX_MONTH_LENGTH] = "october";

İlk durumda, dizinin boyutu, başlatıcının boyutundan alınır. Dize değişmezleri, char0 baytlık sonlandırıcı bir dizi halinde saklanır , bu nedenle dizinin boyutu 8'dir ('o', 'c', 't', 'o', 'b', 'e', ​​'r', 0). İkinci iki durumda, dizinin boyutu, bildirimin bir parçası olarak belirtilir (8 ve MAX_MONTH_LENGTH, ne olursa olsun).

Ne olamaz yaptığı gibi yazma şeydir

char string[];
string = "october";

veya

char string[8];
string = "october";

vb İlk durumda, beyanı stringolduğunu tamamlanmamış hiçbir dizi boyutu belirtilen edildiği için ve gelen boyutunu almak için hiçbir başlatıcı var. Her iki durumda =da işe yaramaz çünkü a) Bir dizinin ifadesi stringbir ödevin hedefi olmayabilir ve b) =Operatör bir dizinin içeriğini bir başkasına kopyalamak için tanımlanmadı.

Aynı şekilde yazı yazamazsın.

char string[] = foo;

foobaşka bir dizi nerede char. Bu başlatma şekli sadece string değişmezleri ile çalışacaktır.

DÜZENLE

Bunu, bir dizi-tarzı başlatıcıya sahip bir dize tutacak dizileri de başlatabileceğinizi söylemek için değiştirmeliyim.

char string[] = {'o', 'c', 't', 'o', 'b', 'e', 'r', 0};

veya

char string[] = {111, 99, 116, 111, 98, 101, 114, 0}; // assumes ASCII

ancak sicim harflerini kullanmak gözler için daha kolaydır.

EDIT ²

Bir dizinin içeriğini bir bildirimin dışında atamak için ya strcpy/strncpy(0-sonlandırılmış dizeler için) ya da memcpy(diğer herhangi bir diziler için) kullanmanız gerekir:

if (sizeof string > strlen("october"))
  strcpy(string, "october");

veya

strncpy(string, "october", sizeof string); // only copies as many characters as will
                                           // fit in the target buffer; 0 terminator
                                           // may not be copied, but the buffer is
                                           // uselessly completely zeroed if the
                                           // string is shorter!

Hatırladığım tek sorun, tam anlamıyla string değişimini atamak char *:

char var1[] = "september";
var1[0] = 'S'; // Ok - 10 element char array allocated on stack
char const *var2 = "september";
var2[0] = 'S'; // Compile time error - pointer to constant string
char *var3 = "september";
var3[0] = 'S'; // Modifying some memory - which may result in modifying... something or crash

Örneğin bu programı alın:

#include <stdio.h>

int main() {
  char *var1 = "september";
  char *var2 = "september";
  var1[0] = 'S';
  printf("%s\n", var2);
}

Bu benim platformumda (Linux) salt okunur olarak işaretlenmiş sayfaya yazmaya çalıştığında çöküyor. Diğer platformlarda 'Eylül' yazdırabilir.

Söylendi - değişmez tarafından başlatma, belirli miktarda rezervasyon yapar, böylece bu işe yaramaz:

char buf[] = "May";
strncpy(buf, "September", sizeof(buf)); // Result "Sep"

Ama bu olacak

char buf[32] = "May";
strncpy(buf, "September", sizeof(buf));

Son olarak - Ben hiç kullanmazdım strcpy:

char buf[8];
strcpy(buf, "very long string very long string"); // Oops. We overwrite some random memory

Bazı derleyiciler güvenli aramaya dönüştürebilse strncpyde çok daha güvenlidir:

char buf[1024];
strncpy(buf, something_else, sizeof(buf)); // Copies at most sizeof(buf) chars so there is no possibility of buffer overrun. Please note that sizeof(buf) works for arrays but NOT pointers.
buf[sizeof(buf) - 1] = '\0';

Her iki parçanın da getiremediği şeylerden biri şudur:

char whopping_great[8192] = "foo";

vs.

char whopping_great[8192];
memcpy(whopping_great, "foo", sizeof("foo"));

Eski gibi bir şey yapacak:

memcpy(whopping_great, "foo", sizeof("foo"));
memset(&whopping_great[sizeof("foo")], 0, sizeof(whopping_great)-sizeof("foo"));

İkincisi sadece memcpy yapar. C standardı, bir dizinin herhangi bir kısmının başlatılması durumunda hepsinin olduğu konusunda ısrar eder. Yani bu durumda, kendin yapmak daha iyidir. Sanırım Treuss'un aldığı şey bu olabilirdi.

Kesinlikle

char whopping_big[8192];
whopping_big[0] = 0;

ikisinden de iyidir:

char whopping_big[8192] = {0};

veya

char whopping_big[8192] = "";

ps Bonus puanları için şunları yapabilirsiniz:

memcpy(whopping_great, "foo", (1/(sizeof("foo") <= sizeof(whopping_great)))*sizeof("foo"));

Bir derleme zamanı atmak için diziden taşmak üzereyseniz sıfır hata ile bölün.

Öncelikle char[], program içinde kolayca kullanabileceğiniz bir değişken / yapı içinde boyutuna sahip olmayacağınız için.

Bağlantıdaki kod örneği:

 char string[] = "october";
 strcpy(string, "september");

stringyığında 7 veya 8 karakter uzunluğunda tahsis edilir. Bu şekilde null sonlandırılıp sonlandırılmadığını hatırlayamıyorum - bağladığınız iş parçacığı olduğunu belirtti.

"Eylül" kelimesini bu dizgenin üzerine kopyalamak, açık bir bellek aşımıdır.

Başka stringbir işleve geçerseniz , diğer işlevin diziye yazabilmesi için bir başka zorluk ortaya çıkar . Çok dizidir ne kadar diğer işlevi söylememe gerek o aşılmasına oluşturmaz. Sonucu stringile birlikte geçebilirsiniz , strlen()ancak konu stringboş bırakılmazsa bunun nasıl patlayabileceğini açıklar .

Sabit bir boyuta sahip bir dize (tercihen sabit olarak tanımlanır) atamaktan daha iyi bir durumdasınız ve daha sonra diziyi ve sabit boyutu diğer işleve geçirin. @John Bode adlı kullanıcının yorumları doğru ve bu riskleri azaltmanın yolları var. Ayrıca, bunları kullanmak için sizin tarafınızdan daha fazla çaba gösterilmesini gerektirir.

Deneyimlerime göre, başlattığım değer, char[]buraya yerleştirmem gereken diğer değerler için genellikle çok küçük. Tanımlanmış bir sabit kullanmak, bu sorunun önlenmesine yardımcı olur.

sizeof stringsize arabellek boyutunu verecektir (8 bayt); strlenHafızadan endişe duymak yerine bu ifadenin sonucunu kullanın .
Çağrısı önce Benzer şekilde, bir çek yapabilir strcpyhedef tampon kaynak dizesi için yeterince büyük olup olmadığını görmek için: if (sizeof target > strlen(src)) { strcpy (target, src); }.
Eğer bir işleve dizi geçmek zorunda Evet, eğer siz de fiziksel boyutunu geçmesi gerekir: foo (array, sizeof array / sizeof *array);. - John Bode

Bence "kötü uygulama" fikri bu formdan kaynaklanıyor:

char string[] = "october is a nice month";

dolaylı olarak kaynak makine kodundan yığına bir strcpy yapar.

Bu dizeye sadece bir link vermek daha verimlidir. Gibi:

char *string = "october is a nice month";

veya doğrudan:

strcpy(output, "october is a nice month");

(ama elbette çoğu kodda muhtemelen önemli değil)

Asla çok uzun bir zaman değil, ancak karakter dizgesini char [] dizgisine çevirmekten kaçınmalısınız, çünkü "string" const char * olur ve siz onu char * a atarsınız. Dolayısıyla, eğer bu karakteri [] veriyi değiştiren yönteme geçirirseniz ilginç davranışlarda bulunabilirsiniz.

Commend dediğim gibi char * ile biraz char [] karıştırdım, biraz farklı olduklarından iyi değil.

Char dizisine veri atama konusunda yanlış bir şey yoktur, ancak bu diziyi kullanmanın niyeti onu 'string' (char *) olarak kullanmak olduğundan, bu diziyi değiştirmemelisiniz unutmak kolaydır.