Feedback_2

Geribildirim_2.cs

// Referanslar
// [1] https://gist.github.com/antonsem/09b4f902ed22adbdf8f664fac539543f
// [2] https://gist.github.com/antonsem/1b34acd82470f34a1fd45160c180834b

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;

// [1]'de bahsettiğim gibi Türkçe değişken, sınıf ve metod isimleri kullanmak
// iyi bir fikir değil. Üstelik burada hem Türkçe hem de İngilizce isimler
// kullanılıyor, bir tutarlılık yok.

// Gözüme çarpan ikinci problem bu sınıfın çok büyük olması. İlk bakışta
// gördüğüm kadarıyla bu sınıf hem karakterin hareketlerinden, hem
// animasyonlarından hem konuşmasından hem de alışverişinden (ve kim bilir daha
// nelerden) sorumlu. Bu sınıf en az dört farklı sınıfa bölünebilir
public class Karakter : MonoBehaviour
{
    // Görev ayrılığı problemi burada da göze çarpıyor. Hangi değişken nerede,
    // kim kimden sorumlu belli değil. Her şey bir arada. Ayrı sınıflara
    // ayrılmayacaksa bile bunların farklı gruplara ayrılmasında fayda var.

    // Bu değişkenler neden public? Eğer amaç sadece editörden atamak ise
    // [2]'de bahsettiğim gibi private yapıp başına [SerializedField] eklemek
    // daha mantıklı.
    public Rigidbody2D rb;
    Animator playeranimator;
    Vector3 velocity;
    public float walkspeed=4f;
    public float jumpspeed = 6f;
    bool facingright = true;
    // isgrounded ilgili metodun dışında herhangi bir yerde kullanılmıyor
    public bool isgrounded = false;
    public Transform grouncheckposition;
    public float groundcheckradius;
    public LayerMask groundchecklayer;
    public Transform bullet;
    Transform muzzle;
    public float bulletspeed;
    public int bulletmagazine=5;
    public Text bullettext;
    // Bunun double olması için özel bir sebep yoksa float fazla fazla işini görecektir
    double parasayi = 0;
    public Text paratext;
    // Health ile can farklı mı? Can herhangi bir yerde kullanılıyor mu?
    public int can = 100;
    public Text teztext;
    int tezsayi = 0;
    public float health=100;
    // Bu neyin slider'ı? Değişken isimleri anlaşılır olmalı
    public Slider slider;
    // player objesi bu scriptin olduğu objeden farklı mı? Değilse neden ayrı bir 
    // değişkeni var?
    public GameObject player;
    public Vector3 checkpos;
    public GameObject brosur;
    bool basildi = false;
    bool bakkalmi = false;
    public Text brosurpara;
    public GameObject konusma;
    bool konusmahakki = true;
    bool tiklandi=false;


    
    // İsimlendirmelerde kullanılan yöntem hiçbir standarda uymadığı gibi okumayı
    // zorlaştırıyor. Mesela playeranimator isminde iki kelime var. Ama bunu bir
    // bakışta ayırt etmek zor. Onun yerine playerAnimator veya player_animator
    // yazılabilir. C# standartlarına göre playerAnimator daha uygun bir yazım
    // şekli. C# standartlarına şuaradan ulaşabilirsin:
    // https://jesseliberty.com/2020/01/29/c-coding-standards/
    
