C# - Leere Variable, wie überprüfen? - Die Hardware-Community für PC-Spieler

Der Maniac · 18.02.2010, 14:52

Hallo Leute!

Ich rätsel grade dadran wie ich eine leere Variable finden kann und dann ne Fehlermeldung rausgegeben wird?!

Ich habe in meinem Programm (Anhang!) Diverse Programmstellen wo eine Zahleneingabe erwartet wird, kommt hier jedoch ein Buchstabe oder einfach nichts, stürzt das Programm ab, und das möchte ich verhindern!

Weiß jemand wie ich das bewerkstelligen kann?

Den entsprechenden Quellcode evtl. so einfügen dasich es nurnoch kopieren brauch

Danke schonmal fürs Kopf zerbrechen

bingo88 · 18.02.2010, 15:33

Hab's jetzt mal ausm Kopf gemacht, müsste aber die Ausnahme vermeiden...

Code:

static void Main()
    {
        int typ;
        Console.Clear();
        Console.WriteLine("************************************************************");
        Console.WriteLine("Bitte waehlen sie den Anwendungstyp aus!");
        Console.WriteLine("Dauerhaft ausfuehren  ==>   1");
        Console.WriteLine("X Mal ausfuehren      ==>   2");
        String input = Console.ReadLine();
        if (input != null)
        {
            typ = Convert.ToInt32(Console.ReadLine());
        }
        Console.WriteLine("************************************************************");

            switch (typ)
            {
                case 1:
                    Dauerhaft();
                    break;

                case 2:
                    X_Mal();
                    break;

                default:
                    Console.WriteLine("Bitte geben Sie NUR eine 1 oder eine 2 ein!");
                    break;
            }
    }

Der Maniac · 18.02.2010, 15:55

Das funktioniert fast...^^

Nur mekkert er jetzt das die Variable "Typ" nichtmehr zugewiesenw erden kann... Wenn ich den geschwiften Klammern nach dem if (input...) wegmache, erkennt er typ wieder, allerdings funktioniert dann der rest nicht mehr >.<

bingo88 · 18.02.2010, 16:31

Hast du mal versucht, int typ = 0 zu schreiben? Java nörgelt nämlich auch, wenn man auf uninitialisierte Variablen zugreift.

rofl...vs...xD · 19.02.2010, 16:54

Das sollte so funktionieren wie du das willst.

Code:

static void Main()
        {
            int typ;
            Console.Clear();
            Console.WriteLine("************************************************************");
            Console.WriteLine("Bitte waehlen sie den Anwendungstyp aus!");
            Console.WriteLine("Dauerhaft ausfuehren  ==>   1");
            Console.WriteLine("X Mal ausfuehren      ==>   2");
            Console.WriteLine("************************************************************");
            String input = Console.ReadLine();
            if (input != "")
            {
                typ = Convert.ToInt32(Console.ReadLine());



                switch (typ)
                {
                    case 1:
                        Dauerhaft();
                        break;

                    case 2:
                        X_Mal();
                        break;

                    default:
                        Console.WriteLine("Bitte geben Sie NUR eine 1 oder eine 2 ein!");
                        break;
                }
            }
            else
            {
                Console.WriteLine("Keine Eingabe");
            }
        }

aurionkratos · 19.02.2010, 16:58

Wenn jemand z.B. Buchstaben eingibt, haut er dir eine Exception raus. Bitte diese abfangen oder vorher mit Int32.TryParse() überprüfen, ob es parsebar ist. Zweiteres wäre der bessere Weg.

EDIT: außerdem möchtest du sicherlich " typ = Convert.ToInt32(input);" anstatt " typ = Convert.ToInt32(Console.ReadLine());" haben, oder?

EDIT 2: Der erste Absatz ist übrigens auch genau das, was der Threadersteller gesucht hat.

rofl...vs...xD · 19.02.2010, 17:23

Zitat:

Zitat von aurionkratos

Wenn jemand z.B. Buchstaben eingibt, haut er dir eine Exception raus. Bitte diese abfangen oder vorher mit Int32.TryParse() überprüfen, ob es parsebar ist. Zweiteres wäre der bessere Weg.

EDIT: außerdem möchtest du sicherlich " typ = Convert.ToInt32(input);" anstatt " typ = Convert.ToInt32(Console.ReadLine());" haben, oder?

EDIT 2: Der erste Absatz ist übrigens auch genau das, was der Threadersteller gesucht hat.

Du hast schon recht mit Buchstaben usw. ABER das hat der Threadhersteller nicht verlangt.

Ich habe mich nur auf sein Problem beschränkt, obwohl das mit Int32.TryParse(); usw sehr sinnvoll wäre.

Und mit deinem ersten EDIT hast du natürlich vollkommen recht.

bingo88 · 19.02.2010, 19:26

Das hab ich doch glatt übersehn... kommt davon, wenn man das nicht ausprobiert

d00mfreak · 20.02.2010, 14:18

Wenn es wirklich nur um eine Unterscheidung zwischen "1", "2", und "Rest" gehen soll, dann sollte doch ein einfacher If-Block wie

Code:

if (input == "1"){
	Dauerhaft();
}
else if (input == "2"){
	X_Mal();
}
else{// if input == alles andere
	Console.WriteLine("Bitte geben Sie eine 1 oder eine 2 ein!");
}

ohne hin- und herkonvertieren sowie Parse-Versuche reichen? Oder ist es für die Aufgabe auch nötig, zwischen einer leeren Eingabe und dem Rest zu unterscheiden?

Der Maniac · 24.02.2010, 13:44

So ich melde mich auch noch mal

Folgendes: Das einzige was einwandfrei funktioniert ist die Lösung von d00mfreak, allerdings wollte ich versuchen Switchcase zu verwenden, sieht schöner aus

Die Lösung von Rofl...vs...xD funktioniert fast, bei Zahlen oder einer leeren Eingabe kein Problem, kommt hier aber ein Buchstabe stürzt das ganze wieder ab mit einer Exception...^^ Knapp am Ziel vorbei also...

Das mit dem Parse klingt interessant, kann das hier jemand näher erläutern bzw. erklären was das macht und wie ich es einbinde? Ich mach jetzt seit grade mal 4 Wochen C#...^^

mfg, Maniac