Laravel Türkiye Discord Kanalı Forumda kod paylaşılırken dikkat edilmesi gerekenler!Birlikte proje geliştirmek ister misiniz?

PhpStorm Return Statement Uyarısı

BoraN7

Herkese merhaba,

Birden fazla yerde kullandığım bir kod bloğum vardı. Bunu bir Service class'ı içerisine almaya karar verdim.

Ancak bu Service 'i hem Controller, hem de console command kullanacak.

İnternetten biraz araştırmalar yaparak, şu şekilde bir kurgu yaptım;

<?php

namespace App\Services;

class SomeService
{
      public function someMethod($variable)
      {
      $out = new \Symfony\Component\Console\Output\ConsoleOutput();
        try {
             // some code...
            if (!app()->runningInConsole()) {
                return back()->with('message', 'Some success message');
            } else {
                $out->writeln("Some success message for CLI!");
            }
        } catch (\Exception $e) {
            if (!app()->runningInConsole()){
                return back()->with('error', 'Error:'.$e->getMessage());
            } else {
                Log::error('Error:'.$e->getMessage());
                $out->writeln("Some errormessage for CLI!");
            }
        }
       }
}

Şimdi kodlar çalışıyor. Eğer panelden bir post isteği ile bu servisi tetiklersem, return back() oluyor. Command'ı tetiklersem de istediğim çıktıyı veriyor. Hata olursa da istediğim gibi Log basıyor.

Ancak, Php Storm return belirtmediğim şeklinde uyarı gösteriyor. Bunu aşmak için ne yapmalıyım?

mgsmus

BoraN7 Hem bir yönteme iki farklı iş yaptırmak hem de bu şekilde bir servis kullanmak yanlış. Servisin görevi bir işi yapmak olmalıdır, uygulamanızı yönetmemeli, yönlendirme ya da çıktı vermek onun görevi değil. Bu görev kullanıldığı, enjekte edildiği yere göre Command ya da Controller tarafından yapılır. Siz servisi iş yapıp sonuç verecek şekilde tasarlayın, sonuca göre enjekte ettiğiniz yerde işlem yapın.

public function handle(SomeService $someService)
{
    if($someService->someMethod($this->argument('path'))) {
        $this->info("Some success message for CLI!");
    }
}

public function index(Request $request)
{
    if($this->someService->someMethod($request->input('path'))) {
        return back()->with('message', 'Some success message');
    }
}

gibi düşünmeniz lazım. Bu şekilde yöntemlerinize de dönüş tipi vermede sorun yaşamazsınız:

public function someMethod(string $path): bool
{
    // true/false dönen bir işlem...
}