在 C 语言中处理有关 strncmp 的一些代码时遇到一些困难

Question

在 C 中关于 strncmp 的一些代码遇到一些困难，只是想知道是否有人遇到同样的问题，我的

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

int endsWith(char* longStr, char* shortStr);

int main()
{
    char* longString = "Grant is The man";
    char* shortString = "man";
    endsWith(longString,shortString);
    printf("%s\n",shortString);
}

int endsWith(char longStr, char shortStr)
{
    int offset;
    for (offset = 0 ; offset < strlen(shortStr) - strlen(longStr) ; offset ++)
        if (strncmp( longStr , shortStr + offset , strlen(longStr)) == 0)
            return 1;

    return -1;
}

返回应该是 man，如果我插入的是不应该返回任何内容或 0。

Answer 1

您有几个问题：

• 不需要循环。只需获取偏移量并从那里比较一次即可。
• 您正在向后计算偏移量。您需要从长长度中减去短长度。
• 无需使用 strncmp()，strcmp() 即可工作。
• 测试函数应返回 1 或 0，而不是 1 或 -1。
• 您永远不会检查函数的结果。

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

int endsWith(char* longStr, char* shortStr);

int main()
{
    char* longString = "Grant is The man";
    char* shortString = "man";
    if (endsWith(longString,shortString)) {
        printf("%s\n",shortString);
    }
}

int endsWith(char longStr, char shortStr)
{
    int offset = strlen(longStr) - strlen(shortStr);
    if (offset < 0) { // shortStr is longer than longStr
        return 0;
    }
    return strcmp(shortStr, longStr + offset) == 0;
}

Answer 2

对于初学者来说，函数定义中函数参数的类型与

int endsWith(char longStr, char shortStr)

先前函数声明中的类型不对应。

至少你需要在函数定义中编写

int endsWith(char *longStr, char *shortStr) 

但是这样声明函数会更正确，

int endsWith( const char *longStr, const char *shortStr );

因为传递的字符串在函数内不会改变。

您使用函数 strnmp 的方法没有多大意义。

而且这个调用

strncmp( longStr , shortStr + offset , strlen(longStr))

是不正确的。例如，为什么使用表达式 shortStr + offset？

如果函数的目的是确定第一个字符串是否以第二个字符串结尾，则可以按以下方式定义该函数

int endsWith( const char *longStr, const char *shortStr );
{
    size_t n1 = strlen( longStr );
    size_t n2 = strlen( shortStr );

    int success = -1;

    if ( !( n1 < n2 ) )
    {
        if ( strcmp( longStr + n1 - n2, shortStr ) == 0 ) success = 1;
    }

    return success;
}

或者如果您想在成功时返回 1，否则返回 0，则函数定义可以如下所示

int endsWith( const char *longStr, const char *shortStr );
{
    size_t n1 = strlen( longStr );
    size_t n2 = strlen( shortStr );

    return !( n1 < n2 ) && strcmp( longStr + n1 - n2, shortStr ) == 0;
}

Answer 3

您的代码存在许多问题。

• 您的 endsWith() 声明与其定义不匹配。

• main() 忽略 endsWith()

  的返回值

• 根本不需要 endsWith() 中的循环。您的方法是实现 indexOf() 函数而不是 endsWith() 函数。您可以通过一次减法计算所需的偏移量，无需循环。

  但是，即使需要循环，您也无法正确计算循环条件。您正在从较短的值中减去较大的值，这将导致负值。但由于 strlen() 返回一个无符号类型，负数将转换为非常大的正值，因此最终循环次数超出了您的需要。

• 您向 strncmp() 传递了错误的参数。由于您想要比较 longStr 的末尾以查看它是否与 shortStr 匹配，因此需要传递 longStr + offset 而不是 >shortStr + offset，并传递 strlen(shortStr) 而不是 strlen(longStr)。

• 如果未找到匹配项，

  endsWith() 应返回 0，而不是 -1。

话虽如此，尝试更像这样的事情：

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

int endsWith(const char* longStr, const char* shortStr);

int main()
{
    const char* longString = "Grant is The man";
    const char* shortString = "man";
    if (endsWith(longString, shortString))
        printf("Ends with %s\n", shortString);
    else
        printf("Does not end with %s\n", shortString);
}

int endsWith(const char* longStr, const char* shortStr)
{
    size_t longLen = strlen(longStr);
    size_t shortLen = strlen(shortStr);
    return (
        (longLen >= shortLen) &&
        (strncmp(longStr + (longLen-shortLen), shortStr, shortLen) == 0)
        ) ? 1 : 0;
}