[RESOLVIDO] Como fazer uma calculadora recebendo o resultado do jogador? UNITY

MayLeone escreveu:Bem, lembre-se que texto não faz operação matemática, portanto todo o seu texto que representa os valores na UI sempre vão ser convertidos para int ou float (se desejar trabalhar com valores decimais).

Para sortear uma operação matemática, existem várias formas, mas um jeito fácil é primeiro sortear os números que farão parte dessa operação (creio que sejam dois né?) e depois sortear a operação.

Por exemplo: randomizar entre o número 0 e o número 99 e guardar esse valor numa variável chamada "firstNumber" e então fazer o mesmo para uma outra variável chamada "secondNumber".
Depois vc pode criar um array de char contendo as operações:

char[] operators = {'+', '-', '*', '/'}; 

E sortea um índice aleatório desse array para escolher uma operação, guardando numa variável:

var op = operators[Random.Range(0, operators.Length)];

Para o resultado, vc o guarda em outra variável, pode ser chamada de "result", fazendo uma verificação de qual operação foi realizada com um switch:

switch (op){
case '+': 
result = firstNumber + secondNumber;
break;

case '-':
result = firstNumber - secondNumber;
break;
}
e assim para as outras operações....

Para mostrar essa conta no textmesh você pode criar uma string chamada "showOperation" e concatenar tudo:

string showOperation = firstNumber.ToString() + " " + op.ToString() + " " + secondNumber.ToString();

E aí vc usa a variável "showOperation" para ser mostrada no TextMesh.

Então se por exemplo firstNumber receber o valor randômico de 67 e secondNumber receber 15, e op for de subtração, o TextMesh vai exibir exatamente o seguinte:

67 - 15
E por dentro, o resultado será 52.
Pronto, tem agora a primeira parte do sistema feita.

Para receber o input do usuário, também existem várias formas, como vc optou para ser por botões, acredito q vc crie eles com cada um tendo o texto de cada número correspondente, né? Por exemplo: o botão do número 2 tem como texto o 2.
Se for assim, fica simples:

Cria uma classe estática e nela um campo estático chamado "userInput" do tipo string:

public static class Controller{
public static string userInput; 
}

Agora para o botão, ele terá um script que quando ele for clicado, vc incrementa dentro do userInput o valor do seu texto,assim:

public void OnClick(){
Controller.userInput += GetComponentInChildren<Text>().text;
}

Então se você clicar no 4, depois no 9, e aí no 3, a variável userInput vai ser 493, por exemplo.

Agora para ver se o usuário acertou a conta ou não, dentro do botão de confirmação faz o seguinte:

