rails プロフィール画像　更新

rails初学者で投稿型のwebアプリを作成中です。
プロフィール画像が更新されるときとされないときがあります。
コードは問題ないと思うのですが他に原因がわかりません。
特にエラーなどは出ていませんのでWi-Fiなどの環境でしょうか？

class UsersController < ApplicationController
  before_action :authenticate_user, {only: [:index, :show, :edit, :update]} #ログインユーザーのみ閲覧
    before_action :forbid_login_user, {only: [:new, :create, :login_form, :login]} #ログアウトユーザーが閲覧
   before_action :ensure_correct_user,{only:[:edit,:update]} #ユーザーを確認
  def index
    @users = User.all
  end

  def show
    @user = User.find_by(id: params[:id])
  end

  def new
    @user = User.new
  end

  def create
    @user = User.new(
      name: params[:name],
      email: params[:email],
      password: params[:password],
      image_name: 'default_user.jpg'
    )
    if @user.save
      session[:user_id]=@user.id
      flash[:notice] = 'ユーザー登録が完了しました'
      redirect_to("/users/#{@user.id}")
    else
      render('users/new')
  end
end

  def edit
    @user = User.find_by(id: params[:id])
    # @user =User.find_by(name: params[:name], email: params[:email])
  end

  def update
    @user = User.find_by(id: params[:id]) # 値を取得する
    @user.name = params[:name] # 名前を上書きする
    @user.email = params[:email] # アドレスを上書きする
    if params[:image]
      @user.image_name = "#{@user.id}.jpg"
      image = params[:image]
      # 画像を送信
      File.binwrite("public/user_images/#{@user.image_name}", image.read)
   end

    if @user.save
      flash[:notice] = 'ユーザー情報を編集しました'
      redirect_to("/users/#{@user.id}")
    else
      render('/users/edit')
  end
  end

  def login_form
  end

  def login
    @user=User.find_by(email: params[:email],password: params[:password])
    if @user
      session[:user_id] = @user.id #情報を保持する
      flash[:noice]="ログインしました"
      redirect_to("/posts/index")
    else
      @error_message="メールアドレスまたはパスワードが間違っています"
      @email=params[:email]
      @password=params[:password]
      render("users/login_form")
    end
  end
  def logout
      session[:user_id] = nil
      flash[:notice] = "ログアウトしました"
      redirect_to("/login")
    end
    def ensure_correct_user
    if @current_user.id !=params[:id].to_i
      flash[:notice]="権限がありません"
      redirect_to("/posts/index")
  end
end
  # ユーザーの削除
  # def destroy
  #   @user = User.find_by(:id params[:id])
  #   @user.destroy
  #   flash[:notice]="削除しました"
  #   redirect_to("/users/index")
  # end
end

<div class="main users-edit">
  <div class="container">
    <div class="form-heading">アカウントの編集</div>
    <div class="form users-form">
      <div class="form-body">
        <% @user.errors.full_messages.each do |message| %>
          <div class="form-error">
            <%= message %>
          </div>
        <% end %>

        <!-- 送信先を指定する,画像設定 -->
        <%=form_tag("/users/#{@user.id}/update",{multipart:true}) do %>
        <p>お名前</p>
        <input name="name" value="<%=@user.name%>">
        <p>画像<p/>
          <input name="image" type="file">
        <p>メールアドレス</p>
        <input name="email" value="<%=@user.email%>">
        <input type="submit" value="保存">
      <%end%>
    </div>
  </div>
</div>
</div>

<div class="main user-show">
  <div class="container">
    <div class="user">
      <img src="<%="/user_images/#{@user.image_name}"%>">
      <h2><%= @user.name %></h2>
      <p><%= @user.email %></p>
      <!-- <div class="user-menus"> -->
      <%if @user.id==@current_user.id%>

        <%=link_to("編集","/users/#{@user.id}/edit")%>
      <%end%>
    </div>
    <%@user.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 %>
    <!-- <%=link_to("削除","/users/#{:id}/destroy",{method:"user"})%> </div> -->
  </div>
</div>

よろしくお願いします。