Question

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


int main(int argc, char** argv)
      {
      string p;
      int c;  
      if (argc >= 2)
      {
          p = get_string("plaintext: ");
      }
      if (argc != 2 || isalpha(argv[1][0]) == false)
      {
          printf("ERROR\n");
          return 1;
      }
      printf("ciphertext: ");
      string k = argv[1];
      int m = strlen(k);
      for(int i = 0,j = 0, n = strlen(p); i < n; i++)
      {
        c = p[i] + k[j % m];
        if(isupper(k[j % m]))
        {
            k[j % m] = k[j % m] - 'A';
        }
        if(islower(k[j % m]))
        {
            k[j % m] = k[j % m] - 'a';
        }
        if(isalpha(p[i]) == false)
        {
            printf("%c", p[i]);
        }
        if(islower(p[i]) && islower(c))
        {
            printf("%c", c);
            j++;
        }
        if(isupper(p[i]) && isupper(c))
        {
            printf("%c",c);
            j++;
        }
        if(isupper(p[i]) && !isupper(c))
        {
            c = ((c - 'A') % 26) + 'A';
            printf("%c", c);
            j++;
        }
        if(islower(p[i]) && !islower(c))
        {
            c = ((c - 'A') % 26) + 'A';
            printf("%c",c);
            j++;
        }
        else(printf("%c",c));
        j++;
        }   
        printf("\n");
      }

我无法弄清楚我的代码有什么问题。当我尝试运行它时，我能够让它提示我输入明文，但如果我试图继续下去，我会遇到分段错误。我相信它对于我的for循环开始有什么意义吗？我不知道我是否正确地声明了变量c。除此之外我真的没有线索。

[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
plaintext: hello
ciphertext: 
Breakpoint 1, main (argc=2, argv=0x7fffffffdf18) at vigenere.c:24
24            for(int i = 0,j = 0, n = strlen(p); i < n; i++)
(gdb) 
(gdb) n
26                  if(isupper(k[j % m]))
(gdb) 
30                  if(islower(k[j % m]))
(gdb) 
32                      k[j % m] = k[j % m] - 'a';
(gdb) 
35                  c = p[i] + k[j % m];
(gdb) 
37                  if(isalpha(p[i]) == false)
(gdb) 
41                  if(islower(p[i]) && islower(c))
(gdb) 
43                      printf("%c", c);
(gdb) 
h44                     j++;
(gdb) 
46                  if(isupper(p[i]) && isupper(c))
(gdb) 
51                  if(isupper(p[i]) && isupper(c) == false)
(gdb) 
57                  if(islower(p[i]) && islower(c) == false)
(gdb) 
65                      (printf("%c",c));
(gdb) 
h66                     j++;
(gdb) 
68              }   
(gdb) 
24            for(int i = 0,j = 0, n = strlen(p); i < n; i++)
(gdb) 
26                  if(isupper(k[j % m]))
(gdb) 
30                  if(islower(k[j % m]))
(gdb) 
32                      k[j % m] = k[j % m] - 'a';
(gdb) 
35                  c = p[i] + k[j % m];
(gdb) 
37                  if(isalpha(p[i]) == false)
(gdb) 
41                  if(islower(p[i]) && islower(c))
(gdb) 
43                      printf("%c", c);
(gdb) 
g44                     j++;
(gdb) 
46                  if(isupper(p[i]) && isupper(c))
(gdb) 
51                  if(isupper(p[i]) && isupper(c) == false)

任何人都可以解释为什么h打印两次，即使if语句是假的？我能够解决问题，但我仍然在努力弄清楚为什么c会为每次迭代输出多次。

Answer 1

您可能会在此处获得针对您的特定问题的答案，但是如果您想解决它并学到一些知识（这是cs50的重点），请执行以下步骤：

尝试首先解决所有问题，如果仍然无法解决，请继续执行下一步
观看Zamyla's walkthrough，一次尝试一次，
如果您遇到的问题似乎太困难了，请离开计算机，拿起笔和一张纸，分析程序应该执行的操作以及它实际执行的操作，
学习使用调试器（在cloud9 cs50 IDE中，它在屏幕的右侧），
如果您的问题看起来真的很真的，请务必关闭计算机，然后用笔和一张纸上床睡觉，您会惊讶地发现躺下解决问题有多么容易：）

应用这些步骤并解决问题时，您将获得一些解决类似问题的技能，并且会感到很聪明：）

如果您是编程的完整入门者，那么前面会遇到一些非常棘手的问题（破解，拼写...），但是如果您执行这些步骤，则可以解决所有问题。我知道，因为我做到了，而且我不是C语言专家，超级聪明的专家，但我对'speller'的解决方案现在排名前10位。
要有耐心，祝你好运：）

Vigenere分段错误

1 个答案: