Question

这是我的代码：

#include<stdio.h>
#define MAXLINE 100

/*print the reverse of the input*/

int getline1(char line[], int maxline);
char *reverse(char);

main(){
    int len;
    char line[MAXLINE];
    char *rp;
    while ((len = getline1(line, MAXLINE)) > 0)
            rp = reverse(line);
            printf("%s", *rp);
    return 0;
}
int getline1(char s[], int lim){
    int c, i;

    for (i = 0; (c=getchar()) != EOF && c != '\n'; i++)
            if (i > lim-1)
                    continue;
            else
                    s[i] = c;
    if (c == '\n'){
            s[i] = c;
            i++;
    }
    s[i] = '\0';
    return i;
}
char *reverse(char ca[]){
    int i;
    int i1 = 0;
    char *rp;
    char reversed[MAXLINE];
    for (i = MAXLINE-1; i >= 0; i--){
            reversed[i1] = ca[i];
            i1++;
    }
    rp = reversed;
    return rp;
}

但是当我尝试编译它时，我收到以下错误：

reverse.cpp: In function ‘int main()’:
reverse.cpp:14:20: error: invalid conversion from ‘char*’ to ‘char’ [-fpermissive]
reverse.cpp:7:7: error:   initializing argument 1 of ‘char* reverse(char)’ [-fpermissive]
reverse.cpp:15:19: warning: format ‘%s’ expects argument of type ‘char*’, but argument 2 has type ‘int’ [-Wformat]

我对C ++没有多少经验。我究竟做错了什么？我只想制作一个指向char数组的指针并返回它。

Answer 1

我只想制作一个指向char数组的指针并返回它。

您似乎想要返回一个字符串。那不是指向char数组的指针。即使您的程序已编译，您也会在返回指向自动对象的指针时调用UB，并且您的代码中还存在很多其他运行时错误。你很幸运，你也发了一个编译时错误，所以编译器不接受你的程序。这个C ++程序实现了您的目标：

#include <string>
#include <iostream>

std::string reverse(std::string val) {
    return std::string(val.rbegin(), val.rend());
}
int main() {
    std::string str;
    while(std::getline(std::cout, str))
        std::cout << reverse(str);
}

我做错了什么？

你正在学习C ++ 11的C89内容。他们真的不同的东西。

如果您希望学习编写C ++代码，则必须学习std::string以及标准库的其余部分。使用char*，char[]和MAGIC_BUFFER_SIZE

时，您无法访问任何地方。

Answer 2

首先声明函数原型

char *reverse(char);

但实际的功能被声明为

char *reverse(char ca[])

那是你的问题。

Answer 3

你想要达到什么目的？代码中存在逻辑错误......

    while ((len = getline1(line, MAXLINE)) > 0)
        rp = reverse(line);
        printf("%s", *rp);

这个部分会在每个/ n字符上调用reverse，但是printf永远不会被调用... 你也有100个字符串，你的反向将在反向字符串结尾处放置前导字符。所以如果你有5个字符串，你将在前95个位置上有垃圾，然后你需要5个字符......

c ++返回并使用char数组指针

3 个答案: