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

Callback function ile alt array oluşturma

mertefe · 2020-09-02T08:52:11+00:00

Merhaba, Kullanıcı yönetim panelinde arama input'una email adresi yazılıp enter'a basıldığında veritabanındaki ilgili satırı çekip inputları doldurmaya çal...

mertefe

Aşağıdaki şekilde tamamen sorunum çözülmüş vaziyette lakin ben bunu $data içindeki address kısmında fonksiyon şeklinde olduğu yere geri dönen bir biçimde yapmak istiyorum. Halen nasıl yapacağımı bulamadım olayları birbirine karıştıryor bile olabilirim şuan. 🙂

public function getUserData($email){
        $users = User::all();
        foreach ($users as $user){
            if(Crypt::decryptString($user->email) === $email){
                $address = array();
                foreach ($user->with('address')->first()->address as $row){
                    array_push($address,[
                        "title" => $row->title,
                        "address" => Crypt::decryptString($row->address)]);
                }
                $data = [
                    "name" => Crypt::decryptString($user->name),
                    "email" => Crypt::decryptString($user->email),
                    "phone" => Crypt::decryptString($user->phone),
                    "address" => $address
                ];
                return response()->json($data);
            }
        }
        return response('',404);
    }

mertefe

Böyle yapınca çalışıyor ama $temp() yerine direk fonksiyonu yazınca çalışmıyor.

            $temp = function () use ($user){
                    $address = array();
                    foreach ($user->with('address')->first()->address as $row){
                        array_push($address, [
                            "title" => $row->title,
                            "address" => Crypt::decryptString($row->address)]);
                    }
                    return $address;
                };
                $data = [
                    "name" => Crypt::decryptString($user->name),
                    "email" => Crypt::decryptString($user->email),
                    "phone" => Crypt::decryptString($user->phone),
                    "address" => $temp()
                ];

mgsmus

mertefe

Yapmak istediğiniz bu mu?

public function getUserData($email)
{
    $user = User::with('address')
        ->where('email', Crypt::cryptString($email))
        ->firstOrFail();

    return response()->json([
        "name" => Crypt::decryptString($user->name),
        "email" => Crypt::decryptString($user->email),
        "phone" => Crypt::decryptString($user->phone),
        "address" => $user->address->map(function ($addr) {
            return [
                'title' => $addr->title,
                'address' => Crypt::decryptString($addr->address)
            ];
        }),
    ]);
}

Ayrıca User ile Adress arasında hasMany kurduysanız ilişkinin adı address değil addresses olmalı

mertefe

mgsmus

->map() kısmı işimi çözdü bunun ne olduğu bilmiyorum ama araştıracağım.

Aşağıdaki kod bloğu işe yaramıyor ne yazık ki çünkü her encrpyt olayında farklı bir string üretiyor e-postalar aynı bile olsa çıktı farklı olduğu için mecburen tüm user'larda gezip decrypt edip öyle karşılaştırmam gerekiyor.

 $user = User::with('address')
        ->where('email', Crypt::cryptString($email))
        ->firstOrFail();

evet addresses konusunda haklısınız google translate çevirirken yalnış yaptım sanırım. şimdi tüm projeyi refaktör etmem gerek 😂

Teşekkür ederim.

mgsmus

mertefe Şimdi ne olacak, yüz bin kullanıcı varsa bir kullanıcıya ulaşmak için hepsini foreach ile dönecek misiniz?

mertefe

mgsmus Peki bunu nasıl daha iyi hale getirebilirim. Ne yapmam gerekir ?

coder2

Hocam tüm verileri encrypt ile şifreleri tutmanız pek sağlıklı kullanım gelmedi. Encrypt edildiği için direk eşleştirme yapamıyor foreach sokuyorsunuz. Çok gerekli değilse verileri açık tutup direk condition ile gerekli fieldleri almanız lazım. Yoksa bu mantıkla veri seti büyüdükçe response süresi uzar ve sonunda cevap veremez hale gelebilir.

mertefe

coder2 Evet hocam farkındayım ama bu kişisel veriye giriyor diye biliyorum. Güvenlik nedeni ile şifrelemem gerekiyor diye düşünüyorum. email yerine şifrelemeden username kullansam bu seferde user register esnasında mecburen email varlığını kontrol etmem gerek yine user'lar üzerinde foreach atmak durumunda kalıyorum. Çok mu paranoyakça davranıyorum bilemedim 😃

mgsmus

mertefe Kişisel veri ile kastettiğiniz KVKK ise; siz veriyi tuttuğunuzda başlıyor, nasıl tuttuğunuzun bir önemi yok 🙂 Şunu bir okuyun isterseniz https://www.kvkk.gov.tr/SharedFolderServer/CMSFiles/7d5b0a2f-e0ea-41e0-bf0b-bc9e43dfb57a.pdf

mertefe

mgsmus Teşekkür ederim hocam 🙂 Okuyup ona göre davranırım.

yusufc

Güvenlik nedeni ile verilerinizi şifrelemeniz gerekmiyor.
Daha doğrusu veritabanında şifreli olarak saklamanız gerekmiyor.
En doğrusu verileri şifre alanı dışında normal şekilde saklamak ve bu eforu diğer güvenlik kontrollerine harcamak olur.
Eğer KVKK'ya göre "veriyi işleyen" konumundaysanız (ki veriyi kayıt ettiğiniz anda veriyi işler konuma gelirsiniz) bu durumda yapılacak en iyi şey diğer güvenlik kontrollerini sağlam tutmaktır.

"Ben emniyetten arıyorum, ihbar var, hemen acil şu kişinin telefonunu bize verin" denildiğinde bi saniye deyip PHPStorm'u açmak ve decryptString yazmak doğru değil 🙂

mertefe

yusufc 😂 Teşekkürler. Projeyi baştan sona inceleyip düzenledim.