printf char array Segmentfehler C

Question

Also habe ich einen Code geschrieben, der Wörter in der richtigen Reihenfolge sortiert. Die Wörter werden über Zeiger gespeichert und ich habe ein anderes Char-Array im Programm initialisiert, um die char* argv zu speichern.

Die letzte for-Schleife ist, was Segmentfehler druckt und ich kann nicht herausfinden, warum.

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

int main(int argc, char* argv[]) { 
    int i, j; 
    char *key; 
    char a[argc-1]; 
    for(i=1; i < argc; i++){ 
    a[i-1]= tolower(argv[i]); 
     } 

    for (i = 2; i < argc; i++) { 
    key = argv[i]; 

    j = i-1; 
    while (j >= 1 && strcmp(argv[j], key) > 0) { 
     argv[j+1] = argv[j]; 
     j--; 
    } 

    argv[j+1] = key; 
    } 

    for(i = 1; i < argc; i++){ 
    a[i-1] = *argv[i]; 
    } 

    for (i = 1; i < argc ; i++){ 
    puts(argv[i]); 
    } 

    for(i = 0; i < argc-1; i++){ 
    printf("%s", a[i]); 
} 
    return 0; 
} 

Eingang

./a.out orange banana apple 

Ausgang

apple 
banana 
orange 
Segmentation fault 

Answer 1

Ihr Compiler sollten Sie auch einen warnging geben:

printf("%s", a[i]); 

Warnung: Format gibt den Typ 'char *', aber das Argum an ent hat 'char' [-Wformat]

eingeben Wenn Sie ändern, dass ein zu %c funktioniert es feinen.

Wie die Warnung sagt, wenn Sie es verwenden %s printf erwartet eine Zeichenfolge oder char* und wird es als solche behandeln. a[i] ist ein Integer-Typ, der auf einen ungültigen Speicherort im Speicher verweisen kann. Der richtige Weg wäre also, entweder% c zu verwenden und ein Zeichen auszudrucken; oder verwenden Sie% s und übergeben Sie ein char * als zweites Argument.

Oder vielleicht möchten Sie Args in a. Ändern Sie die Delkaration.

char *a[argc-1]; 

Dann ändern Sie die Zuordnung.

Answer 2

Es gibt mehrere Probleme in Ihrem Code

1. Die Linie a[i - 1] = tolower(argv[i]) falsch ist, weil tolower() 0 nimmtals Parameter und Sie übergeben char *, so dass es einen Zeiger auf int konvertiert, die in c zulässig ist, aber kein definiertes Verhalten garantiert.

2. Sie sind nicht die '\0' Terminator auf der a Array Einstellung, die für Probleme eine andere Ursache ist, insbesondere, wenn Sie versuchen, es als eine Zeichenfolge zu verwenden, ein char Array, es sei denn kein String ist, dann ist es eine Folge von druckbaren Bytes mit ein abschließendes '\0' Byte.

3. argc - 1 Allokierung wird nicht funktionieren, weil

   • Der Wert argc die Anzahl der übergebenen Parameter an die ausführbare Datei ist.
   • Wenn es die Länge der entsprechenden argv wie von strlen() gefunden hätte, müssten Sie 1 Byte nicht weniger, also wäre es in jedem Fall argc + 1.

Answer 3

Warum verwenden Sie% s

for(i = 0; i < argc-1; i++){ 
printf("%s", a[i]); 

try% c