这个秋千表模型代码设计得很糟糕吗？

Question

上下文：我有一个基于 clojure 的填字游戏应用程序，其主 ui 是一个 JTabbedPane，带有两个选项卡、一个网格和一个线索表。线索表是线索向量的视图，但向量本身并不是数据的权威存储，而是通过 (active-cluelist) 函数从几个内部数据结构动态生成，由选择的线索选项卡触发。

这就是线索表的实现：

(def cluelist [])
(def update-cluelist)
(def model)

(defn make []
  (let [column-names ["Sq" "Word" "Clue"]
        column-widths [48 200 600]
        table-model (proxy [AbstractTableModel] []
                      (getColumnCount [] (count column-names))
                      (getRowCount [] (count cluelist))
                      (isCellEditable [row col] (= col 2))
                      (getColumnName [col] (nth column-names col))
                      (getValueAt [row col] (get-in cluelist [row col]))
                      (setValueAt [s row col]
                                  (let [word (get-in cluelist [row 1])]
                                    (add-clue word s) ; editing a cell updates the main clue data
                                    (def cluelist (assoc-in cluelist [row 2] s))
                                    (. this fireTableCellUpdated row col))))
        table (JTable. table-model)
        ]

; some pure display stuff elided

(def model table-model)
)

(defn update-cluelist []
  (def cluelist (active-cluelist))
  (.fireTableDataChanged model))

另一个讨论中有人指出，手动调用 fireTableDataChanged 是 (update-cluelist) 的主要代码味道，因为没有任何东西TableModel 类之外应该调用该方法。但是，我认为这是从外部源动态生成表的不可避免的结果。这些文档并没有太大帮助 - 他们指出

您的自定义类只需调用以下之一 每次表数据更改时 AbstractTableModel 方法 外部来源。

它隐含地假设 CustomTableModel 类是数据的权威来源。

这里还有一点 clojure/java 阻抗不匹配 - 在 java 中，我会让 cluelist 和 update-cluelist 成为我的 TableModel 的私有成员和方法，而在 clojure 中，cluelist 和表模型是 update-cluelist 可以访问的动态作用域变量。

我的主要问题是没有很多 clojure/swing 代码可供我参考以获取最佳实践。有人对执行此操作的最佳方法有任何建议吗？

Answer 1

建议：使用原子作为线索列表。不断地重新定义线索列表并不是表示可变数据的正确方法。老实说，我希望它在您第二次定义线索列表时抛出异常。

如果您使用原子作为线索列表，则可以从观察者调用 fireTableDataChanged 方法，而不是手动调用它。这意味着无论何时（何地）您更改原子，fireTableDataChanged 都会自动调用，无需显式调用。

def 的问题在于，多次调用 def 在多线程环境中效果不佳，并且 Clojure 尝试将所有内容默认为相当线程安全。据我了解，使用 var 的“正确”方法是单独保留其根绑定（即，不要再次调用 def）并使用 binding 如果您需要在本地更改它。 def 可能按照您使用它的方式工作，但是该语言被设置为在这种情况下支持原子、引用或代理，并且这些在大多数情况下可能会工作得更好（即您得到观察者）。另外，如果您稍后添加线程，则完全不需要担心它们。