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

char tracks[][80] = {
"I left my heart in Harvard Med School",
"Newark, Newark - a wonderful town",
"Dancing with a Dork",
"From here to maternity",
"The girl from Iwo Jima",
};

void find_track(char search_for[])
{
    int i;
    for (i = 0; i < 5; i++)
    {
        if (strstr(tracks[i], search_for))
        printf("Track %i: '%s'\n", i, tracks[i]);
    }

}
int main()
{
    char search_for[80];
    printf("Search for: ");
    fgets(search_for, 80, stdin);
    find_track(search_for);
    getch();
    return 0;

}

これは、Head First C から直接取得したコードです。これは機能しません。一方で、メインの行を変更すると

char search_for[80];

に

char *search_for = "town"

それは私に期待される結果を与えます。コードを直接貼り付けてエラーを見つけるように指示することは、ここではあまり受け入れられないことを理解していますが、基本的なコードの非常に小さな部分なので、うまくいくと思います。

ありがとう

4

2 に答える 2

5

上記のコードの問題は、fgetsが文字列に改行を残すという事実を考慮していないことです。したがって、を入力して押すと、を検索することになりtownます。enter"town\n"

これを解決する安価な方法は、呼び出し後に文字列を修正することですfgets

search_for[strlen(search_for) - 1] = 0;
于 2012-08-18T15:06:04.577 に答える
1

@cnicutar で既に指摘されているように、fgets は文字列に改行文字 (コード 10) を残します。これは、char 配列を反復処理して文字コードを出力し、ASCII コード テーブル (http://www.asciitable.com/) で特殊文字を検索することで、自分で見つけることができます。

int main()
{
    int i;
    char search_for[80];
    printf("Search for: ");
    fgets(search_for, 80, stdin);
    scanf("%79s", search_for);

    // print index, char code, char itself
    for (i=0;i<80;i++) {
        printf("%d: %d %c\n", i, search_for[i], search_for[i]);
    }
    find_track(search_for);
    getch();

    return 0;
}

出力は次のようになります。

...
0: 116 t
1: 111 o
2: 119 w
3: 110 n
4: 10
5: 0 
...

関数 "scanf" を使用して、読み取り文字数を 79 に制限することができます (バッファが最大 80 文字を保持するため、文字列の末尾にある NULL 文字用に 1 文字を予約します)。http://www.cplusplus.com/reference/clibrary/cstdio/scanf/を参照してください。

int main()
{
    char search_for[80];

    printf("Search for: ");
    scanf("%79s", search_for);

    find_track(search_for);

    return 0;
}
于 2012-08-18T15:23:56.093 に答える