public void OnClick(){
var userResult = System.ConvertToInt32(Controller.userInput);
if(userResult == result){
// Acertou
} else {
// Errou
}

MarcosSchultz escreveu:Da certo, mas o jeito da @MayLeone é muitooo melhor, vai por mim.

Tenta fazer um esforço e implementar o que ela explicou, o código vai ficar mais aleatório e vai requerir menos trabalho depois para gerar novas operações... E como a Unity entende operações matemáticas, vai dar pra fazer muitas outras coisas, expandindo as funcionalidadades com algumas pequenas edições.

Código:

[size=14]using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using TMPro;
using UnityEngine.UI;

public class Calculadora : MonoBehaviour
{
    public int PrimeiroNumero;
    public int SegundoNumero;
    public char[] operators = {'+' , '*'};
    public int resultado;
    private string ShowOP;
    public TextMeshProUGUI textDisplay;
    public int textResultado;
    public int ResultadoFinal;
    // Start is called before the first frame update
    void Start()
    {
        PrimeiroNumero = Random.Range (0,9);
        SegundoNumero = Random.Range (0,9);
        var op = operators[Random.Range(0, operators.Length)];
        switch (op){
            case '+':
            resultado = PrimeiroNumero + SegundoNumero;
            break;
            case '*':
            resultado = PrimeiroNumero * SegundoNumero;
            break;
        }
        Debug.Log(op);
        ShowOP = PrimeiroNumero.ToString() + " " + op.ToString() + " " + SegundoNumero.ToString() + " = ";
        textDisplay.text = ShowOP;
        Debug.Log(ShowOP);
    }

    // Update is called once per frame
    void Update()
    {
      
    }
}[/size]


Código:

[size=14]using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using TMPro;
using UnityEngine.UI;

public class BotaoTexto : MonoBehaviour
{
    public TextMeshProUGUI texto;
    public int resultado;    
    // Start is called before the first frame update
    void Start()
    {
        
    }

    // Update is called once per frame
    void Update()
    {
        
    }
    public void AddDigito(string digito){
        if(texto.text.Length < 2){
            texto.text = texto.text + digito;
            Debug.Log(digito);
            resultado = System.Convert.ToInt32(texto);
            Debug.Log(resultado);
            Debug.Log(texto);
        }
    }
}[/size]

Código:

resultado = texto;

//Acessando a variável "resultado" dentro da calculadora:
Calculadora calculadora = GameObject.Find("Nome do objeto que tem a calculadora na cena").GetComponent<Calculadora>();

if(resultado == calculadora.resultado.ToString())

// Acertou

else

// Errou

MayLeone escreveu:Entendo. Não tenho como testar agora os erros que o Convert.ToInt32 podem causar, talvez tenha que usar o TryParse.
De qualquer forma, tente fazer o caminho inverso: Converta o "resultado" para string e compare os valores como string mesmo, sem ser número:

Código:

resultado = texto;

//Acessando a variável "resultado" dentro da calculadora:
Calculadora calculadora = GameObject.Find("Nome do objeto que tem a calculadora na cena").GetComponent<Calculadora>();

if(resultado == calculadora.resultado.ToString())

// Acertou

else

// Errou

Não tenho como testar agora porque estou num PC sem Unity, qualquer dúvida ou erro, só falar.

Código:

resultado = texto;

//Acessando a variável "resultado" dentro da calculadora:
Calculadora calculadora = GameObject.Find("Nome do objeto que tem a calculadora na cena").GetComponent<Calculadora>();

if(resultado == calculadora.resultado.ToString())

// Acertou

else

// Errou

Código:

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

public class BotaoTexto : MonoBehaviour
{
    public TextMeshProUGUI texto;
    public int resultado;
    // Start is called before the first frame update
    void Start()
    {

    }
    // Update is called once per frame
    void Update()
    {
        resultado = texto;
        //Acessando a variável "resultado" dentro da calculadora:
        Calculadora calculadora = GameObject.Find("Calculo").GetComponent<Calculadora>();

        if (resultado == calculadora.resultado.ToString())

        // Acertou

        else

        // Errou
    }
    public void AddDigito(string digito)
    {
        if (texto.text.Length < 2)
        {
            texto.text = texto.text + digito;
            textoString = texto;
        }
    }
}

Ops esse daqui na real escreveu:

Código:

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

public class BotaoTexto : MonoBehaviour
{
    public TextMeshProUGUI texto;
    public string textoString;
    public int resultado;
    // Start is called before the first frame update
    void Start()
    {

    }
    // Update is called once per frame
    void Update()
    {
 
    }
    public void AddDigito(string digito)
    {
        if (texto.text.Length < 2)
        {
            texto.text = texto.text + digito;
            textoString = texto;
            resultado = int.Parse(textoString);
        }
    }
}

MayLeone escreveu:A variável "resultado" do script BotaoTexto precisa ser do tipo string.
Que erro que está dando?

MayLeone escreveu:Vc precisa converter a propriedade "text" de TextMeshProUGUI, e não a classe em si.

Código:

string resultado = GameObject.Find("Calculo").GetComponent<Calculadora>().resultado.ToString();

if(resultado == texto.text){
// Acertou
}else{
// Errou
}


MayLeone escreveu:Faz assim:

Código:

string resultado = GameObject.Find("Calculo").GetComponent<Calculadora>().resultado.ToString();

if(resultado == texto.text){
// Acertou
}else{
// Errou
}


Quando vc tentou fazer:
textoString = texto;
Vc estava tentando armazenar dentro de uma string um objeto do tipo TextMeshProUI, então acarretava esse erro.
O objeto TextMesh possui muitas informações, não é uma string. A string do TextMesh vc acessa da forma como vc estava fazendo: texto.text.
Era essa propriedade q vc deveria usar para armazenar dentro de textoString:
textoString = texto.text;
Daí não daria erros.

Mas pq colocar essa propriedade dentro de uma variável  para ser usada só uma vez para fazer uma verificação?
Nesse caso é preferível usar a propriedade directamente  no if como eu te mostro mais acima.

Código:

[size=14]using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using TMPro;
using UnityEngine.UI;

public class Calculadora : MonoBehaviour
{
    public int PrimeiroNumero;
    public int SegundoNumero;
    public char[] operators = {'+' , '*'};
    public int resultado;
    private string ShowOP;
    public TextMeshProUGUI textDisplay;
    // Start is called before the first frame update
    void Start()
    {
        PrimeiroNumero = Random.Range (0,9);
        SegundoNumero = Random.Range (0,9);
        var op = operators[Random.Range(0, operators.Length)];
        switch (op){
            case '+':
            resultado = PrimeiroNumero + SegundoNumero;
            break;
            case '*':
            resultado = PrimeiroNumero * SegundoNumero;
            break;
        }
        Debug.Log(op);
        ShowOP = PrimeiroNumero.ToString() + " " + op.ToString() + " " + SegundoNumero.ToString() + " = ";
        textDisplay.text = ShowOP;
        Debug.Log(ShowOP);
    }

    // Update is called once per frame
    void Update()
    {
     
    }
}[/size]

Código:

[size=14]using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using TMPro;
using UnityEngine.UI;

public class BotaoTexto : MonoBehaviour
{
    public TextMeshProUGUI texto; 
    public int resultado;
    public float timer;
    // Start is called before the first frame update
    void Start()
    {
       
    }

    // Update is called once per frame
    void Update()
    {
        string resultado = GameObject.Find("Calculo").GetComponent<Calculadora>().resultado.ToString();

    if(resultado == texto.text){
        UnityEngine.SceneManagement.SceneManager.LoadScene("Menu");
    }else{
        timer += Time.deltaTime;
        if(timer >= 3){
        UnityEngine.SceneManagement.SceneManager.LoadScene("Calculadora");
        }
    }
    }
    public void AddDigito(string digito){
        if(texto.text.Length < 2){
            texto.text = texto.text + digito;
        }
    }
}[/size]

MayLeone escreveu:No script do botão tem duas variáveis chamadas resultado, deleta aquela lá de cima do tipo int e só deixa a string dentro do Update.

MayLeone escreveu:No script do botão tem duas variáveis chamadas resultado, deleta aquela lá de cima do tipo int e só deixa a string dentro do Update.

Código:

IEnumerator WaitTime(){
yield return new WaitForSeconds(5);
        string resultado = GameObject.Find("Calculo").GetComponent<Calculadora>().resultado.ToString();
    if(resultado == texto.text){
        UnityEngine.SceneManagement.SceneManager.LoadScene("Menu");
    }else{
        UnityEngine.SceneManagement.SceneManager.LoadScene("Calculadora");
        }
    }

Código:

void Start(){
StartCoroutine("WaitTime");
}

MayLeone escreveu:Vc está colocando a verificação se o jogador acertou ou não dentro do Update, então realmente nem vai dar tempo pra pessoa digitar o número, ele já vai pro else.
Para corrigir isso sugiro que você crie um IEnumerator e passe pra ele um tempo pra pessoa digitar, por exemplo, 5 segundos... Se a pessoa não digitar em 5 segundos o tempo acaba e aí faz a verificação.

Apaga o Update e cria um IEnumerator assim:

Código:

IEnumerator WaitTime(){
yield return new WaitForSeconds(5);
        string resultado = GameObject.Find("Calculo").GetComponent<Calculadora>().resultado.ToString();
    if(resultado == texto.text){
        UnityEngine.SceneManagement.SceneManager.LoadScene("Menu");
    }else{
        UnityEngine.SceneManagement.SceneManager.LoadScene("Calculadora");
        }
    }

Daí dentro do Start desse script você coloca o seguinte:

Código:

void Start(){
StartCoroutine("WaitTime");
}

Agora a pessoa terá tempo para digitar o número (5 segundos, mas vc pode mudar ali no WaitForSeconds) e quando o tempo acabar, verifica se a pessoa acertou ou não o resultado.

Código:

case '-':
result = firstNumber - secondNumber;
if( result < 0)
Start();
break;

Código:

case '/':
if(firstNumber < secondNumber || secondNumber == 0){
Start();
} else {
result = firstNumber / secondNumber;
}
break;

Código:

// Melhorando a divisão para sempre mútiplos:

case '/':
if(firstNumber < secondNumber || secondNumber == 0){
Start();
} else if (firstNumber % secondNumber == 0) {
result = firstNumber / secondNumber;
}  else {
Start();
}
break;

Código:

var op = operators[Random.Range(0, operators.Length)];
 
SortearNovosValores:

        PrimeiroNumero = Random.Range (0,9);
        SegundoNumero = Random.Range (0,9);

        switch (op){
            case '+':
            resultado = PrimeiroNumero + SegundoNumero;
            break;
            case '*':
            resultado = PrimeiroNumero * SegundoNumero;
            break;
case '-':
resultado =  PrimeiroNumero - SegundoNumero;
if (resultado < 0)
goto SortearNovosValores;
break;

case '/':
if (PrimeiroNumero < SegundoNumero || SegundoNumero == 0)
goto SortearNovosValores;
else if (PrimeiroNumero % SegundoNumero == 0)
resultado = PrimeiroNumero / SegundoNumero;
else
goto SortearNovosValores;
    }

MayLeone escreveu:Outra alternativa sem a recursão é utilizando a instrução "goto" dentro do "switch":

Código:

var op = operators[Random.Range(0, operators.Length)];
  
SortearNovosValores:

        PrimeiroNumero = Random.Range (0,9);
        SegundoNumero = Random.Range (0,9);

        switch (op){
            case '+':
            resultado = PrimeiroNumero + SegundoNumero;
            break;
            case '*':
            resultado = PrimeiroNumero * SegundoNumero;
            break;
case '-':
resultado =  PrimeiroNumero - SegundoNumero;
if (resultado < 0)
goto SortearNovosValores;
break;

case '/':
if (PrimeiroNumero < SegundoNumero || SegundoNumero == 0)
goto SortearNovosValores;
else if (PrimeiroNumero % SegundoNumero == 0)
resultado = PrimeiroNumero / SegundoNumero;
else
goto SortearNovosValores;
     }

Código:

[size=14]using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using TMPro;
using UnityEngine.UI;

public class Calculadora : MonoBehaviour
{
    public int PrimeiroNumero;
    public int SegundoNumero;
    public char[] operators = {'+' , '*'};
    public int resultado;
    private string ShowOP;
    public TextMeshProUGUI textDisplay;
    // Start is called before the first frame update
    void Start()
    {
        var op = operators[Random.Range(0, operators.Length)];
        SortearNovosValores:
        PrimeiroNumero = Random.Range (0,9);
        SegundoNumero = Random.Range (0,9);
        switch(op){
            case '+':
            resultado = PrimeiroNumero + SegundoNumero;
            break;
            case '*':
            resultado = PrimeiroNumero * SegundoNumero;
            break;
            case '-':
            resultado =  PrimeiroNumero - SegundoNumero;
            if (resultado < 0){
            goto SortearNovosValores;
            }
            break;
            case '/':
            if (PrimeiroNumero < SegundoNumero || SegundoNumero == 0){
            goto SortearNovosValores;
            }
            else if (PrimeiroNumero % SegundoNumero == 0){
            resultado = PrimeiroNumero / SegundoNumero;
            }else{
            goto SortearNovosValores;
            }
        }
        Debug.Log(op);
        ShowOP = PrimeiroNumero.ToString() + " " + op.ToString() + " " + SegundoNumero.ToString() + " = ";
        textDisplay.text = ShowOP;
        Debug.Log(ShowOP);
    }

    // Update is called once per frame
    void Update()
    {
      
    }
}[/size]


MayLeone escreveu:Faltou uma instrução break embaixo da chave do último goto da estrutura.