Pull to refresh
-1
0
Send message
    [SerializeField]
    public Image iconImage; //Иконка

Почему модификатор доступа public, если есть атрибут SerializeField?

    //Метод "Скрыть попап"
    public void Hide()
    {
        this.closeButton.onClick.AddListener(this.OnCloseClicked);
    }

То есть, в методе Hide еще раз происходит подписка на OnCloseClicked?

    public void SetActive(bool isActive) {
        if (isActive) {
            this.button.interactable = true;
            this.buttonBackground.sprite = this.availableBackground;
            this.getText.SetActive(true);
            this.processingText.SetActive(false);
        } else {
            this.button.interactable = false;
            this.buttonBackground.sprite = this.unavailableBackground;
            this.getText.SetActive(false);
            this.processingText.SetActive(true);
        }
    }

Зачем такое дублирование кода?

button.interactible = isActive;
getText.SetActive(isAcitve);
...

В этом коде много проблемных мест, я бы порекомендовал автору самому почитать о лучших практиках, прежде чем учить этому других

В примере с событиями в методе Dispose опечатка, -= вместо +=

Прекрасная статья, а автор красавчик

Information

Rating
Does not participate
Registered
Activity