0

宿題の一部で立ち往生しています。文字列内の部分文字列の右端のオカレンスを見つけなければなりませんでした。最初の部分は完了しました (単一の単語の文字列で部分文字列を見つけることができます) が、今は 2 番目の部分で問題が発生しています。複数単語の文字列 (別名、スペース付き) を許可するために、変更されたバージョンの getline を使用する必要があります。getline() と変更された strindex() の変更されていないコードも (それぞれ) あります。説明も欲しいのですが、書かれたコードを理解するのに苦労することがあります。

編集：コードを更新しました。ここにあります:)

/* string index */

int strindex(char str[], char substr[]){

    int str_idx, sub_idx, k, c = -1;

    for (str_idx = 0; str[str_idx] != '\0'; str_idx++) {

        for (sub_idx = str_idx, k = 0; substr[k] != '\0' && str[sub_idx] == substr[k]; sub_idx++, k++)
            ;

        if (k > 0 && substr[k] == '\0')
            c = str_idx;

    }
    return c;
    return -1; //never reached?
}

/* getline 
*
* Variable Dictionary
* ctchars - character counter, increments once each time getchar() is called
* str_idx - current index of the string, starts at 0, increments with loop
* 
*/


getline(char str[], int lim){

    int ctchars, str_idx = 0;

    ctchars=getchar();

    for (str_idx; str_idx<lim-1 && ctchars !=EOF && ctchars!='\n'; ++str_idx)
        str[str_idx] = ctchars;

    if  (ctchars == '\n') {
        str[str_idx] = ctchars;
        ++str_idx;
    }


    str[str_idx] = '\0';
    return str_idx;

}
4

1 に答える 1

2

いくつか提案があります。

for (j=i, k = 0; t[k] != '\0' && s[j] == t[k]; j++, k++);

これを壊してください。for 制御構造にすべてを詰め込まないでください。

j = i;
k = 0;
for (;;) {
    if (t[k] == '\0') break;
    if (s[j] != t[k]) break;
    j++;
    k++;
}

変数に適切な名前を付けます。

haystack_inner_index = i; /* get a better name for `i` too */
needle_index = 0;
for (;;) {
    if (needle[needle_index] == '\0') break;
    if (haystack[haystack_inner_index] != needle[needle_index]) break;
    haystack_inner_index++;
    needle_index++;
}
于 2009-10-25T22:38:21.380 に答える