userArray という名前はイケてないですね。。。
userArray ⇒ users
にして、
@users.each do |user| ⇒ User.all.each do |user| 
とかには出来ないでしょうか？@usersが意味をもってるからダメなのかな。

arrayというのはあくまでデータ構造であって、変数名には構造ではなく、データの性質を命名すると良いと思います。
usersがダメならuser_listの方が良いかな。どうでしょう！