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

Laravel eski veri silip yeni veri güncelleme sorunu

talhaguney45

Hayırlı akşamlar efendim 2 adet güncelleme kodum var ama biri çalışmıyor diğerini sildiğimde sorunsuz çalışıyor acaba nerede hata yapıyorum yardımcı olursanız sevinirim şimdiden emeklerine çok teşekkür ediyorum

DB::table('araclar')->where('islem_id','=',$id)->dropColumn(['islem_id','cekici_id']);

  //Yeni Araç Ekle
  foreach ($request->araclar as $arac_id) {
      DB::table('araclar')->where('id','=',$arac_id)->update(['islem_id'=>$islemekle->id,'cekici_id'=>$request->cekici]);
 }

mgsmus

talhaguney45 İlki iyi ki çalışmıyor 🙂

sineld

Kodlarınızı düzenledim, lütfen bu başlığı inceleyiniz:
https://laravel.gen.tr/d/4903-onemli-forumda-kod-paylasirken-dikkat-edilmesi-gerekenler

talhaguney45

mgsmus

mgsmus sağolasın hocam nasıl bir yol izlemeliyim peki ? 😀

talhaguney45

mgsmus hocam peki sizin yaptığınız bir yöntem var mı düzenleme kısmında eski kayıtları araçlar tablosundan işlem id ve cekici id önce silecek sonra yenilerini ekleyecek baya takıldım baya da bi kod denedim ama hala içinden çıkamadım sizide rahatsız ediyorum ama kusuruma bakmayın

coder2

DB::table('araclar')->where('islem_id','=',$id)->dropColumn(['islem_id','cekici_id']);

DB facade içerisinde dropColumn methodu yok, ondan dolayı çalışmayacaktır. Dropcolumn migration içinde kullandığımız ve tabloda varolan bir kolonu silmeye yarayan method ama dediğim gibi DB facade içinde geçmiyor, Blueprint de tanımlı.

  //Yeni Araç Ekle
  foreach ($request->araclar as $arac_id) {
      DB::table('araclar')->where('id','=',$arac_id)->update(['islem_id'=>$islemekle->id,'cekici_id'=>$request->cekici]);
 }

Burada ise comment olarak "Yeni Araç Ekle" yazmışsınız ama update işlemi var, kafa karıştırıcı olmuş. Ben olsam aşağıdaki şekilde kullanırdım.

Vehicle::query()
    ->whereIn('id', $request->input('vehicle_ids'))
    ->update([
        'transaction_id' => $transaction->id,
        'tow_truck_id' => $request->input('tow_truck_id'),
    ]);

talhaguney45

coder2 hocam orada yapılan üst kısımda eski olan araç ve çekici idlerini silmek alttaki update de yeni seçilen araç ve çekici idlerini update ile güncellemek ben asıl yapmak istediğim bu aslında

mgsmus

talhaguney45 Ne yapmaya çalıştığınızı anlamaya çalışıyorum. update yöntemi zaten eski kaydın üstüne yazar, eskileri silinmiş gibi olur.

talhaguney45

mgsmus şöyle söyleyim hocam size id si 2 ve 3 olan araçları getiren çekici işlemini güncelliyoruz ve kullanıcı güncelleme işleminde araç da değiştirebiliyor ve değiştirdiği zaman eski araçların id lerini yenileri ile değiştirmem lazım benim işlemler araçlar ve çekiciler diye tablom var işlem eklendiğinde eklenen araçlar araçlar tablosunda bulunan işlem id ile işlemler tablosuna bağlı yani işlemler tablomda araç id tutmuyorum çünkü bir işlem ile birden fazla araç çekilebiliyor o yüzden kullanıcı da bir işlemi düzenlerken birden fazla araç seçili geliyor ama araç değiştirdiğinde 2 tane update işlemi yapmam gerekiyor eski id leri silip yeni araçlara işlem idlerini ekletmem gerekiyor yada siz daha farklı bir veri tabanı yapısı yapın derseniz ona göre de düzenleyebilirim hocam projeyi siz nasıl derseniz

mgsmus

talhaguney45 Anlayamadım. Tablo yapısını paylaşırsanız daha iyi olur ve lütfen düzgün paylaşmaya çalışın.

talhaguney45

mgsmus
Update fonksiyonum bu şekilde hocam

