Question

我最大的问题是这一行：

for（i = 0; i＆lt; = strlen（enc）; i ++） - ＆gt;

函数decifrar的第7行： 它保持循环，即使用memset来清除内存（它甚至比字符串长度还要大）

注意如果我使用该行中字符串的实际长度，则代码可以正常工作（即用60替换strlen（enc））

void decipher(int k, char *enc){
     char alfa[]="9876543210zyxwvutsrqponmlkjihgfedcbaZYXWVUTSRQPONMLKJIHGFEDCBA9876543210zyxwvutsrqponmlkjihgfedcbaZYXWVUTSRQPONMLKJIHGFEDCBA";
     char *pch;
     int i;
     for(i=0;i<=strlen(enc);i++){
        pch=strchr(alfa, enc[i]);
        if (pch) enc[i] = *(pch + k),enc[i]=tolower(enc[i]);
     }
     printf("%s",enc);
}
int main(){
    int keys[6]={1,4,15,24,12,20},i;
    char *txt="rfgr r hz grkgb fvzcyrf dhr cergraqr fre grfgnqb ab cebtenzn";
    char *txttemp=malloc(sizeof(char)*1024);

    for(i=0;i<6;i++){
        printf("\n\n\t Attempt number: %d\n\n",i+1);
        memset(txttemp,'\0',sizeof(char)*strlen(txt)+30);
        memcpy(txttemp, txt, strlen(txt));
        decipher(keys[i],txttemp);
    }
    return 0;
}

我错过了什么意思？ strlen的用法是错误的吗？

Answer 1

for-loop中的<不是<=。

但作为注释：避免这种模式。 Strlen意味着你计算字符串的长度，但你可能应该已经从某个地方知道它（即当你收到它时：文件长度，返回的字符数等等）。保存并将该值作为长度。这是程序中许多安全漏洞的来源（buffer overflow）。

std::string将此作为一个内置功能，这是我几乎所有时间都推荐使用普通字符*（即如果你可以使用C ++）

For循环不会在strlen上停止

1 个答案: