Ошибка сегментирования, язык 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;

}

[IUnknown]

Еще 2 ошибки:

1) перед тем, как заталкивать число в стек, надо отсечь весь мусор в строке. Это очень просто:

   else if ( ( expr[i] == ' ' || expr[i] == '\t' ) && i > 0 )
    {
     op[j] = '\0';  /* Вот и все, теперь atof будет воспринимать только число, до позиции j */
     push (atof(op));
     return NUMBER;
    }

2) более серьезный просчет:

 static int i; /* Я так понимаю, это чтобы i сохранялось между вызовами? */
 int j;
 char op[10];
 putchar ('.');
 for (i = 0, j = 0; expr[i] != '\0' || expr[i] != '\n' || expr[i] != '='; i++) /* <--- Вот сюда теперь посмотри */

Именно для этого ты описал i статической? Чтобы при втором заходе в getop() все, что уже было прочитано из строки, пропускалось, и читалось со следующего символа? Не получится, ты в начале цикла переменную i обнуляешь, то есть при самом удачном раскладе ты получишь зацикливание, будешь все время читать только первый токен из строки (о чем тебе и говорит gdb). Применяй какой-нибудь другой способ для подобных трюков...