我在想,做这样的事情会更有效率:
setVisible(false) // if the component is invisible
或者像这样:
if(isVisible()){
setVisible(false)
}
答案 0 :(得分:6)
这与效率无关。使用第一个,它更简单,可能已包含setVisible()方法内的可见性检查。
当你编写代码时,不要试图考虑效率,特别是在像这样荒谬的微不足道的情况下。你在多GHz计算机上运行,所以你只是在这样的微优化上浪费你的时间。
答案 1 :(得分:3)
setVisible()
已调用isVisible()
,因此添加冗余呼叫效率不高。在任何情况下,它都不可能是任何一种性能瓶颈。
答案 2 :(得分:0)
你的第二个例子有意义的唯一情况是在打开和关闭可见性的情况下。
if(isVisible()){
setVisible(false);
}else{
setVisible(true);
}