Не работает скрипт — калькулятор в юнити. Что делать?

Question

[Ptichka007]

Сделал простенький калькулятор в юнити, однако когда вписываешь цифры в input Field и нажимаешь на кнопку вывода результата, выходит либо цифра 2, либо цифра 1, либо вообще ноль. Например, если пишешь 10 * 10, то выйдет 1, если напишешь 2 + 2, то выйдет 2, ну вы поняли.

Вот код:

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

public class Simplecalcan : MonoBehaviour
{
    [SerializeField] InputField firstnum;
    [SerializeField] InputField secondnum;
    [SerializeField] InputField thesign;
    [SerializeField] Text text;




     public void Calculator()
    {




        if (thesign.text == "*")
        {
            int firstnum2 = Convert.ToInt32(firstnum);
            int secondnum2 = Convert.ToInt32(secondnum);


            int rezult = firstnum2 * secondnum2;
            string rezult2 = rezult.ToString();
            text.text = rezult2;
        }

        else if (thesign.text == "+")
        {
            int firstnum2 = Convert.ToInt32(firstnum);
            int secondnum2 = Convert.ToInt32(secondnum);

            int rezult = firstnum2 + secondnum2;
            string rezult2 = rezult.ToString();
            text.text = rezult2;



        }

        else if (thesign.text == "/")
        {
            int firstnum2 = Convert.ToInt32(firstnum);
            int secondnum2 = Convert.ToInt32(secondnum);

            int rezult = firstnum2 / secondnum2;
            string rezult2 = rezult.ToString();
            text.text = rezult2;


        }


        else if (thesign.text == "-")
        {
            int firstnum2 = Convert.ToInt32(firstnum);
            int secondnum2 = Convert.ToInt32(secondnum);

            int rezult = firstnum2 - secondnum2;
            string rezult2 = rezult.ToString();
            text.text = rezult2;

        }


    }

}

Comments

[edward_freedom]

пробовал сначала калькулятор сделать на c# без юнити?

int firstnum2 = Convert.ToInt32(firstnum);

Что в переменной firstnum?

[freeExec]

А делали бы через int.Parse сразу бы было понятно

[NoNameDeveloper]

freeExec, Шо вы несете, человек пытается сделать из Input => Int поэтому ничего и не выходит. Нужно парсить именно текст инпута а не сам инпут)
Правильно будет Convert.ToInt32(firstnum.text).

[freeExec]

NoNameDeveloper, если бы он написал int.Parse(firstnum) его бы сразу послали изучать документацию, он бы даже запустить не смог. А не спрашивать на форуме, почему ноль возвращает.

[NoNameDeveloper]

freeExec, Будем честны, когда являешься новичком, эти вещи на кажутся такими очевидными. Поэтому просто нужно объяснять в чем проблема ;)

[freeExec]

NoNameDeveloper, если пройти в отладчике по шагам, то в общем-то всё очевидно. И вопрос бы не звучал как "не работает скрипт калькулятора".

Answer 1

[NoNameDeveloper]

А вот и рабочий укороченный код. Конвертировать нужно не компонент InputField, а текст данного компонента.
Можно использовать также int.Parse(value).

using System;

using UnityEngine;
using UnityEngine.UI;

public class SimpleCalculator : MonoBehaviour
{
    [SerializeField] private InputField _firstInput;
    [SerializeField] private InputField _secondInput;
    [SerializeField] private InputField _signInput;
    [SerializeField] private Text _text;

    // Methods

    public void Calculator()
    {
        int a = int.Parse(_firstInput.text);
        int b = int.Parse(_secondInput.text);

        _text.text = GetResult(a, b).ToString();
    }

    private float GetResult(int a, int b)
    {
        switch(_signInput.text)
        {
            case "+": return Add(a, b);
            case "-": return Substract(a, b);
            case "*": return Multiply(a, b);
            case "/": return Divide(a, b);
            default: throw new Exception("Invalid sign");
        }
    }

    private float Add(int a, int b) => a + b;
    private float Substract(int a, int b) => a - b;
    private float Multiply(int a, int b) => a * b;
    private float Divide(int a, int b) => a / b;
}

Comments

[Ptichka007]

Я решил использовать ваш совет про int.Parse() в своем коде, и все заработало!
Спасибо за помощь!
P.S Ваш код у меня почему-то не работает. Там есть ошибки. (По крайней мере Визуал код студио их видит)

[edward_freedom]

можно так сократить калькулятор
https://qna.habr.com/answer?answer_id=1652237#answ...

[edward_freedom]

NoNameDeveloper и это не красиво

Add(a,b).ToString()

сделай шаблонную функцию и динамически приводи к нужному типу, если сразу не хочешь возвращать стринг
https://docs.microsoft.com/en-us/dotnet/api/system...