一対一で定義したモデルのところからデータを取得したい

Question

実現したいこと

テーブルを作って、一対一で結んだAddressesモデルから、認証しているユーザーのIDを起点にデータを取得したい。

問題

モデルのファイル

User.php

<?php

namespace App;

use Illuminate\Notifications\Notifiable;
use Illuminate\Foundation\Auth\User as Authenticatable;

class User extends Authenticatable
{
    use Notifiable;

    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'name', 'email', 'password',
    ];

    /**
     * The attributes that should be hidden for arrays.
     *
     * @var array
     */
    protected $hidden = [
        'password', 'remember_token',
    ];

    public function addresses()
    {
      return $this->hasOne('App\Addresses');
    }

}

一対一の関係 Addresses.php

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Address extends Model
{
  public function addresses()
    {
        return $this->belongsTo('App\User');
    }
}

コントローラーでデータを引っ張っていきたいのですが、、、

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
use App\Http\Controllers\Controller;
use App\Addresses;


......
  public function test(){
      $id = Auth::id();

      $id = \App\Addresses::find($id);

       $address = $id->address;

      return view('/test', compact('address'));
    }

なのですが、viwwで表示してみようとすると、、、、

Class 'App\Addresses' not found
  

とありました。

質問内容

トレイトの指定がうまくいってるのかなと思うのですが、
なぜかできません。どうすればできるのでしょうか。

Answer 1

Addresses ではなく、正しくは Address ではないでしょうか。

（追記）
モデル名は class Address extends Model と単数形になっているのに
他の箇所では Addresses と複数形になってしまっています。
通常は、テーブル名は複数形（users, addresses など）にしますが
モデル名は単数形（User, Address など）にしますので、Address に統一した方が良さそうです。

ご参考: Eloquentモデル規約

（追記2）

• Adress モデルのメソッド　addresses は user が正しいかと思います。
• コントローラの test メソッドの Address::find は User::find が正しいのではないでしょうか。

Comments

[ryusei_kurata]

モデル名をaddressesにしていて、
returnで、App\Addressesにいくようにしています。

[ryusei_kurata]

なので、Addressesで大丈夫かと！

[Lulucom]

モデル名は通常は単数形（User, Address, 等）にしますし、
編集前の質問でもコントローラで `use App\Address;` と単数形にされていましたので、
すべて `Address` に統一した方が良さそうです。
すみません、回答がわかりづらかったと思いますので、そのように修正しておきます。

[ryusei_kurata]

なるほど！承知しました！それでやってみます！