einfache Zeichenfolge Laufzeitfehler in C?

Question

Dieser Code wird zwar kompiliert, gibt aber beim Ausführen einen Segmentierungsfehler aus. Kann jemand sagen warum?

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

int main() { 
    const char s2[] = "asdfasdf"; 
    char* s1; 

    strcpy(s1, s2); 
    printf("%s", s1); 

    return 0; 
} 

Answer 1

Sie zugewiesenen Platz für einen einzelnen Zeiger, s1, aber nicht der Bytes zeigte auf von s1.

ist eine Lösung Speicher zuweisen dynamisch für s1:

s1 = (char *)malloc(strlen(s2) + 1); 
strcpy(s1, s2); 

Beachten Sie, dass Sie ein weiteres Byte Speicher (die +1 in dem Aufruf von malloc) als die Anzahl der Zeichen zuweisen müssen in s2 weil am Ende ein implizites NULL Byte ist.

Weitere Informationen finden Sie unter C Memory Management (Stack Overflow).

Answer 2

Sie haben keinen Speicher für s1 reserviert. Sie haben einen Zeiger auf s1, aber keinen Speicher, der für den strcpy reserviert ist, um den Wert von s2 in zu kopieren.

char *s1 = malloc(strlen(s2) + 1); 

strcpy(s1, s2); 

Answer 3

Sie müssen das Ziel zuweisen (und using namespace std; ist nicht C, sondern C++, der Rest des Codes ist C).

Answer 4

Das Problem ist, dass s1 keinen Speicher zugeordnet hat. strcpy ruft nicht malloc().

Sie könnten entweder tun:

char s1[10];

oder

char *s1 = malloc(10);

Answer 5

Sie haben kein Gedächtnis s1 nicht zugeordnet. Es ist ein Zeiger auf nichts.

char* s1 = malloc(sizeof(s2)); 
strcpy(s1, s2); 
printf("%s", s1); 
free(s1); 

Answer 6

Sie müssen dem Zeiger s1 Speicher zuweisen. Wenn Sie das nicht tun, wird es irgendwo unbekannt angezeigt und kommt damit zum Segmentierungsfehler. Der korrekte Code sollte sein:

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

int main() { 
    const char s2[] = "asdfasdf"; 
    char* s1 = malloc(21 * sizeof(s2[0])); 
    strcpy(s1,s2); 
    printf("%s",s1); 
    return 0; 
} 

Answer 7

Was sie alle gesagt haben, müssen Sie den Platz für s1 zuweisen. Was für alle anderen arbeiten geschrieben hat ganz gut, aber wenn Sie einen einfacheren Weg wollen für einen vorhandenen String Speicherplatz zuweisen und sie in einen neuen Zeiger kopieren dann strdup wie folgt verwenden:

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

using namespace std; 

int main() { 
    const char s2[] = "asdfasdf"; 
    char* s1; 

    s1 = strdup(s2); 
    printf("%s", s1); 

    return 0; 
} 

Jemand erwähnte strdup früher, das wäre ein Weg, es zu benutzen. Die meisten Systeme sollten es unterstützen, da es in den Standard-C-Bibliotheken enthalten ist. Aber anscheinend tun einige nicht. Also, wenn es einen Fehler zurückgibt, schreibe entweder deinen eigenen mit der bereits erwähnten Methode, oder benutze einfach die bereits erwähnte Methode;)

Answer 8

Niemand hat bisher auf das Potential von strdup (String Duplicate) hingewiesen, um dieses Problem zu lösen.

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

using namespace std; 

int main() { 
    const char s2[] = "asdfasdf"; 
    char* s1; 

    s1 = strdup(s2); // Allocates memory, must be freed later. 
    printf("%s", s1); 

    free(s1);   // Allocated in strdup, 2 lines above 
    return 0; 
}