如何编写适当的strcmp？

Question

以下是我为strcmp编写的代码，我希望它不区分大写和小写字母，但是仍然如此，如何解决此问题？

int strcmp(char str1[], char str2[]) {
    int i = 0;
    for (; str1[i] || str2[i];) {
        if (str1[i] > str2[i]) {
            int j = (str1[i] - str2[i]);
            if (j == 32)
                i++;
            else {
                return +1;
            }
        } else
        if (str1[i] < str2[i]) {
            int q = (str1[i] - str2[i]);
            if (q == -32) {
                i++;
            } else {
                return -1;
            }
        } else
        if (str1[i] == str2[i]) {
            i++;
        }
    }
    return 0;
}

示例：输入：

Aryan
Semi
Kim
kim
Nap

输出：

Aryan
Kim
Nap
Semi
kim

Answer 1

例如：

int strcasecmp(const char *a, const char *b)
{
    size_t i;
    for (i = 0; tolower((unsigned char)a[i]) == tolower((unsigned char)b[i]) && a[i]; ++i);
    return tolower((unsigned char)a[i]) - tolower((unsigned char)b[i]);
}

Answer 2

您的函数存在多个问题：

• 不要命名为strcmp() 。 您不应使用不同的语义重新定义标准函数。 通常对strcmp()进行高度优化，将strcmp传递给排序函数时，您的版本可能甚至不可用。

• 该算法不正确：相隔32个位置的任意2个字符都视为相等，例如“ 0” ==“ P”。

• 比较是非传递性的：您具有"A" < "_"和"_" < "a"但"A" == "a" ，这对于排序非常有问题。

• 您不应该假设ASCII和硬编码区分大小写。 使用<ctype.h> toupper()并将char值<ctype.h>为(unsigned char)以避免对负值进行未定义的行为。

• i应该是size_t 。

• str1和str2应该是const限定的。

这是一个改进的版本：

#include <ctype.h>

int strcmp_case(const char *str1, const char *str2) {
    for (size_t i = 0;; i++) {
        int c1 = toupper((unsigned char)str1[i]);
        int c2 = toupper((unsigned char)str2[i]);
        if (c1 != c2) {
            return (c1 > c2) - (c1 < c2);
        }
        if (c1 == '\0') {
            return 0;
        }
    }
}