Best practices around single statement if