Question

我是编程新手。这是我到目前为止编写的代码。忽略加密本身的细节;我知道需要更多的工作。当我尝试运行该程序时，我收到分段错误错误消息。如果argc != 2我将收到消息，如果argc == 2它打印出“关键字”，但它显示相同的消息并且没有完成程序，所以我认为错误与引用有关的argv [1]。

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

int main (int argc, string argv[])
{
   int i = 0, n = strlen(argv[1]);
   char KeyWord[i]; 

    //makes sure command line has 2 arguements
    if (2 != argc) 

        printf("argc != 2. Try again\n");
        return 1;


    //stores argv[1] as key
    for (i = 0; i < n; i++)
    {
        KeyWord[i] = argv[1][i];  //malloc
        printf("%c", KeyWord[i]);
    }
   printf("\n");

    if (isalpha(KeyWord))
        return 0;
    else
     {   
        printf("try again");
        return 1;
     }

      int j, length;

     printf("input data: ");
     string message = GetString();

     for (i = 0; i < n; i++)   
     {  
        for (j = 0, length = strlen(message); j < length; j++)
        {
            if (islower(message[j]))
            message[j] = message[j] -97 + KeyWord[i];

            if (isupper(message[j]))
            message[j] = message[j] -65 + KeyWord[i];
        }  
        if (i==n) i = 0;
      }  
}

Answer 1

在确保strlen(argv[1])之前，您无法在n的初始化中计算argc == 2。

此外，char KeyWord[i]错误：由于i为0，因此您没有为任何事情分配空间。这应该至少在编译时产生警告，因为数组大小必须大于0.如果你想要动态分配，你的评论建议，你应该在计算字符串的长度后使用malloc。

代码应为：

int i = 0, n;
char *KeyWord; 

// make sure command line has 2 arguments
if (2 != argc) 
{
    printf("argc != 2. Try again\n");
    return 1;
}
n = strlen(argv[1]);
KeyWord = malloc(n+1);
/* ... */

我的Vigenere加密程序中的分段错误

1 个答案: