Is this correct coding style? It doesn't look right to have the if clause code lined up with the if conditions.