Question

我要在JPanel中添加二个JLabel，然后将JPanel添加到JFrame中，JPanel是匿名的，这样写，对吗？

this.add(new JPanel()
{ JPanel.this.add(new JLabel("aaa："));
JPanel.this.add(new JLabel("bbb："));
});
错在哪了？求指教

Answer 1

1. 首先，我个人实在是很不赞同写这种大多数人看不懂的code.

2.这种才是通常会用到的写法
JPanel panel = new JPanel();
panel.add(new JLabel("aaa："));
panel.add(new JLabel("bbb："));

3.如果有什么很特殊的原因单纯是要使JPanel匿名 (虽然我暂时想不太到), 可以考虑使用LayoutManager,比如像下面这样:

import java.awt.GridLayout;

GridLayout layout = new GridLayout();
layout.addLayoutComponent("label2", new JLabel("aaa:"));
layout.addLayoutComponent("label1", new JLabel("bbb:"));

this.add(new JPanel(layout));

4. 如果必须要模仿你那种很“高级”的格式，下面这个才是正确的：
this.add(new JPanel() {
private static final long serialVersionUID = 1L;
{
this.add(new JLabel("aaa："));
this.add(new JLabel("bbb："));
}
});

5.如果不用 serialVersionUID 而是用@SuppressWarning或者干脆无视warning的话, 外面那一层留给variable的bracket也不能去掉, 像下面这段在读code时看到的话我会想诅咒作者的方式：
this.add(new JPanel() {{
this.add(new JLabel("aaa："));
this.add(new JLabel("bbb："));
}});

6.所以总的来说，你的code错误只有两个，第一是少一层{}，第二是不该用JPanel.this,直接用this就可以了。

最后奉劝一句，写出来的code别人和自己都一目了然看得很清楚很明白的才是好的code.