編集履歴

質問編集履歴

冷静になれば自分でもわかるような点がありましたので変更しましが以前windowは真っ白のままでした。加えてcombinationsクラスのソースコードと概要を加えておきました。

2017/09/27 15:10

投稿

スコア20

test CHANGED Viewed

	@@ -1 +1 @@
1	- ゲー~~ムプログラムの実行につ~~いて
1	+ JFrameにコンポーネントが追加されず真っ白い状態となる

test CHANGED Viewed

@@ -1,9 +1,11 @@
-四目並べを行うプログラムを作ったのですがうまく起動してくれません。修正箇所やその他改善点のご指摘をお願い致します。
+四目並べを行うプログラムを作ったのですがうまく起動してくれません。具体的には正しくコンポーネントが追加されていないようで真っ白になってしまします。コンポーネントを追加する順序にミスがあるのでしょうか...?自力では解決しなかったためご指摘のほどお願いします。また、ほかの点で改善点があれば容赦なくお願いします。
 ```Java
 import java.awt.Color;
+import java.awt.Dimension;
 import java.awt.GridLayout;
 import java.awt.event.ActionEvent;
@@ -32,7 +34,7 @@
 	private JPanel backpane;
-	private JLabel tellturn;
+	private JLabel telllimit;
 	private JLabel tellcolor;
@@ -60,12 +62,12 @@
 		jf.setVisible(true);
 	}
 	public GameScreen(int n){
 		/*
 		 * 画面生成
@@ -76,7 +78,7 @@
 		super();
-		//backpane.setPreferredSize(new Dimension(930,700));
+		this.setPreferredSize(new Dimension(930,700));
 		this.setLayout(null);
@@ -88,7 +90,7 @@
 		tellcolor = new JLabel();
-		tellcolor.setBackground(Color.red);
+		tellcolor.setBackground(getPlayer_c());
 		tellcolor.setBounds(0, 0, 200, 200);
@@ -98,19 +100,17 @@
 			//残り時間表示のためのパネル
-		tellturn = new JLabel();
+		telllimit = new JLabel();
-		tellturn.setBorder(new BevelBorder(BevelBorder.RAISED, Color.white, Color.black));
+		telllimit.setBorder(new BevelBorder(BevelBorder.RAISED, Color.white, Color.black));
-		tellturn.setBounds(200, 0, 200, 430);
+		telllimit.setBounds(200, 0, 200, 430);
-		tellturn.setForeground(Color.WHITE);
+		telllimit.setForeground(Color.WHITE);
-		String text = getLabeltext();
-		tellturn.setText(text);
-		this.add(tellturn);
+		this.add(telllimit);
@@ -148,9 +148,9 @@
-			//ゲーム開始
+		//ゲーム開始
-		GameSystem gs = new GameSystem(15);
+		s = new GameSystem(15);
 	}
@@ -168,25 +168,285 @@
 	}
+	public void setTelllimittext(int second){
+		String s= "残り時間　"+ second+"秒";
+		telllimit.setText(s);
+	}
+	@Override
+	public void actionPerformed(ActionEvent e) {
+		String coordinate = e.getActionCommand();
+		for(int k=0;k<cn;k++){
+			for(int j = 0;j<cn;j++){
+				String number = k+"-"+j;
+				if(coordinate.equals(number)){
+					int order = k*20 + j+1;	//入力の番号を作成
+					if(play_turn == true){
+						player_c = Color.RED;
+						s.addA(order);
+					}else{
+						player_c = Color.BLUE;
+						s.addB(order);
+					}
+					changeTurn();
+					if(k == 19){					//入力が底の時
+						processEnter(order,k,j,player_c);
+					}else{							//入力が上部の時
+						if(enter[order+20] == false){
+							System.out.println("無効な入力です");
+						}else{
+							processEnter(order,k,j,player_c);
+						}
+					}
+					s.createCombinations();
+				}
+			}
+		}
+		return;
+	}
-	public String getLabeltext(){
+	public Color getPlayer_c(){
-		//残り時間を表示するためのラベル
-		String text;
 		if(play_turn == true){
-			text = "赤";
+			player_c = Color.RED;
 		}else{
-			text = "青";
+			player_c = Color.BLUE;
 		}
+		return player_c;
+	}
+	public void changeTurn(){
+		if(play_turn == true){
+			player_c = Color.RED;
+			tellcolor.setBackground(Color.red);
+			play_turn = false;
+		}else{
+			player_c = Color.BLUE;
+			tellcolor.setBackground(Color.BLUE);
+			play_turn = true;
+		}
+		tellcolor.setBackground(getPlayer_c());
+	}
+	/*	色塗り処理	*/
+	public void processEnter(int order,int k,int j,Color player_c){
+		/* 過去に入力があればtrueなければfalse */
+		if( enter[order] == true){
-		String s= "残り時間　"+"秒";
+			System.out.println("既に入力されています");
-		return s;
+			return;
+		}else{
+			y_coordinate.get(k).get(j).setBackground(player_c);
+			enter[order] = true;
+		}
+	}
+}
+```
+```Java
+import java.awt.event.ActionEvent;
+import java.awt.event.ActionListener;
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.Iterator;
+import java.util.List;
+import javax.swing.Timer;
+public class GameSystem implements ActionListener{
+	private Timer timer;
+	private int sec;
+	private Combinations<Integer> c ;
+	private GameScreen gs ;
+	private Screen sc;
+	private ArrayList<Integer> number_a = new ArrayList<Integer>();	//player1の入力
+	private ArrayList<Integer> number_b = new ArrayList<Integer>();	//player2の入力
+	private boolean finish = false;
+	public GameSystem(int limit){
+		sec = 0;
+		timer = new Timer(limit*1000,this);
+		timer.start();
+	}
+	public void createCombinations(){
+		if(GameScreen.play_turn == !true){
+			if(number_a.size() > 3){
+				c = new Combinations<Integer>(number_a,4);
+				//con = c.count(number_a.size(),4);
+				judgeWinner();
+			}else{
+				return;
+			}
+		}else{
+			if(number_b.size() > 3){
+				c = new Combinations<Integer>(number_b,4);
+				//con = c.count(number_b.size(),4);
+				judgeWinner();
+			}else{
+				return;
+			}
+		}
+	}
+	/*	勝者判定	*/
+	public void judgeWinner(){
+		Iterator<List<Integer>> i = c.iterator();
+		while(i.hasNext()){
+            List<Integer> t = i.next();
+            Collections.sort(t);			//小さい順に並び替え
+            Collections.reverse(t);			//大きい順へ
+            if(t.get(0)-t.get(3)==3 && t.get(0)-t.get(1)==1	||
+               t.get(0)-t.get(1)==20 && t.get(1)-t.get(2)==20 && t.get(2)-t.get(3)==20 ||
+               t.get(0)-t.get(1)==21 && t.get(1)-t.get(2)==21 && t.get(2)-t.get(3)==21 ||
+               t.get(0)-t.get(1)==19 && t.get(1)-t.get(2)==19 && t.get(2)-t.get(3)==19 ){
+            	finish = true;
+            	System.out.println("ゲーム終了");
+            }
+		}
+	}
+	public void addA(int n){
+		number_a.add(n);
+	}
+	public void addB(int n){
+		number_b.add(n);
 	}
@@ -196,290 +456,270 @@
 	public void actionPerformed(ActionEvent e) {
-		String coordinate = e.getActionCommand();
-		for(int k=0;k<cn;k++){
-			for(int j = 0;j<cn;j++){
-				String number = k+"-"+j;
-				if(coordinate.equals(number)){
-					int order = k*20 + j+1;	//入力の番号を作成
-					if(play_turn == true){
+		if(finish == true){
-						player_c = Color.RED;
-						s.addA(order);
-					}else{
-						player_c = Color.BLUE;
-						s.addB(order);
-					}
-					changeTurn();
+			timer.stop();
-					if(k == 19){					//入力が底の時
-						processEnter(order,k,j,player_c);
-					}else{							//入力が上部の時
-						if(enter[order+20] == false){
-							System.out.println("無効な入力です");
-						}else{
-							processEnter(order,k,j,player_c);
-						}
-					}
-					s.createCombinations();
-				}
-			}
 		}
+	    if (sec == 15){
+	         sec = 0;
+	         gs.changeTurn();
+	    }else{
+	 		gs.setTelllimittext(sec);
-		return;
+	         sec++;
-	}
+	    }
-	public void changeTurn(){
-		if(play_turn == true){
-			player_c = Color.RED;
-			//s.addA(order);
-			play_turn = false;
-		}else{
-			player_c = Color.BLUE;
-			//s.addB(order);
-			play_turn = true;
-		}
-		tellturn.setText(getLabeltext());
-	}
-	/*	色塗り処理	*/
-	public void processEnter(int order,int k,int j,Color player_c){
-		/* 過去に入力があればtrueなければfalse */
-		if( enter[order] == true){
-			System.out.println("既に入力されています");
-			return;
-		}else{
-			y_coordinate.get(k).get(j).setBackground(player_c);
-			enter[order] = true;
-		}
 	}
 }
 ```
 ```Java
-import java.awt.event.ActionEvent;
-import java.awt.event.ActionListener;
+/*http://kuidaored.hatenablog.com/entry/20101120/1290268944様より引用
+*組み合わせをすべて列挙するプログラム
+*/
 import java.util.ArrayList;
-import java.util.Collections;
+import java.util.Arrays;
 import java.util.Iterator;
 import java.util.List;
-import javax.swing.Timer;
-public class GameSystem implements ActionListener{
-	private Timer timer;
-	private int sec;
-	private Combinations<Integer> c ;
-	private GameScreen gs ;
-	private Screen sc;
-	//private int counttime;
-	private ArrayList<Integer> number_a = new ArrayList<Integer>();	//player1の入力
-	private ArrayList<Integer> number_b = new ArrayList<Integer>();	//player2の入力
-	private boolean finish = false;
-	public GameSystem(int limit){
-		sec = 0;
-		timer = new Timer(limit*1000,this);
-		timer.start();
-	}
-	public void createCombinations(){
-		if(GameScreen.play_turn == !true){
-			if(number_a.size() > 3){
-				c = new Combinations<Integer>(number_a,4);
-				//con = c.count(number_a.size(),4);
-				judgeWinner();
-			}else{
-				return;
-			}
-		}else{
-			if(number_b.size() > 3){
-				c = new Combinations<Integer>(number_b,4);
-				//con = c.count(number_b.size(),4);
-				judgeWinner();
-			}else{
-				return;
-			}
-		}
-	}
-	/*	勝者判定	*/
-	public void judgeWinner(){
-		Iterator<List<Integer>> i = c.iterator();
-		while(i.hasNext()){
-            List<Integer> t = i.next();
-            Collections.sort(t);			//小さい順に並び替え
-            Collections.reverse(t);			//大きい順へ
-            if(t.get(0)-t.get(3)==3 && t.get(0)-t.get(1)==1	||
-               t.get(0)-t.get(1)==20 && t.get(1)-t.get(2)==20 && t.get(2)-t.get(3)==20 ||
-               t.get(0)-t.get(1)==21 && t.get(1)-t.get(2)==21 && t.get(2)-t.get(3)==21 ||
-               t.get(0)-t.get(1)==19 && t.get(1)-t.get(2)==19 && t.get(2)-t.get(3)==19 ){
-            	finish = true;
-            	System.out.println("ゲーム終了");
+public class Combinations<T> implements Iterator{
+    private List<List<T>> combinations;
+    private List<T> list;
+    private int[] index;
+    private boolean[] visited;
+    private int r;
+    private boolean overHalf;
+    private Iterator<List<T>> iterator;
+    public Combinations(List<T> l,int r){
+        if(l.size() < 1 || r < 1 || l.size() < r){
+            throw new IllegalArgumentException();
+        }
+        this.combinations = new ArrayList<List<T>>();
+        this.list = l;
+        this.r = r;
+        if(this.r == l.size()){
+            this.combinations.add(list);
+        }
+        else{
+            if(this.r > list.size() / 2){
+                this.r = list.size() - this.r;
+                this.overHalf = true;
             }
-		}
-	}
-	public void addA(int n){
-		number_a.add(n);
-	}
-	public void addB(int n){
-		number_b.add(n);
-	}
-	@Override
-	public void actionPerformed(ActionEvent e) {
-		if(finish == true){
-			timer.stop();
-			e.setSource(this);
-			sc.actionPerformed(e);
-		}
-	    if (sec == 15){
-	         sec = 0;
-	         gs.changeTurn();
-	    }else{
-	         sec++;
-	    }
-	}
+            this.index = new int[this.r];
+            this.visited = new boolean[list.size()];
+            this.compute(0);
+        }
+        this.iterator = this.combinations.iterator();
+    }
+    public Combinations(T[] array, int r){
+    	this(Arrays.asList(array),r);
+    }
+    private void compute(int n){
+        if(n == this.r){
+            List<T> combination = new ArrayList<T>();
+            if(overHalf){
+                for(int i = 0;i < this.list.size();i++){
+                    boolean skip = false;
+                    for(int j = 0;j < this.index.length;j++){
+                        if(i == this.index[j]){
+                            skip = true;
+                        }
+                    }
+                    if(skip){
+                        continue;
+                    }
+                    combination.add(list.get(i));
+                }
+            }
+            else{
+                for(int i = 0;i < this.index.length;i++){
+                    combination.add(list.get(index[i]));
+                }
+            }
+            this.combinations.add(combination);
+        }
+        else{
+            for(int i = 0;i < this.list.size();i++){
+                if(n == 0 || !this.visited[i] && index[n - 1] < i){
+                    this.visited[i] = true;
+                    this.index[n] = i;
+                    this.compute(n + 1);
+                    this.visited[i] = false;
+                }
+            }
+        }
+    }
+    public List<List<T>> getCombinations(){
+        return this.combinations;
+    }
+    public static int count(int n, int r) throws IllegalArgumentException{
+        if(n < 1 || r < 1 || n < r){
+            throw new IllegalArgumentException();
+        }
+        int res = 1;
+        for(int i = n;i > r;i--){
+            res *= i;
+        }
+        for(int i = n - r;i > 1;i--){
+            res /= i;
+        }
+        return res;
+    }
+    public List<T> next(){
+        return this.iterator.next();
+    }
+    public boolean hasNext(){
+        return this.iterator.hasNext();
+    }
+    public Iterator<List<T>> iterator(){
+        return this.iterator;
+    }
+    public void remove(){
+    }
 }
 ```