Segmentierungsfehler (Core Dumped) - C Keine Probleme mehr mit Segmentierungsfehler. Jetzt kann ich debuggen. danke

Question

Der Code ist in Ordnung. Bei der Ausführung bekomme ich nur die Nachricht (Segmentierungsfehler (Core Dump)). Jede Hilfe würde sehr geschätzt werden!

Das Programm soll die Basis der eingegebenen Argumente ändern. zum Beispiel ./convert 10 2 5 6 7 Eingang Basis: 10 Ausgangsbasis: 2 Argumente Eingabebasis: 5 6 7

fange ich gerade c zu lernen ... jede Hilfe würden Sie bieten könnten sei großartig! Danke!

Bearbeitet: Keine Probleme mehr mit Segmentierungsfehler. Vielen Dank.

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




int makeInt(char digits[]) { 
    int num = 0; 
    int i = 0; 
    while (digits[i] != '\0') { 
    num = num * 10 + (digits[i] - '0'); 
    i++; 
} 
return num; 
} 


//converts from original base to base 10 
int makeBase10(char digits[],int a) { 
    int num = 0; 
    int pow = 1; 
    int i = 0; 
    int x = 0; 
    int length = strlen(digits); 
    char k; 
    for (i = length -1; i > -1; i--){ 
     k = digits[i]; 
     if(k <= '9' && k >= '0'){ 
      x = (int)k - '0'; 
     } 
     else{ 
      x = (int)k - 'a' + 10; 

     } 
     num = num + (x*pow); 
     pow = pow * a; 

    } 
    return num; 
} 
//changes number from base 10 to knew base 
// and returns as string 
char * toString(char str[], int num, int b) { 

    int i = 0; 
    char x; 
    int mod = 0; 
    while(0 < num){ 
     mod = num % b; 
     if(mod < 10 && mod > -1){ 
      x = (char)(num + '0'); 
     } 
     else{ 
      x = (char)(num - 10 + 'a'); 
     } 

     str[i] =(int) x; 
     num = num/b; 
    } 
    str[i] = '\0'; 

    int length = i + 1; 
    int j = 0; 
    char *reverse; 
    for(i = length -2; i >= 0; i--){ 

     reverse[i] = str[j]; 
     j++; 
    } 
    reverse[length-1] = '\0'; 
    return reverse; 
} 



int main(int argc, char *argv[]) { 
    int i; 
    int num; 
    int a; 
    int b; 
    char str[33]; //binary could be up to 32 + '\0' 
    a = makeInt(argv[0]); //input base 
    b = makeInt(argv[1]);; //output base 
    printf("make int worked and input works"); 
    num = 0; 
    i = 2; 
    while (i < argc) { 
     num = makeBase10(argv[i], a); 
     printf("%s\n", toString(str, num, b)); 
      i++; 
    } 


} 

Answer 1

char in toStrong Reverse * ist kein Speicher zugewiesen, die zu segmentieren Fehler, ein Teil von diesem argv führen könnte [0] immer der Programmname Sie ausführen, müssen Sie argv verwenden [1] weiter für Ihre Eingaben

Answer 2

Sie verwenden einen nicht initialisierten Zeiger.

int length = i + 1; 
int j = 0; 
char *reverse; // here is the problem 
for(i = length -2; i >= 0; i--){ 

    reverse[i] = str[j]; 
    j++; 
} 
reverse[length-1] = '\0'; 

Wenn ein nicht initialisierte Zeiger verwenden Sie schreiben, was auch immer geschieht Speicheradresse in diesen Variablen gespeichert werden. Es wurde kein vernünftiger Wert angegeben. Daher schreiben Sie in einen Speicherort, der nicht für Apps gedacht ist. Computerspeicher ist segmentiert. Jeder Speicherbereich hat einen Besitzer, der dort schreiben kann. Es ist eine Sicherheitsvorkehrung und verhindert, dass Bugs das System durcheinander bringen. Sie müssen zunächst initialisiert werden entweder mit malloc

char *reverse = malloc(length); 

oder wenn Sie bei der Kompilierung kennen eine maximale Größe, die Sie tun können:

char reverse[MAX_SIZE]; 

P. S. Um umzukehren, durchquert man nicht die ganze Saite bis zum Anfang nur die Hälfte. Anstatt for(i = length -2; i >= 0; i--)
schreiben: for(i = length -2; i >= length/2; i--)