回答編集履歴

コード短縮

2020/03/31 15:44

投稿

スコア13355

answer CHANGED Viewed

@@ -218,29 +218,24 @@
 class DummyDatabaseAccessor implements DatabaseAccessor {
   private static String TEST[] = {"ABC","あいう","１２３"};
   private int nextPid = 0;
-  private Map<Integer,String> rows;
+  private Map<Integer,String> rows = new TreeMap<Integer, String>();
+  DummyDatabaseAccessor() {
+    for(String s : TEST) rows.put(nextPid++, s);
+  }
   @Override
   public MyTableModel selectAll() {
     MyTableModel tableModel = new MyTableModel(this);
-    if(rows == null) {
-      rows = new TreeMap<Integer, String>();
-      for(String s : TEST) {
-        rows.put(nextPid++, s);
-      }
-    }
     for(Map.Entry<Integer,String> entry : rows.entrySet()) {
       tableModel.addRow(entry.getKey(), entry.getValue());
     }
     return tableModel;
   }
   @Override
   public int insert(String text) {
     System.out.println("insert text="+text);
     rows.put(nextPid++, text);
     return 1;
   }
   @Override
   public int update(int pid, String text) {
     System.out.println("update pid="+pid+", text="+text);
@@ -248,7 +243,6 @@
     rows.put(pid, text);
     return 1;
   }
   @Override
   public int delete(int pid) {
     System.out.println("delete pid="+pid);