Kalkulator BMI. Obsługa błędu przy wpisywaniu stringa.

Question

Dzień dobry,

Mam problem przy obsłudze błędu w moim kalkulatorze BMI. Mianowicie kiedy wpisuję w textboxa ciąg literek, to chciałbym, żeby mój "program" dał sobie radę z tym i wyświetlił komunikat "błąd danych"

Nie za bardzo wiem jak zapisać w kodzie, że jeżeli w textboxa jest wpisywany string to wyświetl komunikat " błąd danych"

OTO MÓJ KOD: 

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace CS_BMI_Calc
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {

        }

        private void label3_Click(object sender, EventArgs e)
        {

        }

        private void label5_Click(object sender, EventArgs e)
        {

        }

        private void button4_Click(object sender, EventArgs e)
        {
            txtMasa.Clear();
            txtWzrost.Clear();
            txtBmi.Clear();
            txtBmi.BackColor = Color.White; 
            lblwynik.Text = "";
            lblwynik.BackColor = Color.White; 
        }

        private void button2_Click(object sender, EventArgs e)
        {
            Application.Exit(); 
        }

        private void button1_Click(object sender, EventArgs e)
        {
            if (txtMasa.Text != "" && txtWzrost.Text != ""  && Convert.ToDouble(txtMasa.Text) > 0 && Convert.ToDouble(txtWzrost.Text) > 0)
            {
                txtBmi.BackColor = Color.White; 

                double masa = Convert.ToDouble(txtMasa.Text);
                double wzrost = Convert.ToDouble(txtWzrost.Text) / 100;
                double BMI = masa / (wzrost * wzrost);
                txtBmi.Text = String.Format("{0:f}",BMI);

                if (BMI < 18)
                {
                    lblwynik.Text = "Niedowaga !!!";
                    txtBmi.BackColor = Color.PowderBlue;
                }
                else if (BMI < 24.99)
                {
                    lblwynik.Text = "Waga Prawidłowa";
                    txtBmi.BackColor = Color.Green;
                }
                else
                {
                    lblwynik.Text = "NADWAGA !!!";
                    txtBmi.BackColor = Color.Red; 
                }
            }
            else lblwynik.Text = "Błąd danych"; 
        }

        private void lblwynik_Click(object sender, EventArgs e)
        {

        }
    }
}

 

Answer 1

Można "odwrócić" sytuację i zablokować wpisywanie innych znaków niż dozwolone. 

1. W sekcji kodu z using dopisz using System.Text.RegularExpressions;
    

   using System;
   using System.Collections.Generic;
   using System.ComponentModel;
   using System.Data;
   using System.Drawing;
   using System.Linq;
   using System.Text;
   using System.Text.RegularExpressions; // <-- ;-)
   using System.Threading.Tasks;
   using System.Windows.Forms;

2. W sekcji kodu z public Form1() dopisz
    

           public Form1()
           {
               InitializeComponent();
   
               // Nie można używać menu kontekstowego (prawy myszy)
                 txtMasa.ShortcutsEnabled = false;
               txtWzrost.ShortcutsEnabled = false;
               // TextBox bmi tylko do odczytu  ;-)
               txtBmi.ReadOnly = true;
           }

3. W sekcji kodu dopisz
    

   using System;
   using System.Collections.Generic;
   using System.ComponentModel;
   using System.Data;
   using System.Drawing;
   using System.Linq;
   using System.Text;
   using System.Text.RegularExpressions;
   using System.Threading.Tasks;
   using System.Windows.Forms;
   
   namespace CS_BMI_Calc
   {
       public partial class Form1 : Form
       {
           public Form1()
           {
               InitializeComponent();
               // Nie można używać menu kontekstowego (prawy myszy)
                 txtMasa.ShortcutsEnabled = false;
               txtWzrost.ShortcutsEnabled = false;
               // TextBox bmi tylko do odczytu  ;-)
               txtBmi.ReadOnly = true;
           }
    
           . . .
   
           private void txtMasa_KeyPress(object sender, KeyPressEventArgs e)
           {
               if (OnlyDigits(sender, e))
               {
                   e.Handled = true; 
               }
                          
           }
           private void txtWzrost_KeyPress(object sender, KeyPressEventArgs e)
           {
               if (OnlyDigits(sender, e))
               {
                   e.Handled = true; 
               }
                          
           }
           private bool OnlyDigits(object sender, KeyPressEventArgs e)
           {
               bool bReturn = false;
               // Dozwolone tylko cyfry 0-9 i tzw. znaki kontroli (delete, backspace, left arrow itp.)
               if (!char.IsControl(e.KeyChar) && !char.IsDigit(e.KeyChar) && (e.KeyChar != '.')) bReturn = true;
               // Dozwolona tylko jedna kropka dziesiętna
               if ((e.KeyChar == '.') && ((sender as TextBox).Text.IndexOf('.') > -1)) bReturn = true;
               // Dozwolone tylko dwie cyfry po kropce (np.: waga 71.5 kg, wzrost 172.55 cm)
               if (Regex.IsMatch((sender as TextBox).Text, @"\.\d\d") && !char.IsControl(e.KeyChar)) bReturn = true;
               
               return bReturn;            
           }
   
           . . .
       }
   }
   

4. W pliku Form1.Designer.cs (dwuklik lewym myszy - otworzy się kod)
   
   
   
   w sekcji private void InitializeComponent() { ... }, dopisz
    

           private void InitializeComponent()
           {
   
               . . .
   
               // 
               // txtMasa
               // 
   
               this.txtMasa.KeyPress += new System.Windows.Forms.KeyPressEventHandler(this.txtMasa_KeyPress);
   
   
               // 
               // txtWzrost
               // 
              
               this.txtWzrost.KeyPress += new System.Windows.Forms.KeyPressEventHandler(this.txtWzrost_KeyPress);
   
           }
   

5. Wtedy możesz: 

    else lblwynik.Text = "Błąd danych"; 
   
    na np.: 
   
    else lblwynik.Text = "Proszę podać masę i wzrost"; 

   AA