Könnte jemand mir helfen, diesen strchr() C Segmentierungsfehler zu verstehen?

Question

Ich tauche in C in Zeiger und Strings ein und ich gewöhne mich immer noch an einige Konzepte. Ich habe versucht, eine Version der strchr() Funktion - die gleiche wie in string.h - zu Studienzwecken zu implementieren, aber etwas Grundlegendes ist immer noch nicht richtig.

Hier ist mein Code:

#include <stdio.h> 

char* my_strchr(const char* str, int c){ 
    if (str == NULL){ 
    printf("STR is NULL. Finishing the program\n"); 
    return NULL; 
    } 
    while (*str != '\0'){ 
    if (*str == c){ 
     return (char*) str; 
    } 
    str++; 
    } 
    return NULL; 
} 

int main(){ 
    char *a = "Hello World!"; 
    char *b; 
    char c; 

    printf("Type the character you want to find in the Hello World! string:\n"); 
    scanf(" %c", &c); 

    b = my_strchr(a, c); 

    printf("Character found! %c\n", *b); 

    return 0; 
} 

Ich versuche, herauszufinden, warum dies ist eine Segmentierung Fehler zurückgegeben. Wenn ich gbd verwende, sagt es mir, dass der Fehler in der letzten printf ist, die versucht, die *b zu drucken.

Sobald my_strchr() einen (char*) str zurückgibt, müsste ich diesen Rückgabewert in einer char Zeigervariable speichern, richtig?

Answer 1

Wenn my_strchr das Zeichen in der Zeichenfolge nicht findet, wird NULL zurückgegeben.

In diesem Fall ist bNULL also *b ist undefiniertes Verhalten, das den segfault erklärt.

Vielleicht möchten Sie das Ergebnis my_strchr vor dem Drucken *b, z.B .:

if (b != NULL) { 
    printf("Character found! %c\n", *b); 
} else { 
    printf("Not found...\n"); 
} 

Answer 2

Es gibt eine gewisse Logik Problem zu überprüfen, wie die tuple_cat.

Aber ich denke auch, dass Sie einige Konzepte nicht verstehen, Ihr Code ist aus meiner Sicht nicht sauber.

Ich denke, man habe gerade angefangen in c Codierung so halten Codierung :)

Zuerst ein char * in Ihrer Funktion zurück, sondern Sie definieren Argument der Funktion als

char* my_strchr(const char* str, int c) 

In Standard-C können Sie berühre keine Konstante, die du nicht verändern kannst, das ist der Punkt, an dem du eine Konstante deklarierst.

so die Funktion ändern, um

char* my_strchr(char* str, int c) 

Dann ist die richtige Art und Weise ein Zeichen aus einem String zurück ist nicht

return (char*)str; 

aber nur

return str; 

Am Ende Ihres Funktion.

Auf diese Weise senden Sie die Adresse des ersten Zeichens in char * (string). In einem char * tun Sie das, indem Sie einfach den Variablennamen angeben.

Ich möchte Sie ermutigen lesen: https://www.gnu.org/software/gnu-c-manual/gnu-c-manual.html

RTFM !!! der char * Teil bei 1.3.4 String Konstanten

Wie auch immer viel Glück in Ihrem Lernen.