Question

#include <iostream>
using namespace std;

int main(){
    int findMax(int *);

    const int MAX = 100;
    int values[MAX];
    char ivals[256];
    // Get the space-separated values from user input.
    cin.getline(ivals, 256, '0');
    char *helper;
    // Clean input array and transfer it to values.
    for(int i = 0; i < (MAX) && ivals[i] != 0; i++){
        helper = ivals[i * 2];
            values[i] = atoi(helper);

    }

    int mval = findMax(values);
    cout << values << endl << mval;
    return 0;
}
//Function to find the maximum value in the array
int findMax(int arr[]){
    int localmax = 0;
    for(int i = 0; i < (sizeof(arr)/sizeof(int)); i++){
        if(arr[i] > localmax){
            localmax = arr[i];
        }
    }
    return localmax;
}

该程序的目的是让用户输入以空格分隔的一系列值，以0结尾。然后分析该数组以找到最大值。我想出了如何将原来的char []转换为int []，以便我可以在其上使用findMax（）函数而不会出现错误，但排序循环似乎有自己的问题，当“cout＆lt; ＆lt; values＆lt;＆lt;＆lt; endl＆lt;＆lt; mval;“如果被调用，它只返回一个内存地址，而不是一个非间隔的整数序列。谁能解释我做错了什么？似乎我可能在使用指针时犯了一些错误，但我无法弄清楚是什么。

Answer 1

打印values不会按预期打印数组内容，它会打印数组第一个元素的内存位置。

尝试这样的事情：

#include <iterator>
#include <algorithm>

// ...

copy(&values[0], &values[MAX], ostream_iterator(cout, " "));

很抱歉，我无法发布实际工作代码，但您的原始帖子很多，语法和句法错误很多。

编辑：为了更完整，更平易近人的利益。初学者可以理解，我写了一个小程序，说明了实现这一目标的4种方法。

方法1使用copy和ostream_iterator，如上所述。下面的方法2可能是最基本的＆amp;最容易理解。方法3是C ++ 0x方法。我知道问题是标记为C ++，但我认为添加它可能是有教育意义的。方法4是使用vector和for_each的C ++方法。我已经实现了一个执行转储的仿函数。

分享＆amp;享受

#include <iostream>
#include <iterator>
#include <algorithm>
#include <functional>
#include <vector>
using namespace std;

struct dump_val : public unary_function<int,void>
{
    void operator()(int val)
    {
        cout << val << " ";
    }
};

int main(){
    int vals[5] = {1,2,3,4,5};


    // version 1, using std::copy and ostream_iterator
    copy(&vals[0], &vals[5], ostream_iterator<int>(cout, " "));
    cout << endl;

    // version 2, using a simple hand-written loop
    for( size_t i = 0; i < 5; ++i )
        cout << vals[i] << " ";
    cout << endl;

    // version 3, using C++0x lambdas
    for_each(&vals[0], &vals[5], [](int val) 
    {
        cout << val << " ";
    }
    );
    cout << endl;

    // version 4, with elements in a vector and calling a functor from for_each
    vector<int> vals_vec;
    vals_vec.push_back(1);
    vals_vec.push_back(2);
    vals_vec.push_back(3);
    vals_vec.push_back(4);
    vals_vec.push_back(5);
    for_each( vals_vec.begin(), vals_vec.end(), dump_val() );
    cout << endl;

}

Answer 2

当你传递一个X数组时，它实际上是一个指向你正在传递的X数组的指针。所以当你将values传递给cout时，它只有指针打印出来。

你真的应该考虑使用一些标准算法来简化你的生活。

例如，要打印数组中的所有元素，您只需编写

即可

std::copy(values, values+MAX, std::ostream_iterator<int>(std::cout, "\n"));

找到你可以写的最大元素

int mval = *std::max_element(values, values+MAX);

所以你的代码变成了

#include <iostream>
using namespace std;

int main(){

    const int MAX = 100;
    int values[MAX];
    char ivals[256];
    // Get the space-separated values from user input.
    cin.getline(ivals, 256, '0');
    char *helper;
    // Clean input array and transfer it to values.
    for(int i = 0; i < (MAX) && ivals[i] != 0; i++){
        helper = ivals[i * 2];
            values[i] = atoi(helper);

    }

    copy(values, values+MAX, ostream_iterator<int>(cout, "\n"));
    cout << *std::max_element(values, values+MAX);
    return 0;
}

这样做完全不需要你的findMax方法。

我还会重新编写代码，以便使用向量而不是数组。这使您的代码更短。您可以使用stringstream将字符串转换为数字。

这样的东西应该起作用，并且代码比原始代码少得多。

int main(){


    vector<int> values;
    char ivals[256];

    // Get the space-separated values from user input.
    cin.getline(ivals, 256, '0');

    int temp = 0;
    stringstream ss(ivals);
    //read the next int out of the stream and put it in temp
    while(ss >> temp) {
        //add temp to the vector of ints
        values.push_back(temp);
    }

    copy(values.begin(), values.end(), ostream_iterator<int>(cout, "\n"));
    cout << *std::max_element(values.begin(), values.end());
    return 0;
}

Answer 3

当传递给函数时，int数组被提升为指向int的指针。没有运营商＆lt;＆lt;采用普通阵列。如果你想使用运算符＆lt;＆lt;这样，你需要使用std :: vector。

注意：技术上可以在使用模板传递给函数时区分数组，但这不适用于标准运算符＆lt;＆lt;。

Answer 4

for(int i = 0; i < (sizeof(arr)/sizeof(int)); i++){

sizeof（arr）这里是指向数组的指针的大小。 C ++不会传递实际的数组，这将是非常低效的。你通常只能通过循环一次。像这样声明你的函数：

int findMax(int* arr, size_t elements) {
    //...
}

但是，真的，使用矢量。

哦，坚持，问题。循环遍历数组并打印每个元素。

C ++程序显然打印内存地址而不是数组

4 个答案: