이 코드를 단축 할 방법이 있습니까?

Question

이것은

   if (this->answer->Text == "2"){ 
       this->question->Text = "2+2?"; 
      } 
      else if (this->answer->Text == "4"){ 
       this->question->Text = "3+3?"; 
      } 
      else if (this->answer->Text == "6"){ 
       this->question->Text = "4+4?"; 
      } 
      else if (this->answer->Text == "8"){ 
       this->question->Text = "Finished!"; 
      } 
      else { 
       MessageBox::Show("Wrong!!"); 
      } 

어떤이 코드가 단축하는 것입니다 Q &에 내 코드 비주얼 C++에 게임이다? 배열 사용을 고려하십니까?

Answer 1

내가 윈도우 프로그래머, 당신의 this->question->Text의 유형, 당신은 우리에게하지 않았다 무엇 모르겠어요하고, 그러나 std::string 또는 char *으로 변환 할 수있는 항목이면 작동 할 것입니다.

std::string t = this->answer->Text; 
this->question->Text = t == "2" ? "2+2?" 
        : t == "4" ? "3+3?" 
        : t == "6" ? "4+4?" 
        : t == "8" ? "Finished" 
        : ""; 
if (this->question->Text = "") MessageBox::Show("Wrong!!"); 

Answer 2

switch case 문을 사용하십시오.

Answer 3

if (this->answer->Text == 및 this->question->Text =을 반복하고 있습니다. 그것들을 한 번만 쓰고 상태와 응답을 std :: map에 유지하십시오.

업데이트 :

#include <map> 
#include <string> 

...

std::map<std::string,std::string> answers; 
answers["2"]="2+2"; // "configuration 
answers["4"]="3+3?"; 
//and so on 
std::string text=this->answer->Text; 
    // instead of if ... 
std::map<std::string,std::string>::const_iterator found=answers.find(text); 
if (found!=answers.end()) 
    this->question->Text = answers[text]; //once. Even better found->second 
else 
    MessageBox::Show("Wrong!!");