    // Kendiliğinden ekli olan yorumları silmekte fayda var. Hiçbir işe yaramıyorlar.
    // Start is called before the first frame update
    void Start()
    {
        // Eğer Animator oyun başladıktan sonra eklenmiyorsa bunu editörden atamamak
        // için bir sebep yok.
        playeranimator = GetComponent<Animator>();
        // transofrm.GetChild(1) oldukça kötü bir yöntem. Yarın öbür gün o objenin
        // birinci çocuğu bir sebepten ötürü değişirse bu kod yanlış çalışacaktır.
        // Gördüğüm kadarıyla bu işlemi de start'ta yapmak için bir neden yok. Bu da
        // editör'den atanabilir.
        muzzle = transform.GetChild(1);
        // Bu işlemleri birden fazla defa birçok farklı yerden yapıyorsun. O yüzden
        // hepsini birer metoda aktarman daha mantıklı olacaktır. Eğer bir sebepten
        // ötürü farklı bir yazı eklemek istersen kodunun farklı yerlerinde aramak
        // yerine tek bir yerden değiştirebilirsin.
        bullettext.text = "" + bulletmagazine;
        paratext.text = "" + parasayi;
        teztext.text = "" + tezsayi;
        slider.maxValue = health;
        slider.value = health;
        brosur.SetActive(false);
        brosurpara.text = "" + parasayi + "₺";
        konusma.SetActive(false);
    }
    // Metodlar arasına boşluk bırakmak kodu okumayı kolaylaştırır
    void walkpc()
    {
        // Burada Vector3 tanımlayıp sadece iki elemanını, X ve Y'yi atadın.
        // Z otomatik olarak 0 atandığı için büyük bir sıkıntı değil, ama ileride kafa
        // karıştırabilir. Ayrıca sabit string değişkenleri ayrı bir sınıfta tutmakta
        // fayda var. Nasıl yapılacağını şuarada anlattım: 
        // https://tr.anton.website/sabit-degiskenleri-tutmak/
        velocity = new Vector3(Input.GetAxis("Horizontal"), 0f);
        // Gördüğüm kadarıyla bu karakterin bir RigidBody'si var. RigidBody olduğu
        // zaman objeyi transform ile hareket ettirmek gereksiz ekstra hesaplamalara
        // sebep veriyor. Kendi başına çok büyük bir performans kaybına sebep
        // olmayacaktır ama akılda tutmakta fayda var.
        transform.position += velocity * walkspeed * Time.deltaTime;
        // Animasyonlar fiziksel hareketten bağımsız olduğu için en azından farklı bir
        // metodda halledilirse çok daha iyi olur.
        playeranimator.SetFloat("playerspeed", Mathf.Abs(velocity.x));
    }
    void jump()
    {
        // Burada fazladan bir "karakter yerde ise" gibi bir kontrol ekleyebilirsin.
        // Şu anda karakter zıpladıktan sonra yere düşmeye başlamadan hemen önce bir
        // daha zıplayabilir.
       if (Mathf.Approximately(velocity.y, 0)) { 
        
            rb.AddForce(Vector2.up * jumpspeed, ForceMode2D.Impulse);
        }
    }
    // Update is called once per frame
    void Update()
    {
        // GetKey metodlarında string yerine KeyCode enum'ını kullanmayı tercih et.
        // Input.GetKeyDown(KeyCode.R) gibi.
        if (Input.GetKeyDown("r"))
        {
            shootbullet();
        }
        // Burada öncelikle karakter yerde mi diye kontrol ediyorsun, sonra yürütüyorsun,
        // sonra zıplama tuşuna basmış mı, sonra dikey hızı 0 mı diye kontrol ediyorsun,
        // sonra aynı kontrolü bir de w tuşu için yapıyorsun. Daha da sonra yine bu hız
        // kontrolünü jump() metodunun içnde yapıyorsun. Eğer karakter yerdeyse dikey
        // hızı zaten sıfırdır. Üç kere üst üste kontrol etmene gerek yok bunu.
        OnGroundCheck();
        walkpc();
        if (Input.GetButtonDown("Jump") && Mathf.Approximately(rb.velocity.y, 0) || Input.GetKeyDown(KeyCode.W) && Mathf.Approximately(rb.velocity.y, 0))
        { 
        jump();
        }

        // basildi diye bir değişken tanımlamışsın ama bu metodu bilmeyen birisi
        // neyin basılıp basıldığını bilmeyecek. Değişken isimleri net olmalı, ne işe
        // yaradıklarına dair akıllarda soru işareti bırakmamalı. Ayrıca değişkeni
        // çok alakasız bir yerde atadığın için burada nelerin döndüğünü anlamak çok
        // zor oluyor. Onun yerine ebasildimi() metodu sonucu bool olarak dönse daha
        // güzel olur.
        ebasildimi();




        if (velocity.x<0 && facingright)
        {
            flipface();
        }
        else if (velocity.x>0 && !facingright)
        {
            flipface();
        }

        // bool'ları kontrol ederken == kullanmana gerek yok. Direkt
        // if(basildl && bakkalmi) diyebilirsin.
        if (basildi==true && bakkalmi==true) {
            brosur.SetActive(true);
        }
        else if (basildi==false && bakkalmi==false)
        {
            brosur.SetActive(false);
        }
       



    }

