-3

我已经搜索和阅读了几天试图解决这个问题。

我只是想用我自己的函数来反转 C 中的一个字符串，但我现在很难过，两天后都无法前进！

这是我的代码：

/*
This program takes a string
as input and returns it to
the user but in reverse order.
*/

#include "stdio.h"
#include "stdlib.h"
#define MAXLINE 1000

/* Take in a string and return its length. */
int getline(char s[]); 

/* Copy From to To. */
void copy(char To[], char From[]); 

/* Take the contents of a string and reverse them. */
char * reverse(char s[]);

int main() {

        char mainString[MAXLINE + 1];
        copy(mainString, "Test string.");

    printf("Your string before reversal: %s\n", mainString);
    reverse(mainString);
    printf("Your string after reversal: %s\n", mainString);

    return 0;
}


int getline(char s[]) {
    int i;
    for(i = 0; s[i] != '\0'; i++);
    i++;

    return i;
}

void copy (char To[], char From[]) {
    int i;
    for(i = 0; From[i] != '\0'; i++) {
        To[i] = From[i]; 
    }; 
    To[i] = '\0';
}


char * reverse(char s[]) {

    int string_length;
    string_length = getline(s);

    char myString[MAXLINE];
    int a = 0;

    int i;
    i = string_length;

    while (i >= 0) {
        s[i] = myString[a];
        a++;
        i--;
    };
    copy(s, myString);

    return s;
}
4

3 回答 3

2

删除 getline 函数末尾的 i++ 。这导致返回的字符串长度比应有的长度多 1，这反过来又导致 '\0' 成为反转字符串的第一个字符，这不应该发生。

此外，线 

s[i] = myString[a];

是错的。这会更改 s，但您想更改 myString：

myString[i] = s[a];

（或者，您可以在 while 循环之前执行 copy(myString, s); 以填充 myString，然后将数据从那里反向复制回 s。）

于 2013-06-06T13:02:55.300 回答
1

修复到

int getline(char s[]) {
    int i;
    for(i = 0; s[i] != '\0'; i++);
    return i;
}

char * reverse(char s[]) {
    int string_length;
    string_length = getline(s);
    char myString[MAXLINE];
    int i, a =0;
    i = string_length-1;
    while (i >= 0) {
        myString[a]=s[i];
        a++;
        i--;
    }
    myString[a] ='\0';
    copy(s, myString);

    return s;
}
于 2013-06-06T13:05:23.813 回答
0

reverse函数期望将传入参数的字符串s反转，myString是带有垃圾字符的本地字符串

所以你不能做一个

s[i] = myString[a]

您只需松开 s 中的所有字符

于 2013-06-06T13:06:36.083 回答