1

我有以下代码,它采用未排序的歌曲和艺术家列表,并对它们进行排序和显示。

int main()
{
   SongList totalList; // has a public 2d array 'unsortedSongs' variable
   char songs[100][80] =
   {
      {"David Bowie 'Ziggy Stardust'",},
      {"Smokey Robinson 'You've Really Got A Hold On Me'",},
      {"Carole King 'You've Got A Friend'",},
      // many more songs here totaling to 100
      {"Joni Mitchel 'A Case Of You'",},
      {"Prince 'Kiss'"}

   };
   memcpy(&totalList.unsortedSongs, &songs, sizeof(songs)); // this causes a segmentation fault
   totalList.displaySortedList();
   return 0;
}

我几乎直接从这里的示例中提取了 memcpy 的代码,所以我很困惑为什么这不起作用。有人可以帮我解决这个问题吗?

编辑:

这是 SongList 的初始化

class SongList
{
public:
   char unsortedSongs[100][80];
public:
   void displaySortedList();
   void sortList();
   string rearrange(char[]);
   string getSongsForArtist(int*);
};
4

3 回答 3

4

这一行:

memcpy(&totalList.unsortedSongs, &songs, sizeof(songs));

应该:

memcpy(totalList.unsortedSongs, songs, sizeof(songs));

因为两者songstotalList.unsortedSongs都会衰减为指针,这类似于您引用的参考文献中的第一个示例:

memcpy ( person.name, myname, strlen(myname)+1 );
于 2013-06-18T00:12:34.220 回答
1

http://www.cplusplus.com/reference/cstring/memcpy/

Memcpy 期望源变量和目标变量是指针( void * )

totalList.unsortedSongs 是一个指针。

当您编写 &totalList.unsortedSongs 时,您正在询问指针的地址。有点像“指向指针的指针”......见这里: http ://www.cplusplus.com/doc/tutorial/pointers/

于 2013-06-18T00:23:07.053 回答
0

我刚刚编译了你的代码,它工作正常。

但是,我发现您的初始化列表相当好奇。虽然它有效,但它让我觉得你实际上想要定义一个 char[80] 数组的数组,而不仅仅是一个 char[80] 数组。

因此,我认为您的显示例程可能是错误的,并且您的调试器只是没有向您显示真正出错的地方,因为优化或其他原因。

于 2013-06-18T00:17:30.157 回答