int/counter에 대한 이상한 동작

Question

일부 노래를 클래스의 벡터에 추가하려고합니다. 저장하고있는 값 중 하나는 노래를 나타내는 int입니다. 근본적으로 카운터입니다. 내가 추가하는 첫 번째 노래에는 값 1, 두 번째 값 2 등이 있어야합니다. 그러나 큰 난수 (긍정과 부정)와 같은 다른 이상한 값을 얻고 있습니다. 나는 내가하고있는 일을 머리 속에 감쌀 수 없다.

확인

#include <iostream> 
#include <vector> 
#include <string> 

using namespace std; 

class Jukebox{ 
public: 
    void addSong(string artist, string title, string filename) { 
    song s {++songCounter, artist, title, filename}; 
    Songs.push_back(s); 
    } 

    void printSong (int song) { 
    cout << Songs[song].no << ". "; 
    cout << Songs[song].artist << " - "; 
    cout << Songs[song].title << " : "; 
    cout << Songs[song].filename << endl; 
    } 

private: 
    struct song { 
    int no; 
    string artist; 
    string title; 
    string filename; 
    }; 
    vector<song> Songs; 
    int songCounter; 
}; 

int main() { 
    Jukebox jbox; 
    jbox.addSong("U2", "Magnificent", "U2-Magnificent.mp3"); 
    jbox.addSong("Sting", "Englishman in New York", "Sting-Englishman_in_New_York.mp3"); 
    jbox.addSong("U2", "One", "U2-One.mp3"); 
    jbox.printSong(0); 
    jbox.printSong(1); 
    jbox.printSong(2); 
    return 0; 
} 

업데이트, 아마 바보와이를 구현하기 전에 클래스에 대한 자세한을 읽어야 할 사람 :이 코드입니다. 그러나 나는 내가 읽었고 나는 아직도 그것을 얻지 않는다고 생각한다. 이것은 (작동하지 않을 것이다) 내 수업은 지금 모습입니다 :

확인

class Jukebox(): songCounter(0) 
{ 

public: 
    void addSong(string artist, string title, string filename) { 
    songCounter++; 
    song s {songCounter, artist, title, filename}; 
    Songs.push_back(s); 
    } 

    void printSong (int song) { 
    cout << Songs[song].no << ". "; 
    cout << Songs[song].artist << " - "; 
    cout << Songs[song].title << " : "; 
    cout << Songs[song].filename << endl; 
    } 

private: 
    int songCounter; 
    struct song { 
    int no; 
    string artist; 
    string title; 
    string filename; 
    }; 
    vector<song> Songs; 
}; 

최종 단어. 내가 본 C++ 생성자 클래스의 예제에서 나는 그들이 어떻게 작동하는지에 대해 잘못된 인상을 가지고있었습니다. 이제 좀 더 생각하고 있습니다. 그러나 구문은 여전히 ​​이상하게 보입니다. 그러나 나는 그것을 더 많이 읽으려고 노력했기 때문에 나는 그것을 정말로 이해합니다. 내가 한 일과 일하는 것 같습니다 :

#include <iostream> 
#include <vector> 
#include <string> 

using namespace std; 

class Jukebox { 

public: 
    void addSong(string artist, string title, string filename) { 
    songCounter++; 
    song s {songCounter, artist, title, filename}; 
    Songs.push_back(s); 
    } 

    void printSong (int song) { 
    cout << Songs[song].no << ". "; 
    cout << Songs[song].artist << " - "; 
    cout << Songs[song].title << " : "; 
    cout << Songs[song].filename << endl; 
    } 

    Jukebox(): songCounter(0) {} // Constructor 

private: 
    int songCounter; 
    struct song { 
    int no; 
    string artist; 
    string title; 
    string filename; 
    }; 
    vector<song> Songs; 
}; 

int main() { 
    Jukebox jbox; 
    jbox.addSong("U2", "Magnificent", "U2-Magnificent.mp3"); 
    jbox.addSong("Sting", "Englishman in New York", "Sting-Englishman_in_New_York.mp3"); 
    jbox.addSong("U2", "One", "U2-One.mp3"); 
    jbox.printSong(0); 
    jbox.printSong(1); 
    jbox.printSong(2); 
    return 0; 
} 

Answer 1

생성자에서 songCounter을 초기화하지 않았습니다.

Jukebox(): songCounter(0),//....other members 

초기화하지 않으면 임의의 값이있어 프로그램이 정의되지 않은 상태가됩니다.

unitialized 변수를 사용하는 동안 항상주의해야합니다. 종종 으로 연결되며 정의되지 않은 동작이며 프로그램이 좋은 예입니다.

또한, 나는 당신의 디자인의 확실하지 오전하지만 당신은 당신의 Song 클래스의 모든 객체에 대한 상태를 유지하는 카운터로 사용하려는 경우 아마 그것은 static 구성원이어야합니다.

Song 개체를 만들 때 명시 적으로 적절한 값으로 설정해야합니다.

오케이 그 카운터는 JukeBox이 아니고 Song이 아니므로 여전히 회원이 될 수 있습니다.

Answer 2

변수 songCounter을 초기화하지 않았습니다. 당신이 songCounter을 초기화 할

Jukebox(): songCounter(0) {} 

Answer 3

:

는 Jukebox의 클래스 정의에 다음을 추가 하시겠습니까? C++에서 기본 요소는 기본적으로 0으로 초기화되지 않습니다. 생성자에

: songCounter(0) 

을 추가해야합니다.

Answer 4

당신은 Jukebox에 대한 생성자를 필요로하고 있다는 점에서 당신은 당신이 0으로 songCounter를 초기화해야한다고 생각 0

Answer 5

에 카운터를 초기화해야합니다.수업의 공개 부분에서 :

public Jukebox() : songCounter(0) {}