[wip ]Fix associtation and UI

# 前回PRとの関係
前回PRまで`user has many orders`で作っていました。
しかし、`user has one order`のほうが良いとわかりました。

`user has many orders`ですと、`app/views/arriveds/show.html.erb`で不具合が出ます。
リソース設計としても、このアプリケーションは注文に期限を設けない限り、１ユーザーは１つの注文しかできないのが自然と教えて頂きました。
そのため、修正します。

また、それとは別に、より良いUIにするため注文作成時のビュー変更作業をします。

## １オーダーにする理由について

### `user has many orders`の場合
----------------------------------------------------
「紅茶１個、緑茶１個」を注文したとします。その後で「烏龍茶１個と、追加の緑茶１個」を注文したくなったとします。
これをシステムに反映するには、注文「烏龍茶１個と緑茶１個」を新規に作成することになります。

この場合

* `user`が１つあり、 `order`を２つ持っている。
  * １つめの`order`は、以下２つの`order_detail`を持つ
    * 紅茶１個を意味する`order_detail`
    * 緑茶１個を意味する`order_detail`
  * ２つめの`order`は以下２つの`order_detail`を持つ
    * 烏龍１個を意味する`order_detail`
    * 緑茶１個を意味する`order_detail`

ということになります。
注文する緑茶は２個なのに、それを別々の`order` でわけていることが特に不自然なのだと思います。
（また、前回注文を削除して新規注文を作成するという方法もとれますが、同様に不自然なのだと思います。）
     
### `user has one order`の場合
----------------------------------------------------
同じことをしたくなった場合、以下のように、より自然な形で反映が可能です。

* `user`が１つあり、 `order`を１つ持っている。
* `order`は、以下３つの`order_detail`を持つ
     * 紅茶１個を意味する`order_detail`
     * 緑茶２個を意味する`order_detail` 
     * 烏龍１個を意味する`order_detail`  

# このPRで何をするか
* `user has one order`にして、現状の機能を壊さないように他の場所も修正を入れます。
*  `orders#edit`のビュー（ユーザーインターフェース）を大幅に変更します。

# 終了条件（どうしたらWIPが取れるか）
1. `user has one order`かつ、常に`user`は`order`を持つようにすること。
2. UIを変更すること。
3. 規約や慣習に沿った、読みやすいコードであること。
4. 主要な機能のfeature specとmodelのうち不安な部分のspecを書いてgreenにすること。
これだとサインインしてない状態でも new を最後まで実行してから redirect_to しちゃうんです。redirect_to は return と違ってその場で処理を打ち切るわけではない。

``` ruby
class WelcomeController < ApplicationController
  def show
    redirect_to 'http://google.com'

    puts 'hi!'
  end
end
```

このアクションにアクセスするとこんな感じのログが出ます:

```
Processing by WelcomeController#show as HTML
Redirected to http://google.com
hi!
Completed 302 Found in 1ms (ActiveRecord: 0.0ms)
```

今回のケースではデータを変更するような処理がないからいいんですけど、例えば後ろにレコードを削除するような処理があったら大変なことですね。

これを防ぐには2つの手があって、ひとつが before_action を使う方法、もうひとつが redirect_to した後に忘れず return する方法です。今回のケースでは前者が適切と思うので、調べて直してみてください。
http://guides.rubyonrails.org/action_controller_overview.html#filters