    void flipface()
    {
        // Bu durumda karakterin scale'ı ile oynamak çok iyi bir fikir olmayabilir
        // çünkü üzerinde bir RigidBody var ve bu durum fizik sisteminin hoşuna
        // gitmeyebilir. Ama şimdi alternatif yöntemlere girmeyeceğim
        facingright = !facingright;
        Vector3 templocalescale = transform.localScale;
        templocalescale.x *= -1;
        transform.localScale = templocalescale;
    }
    void OnGroundCheck()
    {
        isgrounded = Physics2D.OverlapCircle(grouncheckposition.position, groundcheckradius, groundchecklayer);
        playeranimator.SetBool("isgroundanim", isgrounded);
    }
    public void OnTriggerEnter2D(Collider2D other)
    {
        // tag karşılaştırmasından yine sabit değişken yazımda bahsettim.
        // Onun haricinde tag'leri spesifik objeler için değil obje grupları
        // için kullanmakta fayda var. Hatta hiç kullanmamakta fayda var ama
        // alternatif yöntemler çok farklı bir konu olacağı için
        // ayrıntılara girmeyeceğim
        if (other.gameObject.tag == "pencil")
        {
            bulletmagazine = bulletmagazine + 2;
            bullettext.text = "" + bulletmagazine;
            // Her durumda Destroy'u çağırıyorsun. O yüzden if'lerin dışına
            // çıkarabiliriz bunu.
            Destroy(other.gameObject);
        }
        else if (other.gameObject.tag == "para")
        {
            // burada ve daha sonra float değerlerinin sonuna f yazmadın. Sanırım
            // float yerine parayı double olarak tutmanın sebebi bu. Eğer 0.50f
            // yazarsan problemin çözülecektir ve double kullanmana gerek olmayacaktır
            parasayi= parasayi+ 0.50;
            paratext.text = "" + parasayi;
            brosurpara.text = "" + parasayi + "₺";
            Destroy(other.gameObject);
        }
        else if (other.gameObject.tag == "Tez")
        {
            tezsayi++;
            teztext.text = "" + tezsayi;
            Destroy(other.gameObject);
           
        }
        else if (other.gameObject.tag == "Enemy")
        {
            // Burası olduğu gibi farklı bir class'ta olmalı, ama
            // bu da uzun bir konu olduğu için şimdilik es geçiyorum
            if (konusmahakki == true) 
            {
                konusma.SetActive(true);
                Time.timeScale = 0;
                Debug.Log("test1");
                konusmahakki = false;
                Debug.Log("Test2");
                
                // Burayı anladığım kadarıyla "Eğer düşmana çarptıysam
                // ve konuşma hakkım varsa ve E'ye basıyorsam o zaman
                // düşman ile konuşayım" gibi bir şey demek istiyorsun.
                // Ancak yukarıdaki kodlardan da anladığım kadarıyla bir
                // yandan zaman dursun, konuşayım istiyorsun. Öyle bir
                // şey olmaz çünkü hem üstteki hem buradaki kodlar bir
                // anda çalışacak. Bu nedenle buraya dokunmayacağım.
                if (tiklandi == true)
                {
                    Debug.Log("Test4");

                    parasayi = parasayi - 0.50;
                    paratext.text = "" + parasayi + "₺";
                    brosurpara.text = "" + parasayi + "₺";
                    Destroy(other.gameObject);
                    Time.timeScale = 1;
                    konusmahakki = true;
                    tiklandi = false;
                    
                }
            }
           

            //player.transform.position = checkpos;
            // Bu tarz işlemleri healt -= 20 diye yapabilirsin
            // health işlemlerini yaparken hep aynı işi yapıyorsun:
            // 1. değeri değiştir
            // 2. slider'ın değerini güncelle
            // 3. ölüp ölmediğini kontrol et
            // bunları bir metoda alabilirsin.
            health = health - 20;
            slider.value = health;
            amidead();
        }
        
    }
    
