Best practices doesn't mean the absolute most efficient code possible, especially at the expense of readability. This solution is bog-standard, 101 level programming. It's impossible for this to be a bad practice besides the lack of brackets on the if-statement which people have mixed opinions about.