0

Cでは、最初の2文字だけをstrcmpするにはどうすればよいですか？次に、別の文字列と連結しますか？このようなもの：

char s[10];
scanf("%s",s);

/* if i input "cs332" or "cs234", anything start with cs */

if (strcmp("cs",???)==0)
    strcat(s,"by professor");
4

6 に答える 6

4

と機能的には同じですが、チェックする文字数を制限するstrncmp関数を探しています。strcmpしたがって、長さ 2 で比較文字列"cs". しかし、ここには他にもいくつかの問題があります。

まず、バッファが十分に大きくありません。「教授による」というテキストを追加するときに、10 文字のバッファーに収まる文字列はありません。

第 2 に、堅牢なコードは無制限の文字列書式指定子を使用することはありません。これはバッファ オーバーフローの問題を引き起こします。ファミリはフォーマットされた入力用であり、ユーザー入力よりもフォーマットされていないものはほとんどありscanfません:-)scanf

堅牢な入力ソリューションが必要な場合は、以前の回答の 1 つを参照してください。

第三に、文字列を連結するとバッファがオーバーフローする可能性があることを常に想定し、これを防ぐコードを導入する必要があります。以下を追加する必要があります。

  • ユーザーが入力した文字列の現在の長さ。
  • 追加文字列の長さ (「教授による」)。
  • null ターミネータ用にもう 1 つ。

バッファが十分な大きさであることを確認してください。

私が使用する方法は、（たとえば）200バイトのバッファを用意getLine()し、リンクされた回答（この回答を自己完結型にするために以下で再現）から十分に小さいサイズ（たとえば100）で使用することです。 「教授による」追加はバッファをオーバーフローさせません。

関数：

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

#define OK       0
#define NO_INPUT 1
#define TOO_LONG 2
static int getLine (char *prmpt, char *buff, size_t sz) {
    int ch, extra;

    // Get line with buffer overrun protection.
    if (prmpt != NULL) {
        printf ("%s", prmpt);
        fflush (stdout);
    }
    if (fgets (buff, sz, stdin) == NULL)
        return NO_INPUT;

    // If it was too long, there'll be no newline. In that case, we flush
    // to end of line so that excess doesn't affect the next call.
    if (buff[strlen(buff)-1] != '\n') {
        extra = 0;
        while (((ch = getchar()) != '\n') && (ch != EOF))
            extra = 1;
        return (extra == 1) ? TOO_LONG : OK;
    }

    // Otherwise remove newline and give string back to caller.
    buff[strlen(buff)-1] = '\0';
    return OK;
}

テストコード:

// Test program for getLine().

int main (void) {
    int rc;
    char buff[10];

    rc = getLine ("Enter string> ", buff, sizeof(buff));
    if (rc == NO_INPUT) {
        // Extra NL since my system doesn't output that on EOF.
        printf ("\nNo input\n");
        return 1;
    }

    if (rc == TOO_LONG) {
        printf ("Input too long [%s]\n", buff);
        return 1;
    }

    printf ("OK [%s]\n", buff);

    return 0;
}
于 2011-05-14T01:48:32.413 に答える
3

strcmpを呼び出すのではなく、文字を直接比較しないのはなぜですか?

例えば

if(s[0]=='c' && s[1]=='s'){ ... }

于 2011-05-14T01:22:01.600 に答える
2 
if (strncmp("cs",???, 2)==0) strcat(s,"by professor");

使用するstrncmp

于 2011-05-14T01:18:53.217 に答える
2

これを行うにはいくつかの方法があります。

文字列比較:

if ( s[0] == 'c' && s[1] == 's' )

これを少し長いコード (長さ 3/4 文字など) に簡単に拡張することはできないため、単純な方法です。

私はあなたがstrncmp()右を使用する必要がある収集したと思いますか？

文字列連結

strcat を使用しないでください。本当。s長さが(宛先)のサイズよりも大きい 2 つの文字列を連結すると、問題が発生します。snprint()代わりに、次のように使用することを検討してください。

char str[80];
snprintf(str, 80, "%s by professor", s);

strncat()または、ヒースが指摘するように使用できます。

char s[80];
strncat(s, " by professor", 80);
于 2011-05-14T01:33:03.477 に答える
1

strncmpを使用できます。

編集：

strcat(s,"by professor");

// s is an array of 10 characters. You need to make sure s is big enough  
// to hold the string that needs to be concatenated + to have a terminating 
// character '\0'.
于 2011-05-14T01:18:59.097 に答える
-1

はい、前述のとおり、strcmp が推奨される方法です。これは、同じことを行う別の方法です。

#define CS 29539
char s[80];

scanf("%60s", s);
if( *(short *)s == CS )
    if( strlcat(s, " by professor", sizeof(s)) >= sizeof(s) )
        fprintf(stderr, "WARNING: truncation detected: %s", s);
于 2011-05-14T02:01:24.467 に答える