display data from multiple tables and foreign keys in laravel - laravel

Question

I have two tables :
bonhomme
+----+--------------+
| id | name |
+----+--------------+
| 1 | Alec Badwin |
| 2 | Maria Quiban |
+----+--------------+
serie
+----+---------------+
| id | name |
+----+---------------+
| 1 | 30 Rock |
| 2 | Good Day L.A. |
+----+---------------+
I have another one called : serie_personnage
+----+-------------+----------+--------------+
| id | bonhomme_id | serie_id | personnage |
+----+-------------+----------+--------------+
| 1 | 1 | 1 | Jack Donaghy |
| 2 | 2 | 2 | Maria Quiban |
+----+-------------+----------+--------------+
I want to display the personnage from serie_personnage and link it in my blade with the name of the bonhomme (in my table bonhomme) associated.
In other words, I want a table in my blade with rows where there is in column 1, the name of the bonhomme and in column 2, the personnage associated.
How can I do that with the relationships in Laravel ? I've tried it but it's not working well.
Controller
public function show($id)
{
$serie = Serie::where('id', $id)->first();
return view('admin.series.castings.show')->withSerie($serie);
}
Model - Serie
public function personnages()
{
return $this->hasMany('App\Models\Admin\SeriePersonnage');
}
public function acteurs()
{
return $this->belongsToMany('App\Models\Admin\Bonhomme', 'serie_personnage', 'serie_id', 'bonhomme_id');
}
Blade
<tbody>
#foreach($serie->acteurs as $acteur)
#foreach($serie->personnages as $personnage)
<tr>
<td>{{ $acteur->nom }}</td>
<td>{{ $personnage->personnage }}</td>
<td>bla</td>
<td>{{ $personnage->rang }}</td>
<td class="has-text-right">
<a class="button is-outlined" href="{{ route('series.show', $serie->id) }}">
Voir
</a>
<a class="button is-outlined" href="#">Editer</a>
</td>
</tr>
#endforeach
#endforeach
</tbody>
My Table in view

Answer 1

I have to help you as French developer as-well! First of all, I would change your controller for a more simple way to get the Serie :
public function show($id)
{
$serie = Serie::findOrFail($id);
return view('admin.series.castings.show')->withSerie($serie);
}
EDIT: I use Model::findOrFail($id); because it throws a Illuminate\Database\Eloquent\ModelNotFoundException. If you don't catch the exception, Laravel will automatically send a 404 HTTP response. It prevent the user to have an error using an obsolete link. See the doc for more informations.
Then I don't think it is a good idea to use foreach loop twice... I would rather use only one foreach loop on the personnages. Then, I would get the actor of the character from the Personnage model. It should looks like the code below:
App\Models\Admin\SeriePersonnage.php
<?php
namespace App\Models\Admin;
use App\Models\Admin\Bonhomme;
use Illuminate\Database\Eloquent\Model;
class SeriePersonnage extends Model
{
/**
* The attributes that are mass assignable.
*
* #var array
*/
protected $fillable = [
//Whatever you want to put here
];
/**
* Gets the actor of the character
*/
public function actor() {
return $this->belongsTo(Bonhomme::class);
}
}
Then in your blade view:
<tbody>
#foreach($serie->personnages as $personnage)
<tr>
<td>{{$personnage->actor->nom}}</td>
<td>{{$personnage->personnage}}</td>
<td>bla</td>
<td>{{$personnage->rang}}</td>
<td class="has-text-right"><a class="button is-outlined" href="{{route('series.show', $serie->id)}}">Voir</a> <a class="button is-outlined" href="#">Editer</a></td>
</tr>
#endforeach
</tbody>
You might also add a personnages() methods in your actor model so that you would be able to get all the characters this actor is playing.
Also I would advice you to use English words for variable names, class names and in fact every word in your code, especially if you post for help on English websites. I would also recommend you to change SeriePersonnage to only Personnage if you only manage series (you might also handle movies, Japanese Anime etc, in this case you can ignore my message). Also it would be great to change the row name of serie_personnage from personnage to nom.
Hope it helped you and I didn't do mistakes :P