better to use if here