0

我有以下包含多个属性（字符串）的 User.h。User.cpp 具有所有定义。

//User.h
#ifndef USER_H
#define USER_H
#include<iostream>
#include <cstring>

using namespace std;

class User{

  string username;

  public:
         User();
         string getUsername() const;                      
         void setUsername(string);

};
#endif

我正在使用另一个类“文件”从随机访问的 .dat 文件中插入新用户/查看用户

//File.h
#ifndef FILE_H
#define FILE_H
#include "User.h"

class File{
    public:

             void loadUser(); 
             bool addUser(User&); 

};
#endif

文件类定义

//File.cpp
#include<cstring>
#include<iostream>
#include<iomanip>
#include<fstream>

#include "User.h"
#include "File.h"

using namespace std;

User tempUser;
fstream usersFile;

void File::loadUser(){
     usersFile.open("users.dat", ios::in | ios::binary);

     usersFile.seekg(0);  

     // commenting the following lines prevented the issue
     usersFile.read(reinterpret_cast<char *>(&tempUser), sizeof(tempUser)); 

     cout<<tempUser.getUsername().c_str(); 

     usersFile.close();
}

bool File::addUser(User& user){

     usersFile.open("users.dat", ios::out | ios::ate | ios::binary);

     // no issue when writing to file
     usersFile.write( reinterpret_cast<const char *>(&user), sizeof(user));

     usersFile.close();

     cout<<"User added";  
}

我在运行时遇到了上述问题。虽然没有编译问题。

在处理内部具有“字符串属性”的对象时是否有任何问题？

请帮忙

4

5 回答 5

1

你不能像那样阅读非 POD 类型。字符串不是 POD 类型。 C++ 中的 POD 类型是什么？

有几种方法可以正确读取字符串，具体取决于它们的存储方式。

对于文本文件：

如果字符串只是一个单词，两边用空格分隔，您可以使用普通的旧 >> 运算符。如果多于一个词，你可以将它存储在它自己的行中，并使用getline。

对于二进制文件：

以空终止形式存储字符串。一次读取一个字符，检查空字符。或者，在字符串前面加上一个整数来存储它的大小。当你读它时，首先读入整数，然后读入那么多字符。

于 2010-10-30T16:48:31.583 回答
1

我认为问题在于您将 C++ 代码与 C 思维方式混合在一起。您真正应该在这里做的是使用提取运算符 ,opeartor>>()以及 C++ IO 流。这与使用 C 标准 IO 和read()函数相反。要编写对象，请使用插入运算符operator<<()而不是 Cwrite()函数。

对于使用字符串，请使用std::string. 这个类提供operator<<()和operator>>()。所以，你可以说std::string s，然后io << s，一些 C++ IO 流对象io >> s在哪里。io这将做正确的事情（tm）。这里的理念是，std::string类比您（用户）更了解如何序列化std::string对象。所以让它去做吧，使用 << 和 >> 操作符。

继续这个想法，作为 的作者，您User比任何人都更了解如何序列化User对象。因此，为您的类的用户提供 << 和 >> 运算符，作为一项服务。“你班级的用户”很可能是一周后的你，那时你已经完全忘记了如何正确序列化一个User对象。（或者，您认为您记得但实际上您忘记了一个细节，从而导致代码中的错误）。例子：

// in User.h
#include <string>
#include <iosfwd>  // forward declarations of standard IO streams

namespace mine {
class User {
    User(const std::string& name) : username(name) { }
    friend std::ostream& operator<<(std::ostream&, const User&);
    friend std::istream& operator>>(std::istream&, User&);

private:
    std::string username;
};

std::ostream& operator<<(std::ostream& out, const User& u)
{
    return out << u.username;
}

std::istream& operator>>(std::istream& in, User& u)
{
    return in >> u.username;
}
}  // namespace mine

从这里开始，将用户保存到您说的文件中

std::ofstream f("filename");
User u("John");
f << u;

而已。读取用户：

std::ifstream f2("filename");
f2 >> u;

将代码包装在命名空间中是一种很好的做法。IDE 通过自动完成功能显示有多少符号可见，从而很好地可视化了这个问题。你会看到全球范围内有多少混乱。通过将代码包装在名称空间中，您可以将其分组在一个范围名称下，从而在全局名称范围中节省更多混乱。就是要整洁。如果您将代码放在自己的命名空间中，那么您可以为函数、类或变量选择任何您想要的名称，只要您之前没有选择过它。如果您不将其放在命名空间中，那么您需要与其他人共享名称。这就像一只臭鼬宣布自己的领土，只是没有床的气味。

在那张纸条上，我建议您using namespace std从标题中删除它。这会将命名空间中的所有符号都带入头std文件的所有文件的范围内。#include这是一个不好的做法。如果你愿意，只using namespace std在实现文件中说，而不是在头文件中。

当然，有些人会说即使这是一个坏主意。我个人认为，如果您知道在该特定实现文件中可能存在名称冲突这一事实，那很好。但至少你知道那个using语句在哪里：它在你的实现文件中，它只会在那个实现文件中引起冲突。它有点像一把枪，（一把塑料水枪，但仍然是一把枪），只有你只能射（弄湿）自己的脚，不能射别人的脚。在我看来，这完全没问题。

于 2010-10-30T18:09:17.883 回答
0

与其构建您自己的自定义序列化代码，不如Google Protocol Buffers之类的东西可以轻松完成您想要的工作。它非常适合将简单的结构化数据从一个地方传递到另一个地方。

于 2010-10-30T16:38:01.107 回答
0

是的，std::string 类不是普通的旧数据，换句话说，它包含指针。如果以这种方式保存/加载字符串类，则不会加载/保存指向的数据，只会加载/保存指针的值。

此外 sizeof(tempUser) 不包括字符串指向的文本的大小。

您的解决方案是改变您读取/写入数据的方式。一种方法是使用 boost::serialization 来处理像 std::string 这样的数据类型。另一种方法是自己将每个字符串写入文本文档中的单独行（不是二进制模式），然后使用 readline 将它们读回。

于 2010-10-30T16:26:00.677 回答
0

string 是一个对象，这意味着您没有编写它的内容。

尝试编写用户并检查文件以了解我的意思。然后您正在阅读的是一些指针，它们指向无效的内存位置。

于 2010-10-30T16:26:25.117 回答