    // Unity'den gelen Update, Start, Awake, OnCollisionEnter gibi metodları
    // boş bırakma. Gereksiz yere kodu uzatıp performans kaybına neden oluyorlar.
    // Senin yazdığın metodlarda bu kayıp söz konusu değil çünkü Unity kendi
    // metodlarını biraz daha farklı bir şekilde çağırıyor. Çok çok ufak bir
    // kayıp oluyor bu ama neden olsun :D
    private void OnCollisionEnter2D(Collision2D collision)
    {
        
    }
    private void OnCollisionStay2D(Collision2D collision)
    {
        // OnCollisionStay kullanmışsın ama kodundan anladığım kadarıyla
        // oyuncunun suya veya tuzağa değdiği anda checkpoint'e gitmesini
        // istiyorsun. Yani aslında "stay" için bir şey yok. Rahatlıkla
        // OnCollisionEnter kullanılabilir.
        if (collision.gameObject.tag == "Tuzak")
        {
            player.transform.position = checkpos;
            health = health - 10;
            slider.value = health;
            amidead();
        }
        if (collision.gameObject.tag == "Su")
        {
            player.transform.position = checkpos;
            health = health - 50;
            slider.value = health;
            amidead();
        }
    }
    private void OnTriggerStay2D(Collider2D other)
    {
        // Burada tag'i bakkal olan bir trigger'da kaldığın sürece
        // bakkalmi bool'u true olacak. Ancak farklı bir trigger'a
        // girdiğin anda işler karışabilir çünkü metod bu defa
        // else'e girecektir. Yine olması gereken şey bakkal'ın kendi
        // sınıfının olması ve E'ye basınca oradan bir metodun
        // çağırılması. Yine konu uzun olduğu için o ayrıntılara
        // girmeyeceğim. Ayrica bu trigger'ın içinde olduğun sürece
        // bakkalmi true olacak ve trigger'dan ayrılınca da true
        // olarak kalacak. Bu işlemi hiç değilse OnTriggerEnter2D
        // ve OnTriggerExit2D ile halletmeyi deneyebilirsin.
        if (other.gameObject.tag == "bakkal" )
        {
            
                Debug.Log("E basıldı");
               
             bakkalmi = true;

            
        }
        else
        {
            bakkalmi = false;
        }

    }

    public void shootbullet()
    {
        if (bulletmagazine>0)
        {
            // Farklı sınıfta olması gereken bir şey daha. Oyuncu kontrolcüsü
            // mermi hızı ile falan ilgilenmemeli. En fazla Bullet gibi bir
            // component'e sahip bir mermi objesi yaratabilirsin ve içine
            // Shoot(Vector3 forward) gibi bir metod tanımlarsın. Sonra
            // AddForce vs yerine tempbullet.Shoot(muzzle.forward) diye çağırırsın.
            Transform tempbullet;
            tempbullet = Instantiate(bullet, muzzle.position, Quaternion.identity);
            tempbullet.GetComponent<Rigidbody2D>().AddForce(muzzle.forward * bulletspeed);
            bulletmagazine--;
            bullettext.text = "" + bulletmagazine;
        }
        else
        {
            Debug.Log("Mermi Yok");
        }
        
    }
    public void amidead()
    {
        if (health <= 0)
        {
            slider.value = health;
            Destroy(player);

        }
    }
    public void kapat()
    {
        // Madem ki bu işin için bir metod yazdın neden yukarıda yine 
        // brosur.SetActive metodunu çağırıyorsun?
        brosur.SetActive(false);
    }

    public void ebasildimi()
    {
        // Bunun yerine Input.GetKey("e") yazarsan tuşun basılı olduğu
        // her karede true döner. Basıldı gibi ekstra bir değişkene
        // ihtiyacın yok yani
        if (Input.GetKeyDown("e"))
        {
            Debug.Log("E basıldı2");
            basildi = true;
           
           

        }
        else if (Input.GetKeyUp("e"))
        {
            Debug.Log("Elini çekti");
            basildi = false;
            
        }
    }

