1

我想将以下 C 代码翻译成 C++。

FILE *fp = NULL;
fp = fopen("./filename", "r");
int i = 0;
fscanf(fp, "%d\n", &i);
uint16_t j = (uint16_t) i;

这就是我想出的：

  ifstream file;
  string filename = "./filename";

  file.open(filename.c_str(), ios::in);
  errno = 0;
  if (file.fail()) {
      int tmp = errno;
      std::cout << file.c_str () << " not found: strerror(" << tmp << "): " << strerror(tmp) );
  }
  int i = 0;
  file >> i >> std::endl;       
  uint16_t j = (uint16_t) i;

我想知道语法是否正确或可改进，更重要的是它是否对各种输入都安全。

4

2 回答 2

2 
int read_int(const std::string file_name) {
    std::ifstream file(file_name); //the file will close itself on destruction
    std::uint16_t i;
    //extract type, don't worry about what it is it will either compile or not
    if(!(file >> i)) { //Catch failure
         //or however you wish to deal with it.
         throw std::runtime_error("can't read file");
    }
    return i;
}

int main() {
    try{
        std::uint16_t i=read_int("./filepath");
        //do with i...
    }
    catch(const std::exception& e) {
         std::cerr << e.what() << std::endl;
         return EXIT_FAILURE;
    }
    return EXIT_SUCCESS;
}

请注意，如果您没有 C++11，则需要使用c_str()来打开文件，但首选字符串方法。

编辑： fstream 关​​闭自己，不需要自己关闭它，如果你必须这样做，功能就在那里，但是依赖 RAII 语义要好得多：

http://en.wikipedia.org/wiki/Resource_Acquisition_Is_Initialization

RAII 规定您应该在构建时打开文件，并在破坏时关闭，这确保没有任何无效（排除 EOF、未找到文件...）fstream 对象防止错误。RAII 是 C++ 中的一个基本结构，应该在涉及资源的地方使用。

fstream 析构函数的文档在这里：

http://en.cppreference.com/w/cpp/io/basic_fstream

破坏 basic_fstream 和相关的缓冲区，关闭文件

于 2012-04-23T15:47:11.197 回答
2

确切的等价物是：

std::ifstream fs( "./filename" );
int i = 0;
fs >> i >> std::ws;
uint16_t j = i;

这是否是您真正想要的另一个问题： "\n"在格式字符串中使用 afscanf表示（至少对我而言）您确实想要读取单个'\n'，而不是任意空格；然而， 中的"\n"意思fscanf是跳到下一个非空格。scanf（在交互式输入的情况下，这可能是一个真正的问题，因为在您遇到非空白字符或文件结尾之前，您不会从您的（或我上面的替换）返回。对于来自文件的输入，这可能不是问题。）

在读取面向行的输入时，经典的解决方案是使用 std::getline，然后std::istringstream解析它。

于 2012-04-23T16:20:35.813 回答