Check if variable null before assign to null?

一笑奈何 提交于 2019-12-19 02:49:19

问题


Is variable assignment expensive compared to a null check? For example, is it worth checking that foo is not null before assigning it null?

if (foo != null) {
     foo = null;
}

Or is this worrying about nothing?


回答1:


This is a micro-micro-optimization (and possibly something handled by the compiler anyways). Don't worry about it. You'll get a far greater return by focusing on your programs actual algorithm.

We should forget about small efficiencies, say about 97% of the time: premature optimization is the root of all evil. -- Donald Knuth




回答2:


This is actually (very, very slightly) less efficient. Variable assignments are roughly equivalent to null checks, plus there's an extra branch possible. Not that it makes much difference.

Or is this worrying about nothing?

You got it.




回答3:


I wouldn't worry about it - it's just extra lines of code to maintain. This is the sort of micro-optimization you should never do unless you have lots of documented proof that it's your bottleneck.




回答4:


First of all, it's micro optimization. So no need to worry much about it.

But to answer your question, you need to reduce it to just one line. (as all your code does is to set it to NULL).

foo = NULL;

Reasons being that,

Comparison is a much much costlier operation than assignment. (As comparison eats up many assembly instructions comparatively. Generally a subtraction and a comparison to zero or an XOR and comparison to zero). Assignment takes up fewer instructions.




回答5:


If you have a decent compiler they will generate identical code. If you have a crappy compiler the one with the if will be worse. On 2009 hardware assignments to variables are very cheap, and conditional branches can sometimes be expensive.




回答6:



foo = null;

if (foo != null)
   foo = null;

If I look at the second block code I would think that you only wanted to set the foo variable to null if it was not null before, and if I look at the first code I would think that you wanted to set the variable foo to null anyway.

I know this is because of the example you wrote, but in the end this kind of micro-optimization only adds confusion (it's not worth it).




回答7:


This will make your code so much harder to read that even if it was an optimization it wouldn't be worth the trouble.

And it's not an optimization. On most modern cpu's if statements are quite expensive.




回答8:


This will have little or no effect. I don't think you could even create a benchmark to demonstrate the difference.

In fact, some would argue that assigning to null at all is a code smell (see the PMD detector for NullAssignment):

Assigning a "null" to a variable (outside of its declaration) is usually bad form. Some times, the assignment is an indication that the programmer doesn't completely understand what is going on in the code. NOTE: This sort of assignment may in rare cases be useful to encourage garbage collection. If that's what you're using it for, by all means, disregard this rule :-)

In general, I'm personally leery of anything that attempts to encourage garbage collection (you almost always get effects that you didn't expect).



来源:https://stackoverflow.com/questions/583454/check-if-variable-null-before-assign-to-null

易学教程内所有资源均来自网络或用户发布的内容,如有违反法律规定的内容欢迎反馈
该文章没有解决你所遇到的问题?点击提问,说说你的问题,让更多的人一起探讨吧!