Калькулятор в командной строке

[NDrago]

В общем задача была такова создать калькулятор командной строки с учетом приоритетов операций,тоесть 2+2*2/2 и тд

#include <iostream>
#include <cstring>

using namespace std;

int main(int argc,char*argv[]) {
	char *x=argv[1];
	int count=1;
	int count2=0;
	for(int i=0; i<strlen(argv[1]); i++)
		if((char)argv[1][i]=='+'||(char)argv[1][i]=='-'||(char)argv[1][i]=='*'||(char)argv[1][i]=='/')
			count2++;
	char *oper=new char[count2];
	for(int i=0; i<strlen(oper); i++)
		oper[i]='0';
	int g=0;
	for(int i=0; i<strlen(argv[1]); i++)
		if((char)argv[1][i]=='+'||(char)argv[1][i]=='-'||(char)argv[1][i]=='*'||(char)argv[1][i]=='/') {
			oper[g]=(char)argv[1][i];
			g++;
		}
	for(int i=0; i<strlen(argv[1]); i++)
		if(x[i]>='0'&&x[i]<='9')
			;
		else count++;
	double *z=new double[count];
	int k=0;
	for(int i=0; i<count; i++)
		z[i]=0;
	int n=strlen(x);
	for(int i=0; i<n; i++)
		if(x[i]>='0'&&x[i]<='9')
			z[k]=z[k]*10+x[i]-'0';
		else k++;

	for(int i=0; i<count; i++)
		cout<<z[i]<<" ";
	cout<<endl;
	for(int i=0; i<count2; i++)
		cout<<oper[i]<<" ";
	cout<<endl;

	double result=0;
	if(strlen(oper)>1) {
		for(int i=0; i<strlen(oper); i++) {
			if((char)oper[i]=='/') {
				result=z[i]/z[i+1];
				cout<<"ERROR"<<endl;
				for(int j=i; j<strlen(oper)-1; j++) {
					oper[j]=oper[j+1];
					oper[j+1]='0';
				}
				for(int k=i; k<count-1; k++) {
					z[k+1]=z[k+2];
					z[k+2]=0;
					z[i]=result;
				}
				z[count-2]=0;
				i--;
				cout<<"ERROR2"<<endl;
			}
		}

		for(int i=0; i<strlen(oper); i++) {
			if((char)oper[i]=='*') {
				result=z[i]*z[i+1];
				for(int j=i; j<strlen(oper)-1; j++) {
					oper[j]=oper[j+1];
					oper[j+1]='0';
				}
				for(int k=i; k<count-1; k++) {
					z[k+1]=z[k+2];
					z[k+2]=0;
					z[i]=result;
				}
				z[count-1]=0;
				i--;
			}
		}
		for(int i=0; i<strlen(oper); i++) {
			if((char)oper[i]=='-') {
				result=z[i]-z[i+1];
				for(int j=i; j<strlen(oper)-1; j++) {
					oper[j]=oper[j+1];
					oper[j+1]='0';
				}
				for(int k=i; k<count-1; k++) {
					z[k+1]=z[k+2];
					z[k+2]=0;
					z[i]=result;
				}
				z[count-1]=0;
				i--;
			}
		}
		for(int i=0; i<strlen(oper); i++) {
			if((char)oper[i]=='+') {
				result=z[i]+z[i+1];
				for(int j=i; j<strlen(oper)-1; j++) {
					oper[j]=oper[j+1];
					oper[j+1]='0';
				}
				for(int k=i; k<count-1; k++) {
					z[i]=result;
					z[k+1]=z[k+2];
					z[k+2]=0;
				}
				z[count-1]=0;
				i--;
			}
		}
	} else {
		if(oper[0]=='*')
			result=z[0]*z[1];
		if(oper[0]=='/')
			result=z[0]/z[1];
		if(oper[0]=='+')
			result=z[0]+z[1];
		if(oper[0]=='-')
			result=z[0]-z[1];
	}
	cout<<"Result:"<<result<<endl;

	return 0;
}

