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

Hata mesajı kontrolü

yalcin

Trait içindeki fonksiyonum webservisden gelen yanıtta hata varsa
return back()->withErrors(['message' => $response->Sonuc->Mesaj]); şeklinde hata return ediyor,
bu fonksiyonumu Controllerda kullanıyorum Auth::user()->MyServices() şeklinde kullanıyorum ve bundan dönen yanıta görede işlem yapıyorum .
Aslında tam istediğim gibi çalışıyor hata varsa ekrana hata bastırıyor ve sonraki işlemleri yapmıyor hata yoksa yapması gerekeni yapıyor ama çok içime sinmedi. yani sanki controller içindede kullandıgım fonksiyondan dönen yanıt hata mesajı içeriyosa kontrol etmem gerekli gibi geliyor . Fonksiyonun yanıtını birde controlller içinde check etmelimiyim ?
tabi birde return back()->withErrors ü nasıl kontrol edicem

BoraN7

yalcin Sorunuzu tam doğru anladım mı emin değilim ama;

Ben genelde Controller dışında iş yapan fonksiyonlarımı app/Services klasörü altında topluyorum.

Örneğin bir hizmetin API'sini kullanıyorsam

XxxApiService.php sınıfını hazırlıyorum.
Metodları bu sınıfın içerisinde hazırlıyorum.
Olumsuz senaryolarda ise metodun içerisinden Exception fırlatıyorum.

Ardından Controller içerisinde o servisin metodunu çağırırken işlemleri try-catch bloğu içerisinde gerçekleştiriyorum.

catch kod bloğu çalıştığında yapılacakları orada belirliyorum.
Genelde sistem logları tuttuğum bir tablom oluyor oraya basıyorum detayları sonra redirect ettiriyorum vs.

Özetle;
Service, Helper, Trait içlerindeki metodlar sadece kendilerine verilen görevi yapıyorlar.
Sonuçta nasıl bir aksiyon olacağına Controller karar veriyor.

Umarım doğru ifade edebilmişimdir : )

mgsmus

BoraN7 Ben de aynı mantıkla yapıyorum. Sadece, kendi yazdığım exception sınıflarının genişlettiği CustomException isimli bir base sınıfım var, raporlanmayacak şekilde ayarladım ve http yanıtı sırasında meydana gelirse otomatik http response a dönüşüyorlar.

yalcin

BoraN7 Olumsuz senaryolarda ise metodun içerisinden Exception fırlatıyorum.

sadeleştirerek yazarsam

 public function apiServisSorgu($userName, $password){
      $client = new SoapClient($this->BildirimService, $this->params);
     try {
       $response = $client->kisiSorgu($requestParameters);

      } catch (\SoapFault $e) {
                   
       Log::error('SOAP Hatası: ' . $e->getMessage());
        return back()->withErrors(['message' => 'SOAP Hatası: ' . $e->getMessage()]);

}

bu traitimin içinde bir metod try blogunda bir kaç if deyimi var başarısız olma durumunda servisden dönen hatam var bunu Controllerimda direk kullanıyorum


public function firmaStore(storeFirmaRequest $request) {
     $validated = $request->validated();

        $userName = $validated['UserName'];
        $password = $validated['Password'];

$response = Auth::user()->apiServisSorgu($userName, $password);

                Services::create([
                    'user_id' => auth()->user()->id,
                    'name' => 'test',
                    'UserName' => $validated['UserName'],
                    'Password' => $validated['Password'],

controller içinde kullanırken Traitim icindeki fonksiyona güveniyorum nası olsa bir hata olursa controllerimdaki

$response = Auth::user()->apiServisSorgu($userName, $password); bu satır da hata vericek diyorum ve hiç bir kontrol yapmadan işlemlere devem ediyorum
Controller içindede try catch kullanmalımıyım ?

BoraN7

yalcin
Örneğin Servis içindeki metod şu şekilde olmalı:

public function apiServisSorgu($userName, $password){
   
     try {
     $client = new SoapClient($this->BildirimService, $this->params);
       $response = $client->kisiSorgu($requestParameters);
       return $response; 
      } catch (\SoapFault $e) {
        throw new \Exception('SOAP Hatası: ' . $e->getMessage(),500);      
}

Ardından kontroller içerisinde

public function firmaStore(storeFirmaRequest $request) {
     $validated = $request->validated();

        $userName = $validated['UserName'];
        $password = $validated['Password'];
try {
$response = Auth::user()->apiServisSorgu($userName, $password);
Services::create([
                    'user_id' => auth()->user()->id,
                    'name' => 'test',
                    'UserName' => $validated['UserName'],
                    'Password' => $validated['Password'],
} catch (\Exception $e) {
        Log::error('SOAP Hatası: ' . $e->getMessage());
        return back()->withErrors(['message' => 'SOAP Hatası: ' . $e->getMessage()]);      
}

Gibi düşünün.
apiServisSorgu size ya sonucu döndürecek ya da Exception fırlatacak.
Sonuç başarılı döndüğünde ne yapılacağı ya da Exception fırlarsa ne yapılacağına Controller karar verecek.

yalcin

ben controller içinde dümzdüz ilerliyordum , nası olsa metod içinde try-catch var hata varsa hata fırlatıp durucak mantığı ile . tamam anladım @BoraN7 teşekkur ettim