public function update(Request $request, $id)
    {
      $request->validate([
        'islem'=>'min:10',
        'Ücret'=>'min:1',
        'adres'=>'min:5'
      ]);



      $islemekle= islem::findOrFail($id);
      $islemekle->cekici_id=$request->cekici;
      $islemekle->yapilan_islem=$request->islem;
      $islemekle->ucret=$request->ucret;
      $islemekle->adres=$request->adres;
      $islemekle->durum=$request->durum;
      $islemekle->save();

      //Eski Araçları Kaldır
      DB::table('araclar')->where('islem_id','=',$id)->dropColumn(['islem_id','cekici_id']);
      //Yeni Araç Ekle
      foreach ($request->araclar as $arac_id) {
      DB::table('araclar')->where('id','=',$arac_id)->update(['islem_id'=>$islemekle->id,'cekici_id'=>$request->cekici]);
     }

      toastr()->success('Başarılı', 'Araç işlemi başarıyla güncellendi ..');
      return redirect()->route('admin.islemler.index');
    }

mgsmus

talhaguney45 Tablo yapısını paylaşın demiştim. Tablo yapısının ne demek olduğunu bilmiyorsanız sorabilirsiniz.

talhaguney45

mgsmus hocam çok pardon ben veri yapısı anlamışım

public function up()
    {
        Schema::create('araclar', function (Blueprint $table) {
            $table->bigIncrements('id');
            $table->string('plaka');
            $table->string('sahibi');
            $table->string('adres');
            $table->string('cep_tel');
            $table->integer('islem_id');
            $table->integer('cekici_id');
            $table->integer('durum');
            $table->softDeletes();
            $table->timestamps();
        });
    }

public function up()
    {
        Schema::create('islem', function (Blueprint $table) {
            $table->bigIncrements('id');
            $table->integer('cekici_id');
            $table->string('yapilan_islem');
            $table->string('ucret');
            $table->string('adres');
            $table->string('durum');
            $table->timestamps();
        });
    }

public function up()
    {
        Schema::create('cekici', function (Blueprint $table) {
            $table->bigIncrements('id');
            $table->string('sahibi');
            $table->string('tel');
            $table->string('durum');
            $table->string('adres');
            $table->string('plaka');
            $table->timestamps();
        });
    }

mgsmus

talhaguney45 Tahmin ettiğim gibi bu tablo yapısı uygun değil. Size önerim envanter mantığı ile hareket etmeniz. Mesela (tüm bilgileri yazmıyorum, örnek vermek amacıyla kısa geçiyorum)

Schema::create('araclar', function (Blueprint $table) {
    $table->id();
    $table->boolean('cekici_mi');
    $table->string('plaka')->index();
    $table->string('sase')->unique();
    $table->timestamps();
});

Schema::create('islemler', function (Blueprint $table) {
    $table->id();
    $table->string('yapilan_islem');
    $table->timestamps();
});

Schema::create('islem_arac', function (Blueprint $table) {
    $table->foreignId('islem_id')->constrained('islemler')->cascadeOnDelete();
    $table->foreignId('arac_id')->constrained('araclar')->cascadeOnDelete();
    $table->boolean('cekici_mi');
    $table->unique(['islem_id', 'arac_id']);
});

app/Models/Islem.php

public function tumAraclar(): BelongsToMany
{
    return $this->belongsToMany(Arac::class, 'islem_arac', 'islem_id', 'arac_id');
}

public function araclar(): BelongsToMany
{
    return $this->belongsToMany(Arac::class, 'islem_arac', 'islem_id', 'arac_id')
        ->wherePivot('cekici_mi', false);
}

public function cekiciler(): BelongsToMany
{
    return $this->belongsToMany(Arac::class, 'islem_arac', 'islem_id', 'arac_id')
        ->wherePivot('cekici_mi', true);
}

public function aracYadaCekiciEkle(Arac|iterable $araclar): bool
{
    if($araclar instanceof Arac) {
        $this->tumAraclar()
            ->attach($araclar, [
                'cekici_mi' => $araclar->cekici_mi,
            ]);

        return true;
    }

    $compiled = [];

    foreach($araclar as $arac) {
        $compiled[$arac->id] = [
            'cekici_mi' => $arac->cekici_mi,
        ];
    }

    $this->tumAraclar()
        ->attach($compiled);

    return true;
}

