为什么一个段出现故障而另一个没有？

Question

嗨，我有一个程序需要将字符串数组与预定义的字符串进行比较，但是当我使用变量 args[0] 时，它在我的函数 strcmp 中工作，如下所示

int gash_execute(char **args){
    int i;

    if(args[0] == NULL){
        return 1;
    }

    for(i = 0; i < gash_command_num(); i++){
        if(strcmp(args[0], functions[i]) == 0){
            return(*function_address[i])(args);     
        } 
    }
    return gash_launch(args);
}

但是，当尝试 strcmp args[i] 如下所示时，我遇到了段错误。 谁能帮我找到解决这个问题的方法？

int gash_execute(char **args){
    int i;

    if(args[0] == NULL){
        return 1;
    }

    for(i = 0; i < gash_command_num(); i++){
        if(strcmp(args[i], functions[i]) == 0){
            return(*function_address[i])(args);     
        } 
    }
    return gash_launch(args);
}

args[] 是一个以前用空格分隔的字符串数组，这个程序用于自定义 shell，所以假设在我的 shell 命令行中我输入“cat echo ls” args[0] 将是“cat”等等。 但是现在我需要实现 i/o 重定向。 所以我需要检查 args 的每个元素来检查它们是否代表符号 "<" ">" "|" 如果其中之一是我们可以从那里拿走

Answer 1

没有看到所有代码，或来自valgrind等工具的报告，我不能肯定地说。 但是我可以告诉你，这个循环充满了潜在的问题。

for(i = 0; i < gash_command_num(); i++){
    if(strcmp(args[i], functions[i]) == 0){
        return(*function_address[i])(args);     
    } 
}

它基于某个函数调用迭代三个数组（ args 、 functions和function_address ），这些函数调用不将这些作为变量（ gash_command_num() ），这些变量与这些数组中实际有多少元素具有未知关系。

它使用了两个可以包含任何内容的全局变量（ functions和function_addresses ）。

如果所有这些事情都是相关的，我会建议明确说明......但我怀疑它们不是。 我怀疑你的循环逻辑是错误的。 它正在比较args[i]和functions[i] 。 我怀疑gash_command_num()实际上是functions的大小，因此循环正在脱离args 。

我怀疑你真正想要做的是看是否args[0]在匹配任何函数名functions ，然后调用相关功能。 如果args[0]是ls那么您要检查是否有ls的内置 shell 函数并使用所有参数调用它。

与其一遍又一遍地搜索列表，并且必须管理两个并行列表，不如使用哈希表来提供更好的服务。 键是函数名，值是函数指针。 C 没有内置的哈希表，但有很多库。 Gnome Lib是 C 缺少的许多其他基本功能的可靠选择。

使用哈希表并消除全局变量，您的代码简化为：

/* For encapsulation */
typedef int(*shell_function_t)(char **);

int gash_execute(char **args, GHashTable *shell_functions){
    int i;

    if(args[0] == NULL){
        return 1;
    }

    shell_function_t func = g_hash_table_lookup(shell_functions, args[0]);
    if( func ) {
        return(*func)(args);
    }
    else {
        return gash_launch(args);
    }
}

扫描管道现在是一个单独的问题。 为此，您确实希望遍历args查找特殊字符。 如果在创建args时确保它以空指针结尾，这args得更简单。

for(int i = 0; args[i] != NULL; i++) {
    ...check if args[i] is special...
}