2016-11-28 6 views
-4

私は現在作業中であり、ループを追加するのに役立つこのプログラムを持っています。その単語は、パズルゲームを推測し、それぞれのプレイヤーは間違った手紙を推測するまで、できるだけ多くの手紙を推測できるはずです。その後、次のプレイヤーに移動します。 今は各プレイヤーに手紙を1回だけ尋ねてから次のプレイヤーに行くなどしてプログラムを終了します。 アドバイスをいただければ幸いです。 これまでのコードです:私のJAVAコード内のループに関するアドバイス

public class PuzzleGuess{ 

public static void main(String []args){ 

    System.out.println("Welcome!!"); 

    System.out.println("Availble letters:"); 

    String[] letter = new String[]{"A", "B", "C", "D", "E", "F", "G", "H", "I", "J", "K", "L", "M", "N", "O", "P", "Q", "R", "S", "T", "U", "V", "W", "X", "Y", "Z"}; 

    for(int a=0;a<26;a++) 

     System.out.print(letter[a]); 

     System.out.println(""); 

    System.out.println("_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _"); 

    String[] Guess = new String[]{"S", "W", "E", "E", "T", "H", "O", "M", "E", "A", "L", "A", "B", "A", "M", "A"}; 

    System.out.println("No of players:"+3); 

    int ch,flag=0,r; 

    String s; 

    Scanner scan = new Scanner(System.in); 

    for(int i=1;i<=3;i++) 

    { 

     System.out.println("players:"+i); 

     System.out.println("Select any one 1) spin wheel 2) Guess"); 

     ch=scan.nextInt(); 

     if(ch==1) 

     { 

      Random t = new Random(); 

      r=t.nextInt(10)+1; 

      if(r==1) 

      { 

       System.out.println("you are landed on $100."); 

      } 

       else if(r==2) 

       { 

       System.out.println("you are landed on $300."); 

       } 

       else if(r==3) 

       { 

       System.out.println("you are landed on $500."); 

      } 

       else if(r==4) 

       { 

       System.out.println("you are landed on $700."); 

       } 

       else if(r==5) 

       { 

       System.out.println("you are landed on $900."); 

       } 

       else if(r==6) 

       { 

       System.out.println("you are landed on $2000."); 

       } 

       else if(r==7) 

       { 

       System.out.println("you are landed on $3000."); 

       } 

      else if(r==8) 

       { 

       System.out.println("you are landed on $5000."); 

       } 

       else if(r==9) 

       { 

       System.out.println("you are landed on $-1000."); 

       } 

       else 

       { 

       System.out.println("you are landed on $0."); 

       } 

     } 

     else if(ch==2) 

     { 

      System.out.println("Select your letter from available letters "); 

      s = scan.next(); 

      for(int g=0;g<16;g++) 

      { 



       if(s.equals(Guess[g])) 

       { 

        flag=1; 



       } 

      } 

      if(flag==1) 

      { 

       System.out.println("correct"); 

       for(int a=0;a<26;a++) 

       { 

        if(s.equals(letter[a])) 

        { 

         letter[a]=null; 

        } 

       } 

       for(int a=0;a<26;a++) 

       System.out.print(letter[a]); 

       System.out.println(""); 



      } 

      else 

      { 

       System.out.println("Incorrect"); 
+1

コードに100行の空白を入れてコードを投稿しないでください。読みにくいです。 – khelwood

+0

_Anyのアドバイスは、StackOverflowの話題にならない種類の質問です。この質問がそこに合うかもしれないので、あなたの質問が自分のサイトの話題に関する要件に合っているかどうかを見るために、CodeReview Stack Exchangeを検討することを検討するべきです。 – Lexi

答えて

0

まず、コードを移動するオブジェクトとメソッドを作成します。 コードを分かりやすくするために、コードを分割する必要があります。

関連する問題