i have code as :
if (somethings == TRUE)
return true; //i am asked to remove it coz it will reach anyhow at end and do same but i feels it is good practice and why should i wait for end?
This is called an “early return” and I don’t think there is a clear consensus.
There are some similar previous questions about early returns, see here for example.