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

Kontrolcülerdeki ajax cevaplarını dinamikleştirmek

ac1919

Selamlar,

Bununla ilgili araştırma yaptım abstract klası oluşturulan örnekler gördüm ama tam olarak nasıl yapıldığını size sormak istedim. Kodlar şu şekilde:

Base AjaxController

 abstract class AjaxController extends Controller
 {
 protected function respondSuccess($message = 'Done!', $code = 200)
 {
 return response()->json([
 'success' => true,
 'message' => $message
 ], $code);
 }

 protected function respondWithError($message = 'Server error', $code = 500)
 {
 return response()->json([
 'success' => false,
 'message' => $message
 ], $code);
 }

Ajax işleminin yapıldığı Controller

class TestController extends AjaxController
{
 public function delete(Request $request, $id)
 {
 $destroy = Article::destroy($id);

 if ($destroy) {
 return $this->respondSuccess();
 }
 
 return $this->respondWithError();

Bu yöntem doğru olur mu ? Eğer olursa abstract sınıfı hazırlamanın doğru yolu nedir ?

mgsmus

Bu yöntemleri abstract bir sınıf oluşturmadan direkt Controller sınıfına da ekleyebilirsiniz, bir trait oluşturup direkt TestController'a da ekleyebilirsiniz ya da hiç kullanmayabilirsiniz de. Ben bir katkısını göremedim.

Ama illa bu şekilde bir şeyler yapmak istiyorsanız:

\App\Providers\AppServiceProvider::boot() içerisine:

Response::macro('success', function ($message = 'Done!') {
 return Response::json([
 'success' => true,
 'message' => $message
 ]);
});

Daha sonra controller içerisinde

return response()->success()

Yani dediğim gibi, bu tür bir kullanımda ben bir katkısını göremiyorum. Öğrenmek için bakabilirsiniz.

ac1919

@mgsmus hocam cevap icin tesekkur ederim

deathisonitsway

Soyut bir sınıf tanımlamanıza gerek yok direkt Controller sınıfına ekleyin zaten her kontrolcü onunla genişliyor.
Yalnız burada eksik olan şu.Bu tür şeyleri kullandığınızda illa son çıktıda vermek zorunda kalıyorsunuz bu uyarıyı.
Halbuki kod bölündüğünde bu tür bir süreci dahil etmek zorlaşıyor.


protected $maps = [];

public function testController()
{
 $this->divide();

 //olabilecek senaryo
 if(!isset($this->maps['foo'])){
 return response()->json();
 }

 return response()->json();

}


private function divide()
{

 if(...) {
 //burada exception isteniyor
 //ama void işlem olduğu için ana methodda bir şey döndüremezseniz.

 //böyle de formata uymuyor.
 throw new Runtime('blabla');
 }

 $this->maps['foo'] = 'bar';

}

hiç hoş görünmüyor kontrolcü..kirli kod.
yani çözüm olarak çıktı yönteminizi exception formatına uyarlamalısınız.


throw new \foo();

procesin herhangi bir yerinde bu şekil bir exception fırlatatarakta aynı modellemenizi yakalayabilmeniz gerekir.

Örneğin; biz kendi apilerimizde bu formata uygun bir şey yazıyoruz.


public function index()
{

 asa;

 return ...

}

normalde bu kodun hata vermesi gerekmektedir.Ancak istediğimiz şey formata uygun dönmesi.yani son kullanıcıya html donmemesi gerekiyor.


{
 "meta": {
 "success": false,
 "status": 500
 },
 "resource": {
 "errorFile": "/var/www/html/UsersController.php",
 "errorLine": 18,
 "errorType": "Undefined",
 "errorMessage": "Use of undefined constant asa - assumed 'asa' (this will throw an Error in a future version of PHP)",
 "errorNo": 2
 }
}

hiç bir şey yapmadıgım halde bu kodun cıktısı bu şekil..Net konuşmamakla beraber,eksik olan budur php dünyasında.
Api frameworklere bakmak lazım bunlar için.Yani ben kodun belirli bir bölgesinde işlemi kesmek istiyorum.
bunun icin gereksinimlerim.

1- formata uygun dönmesi şart
2- hangi dosyada (logda görebileceğim şekilde.testte gerçek çıktıda-prodda logta)
3- dosyanın hangi satırında
4- Kesinlikle http status code (http_response_code()) ile birlikte
5- mesaj acıklayıcı ve net ve dil yapısına uygun.

eğer apinizde bunlar varsa.Apiniz kaliteli olur.
Yoksa uğraşır durursunuz..

ac1919

Tamam hocam vazgectim bu sekilde tasarlamaktan mgsmus da gerek gormedi anladigim kadariyla sorun yasamaya gebe bi yontem altindan da kalkamayabilirim cevaplar icin tesekkur ederim.