According to Stack Overflow it is bad practice to use empty() when check a value that is returned from a function, because when a function has be called there will always be a result. See further explanation in the answer.
Use of empty()
August 29th, 2013