TradesControllerを親クラスにしてIncomeController, CostControllerを作った方がすっきりする気がしました。
差異の部分（`current_user.incomes`, `current_user.costs`)のあたりだけテンプレートメソッドパターンで子クラスに定義してやればシンプルになると思います。