常連

回答者

ベストアンサー

拍手喝采

自己紹介

```rust
struct S<'a> { y: &'a mut i32 }

fn bar() {
    let mut x: i32 = 1;
    let s = S { y: &mut x };
    *s.y = 5;
}

fn baz() {
    let mut x: i32 = 1;
    let s = S { y: &mut x };
    let z = &s;
    *z.y = 5;
}
```

上のコード片で、 `bar()` 関数ではエラーは起きませんが、 `baz()` 関数では次のようなエラーが発生します。

```
error[E0389]: cannot assign to data in a `&` reference
  --> /home/dai/workspace/daily/2018/0222/junk00-11-41.rs:15:5
   |
15 |     *z.y = 5;
   |     ^^^^^^^^ assignment into an immutable reference
```

ここでコンパイラは `z.y` が `immutable reference` であるとしています。

疑問は、

1. どこを見て `z.y` を `immutable reference` としているのか
    `S::y` の型だけ見れば `&mut i32` なのに。
1. 参照経由で `*S::y` を書き換えるには `s` を mutable にするしかないのか
    `baz()` 関数でも `s` を mutable にして `z` を &mut 参照とすればエラーはなくなりました。

以下、1点目の、なぜ `immutable reference` とされているかを理解するために考えたこと・試したことを記述します。少々長くなってしまいましたが、誤解している点などありましたら指摘していただけると嬉しいです。

---

以後、式 `a` が型 `T` であることを `a : T` と書きます (ライフタイムは省略) 。`a` が mutable であることを `mut a` と書きます。

まず、 `S` の定義では確かに `y : &mut i32` と定義されています。`*s.y` は `y` そのものに対する変更 (`y` が参照する先を変更するなど) ではなく `y` の参照先を書き換える行為なので、 `s` が `mut s` である必要はないと考えました。実際 `foo()` 関数によりこれは確かめることができています。

`*s.y = 5;` という処理で `s` の所有権がそのまま絡むような操作はないと思っています。なので `s` への参照 `z` を経由しても同じことができてほしいです。かつ、 `mut s` が要求されていないところでもし `z : &mut S` が要求されるとすると違和感があるので、 `z : &S` で通るはずだと考えます。ところが実際エラーが出ます。

適当に `mut` を付けて次の `qux()` 関数のように書き換えると、コンパイルが通るようになりました。

```rust
fn qux() {
    let mut x: i32 = 1;
    let mut s = S { y: &mut x }; // mut s に
    let z = &mut s; // &mut 参照に
    *z.y = 5;
}
```

つまり `z : &mut S` が要求されています。そして `s` を &mut 借用するために `mut s` も要求されます。これはもしかして、 &mut な借用は同時に一つだけ存在するというルールに違反しないためにこのようになっているのでしょうか。確かに `z : &S` 経由で `s.y : &mut i32` にアクセスできてしまうと、 `s` の immutable な参照ならいくつでも作れてしまうので、 `S::y` という &mut な参照が複数作成できることになります。

もし `S::y : &mut i32` が、 `a : S` または `a : &mut S` のときのみ `S::y : &mut i32` で、 `a : &S` のときは `S::y : &i32` となると見做されるなら、確かに先のように複数作成できてしまうということはなくなります。

この仕組みを考えていたところで、次の関数 `quux()` を試してみました。

```rust
fn quux() {
    let mut x: i32 = 1;
    let mut s = S { y: &mut x };
    let z = &mut s;
    *s.y = 5; // z -> s に変更
}
```

すると、エラーが発生して :

```
error[E0506]: cannot assign to `*s.y` because it is borrowed
  --> /home/dai/workspace/daily/2018/0222/junk00-11-41.rs:29:5
   |
28 |     let z = &mut s;
   |                  - borrow of `*s.y` occurs here
29 |     *s.y = 5;
   |     ^^^^^^^^ assignment to borrowed `*s.y` occurs here
```

...つまり、エラーによると `&mut s` (`&s` も同様に) は `s.y` のみならず `*s.y` まで借用しているということになると思いますが、その認識で正しいでしょうか。それならば `&s` は `*s.y` を `&*s.y` のような形で借用し、`&mut s` は `*s.y` を `&mut *s.y` のような形で借用しているということなのでしょうか。

