在 C 中嵌套 for 和 if 循环

您使用它来检查 text[i] 是否是字母表而不是 isalpha()，因此我假设您将程序的目标环境限制为字母表的字符代码是连续的（如 ASCII）。

        if ((text[i] >= 'a' && text[i] <= 'z') || (text[i] >= 'A' && text[i] <= 'Z'))

在此限制下，您可以简单地减去 'a' 或 'A' 来获取字符的索引，而无需使用循环来查找字符。

这意味着内循环部分

            for (int j = 0; j < 26; j++)
            {
                if (text[i] == alphabet[j])
                {
                    text[i] = cipher[j];
                    break;
                }
                if (text[i] + 32 == alphabet[j])
                {
                    text[i] = cipher[j] - 32;
                    break;
                }
            }

可以写成

            if (text[i] >= 'a' && text[i] <= 'z')
            {
                text[i] = cipher[text[i] - 'a'];
            }
            else
            {
                text[i] = cipher[text[i] - 'A'] - 32;
            }

您可以使用 strchr 函数压缩字符搜索，它的作用与您所做的相同。那里的效率提升不大。

要计算指数，您可以使用直接减法：

// check if character is in the alphabet:
if ((text[i] >= 'a' && text[i] <= 'z') {
    j = text[i]-'a';
    text[i] = cipher[j];
} else if ((text[i] >= 'A' && text[i] <= 'Z') {
    j = text[i]-'A';
    text[i] = cipher[j] + 32;
}

您还可以预先计算一个包含所有 255 个可能的字符值（不包括零）的表，如果字符不能被加密，则将值设置为零，否则设置密码值。

那么：

for (i = 0; text[i]; i++) {
    if (table[text[i]]) {
        text[i] = table[text[i]];
    }
}

当然，如果您需要对长文本进行加密，这是有道理的，否则您在计算表格上的花费比使用表格花费的要多。

此外，由于分支在许多架构上比简单分配更昂贵，请考虑 MikeCAT 的绝妙建议，即“非密码”字符由自身替换，这样 text[i] = table[text[i]] 实际上什么都不做：

for (i = 0; text[i]; i++) {
    text[i] = table[text[i]];
}

这应该会更容易一些

String substitution(String text,String cipher)
{    
    // for each character:
    for (int i = 0,n = strlen(text); i < n; i++)
    {
        // check if character is in the alphabet:
        if (text[i] >= 'a' && text[i] <= 'z') 
        {
            text[i] = cipher[text[i] - 97]; // index of 'a' in alphabet is 97
        }
        else if (text[i] >= 'A' && text[i] <= 'Z'))
        {
            text[i] = cipher[text[i] - 65] - 32; // index of 'A' in alphabet is 65
        }
    }
    return text;
}

如果您要对字符代码顺序和关系（例如大写/小写转换）做出假设，那么 alphabet 数组几乎没有用处。它的唯一目的是将代码与平台定义的字符集排序分离。

在实践中，您的假设对于您可能遇到的任何平台都有效，这可能是一个公平的赌注，但如果您要定义 alphabet，那么您或许应该保持一致并编写其余代码独立于平台字符集。为此，您应该使用 ctype.h 标头。下面的代码做到了这一点，并且还消除了一些其他问题，例如“幻数”。

#include <String.h>
#include <ctype.h>
#include <stdbool.h>

// function for converTing plain text to cipher text
String substitution(String text,String cipher)
{
    static const char alphabet[] = "abcdefghijklmnopqrstuvwxyz";
    static const size_t ALPHABET_LEN = sizeof(alphabet) / sizeof(*alphabet) ;
    
    // for each character:
    size_t text_len = strlen( text ) ;
    for( int i = 0; i < text_len; i++ )
    {
        // check if character is in the alphabet:
        if( isalpha( text[i] ) )
        {
            // find position in alphabet by index,then convert to same index in the cipher String
            for (int j = 0; j < ALPHABET_LEN; j++)
            {
                bool is_upper = isupper( text[i] ) ;
                if( !is_upper && text[i] == alphabet[j])
                {
                    text[i] = cipher[j];
                    break;
                }
                else if( is_upper && tolower(text[i]) == alphabet[j])
                {
                    text[i] = toupper( cipher[j] ) ;
                    break;
                }
            }
        }
    }
    
    return text;
}

这可能是一个见仁见智的问题，但许多编码标准只允许将 break 作为 switch-case 分隔符。用于中止循环的 break 和 conTinue 可能会创建糟糕且混乱的结构和控制流，并且可能使调试更加复杂，因为控制流可以“跳过”断点。避免使用break：

        bool match = false ;
        for (int j = 0; !match && j < ALPHABET_LEN; j++)
        {
            bool is_upper = isupper( text[i] ) ;
            if( !is_upper && text[i] == alphabet[j])
            {
                text[i] = cipher[j];
                match = true ;
            }
            else if( is_upper && tolower(text[i]) == alphabet[j] )
            {
                text[i] = toupper( cipher[j] ) ;
                match = true ;
            }
        }