2つの異なるクラスのオブジェクトを作成すると、JavaでStackOverflowがスローされます

Question

私はこの問題に苦労しています。つまり、OrderSearch.java（メインクラス）とCreateOrder.javaという2つのクラスがあります。私のメインclassにJTableがあり、行がダブルクリックされると、新しいテキストが開きます。CreateOrder.javaとjTableというテキストフィールドがあります。 CreateOrder.javaにSaveButtonがあり、クラスに加えられた変更が保存され、再びJTableが表示されます。

しかし、問題は私がSaveButtonがクリックされたときにいくつかのSQLクエリをリフレッシュするテーブル操作を実行することができないことです。 CreateOrderのsavebuttonをクリックすると、OrderSearchがリフレッシュされます。

問題：CreateOrder.javaでOrderSearch.javaクラスのオブジェクトを作成すると、私にスタックオーバーフローエラーが発生します。 Saveボタンでオブジェクトを作成すると、新しいフレーム全体が再度開きます。

OrderSearch.java

public class OrderSearch extends CreateOrder{ 
//declarations for label,text, and buttons 


public static void main(String[] args) { 

     EventQueue.invokeLater(new Runnable() { 
     public void run() { 
      try { 

       OrderSearch window = new OrderSearch(); 
       window.frmXraymanager.setVisible(true); 
      } catch (Exception e) { 
       e.printStackTrace(); 
      } 
     } 

    public OrderSearch() {  *Stack overflow error here* 
    initialize(); 
    } 

private void initialize() { 

    table = new JTable(); 
    scrollPane.setViewportView(table); 
    table.addMouseListener(new MouseAdapter() { 
     @Override 
     public void mouseClicked(MouseEvent e2) { 

      if (e2.getClickCount() == 2 && !e2.isConsumed()) { 
       e2.consume(); 
      try{ 
      int index = table.getSelectedRow(); 

      String table_Click = table.getModel().getValueAt(table.convertRowIndexToModel(index), 0).toString(); 


      String sql = "SELECT ID, Date, Place, UserName FROM TEST.dbo.Intern WHERE ID = '"+table_Click+"'"; 
      PreparedStatement pst = connection.prepareStatement(sql); 
      ResultSet rs = pst.executeQuery(); 

      if(rs.next()){ 
      String id = rs.getString("ID"); 
      String date = rs.getString("Date").toString(); 
      String place = rs.getString("Place"); 
      String uname = rs.getString("UserName"); 

     frameCreate.setVisible(true); //Frame from CreateOrder.java 

     Number.setText(id); // textfields from CreateOrder.java 
     String date1 = startDate; 
     DateFormat df = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss.S"); 
     Date date2 = df.parse(date1); 

     dateChooser.setDate(date2); 
     jobSite.setText(place); 
     uName.setText(uname); 
     Component.setText(component); 
     Remarks.setText(remarks); 
     rs.close(); 
     pst.close(); 
      } 
      catch(Exception e){ 
       JOptionPane.showMessageDialog(null, e); 
      } 
      } 
     } 
    }); 
    } 

    public void refresh() 
     { 
       query1 = "SELECT * FROM Test.dbo.Intern; 


      try(PreparedStatement pst = connection.prepareStatement(query1); 
      ResultSet rs = pst.executeQuery();){ 



      table.setModel(DbUtils.resultSetToTableModel(rs)); 
      table.setRowHeight(40); 

     } 
     catch(Exception e){ 

      e.printStackTrace(); 
     } 
    } 
    } 

CreateOrder.java

public class CreateOrder { 

    public CreateOrder() { 
    initialize(); 
    } 
OrderSearch one = new OrderSearch(); *Stack overflow error here*  

private void initialize() { 

button_Save = new JButton("Save"); 
    button_Save.addActionListener(new ActionListener() { 

     public void actionPerformed(ActionEvent arg0) { 
      *would like to add refresh() here* 

     } 
    }); 

} 
} 

は、どのように私は再びフレームを開かずに方法refresh()をアクセスもするCreateOrderにOrderSearchのオブジェクトを作成する必要がありますか？

ありがとうございます！

-Ajay

EDIT：

実際にエラーが

OrderSearch one = new OrderSearch(); 

や公共OrderSearch()に、私は完全にそれが無限ループに入るので、私はCreateOrder内のオブジェクトを呼び出すとき、それは理にかなって理解しています.java。しかし、CreateOrder.javaのOrderSearch.javaの内容にアクセスするには、stackoverflowエラーを取得せずに、または新しいフレームOrderSearch.java全体を再度開く必要はありませんか？

Answer 1

いいえ、現在の設定を前提とした方法はありません。これにより、コードをリファクタリングする必要があるという結論に至ります。主な問題は、コードでフレームを作成せずに必要なdbコマンドを実行する方法がないことです。 separate the engine from the uiが必要です。 UIイベントに縛られることなくビジネスロジックを実行できる必要があります。 UIはラッパーではなく、ビジネスロジックのユーザーである必要があります。すべてのビジネスロジックを別のクラスに移動し、適切な場所やイベントでuiから関連するメソッドを呼び出す必要があります。