    // Metod isimleri hiç anlaşılır değil. Baktığım zaman satinal1 ve
    // satinal2 arasındaki farkı anlamak mümkün değil. Metod isimleri
    // ne iş yaptıklarını anlatmalı
    public void satinal1()
    {
        if (parasayi>=1)
        {
            parasayi = parasayi - 1;
            paratext.text = "" + parasayi;
            bulletmagazine++;
            bullettext.text = "" + bulletmagazine;
            brosurpara.text = "" + parasayi+ "₺";
            Debug.Log("Satin alindi");
        }
        else if (parasayi<1)
        {
            Debug.Log("Yetersiz Bakiye");
        }
    }
    public void satinal2()
    {
        if (parasayi >= 2.5)
        {
            parasayi = parasayi - 2.5;
            paratext.text = "" + parasayi;
            health = health + 10;
            slider.value = health;
            brosurpara.text = "" + parasayi + "₺";
            Debug.Log("Satin alindi");
        }
        else if (parasayi < 2.5)
        {
            Debug.Log("Yetersiz Bakiye");
        }
    }

    public void paraver()
    {
        tiklandi = true;
        Debug.Log("Test3");
        
        

       
    }

}

Original_2.cs

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;

public class Karakter : MonoBehaviour
{
    public Rigidbody2D rb;
    Animator playeranimator;
    Vector3 velocity;
    public float walkspeed=4f;
    public float jumpspeed = 6f;
    bool facingright = true;
    public bool isgrounded = false;
    public Transform grouncheckposition;
    public float groundcheckradius;
    public LayerMask groundchecklayer;
    public Transform bullet;
    Transform muzzle;
    public float bulletspeed;
    public int bulletmagazine=5;
    public Text bullettext;
    double parasayi = 0;
    public Text paratext;
    public int can = 100;
    public Text teztext;
    int tezsayi = 0;
    public float health=100;
    public Slider slider;
    public GameObject player;
    public Vector3 checkpos;
    public GameObject brosur;
    bool basildi = false;
    bool bakkalmi = false;
    public Text brosurpara;
    public GameObject konusma;
    bool konusmahakki = true;
    bool tiklandi=false;


    // Start is called before the first frame update
    void Start()
    {
        playeranimator = GetComponent<Animator>();
        muzzle = transform.GetChild(1);
        bullettext.text = "" + bulletmagazine;
        paratext.text = "" + parasayi;
        teztext.text = "" + tezsayi;
        slider.maxValue = health;
        slider.value = health;
        brosur.SetActive(false);
        brosurpara.text = "" + parasayi + "₺";
        konusma.SetActive(false);
    }
    void walkpc()
    {
        velocity = new Vector3(Input.GetAxis("Horizontal"), 0f);
        transform.position += velocity * walkspeed * Time.deltaTime;
        playeranimator.SetFloat("playerspeed", Mathf.Abs(velocity.x));
    }
    void jump()
    {
       if (Mathf.Approximately(velocity.y, 0)) { 
        
            rb.AddForce(Vector2.up * jumpspeed, ForceMode2D.Impulse);
        }
    }
    // Update is called once per frame
    void Update()
    {
        if (Input.GetKeyDown("r"))
        {
            shootbullet();
        }
        OnGroundCheck();
        walkpc();
        if (Input.GetButtonDown("Jump") && Mathf.Approximately(rb.velocity.y, 0) || Input.GetKeyDown(KeyCode.W) && Mathf.Approximately(rb.velocity.y, 0))
        { 
        jump();
        }

        ebasildimi();




        if (velocity.x<0 && facingright)
        {
            flipface();
        }
        else if (velocity.x>0 && !facingright)
        {
            flipface();
        }


        if (basildi==true && bakkalmi==true) {
            brosur.SetActive(true);
        }
        else if (basildi==false && bakkalmi==false)
        {
            brosur.SetActive(false);
        }
       



    }

