C# Programm-Problem

weltbesiedler · 25.05.2011, 18:55

Hab ein kleines Problem. Ich möchte das er in ein Label einen Text schreibt, wenn die Eingabe vom Typ String ist, also ein Wort, keine Zahl. Ich hab versucht das Problem so zu lösen:

Code:

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

        private void textBox2_TextChanged(object sender, EventArgs e)
        {

        }

        private void button1_Click(object sender, EventArgs e)
        {
            
            int Zahl1 = Convert.ToInt16(textBox1.Text);
            int Zahl2 = Convert.ToInt16(textBox2.Text);
            int ergebnis = Zahl1 + Zahl2;
            ergebnis = Zahl1 + Zahl2;
            textBox3.Text = Convert.ToString(ergebnis);
            if (textBox3.Text == string)
                label2.Text = "Nur Zahlen";
          
        }

        private void button2_Click(object sender, EventArgs e)
        {
            int Zahl1 = Convert.ToInt16(textBox1.Text);
            int Zahl2 = Convert.ToInt16(textBox2.Text);
            int ergebnis = Zahl1 + Zahl2;
            ergebnis = Zahl1 - Zahl2;
            textBox3.Text = Convert.ToString(ergebnis);
            if (textBox3.Text == string)
                label2.Text = "Nur Zahlen";

        }
    }
}

Allerdings scheint das Wort "string" nicht zulässig zu sein:

Zitat:

Ungültiger Ausdruck "string".

**xandros** · 25.05.2011, 19:58

String ist ein Keyword, welches in der verwendeten Form als Variable betrachtet wird..... und die Variable wurde nirgends definiert! (waere auch ziemlich ungluecklich, einer Variablen die Bezeichnung eines Keywords zu geben!)

Ich gehe davon aus, dass du in den betreffenden Zeilen lediglich ueberpruefen willst, ob der String eine Zeichenkette oder eine Zahl ist.

Entweder verwendest du dafuer TryParse() oder du bindest z.B. die Bibliothek Microsoft.VisualBasic.dll ein, mit der du dann auf die VB-Funktion IsNumeric() zugreifen kannst.Damit waere es dann z.B. moeglich, die Abfrage auf != IsNumeric umzustellen.

weltbesiedler · 27.05.2011, 12:54

Danke schon einmal für deine Hilfe.
Den Code hab ich nun völlig verworfen und hab versucht einen Taschenrechner zu Programmieren.
Es gibt einen Button und 3 Text Boxen. In 2 Text Boxen gibt man zwei Zahlen ein und in die dritte die Rechenart ( Division, Subtraktion, Multiplikation, Addition ), wenn man etwas anderes eingibt, soll die Meldung erscheinen: "Bitte eine Rechenart eingeben".
Ich hab versucht das bis jetzt so zu lösen:

Code:

  
            int a = Convert.ToInt16 (textBox1.Text);
            int b = Convert.ToInt16(textBox2.Text);
            int c = Convert.ToInt16(textBox3.Text);
            int ergebnis;
            if (textBox3.Text == "Division")
                ergebnis = a / b;
            if (textBox3.Text == "Multiplikation")
                ergebnis = a * b;
            if (textBox3.Text == "Subtraktion")
                ergebnis = a - b;
            if (textBox3.Text == "Addition")
                ergebnis = a + b;
            textBox3.Text = Convert.ToString (ergebnis);

Dann erscheint aber die Fehlermeldung:

Zitat:

Verwendung der nicht zugewiesenen lokalen Variable "ergebnis"

Das kommt wahrscheinlich davon, dass der Variable ergebnis erst ein Wert zugeordnet wird, wenn eine If-Anweisung zutrifft. Und wie programmiere ich, dass, wenn keine Rechenart eingegeben wurde, ein Fenster aufgeht mit der oben genannten Meldung?
Try - Catch hab ich schon probiert, aber das geht nicht.
Edit: Keine Ahnung wieso Zeile, wo die Variable a zugewiesen wird, verschoben ist.
Edit:
Hier mal ein zweiter Versucht, der funktioniert aber auch nicht:

