4

我试图编写一个从文本文件读取到链表的程序

这是列表结构。

#include <iostream> 
#include <string>
#include <fstream>
using namespace std;


struct Video { 
char video_name[1024];      //  video name
int ranking;                // Number of viewer hits
char url[1024];             //  URL
Video *next;  // pointer to Video structure
}  *head = NULL;        // EMPTY linked list

这是读入的代码：

void load()
{
struct Video *temp;
temp = (Video*)malloc(sizeof(Video)); //allocate space for node 
temp = head;
ifstream rankFile ("Ranking.dbm");
if (rankFile.is_open())
{
    while ( rankFile.good() )
    {
        cin.getline(rankFile, temp->video_name, "\n");
        cin.getline(rankFile, temp->ranking, "\n");
        cin.getline(rankFile, temp->url, "\n");
        temp = temp->next; 

    }
    myfile.close();
}

else cout << "Unable to open file"; 

return ;

}

它正在从如下所示的文本文件中读取Ranking.dbm：

bagheera
20
bagheera.com
sushi
60
sushi.com
wicket
99
wicket.com
teek
100
teek.com

但是我收到一条错误消息：在我从文件中读取的Invalid conversion from void* to char*所有 3 条语句中。cin.getline()我需要能够从我的文件 () 中逐行读取Ranking.dbm并将每组 3 行存储到temp->video_name，temp->ranking然后temp->url创建一个新节点并保存接下来的 3 行......依此类推，直到我读入文件中的所有内容。

我怎样才能做到这一点？我是以完全错误的方式解决这个问题还是这只是一个语法错误？我仍然掌握 C++ 的窍门：/

4

2 回答 2

5

这是不正确的使用std::istream::getline()：

cin.getline(rankFile, temp->video_name, "\n");

并且没有任何意义，因为涉及两个输入流：cin和rankFile. 正确的调用（但不是最可取的）是：

rankFile.getline(temp->video_name, 1023);

建议：

  • 使用std::string而不是char[]使用std::getline(in, std::string&).
  • 用于operator>>阅读，int因为您不能std::getline()用于此。
  • 检查每个读取操作的结果。
  • 不要malloc()在 C++ 中使用new和delete.
  • 如果不需要，请不要动态分配。
  • 例如，使用其中一个 STL 容器来保存一个列表，而不是自己实现它std::vector<Video>。

例如：

struct Video { 
    std::string video_name;
    int ranking;
    std::string url;
};

std::vector<Video> load()
{
    std::vector<Video> result;
    std::ifstream rankFile("Ranking.dbm");
    if (rankFile.is_open())
    {
        Video temp;
        std::string line;
        while (std::getline(rankFile, temp.video_name) &&
               rankFile >> temp.ranking &&
               std::getline(rankFile, line) && // need to skip 'ranking's
                                               // unread new-line
               std::getline(rankFile, temp.url))
        {
            result.push_back(temp);
        }
    }
    else
    {
        std::cerr << "Unable to open file"; 
    }

    return result;
}
于 2012-10-07T18:21:59.053 回答
0 
getline(rankFile, temp->video_name); // You should write it this way
于 2012-10-07T18:21:50.963 回答