先の

> もし `S::y : &mut i32` が、 `a : S` または `a : &mut S` のときのみ `S::y : &mut i32` で、 `a : &S` のときは `S::y : &i32` となると見做される

仕組みはこういうことなのでしょうか。

非常に長くなりましたが、よろしくお願いします。

構造体 S 内の &mut T なフィールド経由の更新に &mut S を要求されるのは何故か

[こちらの記事](https://qiita.com/K_Yagi/items/edc46fd976526279312b)にもある通り、

自作のユーティリティクラスを`lib/utils/hoge_util.rb`に

```lang-ruby
class HogeUtil
end
```

と作成し、`config/application.rb`にて

```lang-ruby
# Rails 4の場合
# config.autoload_paths += %W(#{Rails.root}/lib)
# Rails 5の場合はこちらの書き方がよさそう
config.paths.add 'lib', eager_load: true
```

ということはよくやると思います。
しかし`lib`の中にユーティリティクラスを置いてautoloadパスに加えるのは本当に正しいのでしょうか？個人的には`app/utils`ディレクトリを切ってそこに置いた方がいいのではないかと思いっています。理由は

- デフォルトで`lib`の中には`assets`や`tasks`がある
  - `assets`や`tasks`はautoloadパスに加えたくないはず
  - サーバ（`rails s`）を動かすために必要なRubyコードは全部`app`の中にあった方がgrepしやすいし、どちらに入れるか迷わずに済む
  - `lib`に置くと先述の記事のように、Railsをアップグレードしたときハマる可能性がある

そもそも`lib`の下には何を置くべきのでしょうか？
[Railsガイド](https://railsguides.jp/getting_started.html)によると

> lib/	アプリケーションで使用する拡張モジュールが置かれます。

とありますが、ちょっとよく分かりません。別のサイトでは「他のアプリケーションでも使いまわせるコードを置く」と説明しているところもありますが、`assets`や`tasks`はアプリケーション固有である方が普通だと思います。

Railsで自作のユーティリティクラスはlibに置くべき？

Railsのモデルのバリデーションでのエラーをrenderで表示するとURLが変わってしまうので、リロードした時にエラーが起きてしまいます
すみません、追記です。
URLが
/users/new
から
/users
になると言うことです。
なので、リロードすると
アクションがないとエラーが出ます。
これを綺麗に解決したいです。

```ruby.
Rails.application.routes.draw do
  root "users#index"
  resources :users, shallow: true do
    resources :orders, only: [:index] do
      collection do
        get :select
        get :manual_select
        get :manual_new
        post :manual_create
      end
    end
  end
  resources :sessions, only: [:new, :create, :destroy]
end

```


```slim
.row
  .col-xs-12.col-md-4
  .col-xs-12.col-md-4
    .panel.panel-default
      .panel-heading 新規登録
      .panel-body
        = form_for @user do |f|
          .form-group
            = f.label "Email"
            = f.email_field :email,class: "form-control", required: :required, placeholder: "Email"
            = f.label "姓"
            = f.text_field :family_name, required: :required,class: "form-control", placeholder: "姓"
            = f.label "名"
            = f.text_field :first_name, required: :required,class: "form-control", placeholder: "姓"
            = f.label "パスワード"
            = f.password_field :password, required: :required,class: "form-control", placeholder: "6文字以上"
            = f.label "パスワード再入力"
            = f.password_field :password_confirmation,class: "form-control", required: :required, placeholder: "6文字以上"
          = f.submit "送信", class: "btn btn-primary btn-lg btn-block"
          = link_to "ログイン", new_session_path, class: "btn btn-default btn-lg btn-block"
          = link_to auth_at_provider_path(provider: :twitter), class: "btn btn-block btn-social btn-lg btn-twitter"
            i.fa.fa-twitter
            .sns-title = "Twitterログイン"
          = link_to auth_at_provider_path(provider: :facebook), class: "btn btn-block btn-social btn-lg btn-facebook"
            i.fa.fa-facebook
            .sns-title = "Facebookログイン"

```

```ruby.
  def create
    @user = User.new(user_params)
    if @user.save
      #成功した時
    else
      render "new"
    end
  end
```

具体的には上記のような場合です。
URLを変更せずにエラーメッセージを表示させるナイスな方法はありませんでしょうか？

redirectの場合だと綺麗にメッセージが渡せずに悩んでいます。

宜しくお願い致します

Railsのモデルのバリデーションでのエラーメッセージを表示するとURLが変わってしまう件について。。

お世話になっております。
現在RubyOnRailsで開発を行っています。

開発初期ということもあり、テーブル定義が変わることが頻繁にあります。
テーブル定義が変更される度に、Rspecのテストコードで利用しているfixtureのインサート部分でテストがコケてしまいます。
こちらは仕様上仕方のないものだとも思っているのですが、fixtureの修正がなかなか時間のかかる作業なので、何か良い方法があればご教示頂きたいです。

テーブル定義が変わる度に、テストコードで使用しているfixtureを書き換えることになってしまうのをなんとかしたい

###前提・実現したいこと
ここに質問したいことを詳細に書いてください
progateのRoRの練習問題をコピペしてツイートアプリを自分の開発環境上で動かそうとしています。しかしながら、一つひとつ着実にコピーしているはずが、途中で次のようなエラーがでました。

###発生している問題・エラーメッセージ

```
undefined method `image_name' for nil:NilClass

<div class="posts-index-item">
        <div class="post-left">
          <img src="<%= "/user_images/#{post.user.image_name}" %>">
        </div>
        <div class="post-right">
          <div class="post-user-name">
```

###該当のソースコード
```

<div class="main posts-index">
  <div class="container">
    <% @posts.each do |post| %>
      <div class="posts-index-item">
        <div class="post-left">
          
          <img src="<%= "/user_images/#{post.user.image_name}" %>">
        </div>
        <div class="post-right">
          <div class="post-user-name">
           
            <%= link_to(post.user.name,"/users/#{post.user.id}") %>
          </div>
          <%= link_to(post.content, "/posts/#{post.id}") %>
        </div>
      </div>
    <% end %>
  </div>
</div>

```

###試したこと
/public/user_images...などと書き変えてもダメでした
以前におこなったrails db:migrateがうまくいっていないのかと思ってもう一度実行しましたが、正常に実行されているようで、追加のメッセージはやはり出ませんでした。

###補足情報(言語/FW/ツール等のバージョンなど)
Rubyは2.3.1,RoRは5.0.0を使っています。
よろしくおねがいします。


追加情報になります

user.rbのコード
```
class User < ApplicationRecord
    validates :name, {presence: true}
    validates :email, {presence: true, uniqueness: true}
    validates :password, {presence: true}
    
    def posts
        return Post.where(user_id: self.id)
    end
    
end
```

post.rbのコード
```

class Post < ApplicationRecord
    validates :content, {presence: true, length: {maximum: 140}}
    validates :user_id, {presence: true}
    
    def user
        return User.find_by(id: self.user_id)
    end
    
end

```

posts_controller.rbのコード
```
class PostsController < ApplicationController
    before_action :authenticate_user
    
    before_action :ensure_correct_user, {only: [:edit, :update, :destroy]}
    
    def index
        @posts = Post.all.order(created_at: :desc)
    end
    
    def show
        @post = Post.find_by(id: params[:id])
        @user = @post.user
    end
    
    def new
        @post = Post.new
    end
    
    def create
        @post = Post.new(
                         content: params[:content],
                         user_id: @current_user.id
                         )
                         if @post.save
                             flash[:notice] = "投稿を作成しました"
                             redirect_to("/posts/index")
                             else
                             render("posts/new")
                         end
    end
    
    def edit
        @post = Post.find_by(id: params[:id])
    end
    
    def update
        @post = Post.find_by(id: params[:id])
        @post.content = params[:content]
        if @post.save
            flash[:notice] = "投稿を編集しました"
            redirect_to("/posts/index")
            else
            render("posts/edit")
        end
    end
    
    def destroy
        @post = Post.find_by(id: params[:id])
        @post.destroy
        flash[:notice] = "投稿を削除しました"
        redirect_to("/posts/index")
    end
    
   
    def ensure_correct_user
        @post = Post.find_by(id: params[:id])
        if @post.user_id != @current_user.id
            flash[:notice] = "権限がありません"
            redirect_to("/posts/index")
        end
    end
    
end

```