Code:

  int a = Convert.ToInt16 (textBox1.Text);
            int b = Convert.ToInt16(textBox2.Text);
            int c = Convert.ToInt16(textBox3.Text);
            int ergebnis;
            int rechenart = Convert.ToInt16(textBox3.Text);
            ergebnis = a + rechenart + b;
            ergebnis = Convert.ToInt16 (textBox4.Text);
            textBox4.Text = Convert.ToString (ergebnis);

Fehlermeldung:

Zitat:

Format Exception wurde nicht behandelt

**xandros** · 27.05.2011, 14:31

Zitat:

int ergebnis;

Du verwendest hier C# und nicht VB, daher solltest du die Variablen schon korrekt deklarieren und initialisieren.

Code:

int ergebnis = 0;

Zitat:

int rechenart = Convert.ToInt16(textBox3.Text):

Du konvertierst hier ein Character in eine Zahl, was voellig in die Hose gehen muss, wenn dieses Zeichen nicht numerisch ist.....

Zitat:

ergebnis = Convert.ToInt16 (textBox4.Text);
textBox4.Text = Convert.ToString (ergebnis);

Hier nimmst du den Inhalt aus der Textbox4, konvertierst diesen in ein Integer und speicherst es in der Variablen ergebnis. Dann nimmst du den Inhalt aus der Variablen ergebnis, konvertierst ihn in einen String und weist diesen der Textbox4 zu....
Ziemlich ueberfluesig.

weltbesiedler · 27.05.2011, 18:46

So jetzt mal ein Versuch ganz ohne GUI:

Code:

static void Main(string[] args)
        {
            Console.Write("Hallo, welche Rechenart bevorzugen sie? ");
            string name = Console.ReadLine();
            Console.Write("Welche Zahl1: ");
            string Zahl1 = Console.ReadLine();
            Console.Write("Welche zahl2: ");
            string Zahl2 = Console.ReadLine();
            int ergebnis;
            ergebnis = Convert.ToInt16(Zahl1) + Convert.ToInt16(Zahl2);
            if (name == "Division")
                ergebnis = Convert.ToInt16(Zahl1) / Convert.ToInt16(Zahl2);
                Console.WriteLine(ergebnis);
            if (name == "Subtraktion")
                ergebnis = Convert.ToInt16(Zahl1) - Convert.ToInt16(Zahl2);
                Console.WriteLine(ergebnis);
            if (name == "Multiplikation")
                ergebnis = Convert.ToInt16(Zahl1) * Convert.ToInt16(Zahl2);
                Console.WriteLine(ergebnis);
            if (name == "Addition")
                ergebnis = Convert.ToInt16(Zahl1) + Convert.ToInt16(Zahl2);
                Console.WriteLine(ergebnis);
             Console.ReadLine();
                
        }
    }

Bietet C# so etwas wie "elif" in Python? Weil wenn ich eine Rechnung eingebe, erscheint das Ergebnis immer viermal.
Edit: "else if " gibt es anscheined allerdings wirft das bei mir eine Fehlermeldung aus:

Code:

if (name == "Division")
                ergebnis = Convert.ToInt16(Zahl1) / Convert.ToInt16(Zahl2);
                Console.WriteLine(ergebnis);
            else if (name == "Subtraktion")
                ergebnis = Convert.ToInt16(Zahl1) - Convert.ToInt16(Zahl2);
                Console.WriteLine(ergebnis);

Zitat:

Ungültiger Ausdruck "else".
; erwartet
Nur assignment-, call-, increment-, decrement- und "new object"-Ausdrücke können als Anweisungs verwendet werden.

**xandros** · 28.05.2011, 02:41

#elif als Preprozessoranweisung (ist aber nicht das was du suchst!)
#if (C# Reference)

Zitat:

Weil wenn ich eine Rechnung eingebe, erscheint das Ergebnis immer viermal.

