سلام دوستان.
آیا کد زیر رو میشه بهینه تر کرد؟
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');
}
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
اقای نیمارو بخون
آیا مایل به ارسال نوتیفیکیشن و اخبار از طرف راکت هستید ؟