Laravel controllers

Hoi Jorn,

Hier een voorbeeld:
<?php
class NewsController
{
public function show(News $news)
{
$news->load('images');
return view('news.show', ['news' => $news]);
}
}
?>

<?php
//news/show.blade.php content

<h1>{{$news->title}}</h1>
<p>{{$news->content}}</p>
@foreach($news->images as $image)
<img src="{{route('images.show', $image->id)}}" />
@endforeach
?>

Het is een beetje afhankelijk van je code, maar het is hierbij praktisch om je relaties op de modellen te defineren. (Zie: https://laravel.com/docs/5.8/eloquent-relationships )
Dan kun je ze benaderen vanuit je model. Het tonen van de afbeelding is onderdeel van een andere controller, daarvoor kun je dus rustig naar de bijbehorende route linken.

De load is hierbij niet perse nodig omdat het geen versnelling is aangezien je maar 1 model hebt. Wel kun je daarbij er ook voor kiezen om bijvoorbeeld te sorteren, dat is netter om in je controller te doen dan in je view. Je krijgt dan:
<?php
$news->load(['images' => function ($imageQuery) {
$imageQuery->orderBy('position');
}]);
?>

Een gouden regel is om je Controller class zo slank mogelijk te houden. En dat is niet voor niets! Zoals je zelf al aangeeft heb je voor een pagina al gauw een lijstje met verschillende dingen nodig. Ga je die allemaal in je controller samenstellen dan wordt het al snel onoverzichtelijk. De controller moet daarom niets anders doen dan andere classen aanroepen om data te vergaren en dan alle data aan de view meegeven.

zoiets dus:

<?php
$news = $newsModel->getLatestNewsItems();
$images = $bannerModel->getImages();

return view('news.show', [
'news' => $news,
'images' => $images
]);
?>

Ik snap wat jullie bedoelen. Maar met m'n vraag bedoelde ik net even iets anders. Ik wil dat de gebruiker met 1 controller de mogelijkheid heeft om nieuwsberichten te plaatsen. Hierbij heeft de gebruiker de keuze om een stukje tekst te plaatsen, een video of een afbeelding. Dus dat de gebruiker zelf bepaald wat voor bericht het is. Graag wil ik dit in 1 controller/model en 1 database tabel kunnen verrichten omdat het anders zo'n zoek zooitje word.
Ik had zelf al wel aan een structuur gedacht in de html:

   @foreach($posts as $post)
       @if($post->type == 'text')
       //tekst html
       @if($post->type == 'video')
       //iframe ofzo
       @if($post->type == 'image')
       //afbeelding
       @endif
   @endforeach

Je zou beter kunnen werken met een entity. Een entity is een class dat in jou geval één Nieuwsbericht bevat. Het zal properties hebben als $created, $title, $content, $image(s), $video(s), $author etc.

Deze entity kun je vervolgens een method geven met een naam als bijv. getType() welke dan gaat bepalen of het een video, afbeelding of text bevat of misschien wel een mix daarvan.

In je view zou je dan iets krijgen als:

@foreach($newsitems as $newsitem)

  ...

  @if($newsitem->getType() == 'video')
    <video width="400" controls>
      <source src="{{ $newsitem->getVideoUrl() }}" type="video/mp4">
    </video>
  @endif

  ...

@endforeach

Heb je al een keer naar Doctrine ORM gekeken?

Heb ik daarmee gewoon de mogelijkheid om 3 verschillende dingen met 1 soort model in dezelfde database tabel op te kunnen slaan?

Ja. Je kunt gewoon kolommen aanmaken die niet verplicht zijn.

Doctrine ORM kan veel zaken voor je regelen.

- Je kunt met een commandline tool een entity aanmaken (welke je later in je editor weer gaat uitbouwen).
- Als je de entity hebt kan Doctrine de database tabel voor je aanmaken of wijzigen al dan niet met migrations
- Doctrine zet de records uit de database automatisch voor je in een entity/entities.
- Doctrine kan zelfs prima met relaties van andere tabellen werken
- Doctrine zet een entity ook zo weg in de database voor je

[size=xsmall]Toevoeging op 14/04/2019 22:37:47:[/size]

Dit artikel lijkt me wel aardig voor je: https://www.sitepoint.com/laravel-doctrine-best-of-both-worlds/