กำลังเริ่มต้นอักขระถ่าน [] ด้วยการฝึกตัวอักษรที่ไม่ถูกต้องหรือไม่?

ฉันกำลังอ่านหัวข้อเรื่อง"strlen vs sizeof" บน CodeGuruและหนึ่งในคำตอบกล่าวว่า "มันเป็นการปฏิบัติที่ไม่ดี [sic] สำหรับ initialie [sic] charอาร์เรย์ที่มีตัวอักษรสตริง"

นี่เป็นความจริงหรือว่าเป็นเพียงความคิดเห็นของเขา (แม้ว่าจะเป็น "สมาชิกระดับสูง")?

นี่คือคำถามเดิม:

#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;
}

ขวา. ขนาดควรเป็นความยาวบวก 1 ใช่ไหม

นี่คือผลลัพธ์

the size of september is 8 and the length is 9

ขนาดควรเป็น 10 อย่างแน่นอน มันเหมือนการคำนวณขนาดของสตริงก่อนที่จะมีการเปลี่ยนแปลงโดย strcpy แต่ความยาวหลังจาก

มีบางอย่างผิดปกติกับไวยากรณ์ของฉันหรืออะไร

นี่คือคำตอบ :

มันเป็นการปฏิบัติที่ไม่ดีเลยที่จะเริ่มต้นชุดอักขระถ่านด้วยตัวอักษรสตริง ดังนั้นทำอย่างใดอย่างหนึ่งต่อไปนี้:

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

มันเป็นการปฏิบัติที่ไม่ดีเลยที่จะเริ่มต้นชุดอักขระถ่านด้วยตัวอักษรสตริง

ผู้เขียนความคิดเห็นนั้นไม่เคยพิสูจน์ความจริงเลยและฉันพบว่าคำสั่งนั้นทำให้งง

ใน C (และคุณติดแท็กเป็น C) นั่นเป็นวิธีเดียวที่จะเริ่มต้นอาร์เรย์charด้วยค่าสตริง (การเริ่มต้นนั้นแตกต่างจากการกำหนด) คุณสามารถเขียนได้

char string[] = "october";

หรือ

char string[8] = "october";

หรือ

char string[MAX_MONTH_LENGTH] = "october";

ในกรณีแรกขนาดของอาร์เรย์จะถูกนำมาจากขนาดของ initializer สตริงตัวอักษรจะถูกเก็บไว้เป็นอาร์เรย์ของcharด้วยการสิ้นสุด 0 ไบต์ดังนั้นขนาดของอาร์เรย์คือ 8 ('o', 'c', 't', 't', 'o', 'b', 'e', ​​'r', 0) ในสองกรณีที่สองขนาดของอาร์เรย์จะถูกระบุเป็นส่วนหนึ่งของการประกาศ (8 และMAX_MONTH_LENGTHสิ่งที่เกิดขึ้น)

สิ่งที่คุณไม่สามารถทำได้คือเขียนสิ่งที่ชอบ

char string[];
string = "october";

หรือ

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

ฯลฯ ในกรณีแรกประกาศของstringคือไม่สมบูรณ์เพราะไม่มีขนาดอาร์เรย์ได้รับการระบุและมีการเริ่มต้นที่จะใช้ขนาดจากไม่มี ในทั้งสองกรณีตัวดำเนินการ=จะไม่ทำงานเนื่องจาก a) นิพจน์อาร์เรย์เช่นstringอาจไม่ใช่เป้าหมายของการมอบหมายและ b) ตัว=ดำเนินการไม่ได้ถูกกำหนดให้คัดลอกเนื้อหาของอาร์เรย์หนึ่งไปยังอีกแถวหนึ่ง

ด้วยโทเค็นเดียวกันคุณไม่สามารถเขียนได้

char string[] = foo;

ที่เป็นอาร์เรย์ของผู้อื่นfoo charการเริ่มต้นรูปแบบนี้จะทำงานกับตัวอักษรสตริงเท่านั้น

แก้ไข

ฉันควรแก้ไขสิ่งนี้เพื่อบอกว่าคุณยังสามารถเริ่มต้นอาร์เรย์เพื่อเก็บสตริงด้วย initializer สไตล์อาร์เรย์เช่น

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

หรือ

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

แต่จะเป็นการง่ายกว่าที่จะใช้ตัวอักษรสตริง

แก้ไข²

ในการกำหนดเนื้อหาของอาร์เรย์นอกการประกาศคุณจะต้องใช้strcpy/strncpy(สำหรับสตริงที่สิ้นสุดด้วย 0) หรือmemcpy(สำหรับอาร์เรย์ประเภทอื่น ๆ ):

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

หรือ

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!

ปัญหาเดียวที่ฉันจำได้คือการกำหนดสตริงตัวอักษรให้กับ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

ตัวอย่างเช่นใช้โปรแกรมนี้:

#include <stdio.h>

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

สิ่งนี้บนแพลตฟอร์มของฉัน (Linux) ขัดข้องขณะพยายามเขียนไปยังหน้าที่ทำเครื่องหมายว่าอ่านอย่างเดียว บนแพลตฟอร์มอื่น ๆ อาจพิมพ์ 'กันยายน' เป็นต้น

ที่กล่าวว่า - การเริ่มต้นตามตัวอักษรทำให้จำนวนการจองที่เฉพาะเจาะจงดังนั้นสิ่งนี้จะไม่ทำงาน:

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

แต่สิ่งนี้จะ

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

ตามคำพูดสุดท้าย - ฉันจะไม่ใช้strcpyเลย:

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

