не работает сдвиг

[Spook]

int reading(char *path, unsigned long long *x_)
{   unsigned long long x;
   unsigned char simbol;
   ifstream inFile;
   x = *x_;      
   inFile.open(path);
   if (inFile.is_open() == NULL)
   {      cout << "File "<<'\"'<<path<<'\"'<<" not open" << '\n';
      return 1; 
   }
   for (int i = 0; i < 8; i++)
   {
      simbol = inFile.get();   
      if (simbol == EOF)
         break;
      x+=simbol;
      cout<<x<<"\n";
      if (i != 7)
         x<<8;
   }
   inFile.close();
   *x_ = x;
   return 0;
}

[venco]

Вам нужен сдвиг с присваиванием: <<=
А ещё мне кажется сдвигать надо до сложения и каждый раз, но это зависит от того, что вам нужно.

Ещё замечания:
1. Добавьте const к объявлению параметра path:

int reading(const char *path, unsigned long long *x_)

2. Не надо сравнивать is_open() с NULL:

if (!inFile)

3. Переменная simbol (лучше symbol), должна быть объявлена int, иначе проверка на EOF работать не будет.

4. Явно вызывать close() не обязательно - файл закроется в деструкторе. Да и открывать лучше прямо в конструкторе:

ifstream inFile(path);

... ну ещё много чего можно сказать...

[Circiter]

Наверное, можно делать сразу *x_<<=8 выкинув таким образом x. И, возможно, стоит в начале функции написать *x_=0.

[Spook]

venco в сообщении #490529 писал(а):

Вам нужен сдвиг с присваиванием: <<=

точно! на ночь глядя не сообразил

venco писал(а):

ну ещё много чего можно сказать...

предыдущие замечания устранил, если не сложно, дайте другие советы!

-- Сб окт 08, 2011 21:37:06 --

Circiter в сообщении #490539 писал(а):

Наверное, можно делать сразу *x_<<=8 выкинув таким образом x. И, возможно, стоит в начале функции написать *x_=0.

Не слишком силен в указателях, перестраховался) Но попробую и так, чтобы разобраться и чтобы код солиднее выглядел.

-- Сб окт 08, 2011 22:13:31 --

venco в сообщении #490529 писал(а):

3. Переменная simbol (лучше symbol), должна быть объявлена int, иначе проверка на EOF работать не будет.

Я считываю символы из текстового файла, мне нужно, чтобы они хранились как целые положительные числа. Это обеспечивает unsigned char. Подойдет ли unsigned int?

[venco]

Spook в сообщении #490774 писал(а):

venco в сообщении #490529 писал(а):

3. Переменная simbol (лучше symbol), должна быть объявлена int, иначе проверка на EOF работать не будет.

Я считываю символы из текстового файла, мне нужно, чтобы они хранились как целые положительные числа. Это обеспечивает unsigned char. Подойдет ли unsigned int?

unsigned int не подойдёт. Подойдёт int.
istream.get() возвращает значение типа int. Если конец файла, то значение равно -1 (EOF), иначе значение в диапазоне 0-255 - очередной байт из потока ввода.
Судя по сдвигам, поток совсем не текстовый, а бинарный. Для текстовых чисел парсер должен быть сложнее.

[Spook]

venco, Вы правы, нужен просто int. По поводу парсера - сдвиг использую для составления числа из прочитанных символов) Код исправил, работает правильно, спс.