c++ - C ++でchar * sとstrcmp関数を使用する

Question

2 つの char* 参照があり、どちらが少ないかを把握しようとしています。私が持っているコードは次のとおりです。

bool stringComparison::lessThan(char *s1, char* s2) {
  int result = strcmp(*s1,*s2);
  return result < 0;
};

結果はこれまで0未満ではありません。これを機能させるには、「int result」行をどのように変更する必要がありますか?

ありがとう

score 5 · Accepted Answer

strcmp は文字列の比較に使用されます。*s1 と *s2 は文字です。s1 と s2 (星なし) で strcmp を呼び出す必要があります。

編集：

これがstrcmpのリファレンスです

score 0 · Accepted Answer

あなたが私たちに話したよりも多くの問題、または別の問題があります。

あなたが私たちに示したコードでは、s1とs2パラメータは型char*です。strcmp()関数は型の引数を期待します(char*実際にはconst char*)。

を呼び出す場合、それはタイプの 2 つの引数を指定strcmp(*s1,*s2)して呼び出す試みです。あなたが説明したように、それはあなたに間違った結果を与えることはありません。コンパイルされません。少なくとも、コンパイラは型の不一致について不平を言っているはずです。strcmpchar

関数を宣言するヘッダーに#includefor<string.h>またはがない場合、コンパイラが不一致を認識しない可能性がありますが、宣言されていない関数の呼び出しは C++ では無効です。<cstring>strcmp

他の両方の回答が言うように、電話をかける必要があるのは事実ですが、それはあなたが説明している症状を説明していませんstrcmp(s1, s2)。strcmp(*s1, *s2)への呼び出しを修正してstrcmp()も、それ自体で発生しているすべての問題が修正されるわけではありません。

いくつかの可能な説明を考えることができます：

宣言されていない関数の呼び出しについて文句を言わない古いコンパイラを使用しています。新しいコンパイラを入手するか、コンパイラを言語標準により厳密に準拠させるオプションを指定してコンパイラを呼び出します。
コンパイラが警告を生成しており (診断は致命的である必要はありません)、それらを無視しています。警告を無視しないでください。
何らかの理由で、関数の独自の宣言を追加することにしました。strcmp()おそらくint strcmp(char s1, char s2);. そうしないでください。標準関数の宣言が必要な場合は#include、適切なヘッダーのディレクティブを追加します。

score 0 · Accepted Answer

s1そしてs2文字列です。だからあなたのコード

int result = strcmp(*s1,*s2);

読むべき

int result = strcmp(s1,s2);

のオンラインリファレンスを参照してくださいstrcmp。

score 0 · Accepted Answer

あなたの機能の（おそらく）改善されたバージョンを提案できますか？これにより、への呼び出しが修正されstrcmp、不要なresult変数が削除され、const-correctness が追加されます。

bool stringComparison::lessThan(const char *s1, const char *s2)
{
    return strcmp(s1, s2) < 0;
}

がクラスの場合、関数定義の最後にstringComparison別のクラスを追加します。const

4 に答える 4