-1

我是编程新手。这是到目前为止我编写的代码。忽略加密本身的细节；我知道这将需要更多的工作。当我尝试运行该程序时，我收到一条分段错误错误消息。如果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;
      }  
}
4

1 回答 1

0

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);
/* ... */
于 2014-01-23T23:51:15.253 回答