Problem z kalkulatorem C#

Question

Witam, chciałem spróbować napisać prosty kalkulator (jako nauka języka) niestety, podczas rozbudowania jego wersji co chwilę wywala mi błąd --> "System.ArgumentNullException: „Wartość nie może być zerowa.
Nazwa parametru: String”"

 

case '*':
textBox1.Text = (int.Parse(LiczbaPierwsza) * int.Parse(LiczbaDruga)).ToString();
break;

 

Gdy sprawdzam to na początku jest zmienna -"LiczbaPierwsza" ale po kliknięciu np. dodawania to całość idzie do zmiennej - "LiczbaDruga" i nie wiem jak to rozwiązać. no

 

 private void Dzialanie (int liczba)
        {
            if (RodzajDzialania == ' ')
            {
                LiczbaPierwsza += liczba;
                textBox1.Text = LiczbaPierwsza;
            }
            else
            {
                LiczbaDruga += liczba;
                textBox1.Text = LiczbaDruga;
            }
        }

Program działa dopiero po 2 kliknięcia równania :/

Comments

Cześć,

Jeżeli to możliwe, czy możesz pokazać cały kod ?

Pozdrawiam aeragram

---

Mogę :)

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 WindowsFormsApp1
{
    public partial class Form1 : Form
    {

        public Form1()
        {
            InitializeComponent();
        }
        string LiczbaPierwsza, LiczbaDruga;
        char RodzajDzialania;

        private void Form1_Load(object sender, EventArgs e)
        {

        }

        private void button1_Click(object sender, EventArgs e) //Przycisk - "Kliknij mnie"
        {
            MessageBox.Show("Kliknąłeś mnie :o"); //Reakcja na klikniecie przyciska
        }

        private void contextMenuStrip1_Opening(object sender, CancelEventArgs e)
        {

        }

        private void aboutToolStripMenuItem_Click(object sender, EventArgs e) // About w zakladce Informacje
        {
      
		}

        private void openFileDialog1_FileOk(object sender, CancelEventArgs e)
        {

        }

        private void textBox1_TextChanged(object sender, EventArgs e)
        {

        }

        private void openFileDialog1_FileOk_1(object sender, CancelEventArgs e)
        {
		
	}

        private void dateTimePicker1_ValueChanged(object sender, EventArgs e)
        {

        }

        private void textBox1_TextChanged_1(object sender, EventArgs e)
        {
            
        }

        private void dateTimePicker1_ValueChanged_1(object sender, EventArgs e)
        {

        }

        private void TextBox1_TextChanged_2(object sender, EventArgs e)
        {

        }

        private void PlikToolStripMenuItem_Click(object sender, EventArgs e)
        {

        }

        private void Button13_Click(object sender, EventArgs e)
        {
        switch (RodzajDzialania)
            {
                case ('+'):
                    textBox1.Text = (int.Parse(LiczbaPierwsza) + int.Parse(LiczbaDruga)).ToString();
                    break;
                case ('-'):
                    textBox1.Text = (int.Parse(LiczbaPierwsza) - int.Parse(LiczbaDruga)).ToString();
                    break;
                case ('*'):
                    textBox1.Text = (int.Parse(LiczbaPierwsza) * int.Parse(LiczbaDruga)).ToString();
                    break;
                case ('/'):
                    textBox1.Text = (int.Parse(LiczbaPierwsza) / int.Parse(LiczbaDruga)).ToString();
                    break;

            }
            LiczbaPierwsza = "";
            LiczbaDruga = "";
            RodzajDzialania = ' ';
        }

        private void Button11_Click(object sender, EventArgs e)
        {
            RodzajDzialania = '-';
            textBox1.Text = "";
        }

        private void Button7_Click(object sender, EventArgs e)
        {
            RodzajDzialania = '+';
            textBox1.Text = "";
        }

        private void Button5_Click(object sender, EventArgs e)
        {
            Dzialanie(7);
        }

        private void Button6_Click(object sender, EventArgs e)
        {
            Dzialanie(8);
        }

        private void Button12_Click(object sender, EventArgs e)
        {
            Dzialanie(6);
        }

        private void Button8_Click(object sender, EventArgs e)
        {
            Dzialanie(2);
        }

        private void Button10_Click(object sender, EventArgs e)
        {
            Dzialanie(5);
        }

        private void Button9_Click(object sender, EventArgs e)
        {
            Dzialanie(4);
        }

        private void Button4_Click(object sender, EventArgs e)
        {
            Dzialanie(1);
        }

        private void Button16_Click(object sender, EventArgs e)
        {
            Dzialanie(0);
        }

        private void Button3_Click(object sender, EventArgs e)
        {
            Dzialanie(9);
        }

        private void Button14_Click(object sender, EventArgs e)
        {
            RodzajDzialania = '*';
            textBox1.Text = "";
        }

        private void Button2_Click(object sender, EventArgs e)
        {
            Dzialanie(3);
        }

        private void Button15_Click(object sender, EventArgs e)
        {
            RodzajDzialania = '/';
            textBox1.Text = "";

        }
        private void Dzialanie (int liczba)
        {
            if (RodzajDzialania == ' ')
            {
                LiczbaPierwsza += liczba;
                textBox1.Text = LiczbaPierwsza;
            }
            else
            {
                LiczbaDruga += liczba;
                textBox1.Text = LiczbaDruga;
            }
        }
    }
}

 

Niektóre linijki są puste bo kiedyś spełniały jakąś funkcję ale je usunąłem 

Answer 1

Cześć,

Wprawdzie dopiero się uczę języka, ale wydaje mi się, że problem jest taki:

1. deklaracja zmiennej char RodzajDzialania (linia 21) bez zadeklarowania wartości początkowej powoduje, że w zmiennej RodzajDzialania znajduje się na początku domyślna wartość dla char czyli '\0' a nie ' '.

2. po naciśnięciu buttona z liczbą wywołujesz metodę Dzialanie, sprawdzenie warunku RodzajDzialania==' ' (linia 182) daje więc wartość false i przechodzimy do else, gdzie do LiczbaDruga zostanie dodana zmienna liczba.

3. Naciśnięcie Button13 i wykonanie metody Button13_Click (linia 83) zapewne powoduje błąd, ponieważ w LiczbaPierwsza nie ma żadnej wartości (a dokladnie w LiczbaPierwsza nie ma nawet adresu żadnego stringa, tylko jest null)

Pozdrawiam aeragram