代码之家 › 专栏 › 技术社区 › Drin Kurtishi

cs50在线课程中凯撒问题集的问题

caesar-cipher cs50 command-line-arguments segmentation-fault c

Drin Kurtishi · 技术社区 · 2 年前

我正处在这样一个阶段,我们需要正确地设置命令行参数,使其只接受数字,并在用户键入数字以外的任何内容时中断代码。(留空、输入两位数、输入文本、输入两段文本等)

因此,如果用户键入多个命令行参数或一个不是数字的命令行参数,我使程序正常工作。我的问题是,如果我将命令行参数留空,程序会给我一个分段错误,如果我键入像“1d”这样的单个命令行,它将无法正常工作。谁能告诉我哪里出了问题?我卡住了。以下是代码:

#include <cs50.h>
#include <stdio.h>
#include <ctype.h>
#include <string.h>    
    
bool only_digits(string argv[]);
int main(int argc, string argv[])
{
    bool digits = only_digits(argv);
    if(argc != 2 || digits == false)
    {
        printf("Usage: ./caesar key\n");
        return 1;
    }
    else
    {
        return 0;
    }
}
bool only_digits(string argv[])
{
    bool digit = false;
    for(int i = 0; i < strlen(argv[1]); i++)
    {
        if(isdigit(argv[1][i]))
        {
            digit = true;
        }
    }
    return digit;
}

1 回复 | 直到 2 年前

Barmar 2 年前

你有两个问题:

你在打电话吗 only_digits() 在检查是否 argv[1] 存在。因此,如果没有命令行参数,就会出现segfault。打电话解决这个问题 只有_位() 之后 argc 检查
中的条件 只有_位() 这是倒退。它回来了 true 如果任何字符是数字,但它应该返回 符合事实的 只有 全部的 字符是数字。

#include <cs50.h>
#include <stdio.h>
#include <ctype.h>
#include <string.h>
    
bool only_digits(string argv[]);

int main(int argc, string argv[])
{
    if(!(argc == 2 && only_digits(argv)))
    {
        printf("Usage: ./caesar key\n");
        return 1;
    }
    return 0;
}

bool only_digits(string argv[])
{
    for(int i = 0; i < strlen(argv[1]); i++)
    {
        if(!isdigit(argv[1][i]))
        {
            return false;
        }
    }
    return true;
}

推荐文章

Kjell Rilbe · 设置调试命令行参数的更简单方法?

7 年前

Sam Bean · Scala参数解析

7 年前

Farhan Ansari · 如何在命令行上用python传递一个参数,以在.txt上执行操作并生成输出

7 年前

John S. · jq:基于参数的切片阵列

7 年前

matif · 如何更改NotifyIcon。通过命令行参数运行WinForm中的文本?

7 年前

David Roberts · 如何让终端更新脚本中的变量?

7 年前

blazerix · Python 2.7 Argparse可选参数和必需参数

8 年前

abhinav singh · 无法分析Python中的命令行参数

9 年前

user1005246 · 带有参数的PHP脚本在非交互式SSH会话上运行不起作用

9 年前

Rardo · 查找并运行程序。exe并将“/repair”添加到行。(批处理文件)

9 年前