使用指针传递数组值

Question

我有两个函数，一个用于文件读取，另一个用于数字的小排序。使用函数read()我正在读取文件的每一行并将每行放入数组中。文件看起来像：

1 
2 
3 

随着功能sort()我想和值只打印数字大于1

什么是错的：我得到了印两个数组，但我的排序阵列仍打印所有值，而不是只有大于1

我的代码：

#include <iostream> 
    #include <fstream> 
    #include <string> 
    #include <cstdlib> 

    using namespace std; 

    class UZD 
    { 
    private: 
    int numLines; 
    int *value; 
    public: 
     UZD(); 
     int * read(); 
     int sort(); 
    }; 
    // ========================================================= 
    UZD::UZD() 
    { 
    } 
    // ========================================================= 
    int * UZD::read(){ 
    ifstream myfile("stu.txt"); 
    int value[20]; 
    string line[20]; 
    int i=0; 
    while(!myfile.eof() && i < 20) { 
     getline(myfile,line[i]); 
     ++i; 
      } 
    numLines = i; 
    for (i=0; i < numLines; ++i) { 
     value[i] = atoi(line[i].c_str()); 
     cout << i << ". " << value[i]<<'\n'; 
      } 
    return value; 
    } 
    // ========================================================= 
    int UZD::sort(){ 
    int *p; 
    p = read(); 
    int i; 
    if(p[i] > 1) { 
     cout << p <<'\n'; 
     } 
    } 
    // ========================================================= 
    int main() { 
    UZD wow; 
    wow.read(); 
    wow.sort(); 
    } 

Answer 1

有在你的代码中的许多问题，其中最明显的是“返回值”在读（）实现方法具d。值是一个本地数组，并且将会超出read（）的范围。此外，设计似乎有缺陷。你正在调用read（）两次，一次从main（）和一次sort（）。 我写了一个工作代码，使用向量。可能这就是你所期望的：

#include <iostream> 
#include <fstream> 
#include <string> 
#include <cstdlib> 
#include <vector> 

using namespace std; 

class UZD 
{ 
     private: 
       int numLines; 
       vector<int> value; 
     public: 
       UZD(); 
       vector<int> & read(); 
       void sort(); 
}; 
// ========================================================= 
UZD::UZD() 
{ 
} 
// ========================================================= 
vector<int> & UZD::read(){ 
     ifstream myfile("stu.txt"); 
     vector<string> line(20); 
     int i=0; 
     while(!myfile.eof() && i < 20) { 
       getline(myfile,line[i]); 
       ++i; 
     } 
     numLines = i; 
     cout << "After reading file: " << endl; 
     for (i=0; i < numLines; ++i) { 
       value.push_back(atoi(line[i].c_str())); 
       cout << i << ". " << value[i]<<'\n'; 
     } 
     return value; 
} 
// ========================================================= 
void UZD::sort(){ 
     cout << "Inside sort()" << endl; 
     for(int i=0; i<value.size(); ++i){ 
       if(value[i] > 1) 
         cout << value[i] << endl; 
     } 
} 
// ========================================================= 
int main() { 
     UZD wow; 
     wow.read(); 
     wow.sort(); 
     return 0; 
} 

为了清晰起见，我保留了变量名。如果你没有得到任何东西，请告诉我。

Answer 2

你的程序中有很多问题;只是提到其中一些： 在sort，你使用变量i未初始化，这是未定义的行为（可能崩溃）;您编写的while(!myfile.eof()...通常被认为是错误的（see this SO answer;不推荐使用atoi，因为如果该参数不代表数字，则不安全;您将返回一个指向局部变量的指针，范围;你声明成员变量，而是通过本地的（例如，值）...

请看下面的代码这也证明了int*使用和vector<int>;希望它可以帮助

class UZD 
{ 
private: 
    int numLines; 
    int *value = nullptr; 
public: 
    ~UZD() { 
     if (value) 
      delete value; 
    }; 
    void read(); 
    void print(); 
}; 

// ========================================================= 
void UZD::read(){ 
    ifstream myfile("stu.txt"); 
    value = new int[20]; 
    int val; 
    numLines = 0; 
    while(numLines < 20 && myfile >> val) { 
     value[numLines] = val; 
     numLines++; 
    } 
} 
// ========================================================= 
void UZD::print(){ 
    for (int i=0; i<numLines; i++) 
     cout << value[i] << endl; 
} 


class UZD_vector 
{ 
private: 
    vector<int> values; 
public: 
    void read(); 
    void print(); 
}; 

// ========================================================= 
void UZD_vector::read(){ 
    ifstream myfile("stu.txt"); 
    int val; 
    while(myfile >> val) { 
     values.push_back(val); 
    } 
} 
// ========================================================= 
void UZD_vector::print(){ 
    for (auto val : values) 
     cout << val << endl; 
} 

// ================================================== ======= int main（）{

cout << "with int[]:" << endl; 
UZD wow; 
wow.read(); 
wow.print(); 

cout << "with vector:" << endl; 
UZD wow_vector; 
wow_vector.read(); 
wow_vector.print(); 

}

Answer 3

这里是你自己的代码纠正，以防万一你发现矢量太难学了（这不应该是真实的，虽然）

#include <iostream> 
#include <fstream> 
#include <string> 
#include <cstdlib> 

using namespace std; 

class UZD 
{ 
     private: 
       int numLines; 
       int *value; 
       int num; 
     public: 
       UZD(); 
       void read(); 
       void sort(); 
}; 
// ========================================================= 
UZD::UZD():num(20) 
{} 
// ========================================================= 
void UZD::read(){ 
     ifstream myfile("stu.txt"); 
     value = new int[num]; 
     string line[num]; 
     int i=0; 
     while(!myfile.eof() && i < num) { 
       getline(myfile,line[i]); 
       ++i; 
     } 
     numLines = i; 
     for (i=0; i < numLines; ++i) { 
       value[i] = atoi(line[i].c_str()); 
       cout << i << ". " << value[i]<<'\n'; 
     } 
} 
// ========================================================= 
void UZD::sort(){ 
     cout << "After sorting: " << endl; 
     for (int i = 0; i < num; ++i) { 
       if(value[i] > 1) 
         cout << value[i] << endl; 
     } 
} 
// ========================================================= 
int main() { 
     UZD wow; 
     wow.read(); 
     wow.sort(); 
     return 0; 
}