app/Models/Arac.php

public function aracIslemleri(): BelongsToMany
{
    return $this->belongsToMany(Islem::class, 'islem_arac', 'arac_id'. 'islem_id')
        ->wherePivot('cekici_mi', false);
}

public function cekiciIslemleri(): BelongsToMany
{
    return $this->belongsToMany(Islem::class, 'islem_arac', 'arac_id', 'islem_id')
        ->wherePivot('cekici_mi', true);
}

$arac = Arac::create([
    'plaka' => '34ABC123',
    'sase' => '1GCJK33104F173427',
    'cekici_mi' => false,
]);

$cekici = Arac::create([
    'plaka' => '34CDE456',
    'sase' => 'JNKCA21A4XT770979',
    'cekici_mi' => true,
]);

$islem = Islem::create([
    'yapilan_islem' => 'Yanlış yere parkedildiği için çekildi.'
]);

$islem->aracYadaCekiciEkle([$arac, $cekici]);

$islem = Islem::find(1);

// İşleme karışan çekiciler
$cekiciler = $islem->cekiciler;

// Çekilen araçlar
$araclar = $islem->araclar;

// Yeni araç ekle
$yeniArac = Arac::create([
    'plaka' => '34EFG789',
    'sase' => '1D4GP24R57B255713',
    'cekici_mi' => false,
]);

// Eski araçları kaldır
$islem->araclar()
    ->detach();

// Yeni aracı ekle
$islem->aracYadaCekiciEkle($yeniArac);

// Yeni araç ekle
$baskaBirArac = Arac::create([
    'plaka' => '34AB321',
    'sase' => 'JH4DA9340PS000417',
    'cekici_mi' => false,
]);

// Eski araçları ellemeden ekleme yap
$islem->aracYadaCekiciEkle($baskaBirArac);

//  Çekicileri sil
$islem->cekiciler()
    ->detach();

$yeniCekici = Arac::create([
    'plaka' => '34GG121',
    'sase' => 'JKBVNKD167A013982',
    'cekici_mi' => true,
]);

// Yeni çekici ekle
$islem->aracYadaCekiciEkle($yeniCekici);

return $islem;

{
  "id": 1,
  "yapilan_islem": "Yanlış yere parkedildiği için çekildi.",
  "created_at": "2022-06-02T23:47:16.000000Z",
  "updated_at": "2022-06-02T23:47:16.000000Z",
  "cekiciler": [
    {
      "id": 5,
      "cekici_mi": 1,
      "plaka": "34GG121",
      "sase": "JKBVNKD167A013982",
      "created_at": "2022-06-02T23:47:18.000000Z",
      "updated_at": "2022-06-02T23:47:18.000000Z",
      "pivot": {
        "islem_id": 1,
        "arac_id": 5
      }
    }
  ],
  "araclar": [
    {
      "id": 3,
      "cekici_mi": 0,
      "plaka": "34EFG789",
      "sase": "1D4GP24R57B255713",
      "created_at": "2022-06-02T23:47:18.000000Z",
      "updated_at": "2022-06-02T23:47:18.000000Z",
      "pivot": {
        "islem_id": 1,
        "arac_id": 3
      }
    },
    {
      "id": 4,
      "cekici_mi": 0,
      "plaka": "34AB321",
      "sase": "JH4DA9340PS000417",
      "created_at": "2022-06-02T23:47:18.000000Z",
      "updated_at": "2022-06-02T23:47:18.000000Z",
      "pivot": {
        "islem_id": 1,
        "arac_id": 4
      }
    }
  ]
}

Bu şekilde sisteminizde bir araç envanteri oluşturmuş olursunuz ve aynı araçları sonraki işlemlerde de kullanabilirsiniz. Her seferinde aynı aracı oluşturmamış olursunuz. Bu yapıda bir işlemde birden fazla araç ve çekici olabiliyor. Özellikle araçlara şaşe numarası da ekledim, plaka değişme olasılığı var ama şaşe değişmez. Hani olur da aynı araç farklı bir plaka ile gelirse plakasını şase numarasından aracı bulup güncelleyebilirsiniz.

talhaguney45

mgsmus elinize kolunuza sağlık hocam deneyeceğim akşama