Why is my Pre to Postfix code not working?

Posted by Anthony Glyadchenko on Stack Overflow See other posts from Stack Overflow or by Anthony Glyadchenko
Published on 2010-04-27T00:47:04Z Indexed on 2010/04/27 0:53 UTC
Read the original article Hit count: 447

Filed under:
|
|

For a class assignment, I have to use two stacks in C++ to make an equation to be converted to its left to right equivalent: 2+4*(3+4*8) --> 35*4+2 --> 142

Here is the main code:

#include <iostream>
#include <cstring>
#include "ctStack.h"

using namespace std;

int main (int argc, char * const argv[]) {
 string expression = "2+4*2";
    ctstack *output = new ctstack(expression.length());
 ctstack *stack = new ctstack(expression.length());
 bool previousIsANum = false;
 for(int i = 0; i < expression.length(); i++){
  switch (expression[i]){
   case '(':
    previousIsANum = false;
    stack->cmstackPush(expression[i]);
    break;
   case ')':
    previousIsANum = false;
    char x;
    while (x != '('){
     stack->cmstackPop(x);
     output->cmstackPush(x);
    }
    break;
   case '0':
   case '1':
   case '2':
   case '3':
   case '4':
   case '5':
   case '6':
   case '7':
   case '8':
   case '9':
    cout << "A number" << endl;
    previousIsANum = true;
    output->cmstackPush(expression[i]);
    break;
   case '+':
    previousIsANum = false;
    cout << "+" << endl;
    break;
   case '-':
    previousIsANum = false;
    cout << "-" << endl;
    break;
   case '*':
    previousIsANum = false;
    cout << "*" << endl;
    break;
   case '/':
    previousIsANum = false;
    cout << "/" << endl;
    break;
   default:
    break;
  }
 }
 char i = ' ';
 while (stack->ltopOfStack > 0){
  stack->cmstackPop(i);
  output->cmstackPush(i);
  cout << i << endl;
 }
 return 0;
}

Here is the stack code (watch out!):

#include <cstdio>
#include <assert.h>
#include <new.h>
#include <stdlib.h>
#include <iostream>

 class ctstack {
 private:
  long*  lpstack ;              // the stack itself
  long  ltrue ;            // constructor sets to 1
  long  lfalse ;            // constructor sets to 0
          // offset to top of the stack
  long  lmaxEleInStack ;         // maximum possible elements of stack

 public:
  long  ltopOfStack ;

  ctstack ( long lnbrOfEleToAllocInStack )  {     // Constructor
   lfalse = 0 ;                                             // set to zero
   ltrue  = 1 ;                                            // set to one
   assert ( lnbrOfEleToAllocInStack > 0 ) ;     // assure positive argument
   ltopOfStack     = -1 ;          // ltopOfStack is really an index
   lmaxEleInStack = lnbrOfEleToAllocInStack ;    // set lmaxEleInStack to max ele
   lpstack      = new long [ lmaxEleInStack ] ;   // allocate stack
   assert ( lpstack ) ;            // assure new succeeded
  }

  ~ctstack ( ) {              // Destructor
   delete [ ]  lpstack ;                                       // Delete the stack itself
  }

  ctstack& operator= ( const ctstack& ctoriginStack) {   // Assignment
   if ( this == &ctoriginStack )          // verify x not assigned to x
    return *this ;

   if ( this -> lmaxEleInStack <
    ctoriginStack . lmaxEleInStack ) {   // if destination stack is smaller than
    delete [ ]  this -> lpstack ;        // original stack, delete dest and alloc
    this -> lpstack =           // sufficient memory
    new long [ ctoriginStack . lmaxEleInStack ] ;
    assert ( this -> lpstack ) ;           // assure new succeeded

    // reset stack size attribute
    this -> lmaxEleInStack = ctoriginStack . lmaxEleInStack ;
   }
   // copy original to destination stack
   for ( long i = 0 ; i < ctoriginStack . lmaxEleInStack ; i ++ )
    *( this -> lpstack + i ) = *( ctoriginStack . lpstack + i ) ;

   this -> ltopOfStack = ctoriginStack . ltopOfStack ;  // reset stack position attribute

   return *this ;
  }

  long cmstackPush (char lplaceInStack ) {                // Push Method
   if ( ltopOfStack == lmaxEleInStack - 1 )          // stack is full can't add element
    return lfalse ;

   ltopOfStack ++ ;            // acquire free slot

   *(lpstack + ltopOfStack ) = lplaceInStack ;         // add element

   return ltrue ;             // any number other than zero is true
  }

  long cmstackPop (char& lretrievedStackEle ) {           // Pop Method
   if ( ltopOfStack < 0 ) {          // stack has no elements
    lretrievedStackEle = -1 ;         // dummy element
    return lfalse ;
   }

   lretrievedStackEle = *( lpstack + ltopOfStack ) ;     // stack has element  -- return it

   ltopOfStack -- ;                  // stack is pop'd

   return ltrue ;                 // any number other than zero is true
  }

  long cmstackLookAtTop (char& lretrievedStackEle ) {    // Pop Method
   if ( ltopOfStack < 0 ) {          // stack has no elements
    lretrievedStackEle = -1 ;         // dummy element
    return lfalse ;
   }

   lretrievedStackEle = *( lpstack + ltopOfStack ) ;     // stack has element  -- return it

   return ltrue ;                 // any number other than zero is true
  }

  long cmstackHasAnEle (char& lretrievedTopOfStack ) {       // Has element method
   lretrievedTopOfStack = ltopOfStack ;
   return ltopOfStack < 0 ? lfalse : ltrue ;      // 0 - false stack does not have any ele
  }                 // 1 - true stack has at least one element

  long cmstackMaxNbrOfEle (char& lretrievedMaxStackEle ) {    // Maximum element method
   lretrievedMaxStackEle = lmaxEleInStack ;      // return stack size in reference var
   return ltrue ;               // Return Maximum Size of Stack
  }

 } ;

Thanks, Anthony.

© Stack Overflow or respective owner

Related posts about homework

Related posts about c++