só mesmo porque este método ainda nao tinha comentários que chegue:
parece-me desnecessário verificar se o username existe mesmo que não tenha sido usado (ou seja, indo para /user/1, ser redirecionado para /username caso exista, que é o que me parece estas a fazer aqui.

isso pode ser feito mas no futuro, com uma action diferente para os dois casos, e pondo a action original a redireccionar para a do username quando possivel. mas fazer isso fora dessa action vai implicar fazeres este check sempre que quiseres fazer isto, fica mal

ou seja, deixaria isso de lado para já
tendo isso em conta, este if fica bastante generico e podes muda-lo para um metodo no application_controller que para cada user retorna a route com username ou a original, conforme possivel. por exemplo, já ali abaixo tens um if bastante parecido que podia aproveitar isso