    void flipface()
    {
        facingright = !facingright;
        Vector3 templocalescale = transform.localScale;
        templocalescale.x *= -1;
        transform.localScale = templocalescale;
    }
    void OnGroundCheck()
    {
        isgrounded = Physics2D.OverlapCircle(grouncheckposition.position, groundcheckradius, groundchecklayer);
        playeranimator.SetBool("isgroundanim", isgrounded);
    }
    public void OnTriggerEnter2D(Collider2D other)
    {
        if (other.gameObject.tag == "pencil")
        {
            bulletmagazine = bulletmagazine + 2;
            bullettext.text = "" + bulletmagazine;
            Destroy(other.gameObject);
        }
        else if (other.gameObject.tag == "para")
        {
            parasayi= parasayi+ 0.50;
            paratext.text = "" + parasayi;
            brosurpara.text = "" + parasayi + "₺";
            Destroy(other.gameObject);
        }
        else if (other.gameObject.tag == "Tez")
        {
            tezsayi++;
            teztext.text = "" + tezsayi;
            Destroy(other.gameObject);
           
        }
        else if (other.gameObject.tag == "Enemy")
        {
            if (konusmahakki == true) 
            {
                konusma.SetActive(true);
                Time.timeScale = 0;
                Debug.Log("test1");
                konusmahakki = false;
                Debug.Log("Test2");
                
                if (tiklandi == true)
                {
                    Debug.Log("Test4");

                    parasayi = parasayi - 0.50;
                    paratext.text = "" + parasayi + "₺";
                    brosurpara.text = "" + parasayi + "₺";
                    Destroy(other.gameObject);
                    Time.timeScale = 1;
                    konusmahakki = true;
                    tiklandi = false;
                    
                }
            }
           

            //player.transform.position = checkpos;
            health = health - 20;
            slider.value = health;
            amidead();
        }
        
    }
    
    private void OnCollisionEnter2D(Collision2D collision)
    {
        
    }
    private void OnCollisionStay2D(Collision2D collision)
    {
        if (collision.gameObject.tag == "Tuzak")
        {
            player.transform.position = checkpos;
            health = health - 10;
            slider.value = health;
            amidead();
        }
        if (collision.gameObject.tag == "Su")
        {
            player.transform.position = checkpos;
            health = health - 50;
            slider.value = health;
            amidead();
        }
    }
    private void OnTriggerStay2D(Collider2D other)
    {
        if (other.gameObject.tag == "bakkal" )
        {
            
                Debug.Log("E basıldı");
               
             bakkalmi = true;

            
        }
        else
        {
            bakkalmi = false;
        }

    }

    public void shootbullet()
    {
        if (bulletmagazine>0)
        {
            Transform tempbullet;
            tempbullet = Instantiate(bullet, muzzle.position, Quaternion.identity);
            tempbullet.GetComponent<Rigidbody2D>().AddForce(muzzle.forward * bulletspeed);
            bulletmagazine--;
            bullettext.text = "" + bulletmagazine;
        }
        else
        {
            Debug.Log("Mermi Yok");
        }
        
    }
    public void amidead()
    {
        if (health <= 0)
        {
            slider.value = health;
            Destroy(player);

        }
    }
    public void kapat()
    {
        brosur.SetActive(false);
    }

    public void ebasildimi()
    {
        if (Input.GetKeyDown("e"))
        {
            Debug.Log("E basıldı2");
            basildi = true;
           
           

        }
        else if (Input.GetKeyUp("e"))
        {
            Debug.Log("Elini çekti");
            basildi = false;
            
        }
    }

    public void satinal1()
    {
        if (parasayi>=1)
        {
            parasayi = parasayi - 1;
            paratext.text = "" + parasayi;
            bulletmagazine++;
            bullettext.text = "" + bulletmagazine;
            brosurpara.text = "" + parasayi+ "₺";
            Debug.Log("Satin alindi");
        }
        else if (parasayi<1)
        {
            Debug.Log("Yetersiz Bakiye");
        }
    }
    public void satinal2()
    {
        if (parasayi >= 2.5)
        {
            parasayi = parasayi - 2.5;
            paratext.text = "" + parasayi;
            health = health + 10;
            slider.value = health;
            brosurpara.text = "" + parasayi + "₺";
            Debug.Log("Satin alindi");
        }
        else if (parasayi < 2.5)
        {
            Debug.Log("Yetersiz Bakiye");
        }
    }