Weil du die Ausgabeanweisung auch viermal in deinem Code hast und JEDE davon ausgefuehrt wird! Und das geschieht, weil dein Code nicht sauber ist.... da fehlen jede Menge Klammern!

Code:

if (Condition1){
     .....;
     .....;
}
if (condition2){
     .....;
     .....;
}
....

Code:

if (Condition1){
     Anweisung;
     Anweisung;
}
else if (Condition2){
     Anweisung;
     .....
}
else {
     Anweisung;
     .....
}

Schoener als die If-Abfragen ist aber Switch.

Code:

.....
switch(name)
{
    case "Division":
          ......Berechnung;
          break;
    case "....":
          ......Berechnung;
          break;
    case "....":
         ...........
}
Console.WriteLine("Ergebnis :{0}", ergebnis);

hier mal ein Beispiel mit Switch....

Code:

static void Main(string[] args)
        {
            int ergebnis=0; 
            Console.WriteLine("erste Zahl");
            string erste = Console.ReadLine(); 
            Console.WriteLine("zweite Zahl");
            string zweite = Console.ReadLine(); 
            Console.WriteLine("Rechenart (+ - * /)");
            string zeichen = Console.ReadLine(); 
            int ierste=Convert.ToInt16(erste);
            int izweite=Convert.ToInt16(zweite);
            

            switch (zeichen)
            {
                case "+":
                    ergebnis = ierste + izweite;
                    break;
                case "-":
                    ergebnis = ierste - izweite;
                    break;
                case "*":
                    ergebnis = ierste * izweite;
                    break;
                case "/":
                    ergebnis = ierste / izweite;
                    break;
            }
            Console.WriteLine("Ergebnis: {0}", ergebnis);
            string pause = Console.ReadLine();/* dient nur als Warteeingabe, damit das Konsolenfenster nicht geschlossen wird. */
        }

weltbesiedler · 28.05.2011, 11:09

Ok danke, hab das jetzt gelöst. Es haben nur die geschweiften Klammern gefehlt.

**xandros** · 28.05.2011, 17:08

Dann solltest du (rein vom Stil her!) deinen Code noch dahingehend abaendern, dass in den IF-Teilen lediglich die Berechnung durchgefuehrt wird und die Ausgabe einmalig erst am Ende der Ueberpruefungen erfolgt.
Das spart dir dann drei Zeilen Code....

weltbesiedler · 28.05.2011, 18:01

Ok. Jetzt sieht der Code so aus:

Code:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.Write("Hallo, welche Rechenart bevorzugen sie? ");
            string name = Console.ReadLine();
            Console.Write("Welche Zahl1: ");
            string Zahl1 = Console.ReadLine();
            Console.Write("Welche zahl2: ");
            string Zahl2 = Console.ReadLine();
            float ergebnis;
            ergebnis = Convert.ToInt16(Zahl1) + Convert.ToInt16(Zahl2);
            if (name == "Division")
            {
                ergebnis = Convert.ToInt16(Zahl1) / Convert.ToInt16(Zahl2);
                
            }
            else if (name == "Subtraktion")
            {
                ergebnis = Convert.ToInt16(Zahl1) - Convert.ToInt16(Zahl2);
                
            }
            else if (name == "Multiplikation")
            {
                ergebnis = Convert.ToInt16(Zahl1) * Convert.ToInt16(Zahl2);
                
            }
            else if (name == "Addition")
            {
                ergebnis = Convert.ToInt16(Zahl1) + Convert.ToInt16(Zahl2);
                
            }
            Console.WriteLine(ergebnis);
            Console.ReadLine();

        }
    }
}

**xandros** · 28.05.2011, 18:10

AH! Dir ist etwas aufgefallen.....

Zitat:

float ergebnis;

Divisionen ergeben sehr haeufig keine Integer-Ergebnisse.

Sie betrachten gerade: C# Programm-Problem

c# case ungültiger ausdruck

ungültiger ausdruck string