Ik heb een simpel CRM systeem gemaakt met email functie waar je je kan aanmelden en zo gegevens kan delen zoals een post of adres gegevens etc. Zou je willen kijken of dit goed gemaakt is?
Link naar github = https://github.com/systemthinker/demo
LET OP! Een deel van de code maakt gebruike van PHP BLADE addon (dit verklaard de @, maar is verder gewoon functioneerde code)
Voornamelijk ben ik benieuwd naar waar je je code het beste kan verifieren ofwel bij de controllers ofwel bij de views etc. Bijvoorbeeld: Controller:
public function index()
{
if (Auth::check()) {
$currentUser = Auth::user();
$currentUserId = $currentUser->id;
$users = User::orderBy('name', 'asc')->get();
$auth = true;
if ($userPost = $currentUser->post) {
$userPost = true;
return view('posts.index', compact('users', 'userPost', 'currentUserId', 'auth'));
} else {
$userPost = false;
return view('posts.index', compact('users', 'userPost', 'currentUserId','auth'));
}
} else {
$userPost = false;
$auth = false;
$users = User::orderBy('name', 'asc')->get();
return view('posts.index', compact('users', 'userPost', 'auth'));
}
} View:
@foreach($users as $user)
<div class="row">
@if(empty($user->post->path)) <div class="col-sm-1"></div>
@else <div class="col-sm-1"><img style = "padding: 0; max-height:50px; max-width:50px;" src="{{$user->post->path}}"></div> @endif
@if($user->post == null)<div style="padding-top: 10px;" class="col-sm-4"><h5>{{$user->name}}</h5></div>
@else <div style="padding-top: 10px;" class="col-sm-4"><h5><a style="text-decoration: none;" href="{{route('posts.show',$user->id)}}">{{$user->name}}</a></h5></div> @endif
@if($user->post == null) <div style="padding-top: 10px;" class="col-sm-4"><h5>{{$user->email}}</h5></div>
@else <div style="padding-top: 10px;" class="col-sm-4"><h5><a style="text-decoration: none;" href="{{route('posts.show',$user->id)}}">{{$user->email}}</a></h5></div> @endif
<hr>
</div>
<br>
@endforeach
@if($userPost) <a style="text-decoration: none;" href="{{route('posts.edit',$currentUserId)}}"><btn class="btn btn-success btn-block btn-lg">Wijzig je gegevens</btn></a>
@elseif($userPost == false && $auth == true) <a style="text-decoration: none;" href="{{route('posts.create')}}"><btn class="btn btn-success btn-block btn-lg">start hier</btn></a>
@elseif($userPost == false && $auth == false) <a style="text-decoration: none;" href="{{route('login')}}"><btn class="btn btn-success btn-block btn-lg">maak account aan of login</btn></a> @endifis dit nu een beetje netjes of kan dit een stuk beter?
Groet
Robbert
