誰でも私はこのstrchr（）Cセグメンテーションフォールトを理解するのを助けることができますか？

Question

私はCでポインタと文字列に潜っていますが、まだいくつかの概念に慣れています。私はstrchr()関数のバージョンを実装しようとしました。これはstring.hと同じですが、学習の目的では基本的なことはまだ正しくありません。ここで

は私のコードです：

#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; 
} 

私は、これがセグメンテーションエラーを返している原因を把握しようとしています。 gbdを使用すると、エラーが最後にprintfにあり、*bを印刷しようとしていることがわかります。

(char*) strを返すと、charポインタ変数にこの戻り値を格納する必要があります。

Answer 1

my_strchr文字列内の文字が見つからない場合は、NULLを返します。

この場合、bはNULLです。したがって、*bは、segfaultを説明する未定義の動作です。あなたが例えば、*bを印刷する前にmy_strchrの結果をチェックすることをお勧めします

：

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

Answer 2

ありtuple_catのようないくつかのロジックの問題が言われています。

しかし、私はあなたがいくつかの概念を理解していないと思う、あなたのコードは私の視点からきれいではありません。

私はあなたがちょうどそうコーディング:)

まず、あなたの関数内で、char *を返す保つCでコーディングし始めたと思いますが、あなたはあなたができる標準Cでは

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

として関数の引数を定義します定数に触れないでください。それを変更することはできません。それは定数を宣言する点です。

ので、次に文字列から文字を返すための正しい方法は

return (char*)str; 

が、あなたの終わりにちょうど

return str; 

ない

char* my_strchr(char* str, int c) 

に機能を変更します関数。

この方法で、char *（文字列）の最初のcharのアドレスを送信します。 char *では、変数名を与えるだけでそれを行います。

私がお読みになることをお勧め：https://www.gnu.org/software/gnu-c-manual/gnu-c-manual.html

RTFMを！ char *の部分は1.3.4文字列定数

とにかくあなたの学習には幸運があります。