C++程序显然是打印内存地址而不是阵列

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 < <值< < endl < < mval;“被调用时，它只返回一个内存地址，而不是应该是非间隔的整数序列。任何人都可以解释我做错了什么吗？看来我可能使用指针犯了一些错误，但我无法弄清楚什么。

Answer 1

当传递给函数时，int的数组被提升为指向int的指针。采用普通阵列没有运营商< <。如果您想以这种方式使用运算符< <，则需要使用std :: vector。

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

Answer 2

打印values不会打印像您期望的数组中的内容，将打印数组的第一个元素的存储位置。

尝试这样代替：

#include <iterator> 
#include <algorithm> 

// ... 

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

对不起，我不能张贴实际工作的代码，但原来的职位是有许多语法和句法错误一团糟。

编辑：为了更加完整和更平易近人的兴趣，我写了一个小程序，说明了4种方法来实现这一点。

方法1使用copy与ostream_iterator，因为我之前所做的那样。 下面的方法2可能是最基本的&最容易理解。方法3是一个C++ 0x方法。我知道这个问题被标记为C++，但我认为这可能是教育添加此。方法4是使用vector和for_each的C++方法。我已经实现了一个可以进行倾倒的函子。

分享&享受这里

#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 3

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

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

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

但是，真的，使用一个向量。

哦，挂在，问题。遍历数组并打印每个单独的元素。

Answer 4

当您通过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; 
}