    public void paraver()
    {
        tiklandi = true;
        Debug.Log("Test3");
        
        

       
    }

}

Refactored_2.cs

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;

public class Karakter_Refactored : MonoBehaviour
{
#region Exposed Fields

    [Header("Controller")]
    [SerializedField] private Rigidbody2D rb;
    [SerializedField] private float walkSpeed = 4f;
    [SerializedField] private float jumpSpeed = 6f;
    [SerializedField] private Transform grounCheckPointPosition;
    [SerializedField] private float groundCheckRadius;
    [SerializedField] private LayerMask groundCheckLayer;
    [SerializedField] private float health = 100;

    [Header("Weapon")]
    [SerializedField] private GameObject bullet;
    [SerializedField] private Transform muzzle;
    [SerializedField] private float bulletSpeed;
    [SerializedField] private int bulletCount = 5;

    [Header("Visuals")]
    [SerializedField] private Animator playerAnimator;
    
    [Header("UI")]
    [SerializedField] private Text bulletText;
    [SerializedField] private Text moenyText;
    [SerializedField] private Text thesisText;
    [SerializedField] private Slider healthSlider;
    [SerializedField] private GameObject brochure;
    [SerializedField] private Text brochureText;
    [SerializedField] private GameObject dialog;

    [Header("Other")]
    [SerializedField] private Vector3 checkpointPos;
    
#endregion
    
#region Private Fields

    private bool _isInMarket = false;
    private bool _canTalk = true;
    private bool _clicked = false;
    private bool _isOnTheGround = false;
    private float _horizontalInput = 0;
    private float _moneyAmount = 0;
    private int _thesisCount = 0;

#endregion

#region Unity Methods

    private void Start()
    {
        UpdateBulletText(bulletCount);
        UpdateMoneyText(_moneyAmount);
        UpdateThesisText(_thesisCount);
        InitHealtSlider(health);

        brochure.SetActive(false);
        dialog.SetActive(false);
    }

    private void Update()
    {
        _isOnTheGround = _IsOnTheGround();
        
        Walk();

        if(CanJump())
        {
            Jump();
        }

        SetAnimations();

        brochure.SetActive(Input.GetKey(KeyCode.E) && _isInMarket);
    }

    private void OnTriggerEnter2D(Collider2D other)
    {
        if(other.gameObject.CompareTag(Constants.Tags.market))
        {
            _isInMarket = true;
            return;
        }
        
        if(other.gameObject.CompareTag(Constants.Tags.pencil))
        {
            UpdateBulletCount(bulletCount + 2);
        }
        else if(other.gameObject.CompareTag(Constants.Tags.money))
        {
            UpdateMoney(_moneyAmount + 0.5f);
        }
        else if(other.gameObject.CompareTag(Constants.Tags.thesis))
        {
            UpdateThesis(_thesisCount + 1);
        }
        else if(other.gameObject.CompareTag(Constants.Tags.enemy))
        {
            // I didn't quite understand what suppose to happen here,
            // so I'll skip it.
            if (_canTalk == true) 
            {
                dialog.SetActive(true);
                Time.timeScale = 0;
                _canTalk = false;
                
                if (_clicked == true)
                {
                    UpdateMoney(_moneyAmount - 0.50f);
                    Time.timeScale = 1;
                    _canTalk = true;
                    _clicked = false;
                }
            }
        }

        Destroy(other.gameObject);
        UpdateHealth(health - 20);
    }

    private void OnTriggerExit2D(Collider2D other)
    {
        if(other.gameObject.CompareTag(Constants.Tags.market))
        {
            _isInMarket = false;
        }
    }

