функция — C ++ Программа застряла в цикле

Таким образом, эта программа должна взять строку и преобразовать ее в обратную польскую запись, а затем сгенерировать для нее код сборки.

Например. Если бы я должен был ввести «х = у», программа вернет

«RPN: xy =»

«Код:
1 год
2 сто х «

Однако вместо этого программа возвращает бред и продолжает делать это до тех пор, пока не закончится память.

Вот функция ввода

void
getstring()
{
if(datafile) {
file.getline(str,241);
}else{
cout << "Enter a string, please \n\n";
cin.getline( str, 241);
}
nstring++;
}

И это те функции, которые работают с ним.

void
internalize()
{
int i, j;
static char inter[] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
0, 0, 0, 0, 0, 0, 0, 0, 1, 2, 6, 4, 0, 5, 0, 7,
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 3, 0, 0,
0,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,
23,24,25,26,27,28,29,30,31,32,33, 0, 0, 0, 0, 0,
0,34,35,36,37,38,39,40,41,42,43,44,45,46,47,48,
49,50,51,52,53,54,55,56,57,58,59, 0, 0, 0, 0, 0};
char ch, code, k, *p, *q;
cout << "internal form: \n\n";
k=0;
q=inform;
for(p=str;*p;p++){
*q++ = code = inter[(int)*p];
if(k+code == 0){
k = p-str+1;
ch = *p;
}
}
*q = 0;
for(i=j=0, p = inform;p++,j++;j<len){
cout << setw(3) << (int)*p;

if(++i == 16){
cout << '\n';
i = 0;
}
}
if (i !=0){
cout <<'\n';
}
if((err = (0<k))!=0){
cout << '\n**' << (int)k << "-th nonblank char <" <<ch<<"> is illegal. **\n";
}
}

void
makerpn()
{
static char pr[]={0,0,1,2,3,3,4,4};

char n, *p, *r, s, *t;
cout << "\nRPN:\n\n";
t = stak - 1;
r = rpn;

for (p = inform;p++ ; *p){
if(7 < (s = *p)){
*r++ = s;
//a
}else if(s == 1){
*++t = s;
} else{
while(1){
if(t<stak){
*++t = s;
break;
} else if(pr[s] <= pr[n = *t--]){
*r++ = n;
} else if(3 < n+s){
*++t = n;
*++t = s;
}
break;
}
}
while(stak <= t){
*r++ = *t--;
}
*r = '\0';
for(r=rpn;*r;r++){
cout << ext[(int)*r];
}
cout << "\n\n";
}}

void
gencode()
{
void emit(char, char);
char a,j,lop,n1,n2,*p,*t;
cout << "Generated symbolic code \n\n";
j = len = lop = 0;
t = p = rpn;

while(3 <(a = *++p)){
if(7 <a){
*++t = a;
if((++lop == 2) && (0<len)){
n2=*t--;

n1=*t--;
*++t=j+60;
*++t=n1;
*++t=n2;
emit(2,(j++)+60);
}
}else {
if(lop == 1){
emit(a,*t--);
}else{
if( 1 < lop){
n2 = *t--;
n1 = *t--;
emit(1,n1);
emit(a,n2);
}else {
if((a==4) || (a==6)){
n1 = *t--;
emit(a,n1);
}else {
n1=*t--;
emit(2,j+60);
emit(1,n1);
emit(a,j+60);
if( 59 < n1){
--j;}}}}}}
lop = 0;
}

Я предполагаю, что с петлями что-то не так, но я не уверен, что именно. Я настраивал их некоторое время, и все, что мне удалось сделать, это заставить его повторить Hp? вместо HW снова и снова.

0

Решение

for (p = inform;p++ ; *p) будет продолжаться до следующего значения p является nullptrи каждый раунд будет разыменованием p и откажитесь от результата — вы, вероятно, имели в виду это наоборот.

Похожий случай: for(i=j=0, p = inform;p++,j++;j<len) который будет продолжать работать до j++ становится 0 и отбросить результат сравнения j<len каждая итерация.

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

4

Другие решения


По вопросам рекламы [email protected]