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

Ajax ile silme işlemleri

ac1919

Selam, bu kara günde bir umutla herkese iyi günler olması dileğiyle,

Bir kaç gün önce açtığım konuda ajax ile silme işlemlerim doğru değil demiştiniz, bende işlemleri değiştirdim ve yeni konu açarak buraya yazmak istedim belki benim gibi çaylak birine bir faydası olur ve işlemlerin doğru olup olmadığını size sormak istedim. Aşağıda yaptığım ajax işlemleri doğru mu ?

Kontrolcü:

 public function deleteData(Request $request)
 {
 $id = $request->id;

 $user_delete = User::destroy($id);

 if ($user_delete) {
 return response()->json([
 'success' => 'İşlem başarıyla gerçekleşti !',
 ], 200); 
 } else {
 return response()->json([
 'errors' => 'İşlem gerçekleştirilemedi !',
 ]);
 }
 }

Javascript:

$.ajax({
	method: "post",
	url: "{{route('app.routes.deletedata')}}",
	data: form.serialize(),
	dataType: "json",

	success: function(response) {

 if (response.errors) {
 console.log(response.errors);
 }

 if (response.success) {
 console.log(response.success)
 }
	}
});

deathisonitsway

ben olsam şöyle yazarım..


/**
 * delete user data from database
 * 
 * @param Request $request
 */
 public function deleteData(Request $request)
 {
 $id = $request->id;
 
 if(is_numeric($id)){

 //değişkenler camelCase olmalı.
 $userDelete = User::destroy($id);

 if ($userDelete) {
 return response()->json([
 'success' => 'İşlem başarıyla gerçekleşti !',
 ], 200);
 }
 }
 
 return response()->json([
 'errors' => 'İşlem gerçekleştirilemedi !',
 ]);
 }

if else tasarımları içinde return varsa else methoduna gerek yoktur.

json methodu api tasarımı açısından method ismi olarak doğru değildir.Her ne kadar method içerisinde adapter değiştirilsede.
Method ismi refactore tabiidir ve iş yükü oluşturabilir.


request()->output();

olabilir... iyi çalışmalar.

ac1919

@deathisonitsway teşekkür ederim hocam açıklayıcı bilgiler için.

mgsmus

Model::destroy() yöntemi sırasında deleting ve deleted olayları da ateşlenir. Eğer modelin bu olaylarında işlem yaptırıyorsanız hata verme ihtimali var. Ayrıca veritabanı ile ilgili bir hata alma şansınız da var. O yüzden işlemi try...catch... bloğu içerisinde yapın derim.

Ayıca destroy yöntemi delete yönteminin toplu versiyonudur. Model::destroy([1, 2, 3]) şeklinde toplu sildirme yapabilirsiniz. Sonuç olarak ise silinen kayıt sayısını dönderiyor. Bir sebepten ötürü üç kayıttan sadece ikisi silinir biri silinmez ise (örneğin birinde deleting olayı ile işlem yapılıyorsa ve işlem yarıda kesilirse/false dönerse) sizin kontrol şekli yine de true dönecektir çünkü $user_delete değeri 2 olacaktır. Bunu da toplu silme sırasında göz önünde bulundurun.

Ben bazı sorunları yaşamamak için her zaman önce silinecek kaydın olup olmadığını kontrol ediyorum çünkü destory işleminin aksine delete işlemi kayıt olsun olmasın daima true dönüyor.

ac1919

@mgsmus hocam tam da model içerisinde delete işlemi yapmak üzereyim, bilgiler için teşekkür ederim.

Deleted ve deleting işlemleri arasındaki fark tam olarak nedir ? Yani modelden bir kayit silerken iliski kurdugum diger modelden de o id ye ait kayidi silmek icin mesela hangisini kullanmam gerek ?

mgsmus

deleting model silinmeden hemen önce; deleted ise model silindikten sonra ateşlenen olaydır. İlişkili alt kayıtları deleting olayı içerisinde silin ama örneğin kayda ait bir dosya varsa bunu deleted ile yapın.

app/Album.php

protected static function boot() 
{
 parent::boot();

 static::deleting(function($album) {
 foreach ($album->photos as $photo) {
 $photo->delete();
 }
 });

 static::deleted(function($album) {
 Storage::delete($album->cover);
 });
}

app/Photo.php

protected static function boot() 
{
 parent::boot();

 static::deleted(function($photo) {
 Storage::delete($photo->file);
 });
}

$album = Album::findOrFail($id);
$album->delete(); // Albüm, albümün kapak resmi, albüme ait Photo modelleri ve bu modellerde bulunan resimler silinmiş oldu.

ac1919

@mgsmus hocam cok degerli bilgiler tesekkur ederim.

coder2

@mgsmus , hocam Album modelinde boot methodunu override ederken eklediğiniz şu alan ;

 static::deleted(function($album) {
 Storage::delete($album->cover);
 });

tam olarak nasıl çalışıyor ?
cover isimli default bir değer mi var ?

$album = Album::findOrFail($id);
$album->delete(); // Albüm, albümün kapak resmi, albüme ait Photo modelleri ve bu modellerde bulunan resimler silinmiş oldu.

Herhangi bir relation kurmadan dosyaları nasıl siliyor , kafam karıştı açıkcası.

ac1919

@coder2 hocam cover tabloda tutulan yolun sütunudur muhtemelen. Senin soruna ek olarak bende şunu sorayım oradaki değişken nerden geliyor ?

mgsmus

