2

我正在做 Zed Shaw 的Learn C The Hard Way课程。在练习 11 中,在额外学分问题 #2 中,他问:

  1. 使用 i-- 从 argc 开始倒数到 0,使这些循环倒数。您可能需要做一些数学运算才能使数组索引正常工作。

  2. 使用 while 循环将值从 argv 复制到状态。

我尝试:

#include <stdio.h>

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

    int i = argc - 1;
    while(i >= 0){
        printf("arg %d: %s\n", i, argv[i]);
        i--;
    }

    char *states[] = {
        "California", "Oregon",
        "Washington", "Texas"
    };

    int num_states = 4;
    i = num_states - 1;
    while( i >= 0){
        printf( "state %d, %s\n", i, states[i]);
        i--;
    }

    i = 0;
    while(i < argc && i < num_states){
        int j = 0;
        while( (states[i][j++] = argv[i][j++]) != '\0' ){
            i++;
        }
        states[i][j] = '\0';
    }

    i = num_states - 1;
    while( i >= 0){
        printf( "state %d, %s\n", i, states[i]);
        i--;
    }

    return 0;
}

我得到一个Segmentation Fault. 我知道您不能在 C 中复制数组,它们是 const 指针或类似的东西(或者我读过)。这就是为什么我尝试逐个字符地复制:

while(i < argc && i < num_states){
    int j = 0;
    while( (states[i][j++] = argv[i][j++]) != '\0' ){
        i++;
    }
    states[i][j] = '\0';
}

然而它不起作用。我该怎么做?当我编译时,编译器给了我这个警告:

$ make ex11
cc -Wall -g    ex11.c   -o ex11
ex11.c: In function ‘main’:
ex11.c:26:28: warning: operation on ‘j’ may be undefined [-Wsequence-point]

我不明白为什么它说这j是未定义的。 valgrind说:

$ valgrind ./ex11 this is a test
==4539== Memcheck, a memory error detector
==4539== Copyright (C) 2002-2012, and GNU GPL'd, by Julian Seward et al.
==4539== Using Valgrind-3.8.0 and LibVEX; rerun with -h for copyright info
==4539== Command: ./ex12 this is a test
==4539== 
arg 4: test
arg 3: a
arg 2: is
arg 1: this
arg 0: ./ex11
state 3, Texas
state 2, Washington
state 1, Oregon
state 0, California
==4539== 
==4539== Process terminating with default action of signal 11 (SIGSEGV)
==4539==  Bad permissions for mapped region at address 0x400720
==4539==    at 0x4005F1: main (ex11.c:26)
==4539== 
==4539== HEAP SUMMARY:
==4539==     in use at exit: 0 bytes in 0 blocks
==4539==   total heap usage: 0 allocs, 0 frees, 0 bytes allocated
==4539== 
==4539== All heap blocks were freed -- no leaks are possible
==4539== 
==4539== For counts of detected and suppressed errors, rerun with: -v
==4539== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 2 from 2)
Segmentation fault

[编辑 1] 为此切换了我的代码

int i = 0;
while( i < argc && i < num_states ){
    states[i] = argv[i];
    i++;
}

它确实有效,但编译器给了我一个警告。另外,我在发布这个问题后意识到我以前的问题:

while( (states[i][j++] = argv[i][j++]) != '\0' ){
    i++;
}

完全是错误的。因为j++每个循环执行两次,并且i++应该在外循环中的该循环之外。而且正如下面评论中提到的,我尝试使用数组数组进行的逐字节复制不起作用,因为我实际上有指针数组。

也就是说,有没有办法在没有编译器警告的情况下做到这一点?

4

2 回答 2

2

一个问题(导致 SEGV 的问题)是,虽然states是 的数组char *,但您正在使用一堆字符串文字(它们是const char *)对其进行初始化。因此,您至少应该在该行收到警告。当您尝试写入字符串常量(位于只读内存中)时,这会咬您一口。

现在,如果您真的想一次将字符串从一个字节复制argvstates一个字节(而不仅仅是复制指针——例如states[i] = argv[i]),您需要确保它states[i][j]是可写的。您可以通过创建states一个字符数组数组而不是一个指向字符的指针数组来做到这一点:

char  states[][16] = {
    "California", "Oregon",
    "Washington", "Texas"
};

当然,这些是固定大小的数组,所以你需要担心它们会溢出,但是你已经遇到了 states 数组本身溢出的问题。

于 2012-08-20T03:13:57.440 回答
0

我自己刚刚完成了那个练习,并且在完成之后对复制进行了更多的研究(正如 Zed 建议的那样)。这就是我所做的,没有问题。

    i = 0;
    num_states = NELEMS(states);

    while(i < argc  && i <= num_states) {
            states[i] = argv[i];
            i++;
    }
于 2014-07-07T22:00:06.420 回答