類指針單元化局部變量

Question

最近在我的C++類中，我們已經瞭解了指針和類。

我正在嘗試製作一個有class Student的程序，我們將指出每個學生的姓名和考試分數。

輸入姓名和考試分數後，他們被排序，然後按照從高到低的順序排列。

我相信我所有的語法都是正確的，但是我仍然在學習。我遇到的問題是，我第一次使用我的類，我得到一個未初始化的局部變量錯誤，有關如何解決此問題的任何幫助？由於存取什麼都不做，

struct Student { 
    double score; 
    std::string name; 
}; 

：

#include "stdafx.h" 
#include <iostream> 
#include <string> 
#include <array> 
using namespace std; 

class Student { 
private: 
    double score; 
    string name; 
public: 
    void setScore(double a) { 
     score = a; 
    } 
    double getScore() { 
     return score; 
    } 
    void setName(string b) { 
     name = b; 
    } 
    string getName() { 
     return name; 
    } 
}; 

void sorting(Student*, int); 

int main() 
{ 
    Student *students; 
    string name; 
    int score; 
    int *count; 
    count = new int; 
    cout << "How many students? "; 
    cin >> *count; 
    while (*count <= 0) { 
     cout << "ERROR: The number of students must be greater than 0.\n"; 
     cin >> *count; 
    } 
    for (int i = 0; i < *count; i++) { 
     cout << "Please enter the students name: "; 
     cin >> name; 
     students[i].setName(name); 
     cout << "Please enter " << students[i].getName() << "'s score: "; 
     cin >> score; 
     while (score < 0) { 
      cout << "ERROR: Score must be a positive number.\n"; 
      cin >> score; 
     } 
     students[i].setScore(score); 
    } 
    sorting(students, *count); 
    for (int i = 0; i < *count; i++) { 
     cout << students[i].getName() << ": " << students[i].getScore() << endl; 
    } 
    system("PAUSE"); 
    return 0; 
} 

void sorting(Student *s, int size) { 
    for (int i = 0; i < size; i++) { 
     for (int j = i; j < size; j++) { 
      if (s[j].getScore() > s[(j + 1)].getScore()) { 
       int tmp = s[(j + 1)].getScore(); 
       s[(j + 1)].setScore(s[j].getScore()); 
       s[j].setScore(tmp); 
       string tmp1 = s[(j + 1)].getName(); 
       s[(j + 1)].setName(s[j].getName()); 
       s[j].setName(tmp1); 
      } 
     } 
    } 
} 

Answer 1

首先，你Student類可以簡化爲這一點。我還添加了std::前綴，因爲using namespace std is considered a bad practice。

現在，而不是使用指針來保存學生，包括vector和使用：

std::cout << "How many students? "; 
int count; 
std::cin >> count; 

std::vector<Student> students(count); 

加載程序也可以簡化鑑於缺乏accesors的：

for (auto& student : students) { 
    std::cout << "Please enter the students name: "; 
    std::cin >> student.name; 
    std::cout << "Please enter " << student.name << "'s score: "; 
    std::cin >> student.score; 
    while (score < 0) { 
     std::cout << "ERROR: Score must be a positive number.\n"; 
     std::cin >> student.score; 
    } 
} 

而且實際上，一旦你有這個，你可以把它放在istream& operator>>(istream&, Student&)，並將其減少到：

std::copy_n(std::istream_iterator<Student>(std::cin), students.size(), students.begin()); 

現在不再需要臨時變量了（即使您想使用它們，也應該在使用之前定義，因此在循環內部）。

最後一件事是你的排序例程。首先，有std::sort，您可以使用如果您只是提供了一個比較：

std::sort(
    begin(students), 
    end(students), 
    [](Student const& a, Student const& b) { return b.score < a.score; } 
); 

如果你堅持自己寫的排序例程，至少使用std::swap。