为什么我的程序在调用函数时停止？

Question

我正在编写一个函数来反转给定的单词或输入，但是当我调用它时程序一直停止。

密码是：

#include <stdio.h>
#define MAX 1000

void reverse(char normal[], char reversed[]);

main() {

    int c, i;
    char line[MAX], reversed_line[MAX];

    for (i=0; (c = getchar()) != EOF && c != '\n'; i++) {
        line[i] = c;
        if (c == '\n')
            i++;
    }
    line[i] = '[=10=]';
    copy(line, reversed_line);
    printf("%s", reversed_line);

    return 0;
}


void reverse(char normal[], char reversed[]) {

    int i, len;

    for (i = 0; normal[i] != '[=10=]'; i++)
        len++;
        
    i = 0;
    while ((reversed[len-i] = normal[i]) != '[=10=]')
        i++;
}

程序停止的行是copy(line, reversed_line);，它给出的错误是[Error] ld returned 1 exit status

任何帮助将不胜感激

Answer 1

首先你不执行你的程序只尝试编译和link它。

错误消息表明 linking 失败，因为 linker 没有找到函数 copy，它确实从未在您的代码中定义。标准库中没有 'copy' 函数。如果你想复制字符串或一些内存区域，你应该使用 strcpy、strncpy、memcpy 或 memmove.

但是即使程序 links 你的代码也有很多问题。

这是程序的工作版本：

#include <stdio.h>
#define MAX 5

char *reverse(char *normal, char *reversed);

int main(void) {

    int c;
    size_t i = 0;
    char line[MAX], reversed_line[MAX];

    while((c = getchar()) != EOF)
    {
        line[i] = c;
        if(c != '\n') i++;
        if(i >= MAX - 1) break;
    }
    line[i] = '[=10=]';

    reverse(line, reversed_line);
    printf("%s", reversed_line);

    return 0;
}

char *reverse(char *normal, char *reversed) 
{
    size_t size = 0;

    if(normal && reversed)
    {
        while(normal[size]) size++;
        reversed[size] = 0;
        while(size)
        {
            reversed[size - 1] = *normal++;
            size--;
        }
    }
    return reversed;
}

https://godbolt.org/z/jb7b6hjfT

Answer 2

对于这个 for 循环中的初学者

for (i=0; (c = getchar()) != EOF && c != '\n'; i++) {
    line[i] = c;
    if (c == '\n')
        i++;
}

您没有检查 i 是否大于或等于 MAX。而且这个 if 语句

    if (c == '\n')
        i++;

由于循环的条件，永远不会执行。

您声明了函数 reverse

void reverse(char normal[], char reversed[]);

但是您尝试调用另一个函数而不是函数 copy。

copy(line, reversed_line);

这个循环也在函数中 reverse

while ((reversed[len-i] = normal[i]) != '[=14=]')
        i++;

正试图在参数reversed指向的字符数组的第一个位置写入终止零字符'[=23=]'。

您需要的是以下内容

#include <stdio.h>

#define MAX 1000

char * reverse_copy( const char normal[], char reversed[] );

int main(void) 
{
    char line[MAX], reversed_line[MAX];

    size_t i = 0;
    
    for  ( int c; i + 1 < MAX && ( c = getchar() ) != EOF && c != '\n'; i++ ) 
    {
        line[i] = c;
    }
    
    line[i] = '[=15=]';
    
    puts( reverse_copy( line, reversed_line ) );
    
    return 0;
}

char * reverse_copy( const char normal[], char reversed[] ) 
{
    size_t len = 0;

    while ( normal[len]  != '[=15=]' ) ++len;

    reversed += len;
    
    *reversed = '[=15=]';
    
    for ( ; len != 0; --len )
    {
        *--reversed = *normal++;
    }

    return reversed;
}

如果要输入例如字符串

Hello World!

那么程序输出将是

!dlroW olleH

Answer 3

您在代码中做了一些不必要的事情。例如，当您已经知道长度时，为什么在 reverse 的 while 循环中有这个条件？

while ((reversed[len - i] = normal[i]) != '[=10=]')

我重写了你的整个代码以优化一些东西。也许这是对的有益补充。我在代码中包含了很多解释注释。

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

#define MAX 1000

void reverse(char *str, char *rev_str, size_t len)
{
    /* check if `str` is a valid string */
    if (str == NULL) {
            fprintf(stderr, "Error: invalid string\n");
            exit(EXIT_FAILURE);
    }

    /* take null character into account */
    for (int i = 0; i < len - 1; ++i)
            rev_str[len - i - 2] = str[i];
    rev_str[len - 1] = '[=11=]';
}

int main(void)
{
    char str[MAX];
    fputs("Input: ", stdout);
    /* just use `fgets` and prevent buffer overflow */
    fgets(str, MAX, stdin);

    /* `len` includes null character */
    size_t len = strlen(str);
    /* change '\n' to '[=11=]' */
    str[len - 1] = '[=11=]';

    /* `sizeof(rev_str)` doesn't always need to be `MAX` */
    char rev_str[len];
    reverse(str, rev_str, len);
    printf("Output: %s\n", rev_str);

    return EXIT_SUCCESS;
}

为什么我的程序在调用函数时停止？

Why does my program stop when I call a function?

c

reverse

c-strings

function-definition