0

第一次在这里写。我已经看到了像我这样的其他几个问题,但无法解决我的问题。我有 4 个课程:歌曲、播放列表、专辑、艺术家。播放列表由一系列歌曲组成。专辑继承了播放列表并具有更多功能,艺术家由专辑数组和其他一些东西组成。

Album(char* _name, Song* songs,int _sales ,int _year,int n):Playlist(songs,n)
    {
        name = new char[strlen(_name)+1];
        strcpy(name,_name);
        salesCount=_sales;
        year = _year;
    };
Playlist::Playlist(Song* _songlist, int n)
{
    if(n > 20)
    {
        cout<<"The number of song must be lesser or equal to 20"<<endl;
        return;
    } 

    SongList = new Song[n];
    for(int i = 0 ; i<n; i++)
    {
        SongList[i] = _songlist[i];
    }
numberOfSongs=n;
}

当我使用这个构造函数创建一个专辑时,我没有任何问题,但是当我尝试创建一个专辑数组时,它由使用这个构造函数创建的专辑组成,我遇到了内存访问冲突。有任何想法吗 ?

好的,所以我稍微更新了我的代码。这是我在主要方法中尝试做的事情:

Song newSong("My Song", 9, 231),mySong("Your Song", 8 , 180),yourSong("His Song",7,135), herSong("Her Song",8,431);
Song songs[4] = {newSong, mySong,yourSong,herSong};
Album yourAlbum("My Album",songs,200000, 2010, 4);
yourAlbum.PrintInfo(); //<------- WORKS
Album albums[1]; //<------ It calls the deffault constructor of Album and after it gives me //"Source not available"
//When I try to declare it like this Album albums[1] = { yourAlbum } ; it gives me access violation

Artist myArtist("blake", albums);
myArtist.PrintArtistInfo();
4

2 回答 2

4
Playlist::Playlist(Song* _songlist) {
    numberOfSongs = sizeof(*_songlist)/sizeof(_songlist);  // <-- THIS
    ...

导致将意外(无意义)值分配给,numberOfSongs 因为它等于:

numberOfSongs = sizeof(Song) / sizeof(Song*);

如果你需要你的函数知道动态分配数组的大小,你应该自己跟踪这个大小并将它传递给这个函数。

另请注意,由于您使用 C++ 进行编程,因此您应该使用该语言提供的功能。您应该使用std::string代替 C 样式的字符串和 STL 容器,例如std::vector代替 C 样式的数组(在 的情况下std::vector,对象内部保存有关其长度的信息,因此您不会处理此类问题)。

于 2013-03-27T14:20:35.460 回答
0

这里的问题是您正在分配一个sizeof(*_songlist)/sizeof(_songlist);元素数组。

也就是说,Song您分配的项目数等于用于存储 aSong的字节数除以用于存储指针(指向 a Song)的字节数。这是一个相当无意义的值。

此外,这个数字是一个常数值,它不依赖于您传递给它的数组中的歌曲数量。因此,无论传递什么,此函数的任何运行都将始终分配相同数量的内存。如果此数字大于实际传递的歌曲数,您将在数组之外进行索引,这是未定义的行为。这可以解释你的分段错误。

另一方面,如果您传递的歌曲多于该常量,您将使用适合您分配的数组的歌曲子集初始化您的数组,从而丢失一些数据。为了解决这个问题,要么传递一个(对 a)std::vector 歌曲的引用,要么添加一个额外的参数,其中包含数组中歌曲的实际数量,并将其用作数组的大小。

于 2013-03-27T14:42:40.900 回答