@coder2
Ben Album ile Photo modeli arasında ilişki olduğunu varsaydım, konu o olmadığı için kod olarak belirtmedim. Album ile Photo arasında one-to-many ilişkisi olduğunu varsayın.
Oradaki $album değişkeni o sırada işlem yapılan Album modelinin örneği (instance) oluyor. Örneğin o sırada silinen Album id'si 5 olan album olsun, bu durumda $album = Album::find(5) gibi düşünebilirsiniz. Album::$cover alanı da örnek, size göstermek için modelde öyle bir alan olduğunu varsaydım. Aşağıda neri'ye o değişkenin ne olduğunu, nasıl olduğunu anlattım.

@neri
Ordaki değişken, yani $album, bir yerden gelmiyor. Onu siz veriyorsunuz. Oradaki değişken zaten $album değil, function($album) {} şeklinde fonksiyonun komple kendisi deleted yönteminin değişkeni. Buna anonim fonksiyon denir, üstteki gibi anonim fonksiyonun başka bir fonksiyonun parametresi olarak kullanımına da genellikle callback denir. Javascript'te karşınıza çok çıkar. Yani üstteki örnekte aslında static::deleted($callback) olmuş oluyor. deleted yöntemi ateşlenip işlemler yapıldıktan sonra Laravel sizin bu anonim fonksiyonun parametresine modelin o anki örneğini yükletip geri dönderiyor. Dönen anonim fonksiyon olduğu için direk çalışıyor, $album değişkeni de coder2'ye verdiğim cevaptaki gibi Album modelinin o anki örneği oluyor. İçeride kullanabilmeniz için. Şöyle bir örnek vereyim mesela:

class Message
{
 public function get()
 {
 return "Merhaba Dünya";
 }
}

class Messenger
{
 public function give($callback)
 {
 return $callback(new Message);
 }
}

$messenger = new Messenger();

$messenger->give(function($message) {
 echo $message->get(); // Merhaba Dünya
});

https://www.php.net/manual/en/functions.anonymous.php

coder2

@mgsmus , şimdi taşlar yerine oturdu hocam. Teşekkürler.

deathisonitsway

şimdi mgsmsun yazdıklarını görünce hata çok bariz burada @neri..

destroy modeli sayı dönüyor.yani kaç kayıt sildiyse o kadar counter yapıp o sayıyı döndürüyor sana.
yani 0 icin false yakalarsın ama diğerleri için tümden true gelir.Arada birini silemezse yine true gelcek sana.
O yüzden bu destroy modelini kullanırken dikkat etmelisin.

Destroy methodunun alacağı parametreler;


destroy([1,2,3]);

destroy(1);

destroy(Collection::class) // ılluminate/support/collection instance

delete methodu da her türlü true dönderiyor.Onada güvenmemelisin.

öncesi ve sonrası olaylar cok basit..laravel bazı methodlar için fireModelevent çalıştırıyor.bunla deleting deleted saving saved updating updated replicating gibi method kullanımlarıyla yapıyor.bu methodları tespit ettiğinde callback değeri event mekanizmasına listen aracılığı ile kaydediyor.

ve daha sonra dispatch yapıyor bu callback değeri.yani oncesinde [ing] ler sonrasında [ed] ler çalıştırılıyor.

en iyi tutarlılığa sahip model event çalıştırılacak kısım bence save methodu.
save kaydedilip kaydedilmediğini yada şöyle söyleyeyim.

model event icin saving calıstırıyor yani oncesinde bir event..başarısızsa boolean false dönderiyor.Eğer true gelirse ana queryi calıstırıyor yani save provess...eğer buradan kayıt başarısız dönerse false dönderiyor ve model saved calıstırılmıyor.

delete felan komple her türlü true dönderiyor.

ac1919

Dostlar cok aciklayici olmus cevaplar satir satir yazi yazan emiginize tesekkur ederim. Kodlari sayenizde refactor edebiyoruz daha duzgun kod yaziyoruz

Aslinda bu konuyla ilgili sunu da merak ediyorum bazi paketlerin birden fazla tablosu oluyor ama delete islemi yapinca tablolar birbiryle iliskili olsa da diger kayit silinmeyebiliyor, bu delete islemini paketin kendi modelinde yapamayiz galiba cunku composer update yapinca islemler resetlenir saniyorum. AppServiceProvider icinde mi yapmaliyiz ?

deathisonitsway

Esasen relationshipe sahip tablolar da mysql olarak foreign key kullanmalısın.yani tablo bir üst tabloya foreign key olarak bağlı olmalı.
Eğer birbirine bağlı verilerden herhangi birine sahip ve kullanıcı bunu silmek istiyorsa foreign key engel olacaktır buna.
Tam tersi durumlar içinde, yani foreign keyi olan bir veri için tüm bağlı ilişkileri silsin istiyorsan bu seferde foreign keye bağlı option olarak ON_DELETE_CASCADE yöntemini kullanacaksın..

Hiç bir foreign keye bağlı olmayan veriler için birden fazla tablo ilişkisel yada ilişkisel olmayan(multiple data) veri silme için..

https://laravel.com/docs/5.8/database#database-transactions

laravelin transaction yöntemini kullanmalısınız.Kacınılmazdır ve veri kaybını önleyecektir.
Transaction kullandığınızda herhangi bir tabloda çoklu kayıtlarınızda herhangi bir arıza olduğunda database rollback yaparak size hata donecektir.

UYARI: sakın composer satıcı dosyalarına müdahale etmeyin.Aptalca bir yöntemdir.

İyi calısmalar.

ac1919

@deathisonitsway tamam hocam bilgiler için teşekkür ederim.