wouldn't be better to write it in one line of code