Автокореляционная функция

[DaRKSPiRT]

товарищи, задача такова: есть текстовый файл *.txt с рядом чисел формата *.*** ; необходимо считать из этого файла числа в буфер. можно vector<int> chisla; и посчитать автокореляционную функцию. цикл самой функции я написал. программа есть но на выходе выдает белеберду а не числа.
помогите советом может надо представлять данные из файла в каком то другом виде? C++ Builder 6.0

TForm1 *Form1;
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{
}
//---------------------------------------------------------------------------
void __fastcall TForm1::Button3Click(TObject *Sender)
{
   FILE *Sig_in,*Res;
   short int x;
   vector<short int> buffer;
   vector<short int> result;
   if((Sig_in=fopen(Input_File->Text.c_str(),"rb")) == NULL)
   {
      ShowMessage("Ошибка открытия файла "+ Input_File->Text);
      Input_File->Text="";
   }
   else
   {
      while(fread(&x,2,1,Sig_in)!=NULL) buffer.push_back(x);
      fclose(Sig_in);
      int num_of_element=buffer.size();
      for(int i=0; i<num_of_element; i++) buffer.push_back(buffer[i]);
      for (int i=0; i<num_of_element; i++)
      {
         short int sum=0;
         for(int j=0; j<i; j++) sum+=buffer[num_of_element-j+i]*buffer[j];
         result.push_back(sum);
      }
      if((Res=fopen(Output_File->Text.c_str(),"wb"))==NULL)
      {
         ShowMessage("Ошибка создания файла "+ Output_File->Text);
         Output_File->Text="";
      }
      else
      {
         for(int i=0; i<num_of_element; i++) fwrite(&result[i], 2, 1, Res);
         fclose(Res);

      }
   }
   Label3->Caption="Расчет окончен";
       Label3->Visible=True;
}
void __fastcall TForm1::Button1Click(TObject *Sender)
{
   OpenDialog1->Execute();
}
void __fastcall TForm1::OpenDialog1SelectionChange(TObject *Sender)
{
   Input_File->Text=OpenDialog1->FileName;
}
void __fastcall TForm1::Button2Click(TObject *Sender)
{
   SaveDialog1->Execute();
}
void __fastcall TForm1::SaveDialog1CanClose(TObject *Sender,
      bool &CanClose)
{
   Output_File->Text=SaveDialog1->FileName;
}

[merax]

Один вопрос - одна тема !!!

[B_N]

Сообщение от DaRKSPiRT

......но на выходе выдает белеберду а не числа.

В чём заключается "белиберда"? Как должны выглядеть "числа" в файле?

[filosof_x86]

Расширение txt наталкивает на мысль о том, что это обычный текстовый файл, с которым вы пытаетесь работать бинарно (“rb”/”wb” и fread/fwrite). Вероятно, что ошибка у вас начинается здесь.

Для более точного ответа хорошо бы привести пример входных и выходных данных, а также то, что ожидается увидеть в buffer после считывания входных данных. И еще не очень понятен что за формат чисел (не десятичная ли дробь?) во входном файле, а также как они там записаны (бинарно или текстово).

[DaRKSPiRT]

пример входного файла:
23
25
53
64
24
и т.д. (можно в строчку через пробел)
пример выходного аналогичен.

[filosof_x86]

Как и предполагал, файл у вас текстовый, а работаете вы с ним как с бинарным. К тому же, если стали использовать stl, то так и используйте ее и для работы с файлами. Вот пример кода:

#include <iostream>
#include <fstream>
#include <vector>

int main()
{
  std::vector<int> buffer;
  std::ifstream in("in.txt");
  
  if(in) {
    typedef std::istream_iterator<int> ii;
    std::copy(ii(in), ii(), std::back_inserter(buffer)); // загружаем все целые числа в вектор
    in.close();
    
    typedef std::vector<int>::iterator it;
    // выводим и затем увеличиваем значение каждого элемента на единицу
    for(it p = buffer.begin(); p != buffer.end(); p++) std::cout << (*p)++ << std::endl; 

    // теперь сохраним результаты в другой файл
    std::ofstream out("out.txt");
    if(out) {
      std::copy(buffer.begin(), buffer.end(), std::ostream_iterator<int>(out, "\n")); // "\n" - то, что добавляется после вывода 1 элемента
      out.close();
    } else std::cout << "error: out.txt" << std::endl;

  } else std::cout << "error: in.txt" << std::endl;
  
  return 0;
}

in.txt

1
2 3
55     66		77

out.txt

2
3
4
56
67
78

[DaRKSPiRT]

такие ошибки:

Build
[C++ Error] Unit1.cpp(41): E2316 'istream_iterator' is not a member of 'std'
[C++ Error] Unit1.cpp(41): E2040 Declaration terminated incorrectly
[C++ Error] Unit1.cpp(42): E2268 Call to undefined function 'ii'
[C++ Error] Unit1.cpp(52): E2316 'ostream_iterator' is not a member of 'std'
[C++ Error] Unit1.cpp(52): E2188 Expression syntax

[filosof_x86]

Я видимо неудачно сюда перенес код, не хватает строчки

#include <iterator>

[DaRKSPiRT]

Все замечательно сработало. Остался маленький нюанс относящийся к вычисмлению самой АКФ.

for(int j=0; j<num_of_element; j++) sum+=buffer[j+i]*buffer[j];\вычисление функции с задержкой i.
result.push_back(sum); \записывает в буфер

вот как это выглядело в предыдущем примере. как бы по грамотнее это реализовать в новом коде?

[filosof_x86]

Строчку

for(int i=0; i<num_of_element; i++) buffer.push_back(buffer[i]);

лучше заменить на

buffer.insert(buffer.end(),buffer.begin(),buffer.end());

Блок

for (int i=0; i<num_of_element; i++)  
{  
    short int sum=0;  
    for(int j=0; j<i; j++) sum+=buffer[num_of_element-j+i]*buffer[j];  
    result.push_back(sum);  
}

Можно переделать с использованием итераторов. Пример работы с итераторами есть в моем куске кода выше.