2016-04-11 130 views
0

local variable 'newElement' is redundantというコンパイラ警告が表示されます。次のコードをより効率的に記述して、この警告のコードを削除するにはどうすればよいですか?コンパイラの警告:ローカル変数は冗長です

Element newElement = new Element(left,elements.next); 
elements.next = newElement; 
+3

'elements.next = new Element(left、elements.next);'私はそれが改善であると確信していません。 –

答えて

1

は、リストa -> b -> cを持っているし、今あなたがそれにdを追加したいと言います。あなたが書いたものはdを指して、dを指し示します。

このブロック

Element newElement = new Element(left,elements.next); 
    elements.next = newElement; 

は本当に意味の束がありません

elements.next = new Element(left,elements.next); 

に相当します。

リンクされたリストのようなものを作ろうとしている場合は、

Element newElement = new Element(left,null); 
elements.next = newElement; 
+0

はい私はリンクリストで使用しています。ありがとうございました。 – Doom3030

+0

@ Doom3030私は解決策があなたと関連していると思う –

1

時には冗長な変数を持つことが(おそらくコード明確にするため)ことが望ましいです。このような場合、警告を抑制することができます。

@SuppressWarnings("UnnecessaryLocalVariable") 
関連する問題