یلدا ادامه داره... ❤️ ۴۰ درصد تخفیف همه دوره‌ها

استفاده از تخفیف‌ها
ثانیه
دقیقه
ساعت
روز
Armin Rahmati
3 سال پیش توسط Armin Rahmati مطرح شد
5 پاسخ

بهینه کردن کد

سلام دوستان.
آیا کد زیر رو میشه بهینه تر کرد؟

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');
    }

ثبت پرسش جدید
نیما
تخصص : جوجه برنامه نویسی که میخواد فو...
@Nima.nori 3 سال پیش مطرح شد
0

من که فکر کاملا بهینه خوانا و CleanCode هستش


حسین شیری نژاد
تخصص : programmer
@hosseinshirinegad98 3 سال پیش مطرح شد
1
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;
}

Armin Rahmati
@arminrahmati999 3 سال پیش مطرح شد
0

@Nima.nori
تشکر از نظر شما.
@hosseinshirinegad98
میشه راجب کدتون یه توضیحی بدین.
به نظرم کدی که خودم نوشتم بهینه تر هست.


حسین شیری نژاد
تخصص : programmer
@hosseinshirinegad98 3 سال پیش مطرح شد
0

متد isValid برسی میکنه که فایل به صورت کامل آپلود شده
برای if سعی کنیداز آکولاد استفاده کنید درسته که برای یه دستور نیاز نیست
و متد شما در هر صورت اینجا یه مقداری باید برگردونه
در پارامتر تابع بجای سینگل کوت از مقدارnull استفاده نمایید
کدشما هم خوب بود ولی حتما فاصله گذادی بین لاینهارو دقت کنید این کار رو میتونید از خود فریم ورد یاد بگیرید


حسین
تخصص : وب
@hoseen.r8613 3 سال پیش مطرح شد
0

سلام چطورین خب بخواهین بهینه شه به نظرم اینکارو کن
جاسازی کن

<?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
اقای نیمارو بخون


برای ارسال پاسخ لازم است وارد شده یا ثبت‌نام کنید

ورود یا ثبت‌نام