“文字列”にはCの関数が含まれています

いくつかのコメント：

• 最後の行の後に改行を追加する必要があります：

   $ ./nh First char match index: 18 Char: t Exist count: 1 Exist count: 2 Exist count: 3 Exist count: 4 Position: 18 $ 

• 何がわかりません使用するコンパイラですが、gccおよび-Wall -Wextra -pedanticでコンパイルすると、次のようになります。

  gcc -O2 nh.c -lm -o nh -Wall -Wextra -pedantic nh.c: In function ‘contains’: nh.c:25:15: warning: unused variable ‘j’ [-Wunused-variable] size_t i, j; ^ 

• コードのフォーマットはより一貫している必要があります。たとえば、この行では、needleの前に空白を入れますが、前に空白を入れないでくださいhaystack：

  size_t contains(const char * needle, const char *haystack); 

• %luはsize_tタイプの移植可能な指定子ではありません。C99で導入された%zuを使用する必要があります。

• あなたが言った：

ある場合は位置を返し、そうでない場合は0を返します。位置は0です。この場合、位置は特定されません。

これは本当に良くありません。たとえば、これを使用すると0が返されます：

char *needle = "This"; char *haystack = "This is a dinosaurtest."; 

これを使用すると、ゼロも返されます：

char *needle = "non-existent"; char *haystack = "This is a dinosaurtest."; 

この2つの例では、成功と失敗の違いがわかりません。実際、atoi()にも同じ問題があります。使用しているオペレーティングシステムがわかりません。ただし、可能な場合は戻り値の型としてssize_tを使用し、失敗した場合は-1を返すことができます。

@Arkadiusz Drabczykによる前の回答への追加：

containsの単純で簡単な実装は次のようになります。このように実行します：

ssize_t contains(const char * needle, const char *haystack) { char *needle_in_haystack; if(!needle || !haystack) return -1; needle_in_haystack = strstr(haystack, needle); return needle_in_haystack ? needle_in_haystack - haystack : -1; } 

次に、このプログラム（上記のようにいくつかの変更を加えたもの）が機能するはずです：

#include <stdio.h> #include <stdlib.h> #include <string.h> #include <sys/types.h> ssize_t contains(const char * needle, const char *haystack) { char *needle_in_haystack; if(!needle || !haystack) return -1; needle_in_haystack = strstr(haystack, needle); return needle_in_haystack ? needle_in_haystack - haystack : -1; } int main(void) { char *needle = "test"; char *haystack = "This is a dinosaurtest."; char *haystack2 = "This does not contain the string."; printf("Position: %zd\n", contains(needle, haystack)); printf("Position: %zd\n", contains(needle, haystack2)); return EXIT_SUCCESS; } 

出力：

位置：18
位置：-1

コメント

• 入力がNULLでないことのチェックを削除し、juそのための言語拡張を使用します（GCCでは__attribute__((nonnull))）。 NULLは、この関数の入力としては決して期待できないものであり、1行または2行の不要なコードが追加されます。関数のドキュメントに次のように記述したいと思います。"この関数への入力がNULLポインターの場合、動作は未定義です。"。
• @CacahueteFrito元のコードで実行されたので、互換性を追求したいと思います（OPがどのように使用していたかは誰にもわかりません）。
• div id = “1078bb5adc”>