在 C 中使用 if/else 语句的命令行参数

Question

关闭。这个问题需要debugging details .它目前不接受答案。












想改进这个问题？将问题更新为 on-topic对于堆栈溢出。

6年前关闭。




Improve this question




我是编程新手，这让我感到困惑。

我正在编写一个由 main 调用的函数它接受命令行参数并将它们存储在 struct 中以后使用。此特定示例用于图像编辑，但可以在任何地方使用。

所需性能:函数从命令行获取参数。识别并检查三个特定参数:-h , -o和 -t .如果存在，它们将更改 struct值(value)观。论据 -o和 -t将紧随其后的参数存储到它们各自的struct中字段。任何不是 -h 的参数或没有 -o或 -t前面假定为输入文件名并存储在 flag->inputFile .如果考虑了所有参数，则 flag->inputFile应该保留 NULL并且可以在主函数中进行测试，如果这是真的，程序终止。

问题:当没有指定输入文件时(使用上述参数)flag->inputFile不断设置为 -o当它作为参数包含时。

解决方案:感谢 Scott，这个问题已经通过替换几个 if 得到了回答。带有 else if 的语句现在似乎已经解决了这个问题，并且该功能似乎可以按预期工作。
我对发生的事情的理解是 else语句在 i 的每次迭代中运行除非 -t包含参数，因为它是紧接在 else 之前的语句。

我使用的编译器是 gcc，这是我的代码。 (我知道我的结构没有打包，我仍在努力解决这个问题，但看不到它会如何导致我所看到的。段错误，是的，但不是这个？)

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

struct PROMPTFLAGS {
    int help;    // Change from NULL if -h argument present
    char * outputFile;    // Store argument after -o as the output file
    char * inputFile;    // Argument not paired with a flag stored here as input file
    float threshold;    // Stores a value to use in image manipulation in main
};

struct PROMPTFLAGS * parsargs(int argc, char * argv[]) {
    struct PROMPTFLAGS* flag = malloc(sizeof(struct PROMPTFLAGS));
    int i;
    printf("argc: %d\n",argc);
    for (i = 1; i < argc; i++) {
        char * arg = argv[i];
        int str_aft = i+1;    // Allows the next string to be used in this iteration of the loop
        if (strcmp(arg,"-h") == 0) {
            flag->help = 1;
        }
        if (strcmp(arg,"-o") == 0) {    // Changing this to 'else if' seems to be a fix
            flag->outputFile = argv[str_aft];
            i++;        // Skips over next arg since already read by argv[str_aft]
        }
        if (strcmp(arg,"-t") == 0) {    // Changing this to 'else if' seems to be a fix
            flag->threshold = strtod(argv[str_aft],NULL);
            i++;        // Skips over next arg since already read by argv[str_aft]
        }
        else {
            flag->inputFile = arg;
        }
    }
    return flag;
}

int main(int argc, char* argv[]) {

    struct PROMPTFLAGS * flags;
    flags = parsargs(argc, argv);
    printf("Help = %d\n",flags.help);
    printf("Output = %s\n",flags.outputFile);
    printf("Input = %s\n",flags.inputFile);
    printf("Threshold = %s\n",flags.threshold);

return 0;
}

我为这个问题的第一个版本的糟糕格式道歉，并希望这个编辑更好。我已经使函数的预期结果和遇到的问题更加清晰，并删除了我通过代码获得的大部分测试打印并添加了一些注释。我还包括了我的问题的解决方案(由另一个用户提供)以及我对损坏代码中发生的事情的理解。

如果人们仍然认为这是一个糟糕的问题或对其他人没有用，那么我很乐意将其删除，但已对其进行编辑并保留它，希望它可以帮助其他人。

这是我关于堆栈溢出的第一篇文章，我感谢大家在我学习编码和发布问题的最佳方式时的帮助和耐心。

Answer 1

你设置flag->inputFile = arg每当arg不是 "-t" (测试 else 后的 "-t" )。我不确定您何时要分配给该字段，但我确信这不是正确的逻辑。例如，如果您想在 arg 时执行此操作。不是您要查找的其他特定标志，您应该使用 if ... else if ... else if ... else .