visual-c++ - このコードを短くする方法はありますか?

Question

これは、Visual C++ の Q&A ゲームの私のコードです

             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!!");
             }

このコードを短縮するものはありますか? 配列の使用を検討しますか?

Answer 1

私は Windows プログラマーではありません。あなたの のタイプが何かわかり 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

とを繰り返し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!!"); 

Answer 3

switch caseステートメントを使用してみてください。