после долгих мучений смог сделать это, для большинства заданных операций все работает, но почему-то на операциях типа 2+2*2/2
или 2*2+2/2 зависает,как мне удалось выяснить проблема находится в цикле деления, он становится бесконечным,для наглядности вставил туда 2 вывода эррора, первый становится бесконечным,а второй не выводится вообще, пока не могу понять в чем проблема, помогите пожалуйста найти баг,буду благодарен

[alexzk]

Перепешите с использованием хотя бы функций. Т.е. 1 функция выполняет 1 операцию, например, выделяет входное число.
Декомпозируйте задачу на блоки. В этой вашей копипасте сложно разбираться.

Смотрите - 2 цикла совершенно одинаковых, различие в знаке операции, это явный кандидат на функцию

for(int i=0; i<strlen(oper); i++) {
			if((char)oper[i]=='-') {
				result=z[i]-z[i+1];
				for(int j=i; j<strlen(oper)-1; j++) {
					oper[j]=oper[j+1];
					oper[j+1]='0';
				}
				for(int k=i; k<count-1; k++) {
					z[k+1]=z[k+2];
					z[k+2]=0;
					z[i]=result;
				}
				z[count-1]=0;
				i--;
			}
		}
		for(int i=0; i<strlen(oper); i++) {
			if((char)oper[i]=='+') {
				result=z[i]+z[i+1];
				for(int j=i; j<strlen(oper)-1; j++) {
					oper[j]=oper[j+1];
					oper[j+1]='0';
				}
				for(int k=i; k<count-1; k++) {
					z[i]=result;
					z[k+1]=z[k+2];
					z[k+2]=0;
				}
				z[count-1]=0;
				i--;
			}

(а еще, - можно заменить сложением с отрицательным числом, и вообще исключить -)

В общем случае, если вы что-то делает 2 раза - это должна быть отдельная функция.

[NDrago]

Сообщение от alexzk

Перепешите с использованием хотя бы функций. Т.е. 1 функция выполняет 1 операцию, например, выделяет входное число.
Декомпозируйте задачу на блоки. В этой вашей копипасте сложно разбираться.

Смотрите - 2 цикла совершенно одинаковых, различие в знаке операции, это явный кандидат на функцию

for(int i=0; i<strlen(oper); i++) {
			if((char)oper[i]=='-') {
				result=z[i]-z[i+1];
				for(int j=i; j<strlen(oper)-1; j++) {
					oper[j]=oper[j+1];
					oper[j+1]='0';
				}
				for(int k=i; k<count-1; k++) {
					z[k+1]=z[k+2];
					z[k+2]=0;
					z[i]=result;
				}
				z[count-1]=0;
				i--;
			}
		}
		for(int i=0; i<strlen(oper); i++) {
			if((char)oper[i]=='+') {
				result=z[i]+z[i+1];
				for(int j=i; j<strlen(oper)-1; j++) {
					oper[j]=oper[j+1];
					oper[j+1]='0';
				}
				for(int k=i; k<count-1; k++) {
					z[i]=result;
					z[k+1]=z[k+2];
					z[k+2]=0;
				}
				z[count-1]=0;
				i--;
			}

(а еще, - можно заменить сложением с отрицательным числом, и вообще исключить -)

В общем случае, если вы что-то делает 2 раза - это должна быть отдельная функция.

спасибо,поправлю,а в данном коде вообще невозможно найти ошибку?

[min@y™]

Сообщение от alexzk

В общем случае, если вы что-то делает 2 (или более) раза - это должна быть отдельная функция.

[alexzk]

Сообщение от NDrago

спасибо,поправлю,а в данном коде вообще невозможно найти ошибку?

Бесплатно? - нет

[SAMOUCHKA]

ищи синтаксический анализ. Есть описание в книге "Полный справочник С++" автор Шилдт. где то в последних главах описан

[Alex11223]

Сообщение от alexzk

В общем случае, если вы что-то делает 2 раза - это должна быть отдельная функция.

Главное не забывать, что это не единственное предназначение функций и создавать их стоит и во многих других случаях.

Я шарю еще больше, да?

[min@y™]

Сообщение от Alex11223

Главное не забывать, что это не единственное предназначение функций и создавать их стоит и во многих других случаях.
Я шарю еще больше, да?

[pu4koff]

Дичь какая-то. Найдите в интернетике готовую реализацию обратной польской нотации и сдайте лабу