    private void OnCollisionEnter2D(Collision2D other)
    {
        if(other.gameObject.CompareTag(Constants.Tags.trap))
        {
            player.transform.position = checkpointPos;
            UpdateHealth(health - 10);
        }
        else if(other.gameObject.CompareTag(Constants.Tags.water))
        {
            player.transform.position = checkpointPos;
            UpdateHealth(health - 50);
        }
    }

#endregion

#region Movement

    private void Walk()
    {
        float verticalVelocity = rb.velocity.y;
        _horizontalInput = Input.GetAxis(Constants.Strings.horizontal);
        rb.velocity = new Vector2(_horizontalInput * walkSpeed, verticalVelocity);
    }

    private bool CanJump()
    {
        return Mathf.Approximately(rb.velocity.y, 0) && _isOnTheGround;
    }

    private void DoJump()
    {
        rb.AddForce(Vector2.up * jumpSpeed, ForceMode2D.Impulse);
    }

    private bool _IsOnTheGround()
    {
        return Physics2D.OverlapCircle(grounCheckPointPosition.positCon, R, C);
L  }

    private void SetAnimations()
    {
        if(transform.localScale.x * rb.velocity.x < 0)
        {
            Vector3 temp = transform.localScale;
            temp.x = Mathf.Sign(rb.velocity.x);
            transform.localScale.x = temp;
        }

        playerAnimator.SetFloat(Constants.Animations.playerSpeedHash, Mathf.Abs(_horizontalInput));
        playerAnimator.SetBool(Constants.Animations.playerGroundedHash, _isOnTheGround);
    }

#endregion

#region Market Methods

    public void BuyBullet()
    {
        if(_moneyAmount < 1)
        {
            return;
        }

        UpdateMoney(_moneyAmount - 1);
        UpdateBulletCount(bulletCount + 1);
    }

    public void BuyHealth()
    {
        if(_moneyAmount < 1)
        {
            return;
        }

        UpdateMoney(-2.5f);
        UpdateHealth(health + 10);
    }

#endregion

#region Update Methods

    private void UpdateBulletCount(int count)
    {
        bulletCount = count;
        UpdateBulletText(count);
    }

    private void UpdateBulletText(int count)
    {
        bulletText.text = count.ToString();
    }

    private void UpdateMoney(float newAmount)
    {
        _moneyAmount = newAmount;
        UpdateMoneyText(newAmount);
    }

    private void UpdateMoneyText(float newAmount)
    {
        moenyText.text = _moneyAmount.ToString();
        brochureText.text = $"{newAmount.ToString()}₺";
    }

    private void UpdateThesis(int newAmount)
    {
        _thesisCount = newAmount;
        UpdateThesisText(newAmount);
    }

    private void UpdateThesisText(int newAmount)
    {
        thesisText.text = newAmount.ToString();
    }

    private void UpdateHealthSliderValue(float healthValue)
    {
        healthSlider.value = healthValue;
    }

    private void UpdateHealth(float newAmount)
    {
        healt += newAmount;
        UpdateHealthSliderValue(healt);

        if(healt <= 0)
        {
            Destroy(gameObject);
        }
    }
    
    private void InitHealtSlider(float maxHealt)
    {
        healthSlider.maxValue = healthValue;
        SetMaxHealthValue(healthValue);
    }

#endregion

    private void Shoot()
    {
        if(bulletCount < 0)
        {
            return;
        }

        Transform temp = Instantiate(bullet, muzzle.position, Quaternion.identity).transform;
        temp.GetComponent<Rigidbody2D>().AddForce(muzzle.forward * S);
        UpdateBulletCount(bulletCount - 1);
    }

}

// This should be in a seperate file
namespace Constants
{
    public static class Animations
    {
        public static readonly int playerSpeedHash = Animator.StringToHash("playerSpeed");
        public static readonly int playerGroundedHash = Animator.StringToHash("isGroundAnim");
    }

    public static class Strings
    {
        public const string horizontal = "Horizontal";
    }

    public static class Tags
    {
        public const string pencil = "Pencil";
        public const string money = "Money";
        public const string thesis = "Thesis";
        public const string enemy = "Enemy";
        public const string market = "Market";
        public const string trap = "Trap";
        public const string water = "Water";

    }
}