Question

我目前正在尝试将整数字符串（例如＆＃34; 509＆＃34;作为类型char）转换为C中的int。但是，一旦我添加了检查的代码部分以查看值应该是负数我得到分段错误。我尝试做一些研究，发现它因为使用指针错误或访问内存而我没有获得许可。但我似乎无法弄清楚我哪里出错了。这是我的第一个C课，所以我是新手，任何帮助都会非常感激。谢谢！

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

int toInteger(char *string){
    int length = strlen(string); 
    int value = 0; 
    if(strcmp(string[0], "-") == 0){
        for(int i = 1; i < length; i ++){
            if((string[i] - '0') < 0 || (string[i] - '0') > 9){
                printf("string must be entirely numeric values.\n"); 
            }
            else{
                value = value * 10 + (string[i] - '0');
            }
        }
        value = value * -1; 
    }
    else{

        for(int i = 0; i < length; i ++){
            if((string[i] - '0') < 0 || (string[i] - '0') > 9){
                printf("string must be entirely numeric values.!\n"); 
            }else{
            value = value * 10 + (string[i] - '0');
            }
        }
    }
    return value;   
}

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

int x = argc; 
char *variable = argv[1]; 
char *function = argv[2];

if(strcmp(function,"1") == 0){
        int asInteger = toInteger(variable);
        printf("%d\n",asInteger);
    }
else {
    printf("incorrect function number"); 
}
return 0; 
}

当函数只是这个

时代码工作

int toInteger(char *string){
int length = strlen(string); 
int value = 0; 

    for(int i = 0; i < length; i ++){
        if((string[i] - '0') < 0 || (string[i] - '0') > 9){
            printf("string must be entirely numeric values.!\n"); 
        }else{
        value = value * 10 + (string[i] - '0');
        }
    }
    return value;   
}

但是一旦我添加了另一个循环以检查否定符号，它就开始给我分段错误：11

Answer 1

Matt McNabb已暗示

if(strcmp(string[0], "-") == 0){

错了。 strcmp需要两个字符串，你给它一个字符串和一个字符串。做

if(string[0] == '-')){

是的，永远不要忽视警告，编译器正试图帮助你。虽然它没有帮助是有争议的，但如果它已经变胖了，你会尝试修复它

修复分段错误：11

1 个答案: