3

我正在尝试从二进制文件中计算零和一的数量。问题是,我得到了正确的不。零,但结果等于否。的零。我正在做的是按 char 读取文件 char 。由于最多可以有 256 个字符,我将结果存储在一个临时数组中,用于零和一,如果再次出现字符,则从那里检索。

#include<iostream>
#include<cstdio>
#include<cstdlib>
using namespace std;
void func(int* a1 ,int* a2)
{
    for(int i=0;i<256;i++)
    for(int j=0;j<8;j++)
    {
        if( (i & 1) ==1 )
        {
            a1[i]+=1;
        }
        else if( (i & 1) ==0 )
        {
            a2[i]+=1;   
        }
        i>>1;
    }
}
int main()
{
    int zero[256];
    int one[256];
    int tzero[256];
    int tone[256];
    for(int i=0;i<256;i++)
    {
        zero[i]=0;
        one[i]=0;
        tzero[i]=0;
        tone[i]=0;
    }
    func(tone,tzero);
    FILE* input;
    FILE* output;
    output=fopen("ascii.txt","w");
    input=fopen("one.bin","r");
    int c;
    while((c=fgetc(input))!=EOF)
    {
        fprintf(output,"%d\n",c);
        zero[c]+=tzero[c];
        one[c]+=tone[c];
    }
    int zeroes=0;
    int ones=0;
    for(int i=0;i<=255;i++)
    {
        zeroes+=zero[i];
        ones+=one[i];
    }
    cout<<"zeroes:"<<zeroes<<endl;
    cout<<"ones:"<<ones<<endl;
    fclose(input);

    fclose(output);

}
4

2 回答 2

1

c计数零和一的循环通过做破坏了值

c >>= 1;

在所有八班完成后,c始终为零,因此以下代码会增加错误计数:

// The value of c is always zero
tzero[c]=z;
tone[c]=o;
one[c]+=tzero[c];
zero[c]+=tzero[c];

您应该保存c位计数循环之前的值,并在循环结束后恢复它。

tzero[]更好的是,预先计算 和 的值,tone[]而无需等待它们出现在文件中。这将使您的主循环的主体非常简短和干净:

while((c=fgetc(input))!=EOF) {
    one[c] += tzero[c];
    zero[c] += tzero[c];
}
于 2013-09-16T15:29:20.810 回答
0

如果您的目标只是按位计算文件10的位,则可以通过使用 C++ 文件流而不使用查找表来大大简化事情:

#include <iostream>
#include <fstream>

int main(int argc, char** argv)
{
  std::ifstream fpInput("Input.txt");
  unsigned unOnes = 0;
  unsigned unZeros = 0;
  char chTemp;

  chTemp = fpInput.get();
  while (fpInput.good())
  {
    for (unsigned i = 0; i < 8; ++i)
    {
      if (chTemp & 0x1<<i) unOnes++;
      else unZeros++;
    }

    chTemp = fpInput.get();
  }

  fpInput.close();

  std::cout << "Found " << unOnes << " ones." << std::endl;
  std::cout << "Found " << unZeros << " zeros." << std::endl;

  return 0;
}

如果您传递正确的优化标志,一个好的编译器应该扩展中间循环。

于 2013-09-16T15:47:42.653 回答