Ошибка сегментирования, язык C Опции

[Игорь]

Хочу написать програмку-калькулятор (по книге Кернигана и Ритча).
Суть простая: вводится выражение в форме обратной польской нотации, оно вычисляется и выводится результат.

Программу разбил на функции:

1) GetLine - получение строки (работает нормально).
2) calculate - вычисление и получение результата, вот тут проблема, при входе в функцию появляется та самая ошибка сегментирования.
3) getop - получение следующего элемента из строки. (тоже самое).
4) pop и push - работа со стеком. (работают тоже не правильно, но по крайней мере из-за них программа не вылетает).

Компилятор GCC.

calculate (Показать/Скрыть)

double calculate (char expr[])
{

 char type;
 double op2;

 printf ("Начало вычисления"); // я это специально здесь поставил, но до этого места даже не доходит.

 while ( ( type = getop(expr) ) != EOE )
  switch ( type )
   {
    case NUMBER:
     ;
    break;
    case '+':
     push (pop() + pop());
    break;
    case '-':
     op2 = pop();
     push (pop() - op2);
    break;
    case '*':
     push (pop() * pop());
    break;
    case '/':
     if ( (op2 = pop()) != 0.0)
      push (pop() / op2);
     else
      printf ("Ошибка: деление на нуль.\n");
    break;
    default:
     printf ("Ошибка: неизвестная операция.\n");
    break;
   }

 return pop();

}

getop (Показать/Скрыть)

int getop (char expr[])
{

 static int i; /* getop будет возвращать найденную операцию или NUMBER если найдено число, потом после того
                  как calculate выполнит необходимые действия работа будет продолжена там, где остановилась.*/  
 int j;
 char op[10];

 for (i = 0, j = 0; expr[i] != '\0' || expr[i] != '\n' || expr[i] != '='; i++)
  {
   if ( expr[i] >= '0' && expr[i] <= '9' )
     op[j++] = expr[i];
   else if ( expr[i] = '.' && j > 0 )
    op[j++] = expr[i];
   else if ( expr[i] == '+' || expr[i] == '-' || expr[i] == '*' || expr[i] == '/' )
    {
     i += 2;
     return expr[i];
    }
   else if ( ( expr[i] == ' ' || expr[i] == '/t' ) && i > 0 )
    {
     push (atof(op));
     return NUMBER;
    }
   else
    printf ("Ошибка: неверные данные.\n");
  }

  return EOE;

}

[Игорь]

Спасибо за найденные ошибки, с помощью gdb нашёл ещё несколько в логике getop, поэтому немножко её изменил. Однако, проблемы ещё остались.

Строка для теста: 1.5 2 +

else if ( ( expr[i] == ' ' || expr[i] == '\t' ) && j > 0 )
 {
   op[j] = '\0';	
   push (atof(op));  // когда я проверил stack[0] он оказался равен 0 вместо 1.5 
   j = 0;
 }

for (j = 0; expr[i] != '\0' || expr[i] != '\n' || expr[i] != '='; i++)  /* здесь при втором вхождении в функцию, expr[i] = '\0',
                                                                           но цикл всё равно идёт дальше. */

Применяй какой-нибудь другой способ для подобных трюков...

Т.е. если я захочу сохранить значение переменной после выхода из функции я как-то ещё поступить? Как?
И можно ли использовать, например, expr в getop не передавая как аргумент и не делая глобальной переменной?

Сообщение отредактировано: Игорь - 6.07.2011 16:54