Почему сменная не получает данные из другого скрипта?

Question

[Юра Березовский]

Скрипт должен снимать со счета игрока стоимость скина, но в результате ничего не происходит.
Функция которая должна снимать деньги:

void Update ()
    {
        if (oppAcc == true) { 
             if (playerInfo.gold >= mfb && playerInfo.crystals >= cfb)
             {
            playerInfo.gold = playerInfo.gold - mfb;
            playerInfo.skin = buttonClickedOnObjectForBuy;
            playerInfo.crystals = playerInfo.crystals - cfb;

             }
             if (playerInfo.gold < mfb || playerInfo.crystals < cfb)
             {
            playerInfo.skin = 999;
             }
        }


    }

Игрок:

public class PlayerInfo : MonoBehaviour {
    public int skin;
    public int weapon;
    public int gold ;
    public int crystals ;
    public int swords;

	// Use this for initialization
	void Start () {
	
	}
	
	// Update is called once per frame
	void Update () {
        gold = 10000;
        crystals = 10000;
	}
}

Comments

[RedSkotina]

// Update is called once per frame
Может быть в этом проблема?

[devalone]

Юра Березовский:

это комментарий

Спасибо, КЭП

[Юра Березовский]

devalone: к сожалению, видимо не в этом проблема, потому что и в других методах функция не работает

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

public class Shop : MonoBehaviour {
    
    // Scripts
    
    public PlayerInfo playerInfo;
    //Canvas
    public Canvas really;
    public Canvas shop;
    public Button yesBut;
    public Button noBut;
    public Button buySkin;

    //variables
    public int reallyCode;
    public int mfb;
    public int cfb;
    public int buttonClickedOnObjectForBuy;
    public bool oppAcc;

    


    void Start ()
    {
        
        playerInfo = playerInfo.GetComponent<PlayerInfo>();
        really = really.GetComponent<Canvas>();
        yesBut = yesBut.GetComponent<Button>();
        noBut = noBut.GetComponent<Button>();
        buySkin = buySkin.GetComponent<Button>();
        shop = shop.GetComponent<Canvas>();
        //enabled
        shop.enabled = true;
        really.enabled = false;
        
	}

	
	void Update ()
    {
         
            
        


    }
    public void BuyClicked()
    {
        if (buttonClickedOnObjectForBuy > 0)
        {
            really.enabled = true;
            shop.enabled = false;
        }
    }
    public void YesBut()
    {
        
        really.enabled = false;
        shop.enabled = true;
        if (playerInfo.gold >= mfb && playerInfo.crystals >= cfb)
        {
            playerInfo.gold = playerInfo.gold - mfb;
            playerInfo.skin = buttonClickedOnObjectForBuy;
            playerInfo.crystals = playerInfo.crystals - cfb;

        }
        if (playerInfo.gold < mfb || playerInfo.crystals < cfb)
        {
            playerInfo.skin = 999;
        }
    }
    public void NoBut()
    {
        really.enabled = false;
        shop.enabled = true;


    }
    public void Skin1()
    {
        buttonClickedOnObjectForBuy = 1;
        mfb = 1000;
        cfb = 5;
    }
    public void Skin2()
    {
        buttonClickedOnObjectForBuy = 2;
        mfb = 4000;
        cfb = 10;
    }
}

[RedSkotina]

Юра Березовский: я не разбираюсь в юнити, но если unity не имеет дебагера, то вы пробовали использовать стратегию дебага принтами (когда вы вы тыкаете вывод переменых во все проблемные точки)?
И мне кажется подозрительным что вы инициализируете значения голды и кристалов по умолчанию в ноль и при этом непонятно как вызывается функции Skin1 и Skin2

Answer 1

[devalone]

Когда ты пишешь такой код, где-то в мире умирает котёнок, не будь извергом!
if (oppAcc == true) {
можно просто
if (oppAcc) {
и что такое oopAcc? Что такое mfb, cfb? Пиши нормально.
playerInfo.skin = 999;
Почему скин 999? Что такое 999? id? Почему не enum? Ну ладно, это мелочи. Что этот код по покупке делает в update? Читаем документацию https://docs.unity3d.com/ScriptReference/MonoBehav... update вызывается в каждом кадре, у вас будет покупаться очень много вещей, пока переменная oppAcc будет true. Также что вот этот код делает в update?

void Update () {
        gold = 10000;
        crystals = 10000;
  }

Возможно проблема в этом. А может и ещё где-то.

Comments

[devalone]

Юра Березовский: учить матчасть, четверть-половина книги Шилдта хватит, думаю, ну или можете выбрать любую другую книгу по языку, погуглите, их много хороших. Ну и конечно больше писать код, читать статьи по тому, как сделать код читабельным и стараться писать лучше. Если смотрите туторилы, посоветую выбирать их тщательнее, т.к. зачастую там учителя, которые сами неделю назад написали первую строчку кода, однозначно годные на официальном сайте(на английском).

[devalone]

Юра Березовский, зачем комменты удалять?

[Юра Березовский]

devalone: Подумал что здесь не место шуткам и удалил(удалил за секунду до того как вы ответили). Спасибо за ответ.

Answer 2

[Арк Тарусов]

А где у вас собственно съем денег то?
например
gold-=skin;
Это раз.

Два, у вас в принципе сумма на счету у игрока не может уменьшиться так как вы ее каждый кадр приравниваете к 10000.
Даже если вы вычтите стоимость скина, в следующем кадре у игрока опять будет 10000

Comments

[Юра Березовский]

Даже если вы вычтите стоимость скина, в следующем кадре у игрока опять будет 10000

Ну это я для отладки писал. А не получало, потому что Первый скрипт не имел доступа к данным PlayerInfo, через то что я неправильно кешировал скрипт =)

Answer 3

[Дмитрий Еремин]

что такое

• oppAcc
  
• mfb
  
• cfb
  

?
Где определяется buttonClickedOnObjectForBuy?

Comments

[Юра Березовский]

opAcc - operating access
mfb - money for buying
cfb - crystals for buying

public void Skin1()
    {
        buttonClickedOnObjectForBuy = 1;
        mfb = 1000;
        cfb = 5;
    }
    public void Skin2()
    {
        buttonClickedOnObjectForBuy = 2;
        mfb = 4000;
        cfb = 10;
    }
}

public void YesBut()
    {
        oppAcc = true;
        really.enabled = false;
        shop.enabled = true;

    }