Java IO、ファイルからの読み取りと2文字目の配列への印刷

Question

私は、ゲームデザインクラスの課題に取り組んで、パックマンクローンを構築しています。

WWWWWWWWWWWWWWWWWWWWWWWWWWWW 
W............WW............W 
W.WWWW.WWWWW.WW.WWWWW.WWWW.W 
W*WWWW.WWWWW.WW.WWWWW.WWWW*W 
W.WWWW.WWWWW.WW.WWWWW.WWWW.W 
W..........................W 
W.WWWW.WW.WWWWWWWW.WW.WWWW.W 
W.WWWW.WW.WWWWWWWW.WW.WWWW.W 
W......WW....WW....WW......W 
WWWWWW.WWWWW.WW.WWWWW.WWWWWW 
WWWWWW.WWWWW.WW.WWWWW.WWWWWW 
WWWWWW.WW..........WW.WWWWWW 
WWWWWW.WW.WWWWWWWW.WW.WWWWWW 
WWWWWW.WW.WWWWWWWW.WW.WWWWWW 
..........WWWWWWWW.......... 
WWWWWW.WW.WWWWWWWW.WW.WWWWWW 
WWWWWW.WW.WWWWWWWW.WW.WWWWWW 
WWWWWW.WW..........WW.WWWWWW 
WWWWWW.WW.WWWWWWWW.WW.WWWWWW 
WWWWWW.WW.WWWWWWWW.WW.WWWWWW 
W............WW............W 
W.WWWW.WWWWW.WW.WWWWW.WWWW.W 
W*WWWW.WWWWW.WW.WWWWW.WWWW*W 
W...WW................WW...W 
WWW.WW.WW.WWWWWWWW.WW.WW.WWW 
WWW.WW.WW.WWWWWWWW.WW.WW.WWW 
W......WW....WW....WW......W 
W.WWWWWWWWWW.WW.WWWWWWWWWW.W 
W.WWWWWWWWWW.WW.WWWWWWWWWW.W 
W..........................W 
WWWWWWWWWWWWWWWWWWWWWWWWWWWW 

アイデアは、Java IOからの読者によって行ずつ、これが読み込まれていることである。

は現在、私はパックマン迷路

は下記を参照されたテキストファイルを持っていますパッケージを作成し、次に2次元配列を生成するために使用すると、配列内のデータを使ってペイントクラスを使用してイメージをどこに印刷するかをループで指定できます。

現在のところ、私の問題はペイント方法ですが、それは全く問題なく動作しているようですが、現時点で何が問題なのかはわかりません。誰かが私を正しい方向に向けることができますか？

（私のコードは、書式設定、私はまた、Java IOパッケージに新たなんだ、ここで必要なインデントによって、私は例外処理を見てきた最初の時間を少しを台無しにされています！）任意の助けを事前に

感謝！

//imports 
import java.awt.*; 
import java.io.*; 
import javax.swing.*; 


public class Maze extends JFrame 
{ 

//V.Decleration 
private static final Dimension WindowSize = new Dimension (600,600); 
static char[][] Amaze = new char[28][31]; 


//default constructor 
public Maze() 
{ 
this.setTitle("Pacman"); 
setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
Dimension screensize = java.awt.Toolkit.getDefaultToolkit().getScreenSize(); 
int x = screensize.width/2 - WindowSize.width/2; 
int y = screensize.height/2 - WindowSize.height/2; 
setBounds(x, y, WindowSize.width, WindowSize.height); 
setVisible(true); 
} 

public void paint (Graphics g) 
{ 

String line = null; 

try 
    { 
    BufferedReader reader = new BufferedReader(new FileReader("G:\\Game  Design\\Pacman\\src\\Maze.txt")); 


    for (int i=0; i<=31; i++) 
{ 
do 
{ 
    try 
    { 

    line=reader.readLine(); 
    for (int y=0; y<=28; y++) 
    { 

     Amaze[y][i]=line.charAt(y); 
    } 

    }catch (IOException e) { } 
} 

while (line!= null); 
    try 
     { 
     reader.close(); 
     } catch (IOException e) { } 
    } 
} catch (FileNotFoundException e) {} 

} 

//main 
public static void main (String [] args) 
{ 
Maze maze = new Maze(); 

for (int i=0;i<=28;i++) 
System.out.print(Amaze[i][31]); 
} 

} 

Answer 1

三件の提案：

1. 一度に一つだけ仕事をするあなたの機能を分離するためにあまり誤解を招く可能性があります。そのため、ファイルを読み取る機能と、結果をペイントする機能が必要です。
2. デバッグしようとしているときに、System.out.println()ステートメントをコードに追加してください。それは、それぞれの部品があなたが意図したことをしたかどうかを確認する良い方法です。たとえば、変数を読み取った後にline変数を印刷すると、ファイルを正しく読み取っているかどうかがわかります。
3. 例外を常に印刷してください。彼らはあなたに何が間違っていたのか、どこにどこに行くのかを伝えますこのコードは、ロードし、あなたの迷路印刷します、と述べ

：

import java.io.*; 

public class Read2DArray { 
    private final int WIDTH = 28; 
    private final int HEIGHT = 31; 

    private char[][] maze = new char[WIDTH][HEIGHT]; 

    public static void main(String[] args) { 
     Read2DArray array = new Read2DArray(); 
     array.loadFile("maze.txt"); 
     array.printArray(); 
    } 

    public void loadFile(String fname) { 
     try { 
      BufferedReader reader = new BufferedReader(new FileReader(fname)); 

      String line; 
      int col = 0, row = 0; 
      while((line = reader.readLine()) != null && row < HEIGHT) { 
       for(col = 0; col < line.length() && col < WIDTH; col++) { 
        maze[col][row] = line.charAt(col); 
       } 
       row++; 
      } 
      reader.close(); 
     } catch(IOException e) { 
      e.printStackTrace(); 
     } 
    } 

    public void printArray() { 
     for(int row = 0; row < HEIGHT; row++) { 
      for(int col = 0; col < WIDTH; col++) { 
       System.out.print(maze[col][row]); 
      } 
      System.out.println(); 
     } 
    } 
} 

Answer 2

ループには小さすぎる配列を作成しています。すなわち：

for (int i=0; i<=31; i++) 
for (int y=0; y<=28; y++) 

使用i<31とy<28、または[29][32]であるためにあなたの配列を増やす：new char[28][31];だけforループ27および30あなたの最大のインデックスを可能にするがあります。これらのいずれかがあなたの現在の問題を解決するはずです。