ในขณะที่คอมไพเลอร์บางตัวสามารถเปลี่ยนเป็นการเรียกที่ปลอดภัยได้อย่างปลอดภัยstrncpyมากขึ้น:

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';

สิ่งหนึ่งที่ไม่มีเธรดเกิดขึ้นคือ:

char whopping_great[8192] = "foo";

เมื่อเทียบกับ

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

อดีตจะทำสิ่งที่ชอบ:

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

หลังเท่านั้น memcpy มาตรฐาน C ยืนยันว่าหากส่วนใดส่วนหนึ่งของอาร์เรย์ถูกเตรียมใช้งานมันคือ ดังนั้นในกรณีนี้ควรทำด้วยตัวเอง ฉันคิดว่านั่นอาจเป็นสิ่งที่ทริชได้มา

ได้อย่างแน่นอน

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

ดีกว่าอย่างใดอย่างหนึ่ง:

char whopping_big[8192] = {0};

หรือ

char whopping_big[8192] = "";

ps สำหรับคะแนนโบนัสคุณสามารถทำได้:

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

เพื่อทิ้งเวลารวบรวมโดยไม่มีข้อผิดพลาดหากคุณกำลังจะล้นอาร์เรย์

หลักเพราะคุณจะไม่มีขนาดของchar[]ในตัวแปร / สร้างที่คุณสามารถใช้ภายในโปรแกรม

ตัวอย่างโค้ดจากลิงค์:

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

stringถูกจัดสรรบนสแต็กโดยมีความยาว 7 หรือ 8 อักขระ ฉันจำไม่ได้ว่าเป็นโมฆะด้วยวิธีนี้หรือไม่ - เธรดที่คุณเชื่อมโยงเพื่อระบุว่าเป็น

การคัดลอก "กันยายน" บนสตริงนั้นเป็นหน่วยความจำที่เห็นได้ชัดเกิน

ความท้าทายอีกอย่างเกิดขึ้นถ้าคุณส่งผ่านstringไปยังฟังก์ชันอื่นเพื่อให้ฟังก์ชันอื่นสามารถเขียนลงในอาร์เรย์ได้ คุณจำเป็นต้องบอกฟังก์ชั่นอื่น ๆ นานเท่าใดอาร์เรย์เป็นดังนั้นมันจะไม่ทำให้เกิดการใช้จ่ายเกิน คุณสามารถผ่านstringไปพร้อมกับผลลัพธ์ของstrlen()เธรด แต่เธรดจะอธิบายว่าสิ่งนี้สามารถระเบิดได้อย่างไรหากstringไม่สิ้นสุดด้วย null

คุณควรจัดสรรสตริงที่มีขนาดคงที่ (ดีกว่ากำหนดเป็นค่าคงที่) จากนั้นส่งอาร์เรย์และขนาดคงที่ไปยังฟังก์ชันอื่น @John Bode แสดงความคิดเห็นถูกต้องและมีวิธีการลดความเสี่ยงเหล่านี้ พวกเขายังต้องใช้ความพยายามมากขึ้นในส่วนของคุณเพื่อใช้พวกเขา

จากประสบการณ์ของฉันค่าที่ฉันได้รับการเริ่มต้นchar[]มักจะเล็กเกินไปสำหรับค่าอื่น ๆ ที่ฉันต้องมีใน การใช้ค่าคงที่ที่กำหนดช่วยหลีกเลี่ยงปัญหานั้น

sizeof stringจะให้ขนาดของบัฟเฟอร์ (8 ไบต์) ใช้ผลลัพธ์ของนิพจน์นั้นแทนstrlenเมื่อคุณกังวลเรื่องความจำ
ในทำนองเดียวกันคุณสามารถทำให้การตรวจสอบก่อนที่จะเรียกร้องให้เพื่อดูว่าบัฟเฟอร์เป้าหมายของคุณมากพอสำหรับสตริงแหล่งที่มา:strcpy ใช่ถ้าคุณต้องส่งอาเรย์ไปยังฟังก์ชั่นคุณจะต้องผ่านขนาดจริงเช่นกัน: . - John Bodeif (sizeof target > strlen(src)) { strcpy (target, src); }
foo (array, sizeof array / sizeof *array);

ฉันคิดว่าความคิด "การปฏิบัติที่ไม่ดี" มาจากความจริงที่ว่าแบบฟอร์มนี้:

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

ทำให้ strcpy โดยนัยจากซอร์สโค้ดเครื่องไปยังสแต็ก

การจัดการลิงก์ไปยังสตริงนั้นมีประสิทธิภาพมากกว่า ชอบกับ:

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

หรือโดยตรง:

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

(แต่แน่นอนในรหัสส่วนใหญ่มันอาจไม่สำคัญ)

ไม่นานนัก แต่คุณควรหลีกเลี่ยงการเริ่มต้นถ่าน [] ถึง string เพราะ "string" เป็น const char * และคุณกำหนดให้ถ่าน * ดังนั้นหากคุณส่งถ่าน [] ไปยังวิธีการที่เปลี่ยนแปลงข้อมูลคุณสามารถมีพฤติกรรมที่น่าสนใจ

ดังที่ฉันได้กล่าวไว้ว่าฉันผสมถ่าน [] กับถ่าน * นั่นไม่ดีเท่าที่พวกเขาต่างกันเล็กน้อย

ไม่มีอะไรผิดปกติเกี่ยวกับการกำหนดข้อมูลให้กับอาร์เรย์ถ่าน แต่เนื่องจากความตั้งใจในการใช้อาร์เรย์นี้คือการใช้มันเป็น 'string' (char *) จึงเป็นเรื่องง่ายที่จะลืมว่าคุณไม่ควรแก้ไขอาร์เรย์นี้