do do not do do do do do do do this program.이 프로그램을 수리 해 주실 수 있습니까?

Question

이 프로그램을 고칠 수 있습니까? 그것은 말할 수 없습니다.

int main() 
{ 
    using namespace std; 
    int x, y, z, a; 
    cout << "Please enter a number" << endl; 
    cin >> x; 
    cout << "Please enter another number" << endl; 
    cin >> y; 
    cout << "What do you want to do with these numbers?" << endl; 
    cout << "1 = +" << endl; 
    cout << "2 = -" << endl; 
    cout << "3 = *" << endl; 
    cout << "4 = /" << endl; 
    cin >> a; 

    do { 
     z = add(x, y); 
    } while (a == 1); 

    do { 
     z = sub(x, y); 
    } while (a == 2); 

    do { 
     z = mul(x, y); 
    } while (a == 3); 

    do { 
     z = dis(x, y); 
    } while (a == 4); 

    cout << z; 
    return 0; 
} 

do while 문을 사용해 보았지만 제대로 작동하지 않습니다. --- EDIT --- 추가 된 들여 쓰기가 추가되었는데, 왜 그렇게 분무합니까?

Answer 1

당신은 그 입력하려고 입력 어느 옵션이 할-while 루프 결코 종료 때문에 조건은 항상 만족 될 것입니다. 전의. 내가 1을 입력하면

do { 
    z = add(x, y); 
} while (a == 1); 

이 표시되고 a는 항상 1이므로 종료하지 않습니다. 다른 조건에서도 마찬가지입니다. 대신 switch 문을 사용할 수 있습니다. 예 :

switch(a) { 
    case 1 : z = add(x, y); 
      break; 
    case 2 : z = sub(x, y); 
      break; 
    case 3 : z = mul(x, y); 
      break; 
    case 4 : z = div(x, y); 
      break; 
    default : cout<<"Please choose a valid option to proceed."; 
} 
cout << z; 
return 0; 

또한 이러한 경우 do-while을 사용하지 마십시오. 루프를 빠져 나가더라도 do-while은 들어 와서 조건을 하위 순차적으로 확인하기 때문에 항상 (x, y)로 나뉩니다. 당신은 그러나이 같은 while 루프 뭔가를 사용할 수 있습니다

while(a==1) { 
    z = add(x, y); 
    a = -1; 
} 

while(a==2) { 
    z = sub(x, y); 
    a = -1; 
} 

while(a==3) { 
    z = mul(x, y); 
    a = -1; 
} 

while(a==4) { 
    z = div(x, y); 
    a = -1; 
} 

Answer 2

귀하의 코드가 무한 루프에 진입

, 그래서

int main() 
{ 
using namespace std; 
int x, y, z, a; 
do 
{ 
cout << "Please enter a number" << endl; 
cin >> x; 
cout << "Please enter another number" << endl; 
cin >> y; 
cout << "What do you want to do with these numbers?" << endl; 
cout << "1 = +" << endl; 
cout << "2 = -" << endl; 
cout << "3 = *" << endl; 
cout << "4 = /" << endl; 
cin >> a; 
switch(a) 
{ 
    case 1: 
    z = add(x, y); 
    break; 
    case 2: 
    z = sub(x, y); 
    break; 
    case 3: 
    z = mul(x, y); 
    break; 
    case 4: 
    z = dis(x, y); 
    break; 
} 
cout << z; 
} while(1 >= a <= 4) 
return 0; 
} 

lokoko의 제안에 따라 수정처럼 수정

Answer 3

문 :

while(1 >= a <= 4) 

는 [잘못 아니라, 그것은 올바른 C 구문이다, 그러나 않기 때문에 그것은 당신이 원하는 것을 가장 가능성이 아니에요 ] :

while((1 >= a) <= 4); 

그리고 (1 >= a)의 결과 중 하나 0 또는 1 때문에

는 항상 <= 4입니다. 당신은 할 필요가 :

이

while ((1 >= a) && (a <= 4));