0

ボタンとテキストフィールドを含むフレームがあります。

私のporpuseはテキストフィールドからID番号を読み取り、ボタンをクリックすると、テーブルにそのIDとその名前とマークが表示されます。

私の「UserSearchFile.txt」はこれです：

12 joe 120
14 ted 220
19 alex 560
22 julia 668

私のjButton6ActionPerformedコード全体は次のとおりです。

int idS=Integer.parseInt(jTextField2.getText());
      final Vector data = new Vector();
      final Vector column = new Vector();
 File f=new File("D:\\UserSearchFile.txt");
    try{
    FileReader fr=new FileReader(f);
    BufferedReader br1=new BufferedReader(fr);
    String s;
while ((s = br1.readLine()) != null) {
                String[] st = s.split(" ");
                String id = st[0];
                String name = st[1];
                String mark = st[2];
                if (id.equals(String.valueOf(idS))) {
try {
            String line2;
FileInputStream fis = new FileInputStream("D:\\UserSearchFile.txt");
        BufferedReader br2 = new BufferedReader(new InputStreamReader(fis));
      StringTokenizer st1 = new StringTokenizer(br2.readLine(), " ");
              while (st1.hasMoreTokens())
               column.addElement(st1.nextToken());
                while ((line2 = br2.readLine()) != null) {
                       StringTokenizer st2 = new StringTokenizer(line2, " ");
                        while (st2.hasMoreTokens())
                                data.addElement(st2.nextToken());
                }
                br2.close();
        } catch (Exception e) {
                e.printStackTrace();
        }
                }
            }
        } catch (IOException ex) {
           Logger.getLogger(MainFrame1.class.getName()).log(Level.SEVERE, null, ex);
        }

 jTable1.setModel(new AbstractTableModel() {

        public int getRowCount() {
           return data.size() / getColumnCount();
        }

        public int getColumnCount() {
            return column.size();
        }

        public Object getValueAt(int rowIndex, int columnIndex) {
            return (String) data.elementAt((rowIndex * getColumnCount())
                        + columnIndex);
        }
    });
    jTable1.setVisible(true);

私を助けて、きれいでシンプルな書き方を教えてください。

4

1 に答える 1

4

変数とメソッドに意味のある名前を付けることから始めます。jTable1、br1、br2およびjButton6ActionPerformedは使用できない名前です。

次に、複雑なメソッドを 2 つまたは 3 つの操作に分割してみたり、それ自体を 2 つまたは 3 つの操作に分割したりします。各操作はメソッド呼び出しにする必要があります。例えば：

private void readButtonClicked() {
    String id = idTextField.getText();
    Student student = findStudentWithId(id);
    showStudentInGUI(student);
}

private Student findStudentWithId(String id) {
    List<String> lines = readLinesInFile();
    List<Student> students = transformLinesIntoStudents(lines);
    Student studentWithId = findStudentWithId(students, id);
}

private Student findStudentWithId(List<Student> students, String id) {
    for (Student student : students) {
        if (student.getId().equals(id)) {
            return student;
        }
    }
    return null;
}

private List<Student> transformLinesIntoStudents(List<String> lines) {
    List<Student> students = new ArrayList<Student>(lines.size());
    for (String line : lines) {
        students.add(parseStudentLine(line);
    }
    return students;
}

...
于 2012-12-23T16:55:59.993 に答える