एक स्ट्रिंग के लिए malloc स्थान के लिए सही तरीका और फिर उस स्थान में अक्षर डालें?

Question

मेरे पास दो स्ट्रिंग्स, str1 और str2 हैं। मैं ढेर में एक जगह पर उनके साथ संयोजन करना चाहता हूँ। मैं का उपयोग कर उनके लिए अंतरिक्ष malloc:

char *concat = (char*) malloc(strlen(str1) + strlen(str2) + 1); 

मैं सिर्फ कर सकते हैं:

strcat(concat, str1); 
strcat(concat, str2); 

और concat मुझे concatted दो तार के साथ ढेर पर जगह दे देंगे? मैं पूछ रहा हूं क्योंकि ऐसा लगता है कि स्ट्रैट वास्तव में malloc का उपयोग करके आवंटित स्थान के अंत में str1 जोड़ देगा। क्या ये सही है? तो, फिर, str1 स्थिति strlen (str1) + strlen (str2) + 1.

कारण मैं पूछ रहा हूं कि मैं ऊपर की विधि का उपयोग कर रहा हूं, लेकिन मुझे valgrind में त्रुटि मिल रही है: सशर्त कूद या चाल uninitialised मूल्य (रों)

Answer 1

क्या strcat(dest, src) लेकिन वास्तव में ऐसा एक अशक्त बाइट dest से शुरू होकर आगे बढ़ते हुए, और फिर वहाँ src स्ट्रिंग लिखने के लिए खोज है।

malloc के बाद, स्मृति की सामग्री अपरिभाषित है, इसलिए आपका वर्तमान कोड किसी भी चीज को कर सकता है, उनमें से अधिकांश गलत हैं। यदि आप concat[0] = 0strcat से पहले करते हैं, तो आपका कोड काम करता है लेकिन str1 की लंबाई को तीन बार - strlen के लिए खोजना होगा, फिर पहले strcat के लिए, और दूसरे strcat के लिए अंतिम होगा।

इसके बजाय, हालांकि, मैं memcpy:

size_t len1 = strlen(str1), len2 = strlen(str2); 
char *concat = (char*) malloc(len1 + len2 + 1); 

memcpy(concat, str1, len1); 
memcpy(concat+len1, str2, len2+1); 

इस तथ्य यह है कि आप शुरू जहां दोनों तार के बाइट्स जाना चाहता हूँ से पता का लाभ लेता है, और कितने देखते हैं का उपयोग करें। त्रुटि जांच का उल्लेख करने के लिए

Answer 2

पर निर्भर करता है आप एक strcpy और फिर एक strcat क्या करना चाहते हैं: ('\ 0') पता करने के लिए

strcpy(concat, str1); 
strcat(concat, str2); 

strcat वहाँ एक अशक्त टर्मिनेटर किया जा रहा है पर निर्भर करता है, जहां शुरू करने के लिए। यदि आप सिर्फ मॉलोक और स्ट्रैक करते हैं, तो यह कुछ ग़लत चीजें करने जा रहा है।

और नहीं, न तो स्ट्रैपी और न ही स्ट्रैट किसी भी तरह का आवंटन आवंटन या पुनर्वितरण करेगा।

Answer 3

मैं व्यक्तिगत रूप से निम्नलिखित करना होगा:

size_t length = strlen(str1) + strlen(str2) + 1; 
char *concat = malloc(sizeof(char) * length); 

if(concat == NULL) 
{ 
    // error 
} 

snprintf(concat, length, "%s%s", str1, str2);