Ложное срабатывание else

@NPD · Регистрация: 13.11.2016

Author24 — интернет-сервис помощи студентам

Всем привет! Делаю программу для тренировки устного счёта, значения меняются, выводиться, но при верном ответе срабатывает else. (на случай серьёзных косяков: при верном ответе должен к значению vo прибавить один, при неверном выдать сообщение)

C#

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
 
namespace Устный_счёт
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }
 
        private void label1_Click(object sender, EventArgs e)//первое число=========================================================================================
        {
 
        }//=========================================================================================================================================================
 
        private void label3_Click(object sender, EventArgs e)//втрое число==========================================================================================
        {
 
        }//=========================================================================================================================================================
 
        private void textBox1_TextChanged(object sender, EventArgs e)//===================================================================значение ведёное человеком
        {
 
        }//=========================================================================================================================================================
 
        private void label6_Click(object sender, EventArgs e)//==============================================================================================проверка
        {
 
        }//=========================================================================================================================================================
 
        public void pvd()
        {
            Random nd = new Random();
                int vo = 0;
                int a = nd.Next(1, 9);
                int b = nd.Next(1, 9);
                label1.Text = a.ToString();
                label3.Text = b.ToString();
                int c = Convert.ToInt32(textBox1.Text);
                if ((a + b) == c) { vo++; }
                else
                {
                    MessageBox.Show("Ошибка, начинай заново.");
                }
                return;
        }
 
        private void button1_Click(object sender, EventArgs e)
        {
            pvd();
        }
    }//end
}//end

И ещё, как показать кол-во верных ответов в сообщении.

@Sergey84 · 17.11.2016, 21:02

А можно, хотя бы, прокомментировать функцию pvd и всё что в ней?

@NPD · 17.11.2016, 21:16 **[ТС]**

Я начинающий прошу прощение.

А как собственно прокомментировать?

Добавлено через 1 минуту
А понял!

Добавлено через 4 минуты

C#

public void pvd()
        {
            Random nd = new Random();//рандом
                int vo = 0;// верные ответы
                int a = nd.Next(1, 9);//новое значение 1 слагаемому 
                int b = nd.Next(1, 9);//новое значение 2 слагаемому 
                label1.Text = a.ToString();//вывод в лабелб т. е. отображение
                label3.Text = b.ToString();//вывод в лабель т. е. отображение
                int c = Convert.ToInt32(textBox1.Text);//ответ данный пользователем
                if ((a + b) == c) { vo++; }//верных ответов + 1
                else
                {
                    MessageBox.Show("Ошибка, начинай заново.");//сообщение 
                }
                return;//...
        }

@korochinskiy · 17.11.2016, 21:23

Поставьте точку останова на if и посмотрите значения в переменных a, b, c. Возможно вы где-то ошиблись с вводом - выводом данных, ну или конвертация проходит не правильно...
Потом... Если вы хотите посчитать количество правильных ответов в переменной vo, то у вас ничего не выйдет) т.к. она будет обнуляться после каждого нажатия на button1. Вынесите её в видимость класса.

