بهینه کردن کد
سلام دوستان.
آیا کد زیر رو میشه بهینه تر کرد؟
protected function uploadImage($image = '')
{
$path = 'upload/images';
if (request()->hasFile('image') && $files = request()->file('image'))
$image = $files->store($path, 'public_files');
return $image;
}public function store(CreateArticleRequest $request)
{
$data = $request->validated();
$image = $this->uploadImage();
$data['image'] = $image;
$article = auth()->user()->articles()->create($data);
$article->categories()->attach($data['categories']);
session()->flash('status', 'مقاله با موفقیت ایجاد شد');
return redirect()->route('articles.index');
}
پاسخها
5
من که فکر کاملا بهینه خوانا و CleanCode هستش
protected function uploadImage($image = null)
{
$path = 'upload/images';
if (request()->hasFile('image') && request()->file('image')->isValid()) {
$image = request()->file('image')->store($path, 'public_files');
}
return $image;
}گفتگوهای مرتبط
شاید پاسخ شما همینجا باشد
@Nima.nori
تشکر از نظر شما.
@hosseinshirinegad98
میشه راجب کدتون یه توضیحی بدین.
به نظرم کدی که خودم نوشتم بهینه تر هست.
متد isValid برسی میکنه که فایل به صورت کامل آپلود شده
برای if سعی کنیداز آکولاد استفاده کنید درسته که برای یه دستور نیاز نیست
و متد شما در هر صورت اینجا یه مقداری باید برگردونه
در پارامتر تابع بجای سینگل کوت از مقدارnull استفاده نمایید
کدشما هم خوب بود ولی حتما فاصله گذادی بین لاینهارو دقت کنید این کار رو میتونید از خود فریم ورد یاد بگیرید
سلام چطورین خب بخواهین بهینه شه به نظرم اینکارو کن
جاسازی کن
<?php
// code
//Heredoc variable
$s1=<<<EOD
اینجا کدتو بزن
EOD;
echo $s1حالا برو یه جا دیگه اکو بزن
کد اجرا میشه
یا اینو دیدگاهشو بخون
https://roocket.ir/discuss/%D8%AC%D8%A7%D8%B3%D8%A7%D8%B2%DB%8C-%DA%A9%D8%AF-%D8%AF%D8%B1-php
اقای نیمارو بخون