simplify code

Author: fsaadatmand

chapter04/4-10.c

@@ -2,6 +2,7 @@
  * Exercise 4-10. An alternative organization uses getline to read an entire
  * input line; this makes getch and ungetch unnecessary. Revise the calculator
  * to use this approach.
+ *
  * Faisal Saadatmand
  */
 
@@ -18,7 +19,6 @@
 #define MAXVAL   100          /* maximum depth of val stack */
 #define BUFSIZE  100
 #define MAXVAR   26
-#define TOP      val[sp - 1]  /* top element in stack */
 
 /* functions */
 int    getop(char []);
@@ -40,7 +40,6 @@ double val[MAXVAL];          /* value stack */
 double mem[MAXVAR];          /* variables values */
 char   buf[BUFSIZE];         /* buffer from ungetch */
 int    bufp;                 /* next free position in buf */
-int    peak;                 /* flag: peak at top of the stack */
 int    variable;             /* current input variable */ 
 double printed;              /* last printed value */
 
@@ -127,18 +126,18 @@ int getLine(char s[], int lim)
 /* printTOP: print top of the stack without pop */
 void printTOP(void)
 {
-	if (sp < 1)
-		printf("stack empty\n");
-	printf("\t%.8g\n", TOP);
+	double top;
+
+	top = pop();
+	printf("\t%.8g\n", top);
+	push(top);
 }
 
 /* duplicateTop: duplicate the top element in the stack */
 void duplicateTop(void)
 {
 	double top;
 
-	if (sp < 1)
-		return;
 	push(top = pop());
 	push(top);
 }
@@ -148,11 +147,6 @@ void duplicateTop(void)
  {
 	 double top1, top2;
 
-	 if (sp < 2) {
-		 if (sp == 1)
-			 printf("error: 1 element in stack\n");
-		 return;
-	 }
 	 top1 = pop();
 	 top2 = pop();
 	 push(top1);
@@ -162,8 +156,7 @@ void duplicateTop(void)
 /* clear: clears the entire stack */
 void clearStack(void)
 {
-	while (sp > 1)
-		pop();
+	sp = 0;
 }
 
 /* mathfunction: call the appropriate math function according to value of s,
@@ -192,7 +185,6 @@ int mathfunction(char s[])
  * location in mem and push back to top of stack */
 void storeVariable(void)
 {
-//	if (isalpha(variable) && islower(variable)) {
 	if (variable >= 'a' && variable <= 'z') {
 		pop();
 		push(mem[variable - 'a'] = pop());
@@ -233,7 +225,6 @@ int main(void)
 				push(printed);
 			} else if (!strcmp(s, "mc")) {
 				clearMemory();
-				peak = 1;
 			} else if (!mathfunction(s))
 				printf("error: unknown command %s\n", s);
 			break;
@@ -261,27 +252,23 @@ int main(void)
 			else
 				printf("error: zero divisor\n");
 			break;
-		case '!':
-			peak = 1;
+		case '?':
+			printTOP();
 			break;
-		case '#':
+		case 'd':
 			duplicateTop();
 			break;
-		case '&':
+		case 's':
 			swapTopTwo();
 			break;
-		case '~':
+		case 'c':
 			clearStack();
 			break;
 		case '=':
 			storeVariable();
 			break;
 		case '\n':
-			if (peak) {
-				printTOP();
-				peak = 0;
-			} else
-				printf("\t%.8g\n", printed = pop());
+			printf("\t%.8g\n", printed = pop());
 			break;
 		default:
 			if (islower(type))