我有一个 C++ 作业。作业要求将一个c程序转换成c++。 下面是问题:

You are requested to convert the following C function into a C++ function and then embed it into a complete program and test it. Note that this function copies a binary file of integers and not a text file. The program must accept the arguments (the file to copy and the file to be copied to) from the command line.

/* ==================== cpyFile =====================

This function copies the contents of a binary file
of integers to a second file.
Pre fp1 is file pointer to open read file
fp2 is file pointer to open write file
Post file copied
Return 1 is successful or zero if error
*/
int cpyFile (FILE *fp1, FILE *fp2)
{
  /* Local Definitions */
  int data;

  /* Statements */
  fseek (fp1, 0, SEEK_END);
  if (!ftell (fp1))
  {
    printf ("\n\acpyFile Error : file empty\n\n");
    return 0;
  } /* if open error */
  if (fseek (fp1, 0, SEEK_SET))
    return 0;
  if (fseek (fp2, 0, SEEK_SET))
    return 0;

  while (fread (&data, sizeof (int), 1, fp1))
    fwrite (&data, sizeof (int), 1, fp2);
  return 1;
} /* cpyFile */

我尽了最大的努力并设法转换了它，但不幸的是，当我使用它时，复制后得到的文件是空的。以下是我的回答:

#include <fstream>
#include <cstdlib>
#include <iostream>
using namespace std;
int main(int argc,char* argv[])
{
  if(argc!=3)
  {cerr<<"invalid number of arguments. must be 3."<<endl;exit(1);}

  fstream fp1(argv[1],ios::in);
  if(!fp1)+{cerr<<argv[1]<<" could not be opened"<<endl;exit(1);}

  fstream fp2(argv[2],ios::out);
  if(!fp2)+{cerr<<"file could not be found."<<endl;exit(1);}


  int data;

  fp1.seekg (0,ios::end);
  if (!fp1.tellg ())
  {
    cout<<"\n\acpyFile Error : file empty\n\n";
    return 0;
  } /* if open error */
  if (fp1.seekg (0, ios::beg))
    return 0;
  if (fp2.seekg (0, ios::beg))
    return 0;

  while (fp1.read (reinterpret_cast<char*>(&data), sizeof (int)))
  {
    fp2.seekp(0);
    fp2.write (reinterpret_cast<char*>(&data), sizeof (int));
  }
  return 1;
}

我已尽力而为，一切正常，除了当我复制一个二进制文件时，我得到的文件是空的，我不知道为什么。

最佳答案

正如其他人所说，您需要以二进制模式打开文件

fstream fp1(argv[1], ios::in | ios::binary); // combine ios::in with ios::binary

fstream fp2(argv[2], ios::out | ios::binary); // combine ios::out with ios::binary

或者你可以让它们ifstream(in file stream只读)和ofstream(out file stream，仅用于写入)并删除 ios::in 和 ios::out 因为 ifstream 隐含了 ios::in 和 ofstream 意味着 ios::out:

ifstream fp1(argv[1], ios::binary);

ofstream fp2(argv[2], ios::binary);

你需要这样做，因为如果你不这样做，当你从 \r\n 或 中读取或写入文件时，文件将被翻译\r 到 \n 等，这会弄乱您的二进制数据，而这些二进制数据可能恰好包含这些字节。

这个:

if (fp1.seekg (0, ios::beg))
    return 0;

if (fp2.seekg (0, ios::beg))
    return 0;

将始终使您的代码返回，因为 seekg 返回您调用它的对象。在这方面，它不等同于 fseek，因为 fseek 在成功时返回 0。所以你永远不会进入 while 循环。将那些从 if 语句中取出，使其看起来像这样:

fp1.seekg(0, ios::beg);
fp2.seekg(0, ios::beg);

或者如果你必须进行检查，你想做

if (!fp1.seekg (0, ios::beg)) // notice the added !
    return 0;

if (!fp2.seekg (0, ios::beg)) // notice the added !
    return 0;

此外，这个(在 while 内):

fp2.seekp(0);

将要写入的点设置为文件的开头。所以除了在文件的开头，你永远不会写任何东西。只需完全删除该行即可。

此外，您在循环内有一个 return，这使得它在第一次迭代时返回。将 return 1; 移到循环之外，这样您只能在循环完成后返回。 没关系，由于不寻常的大括号样式而导致误读。

关于c++ - 需要帮助将 c 转换为 c++(简单错误但无法修复)，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/8828235/