java - 这是JSlider中的错误吗？

我在使用JSlider类时遇到了一些问题-特别是带有刻度标签。

第一次使用setMajorTickSpacing和setMinorTickSpacing时，一切都会按预期进行。但是，随后对setMajorTickSpacing的调用会更新刻度线，但不会更新标签。我编写了一个简单的示例来演示此行为:

import java.awt.event.*;
import javax.swing.*;

public class SliderTest {
    public static void main(String args[]) {
        JFrame frame = new JFrame();
        frame.addWindowListener(new WindowAdapter() {
            public void windowClosing(WindowEvent we) {
                System.exit(0);
            }
        });
        frame.setSize(300, 250);

        JSlider slider = new JSlider(0, 100, 0);
        slider.setMajorTickSpacing(10);
        slider.setMinorTickSpacing(1);
        slider.setPaintLabels(true);
        slider.setPaintTicks(true);

        slider.setMajorTickSpacing(25);
        slider.setMinorTickSpacing(5);

        frame.add(slider);
        frame.pack();
        frame.setVisible(true);
    }
}

有两种简单的解决方法似乎可以解决此问题-在第二次调用slider.setLabelTable(null)之前使用slider.setLabelTable(slider.createStandardLabels(25))或setMajorTickSpacing。鉴于此，标签表似乎没有正确更新。

我不确定这是否是预期的行为。我的第一个直觉是，更新刻度间隔也应同时更新标签，但是也存在将两者分开的论点。

所以我想知道它是什么-这是JSlider中的错误还是预期的行为？如果这是预期的行为，那么做出该选择的突出原因是什么？

最佳答案

通过查看setMajorTickSpacing源代码，您可以轻松发现此问题的原因:

public void setMajorTickSpacing(int n) {
    int oldValue = majorTickSpacing;
    majorTickSpacing = n;
    if ( labelTable == null && getMajorTickSpacing() > 0 && getPaintLabels() ) {
        setLabelTable( createStandardLabels( getMajorTickSpacing() ) );
    }
    firePropertyChange("majorTickSpacing", oldValue, majorTickSpacing);
    if (majorTickSpacing != oldValue && getPaintTicks()) {
        repaint();
    }
}

如果您两次调用此方法-labelTable值将不再为null，也不会更新。根据方法的注释，这可能是预期的行为:

 * This method will also set up a label table for you.
 * If there is not already a label table, and the major tick spacing is
 * {@code > 0}, and {@code getPaintLabels} returns
 * {@code true}, a standard label table will be generated (by calling
 * {@code createStandardLabels}) with labels at the major tick marks.

因此，每次要更新标签时，都必须手动更新标签(除非您用自己的标签来进行更新来覆盖此方法)。