C#

        private int vo = 0;
        public void pvd()
        {
           //и т.д.

Показывать колличество правильных ответов как хотите (label или MessageBox например)?

@Diamante · 17.11.2016, 21:34

так у вас по кнопке происходит сразу все, и генерация рандома и считывание из текстбокса и сравнение,т.е. значение в текстбоксе остаётся с прошлого раза, а кнопка уже того, отработала, у вас не тренажер памяти, а угадайка

@NPD · 17.11.2016, 21:37 **[ТС]**

А можно подробнее где поставить точку?

Добавлено через 1 минуту
Diamante, а как сделать уже не угадай ка значение, а тренер устного счёта.

Я просто новичок, много что не понимаю.

@korochinskiy · 17.11.2016, 22:13

Ну на if же вашем.

Сообщение от NPD

if ((a + b) == c) { vo++; }//верных ответов + 1

Там где нумерация строк щелкните мышкой. Должна появиться красная точка. Там произойдет остановка программы после того как вы нажмете button1. После этого подведте мышку к вашим переменным a, b, c и отобразиться их значение в данный момент.

Diamante, прав. У вас получается угадайка...
Это в отдельную функциюю

C#

            Random nd = new Random();
                int vo = 0;
                int a = nd.Next(1, 9);
                int b = nd.Next(1, 9);
                label1.Text = a.ToString();
                label3.Text = b.ToString();

Первый раз вызов ее из этой части

C#

         
public Form1()
        {
            InitializeComponent();
        }

И каждый раз после нажатия button1 перед return в pvd()

@NPD · 18.11.2016, 10:19 **[ТС]**

C#

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
 
namespace Устный_счёт
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
            random();
        }
 
        public static string myvariable;
        
        public int a = 0;
        public int b = 0;    
 
        private void label1_Click(object sender, EventArgs e)//первое число=========================================================================================
        {
 
        }//=========================================================================================================================================================
 
        private void label3_Click(object sender, EventArgs e)//втрое число==========================================================================================
        {
 
        }//=========================================================================================================================================================
 
        private void textBox1_TextChanged(object sender, EventArgs e)//===================================================================значение ведёное человеком
        {
 
        }//=========================================================================================================================================================
 
        private void label6_Click(object sender, EventArgs e)//==============================================================================================проверка
        {
 
        }//=========================================================================================================================================================
 
        public int vo = 0;
        public void pvd()
        {
            int c = Convert.ToInt32(textBox1.Text);//ответ данный пользователем
            if ((a + b) == c) { vo++; }//верных ответов + 1
            else
            {
                MessageBox.Show("Ошибка, начинай заново.");//сообщение 
            }
            random();
            return;//...
        }
 
        private void button1_Click(object sender, EventArgs e)
        {
            pvd();
            return;
        }
        private void random()
        {
            Random nd = new Random();
            int vo = 0;
            int a = nd.Next(1, 9);
            int b = nd.Next(1, 9);
            label1.Text = a.ToString();
            label3.Text = b.ToString();
 
        }
    }//end
}//end

Так?
Всё равно пишет что ошибка.

@schlawiner · 18.11.2016, 10:27

У тебя глобальные переменные есть

C#

1 2	public int a = 0; public int b = 0;

а в методе random() ты создаешь заново, но локальные переменные.
а в методе pvd() у тебя в условии

C#

1	if((a+b) == c)

используются глобальные переменные, которые всегда равны 0

@NPD · 18.11.2016, 11:37 **[ТС]**

schlawiner, уважение +.

@Sergey84 8 / 8 / 7 Регистрация: 21.08.2015 Сообщений: 90
	17.11.2016, 21:02	2
	А можно, хотя бы, прокомментировать функцию pvd и всё что в ней? 1

@Diamante 3564 / 2505 / 1174 Регистрация: 14.08.2016 Сообщений: 8,214
	17.11.2016, 21:34	5
	так у вас по кнопке происходит сразу все, и генерация рандома и считывание из текстбокса и сравнение,т.е. значение в текстбоксе остаётся с прошлого раза, а кнопка уже того, отработала, у вас не тренажер памяти, а угадайка 1

@NPD 0 / 0 / 0 Регистрация: 13.11.2016 Сообщений: 25
	17.11.2016, 21:37 [ТС]	6
	А можно подробнее где поставить точку? Добавлено через 1 минуту Diamante, а как сделать уже не угадай ка значение, а тренер устного счёта. Я просто новичок, много что не понимаю. 0

@NPD 0 / 0 / 0 Регистрация: 13.11.2016 Сообщений: 25
	18.11.2016, 11:37 [ТС]	10
	schlawiner, уважение +. 0

Ложное срабатывание else

Решение