Is it bad to do Inner assignments?

Question

We were having this discussion wiht my colleagues about Inner assignments such as: return result = myObject.doSomething(); or if ( null == (point = field.getPoint()) ) Are these acceptable or should they be replaced by the following and why? int result = myObject.doSomething(); return result; or Point point = field.getPoint(); if ( null == point) JavaScript questions and answers, JavaScript questions pdf, JavaScript question bank, JavaScript questions and answers pdf, mcq on JavaScript pdf, JavaScript questions and solutions, JavaScript mcq Test , Interview JavaScript questions, JavaScript Questions for Interview, JavaScript MCQ (Multiple Choice Questions)

Answer 1

The inner assignment is harder to read and easier to miss. In a complex condition it can even be missed, and can cause error. Eg. this will be a hard to find error, if the condition evaluation prevent to assign a value to the variable: if (i == 2 && null == (point = field.getPoint())) ... If i == 2 is false, the point variable will not have value later on.