1) please use brackets for each if-statement even if it's 1-line;
2) it's better to show one operation per line;