Die ersten Schritte - Verbesserungs- und Codingvorschläge.

Erleon · 22. Oktober 2012

Nabend Community,
Ich habe beschlossen mir die Sprache C# anzueignen , da mir die Sprache ( Lazarus , also delphi ) aus der Schule gar nicht gefällt.
Ich habe mir heute mal die Grundlagen der Konsolenanwendungen angeschaut und gemerkt das es ähnlich wie Delphi ist. Daher hab ich auch schnell kapiert wie was funktioniert.
Ich hab mich dann einfach mal rangesetzt und einen kleinen, also wirklich kleinen , Taschenrechner erstellt. Da wollte ich doch glatt mal nachfragen was man da besser machen könnte. Des Weiteren suche ich leichte Programme zum Coden. Also Programme die mich fördern , aber auch solche die nicht absolut Unmöglich für einen Anfänger sind.
Hier der Download ( Visual C# 2010): http://ul.to/9waz8gvi
Hier der Quelltext zu meinen rechner:

Spoiler anzeigen

Code

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 WindowsFormsApplication1
{
public partial class Form1 : Form
{
string operation;
int eingabe1;
int eingabe2;
public Form1()
{
InitializeComponent();
}
private void beendenToolStripMenuItem_Click(object sender, EventArgs e)
{
Environment.Exit(0);
}
private void button1_Click(object sender, EventArgs e)
{
if (textBox1.Text == "")
{
textBox1.Text = " Erst text eingeben";
}
else
{
eingabe1 = Convert.ToInt32(textBox1.Text);
textBox1.Text = "";
operation = "+";
}
}
private void button4_Click(object sender, EventArgs e)
{
int ergebnis = 0;
eingabe2 = Convert.ToInt32(textBox1.Text);
switch (operation)
{
case "+":
ergebnis = eingabe1 + eingabe2;
textBox1.Text =Convert.ToString(ergebnis);
break;
case "-":
ergebnis = eingabe1 - eingabe2;
textBox1.Text = Convert.ToString(ergebnis);
break;
case "*":
ergebnis = eingabe1 * eingabe2;
textBox1.Text = Convert.ToString(ergebnis);
break;
}
}
private void button2_Click(object sender, EventArgs e)
{
eingabe1 = Convert.ToInt32(textBox1.Text);
textBox1.Text = "";
operation = "-";
}
private void button3_Click(object sender, EventArgs e)
{
eingabe1 = Convert.ToInt32(textBox1.Text);
textBox1.Text = "";
operation = "*";
}
}
}

Alles anzeigen

BlackManta · 22. Oktober 2012

Schaut doch schon mal gut aus
Du könntest dich auch mit algorithmen beschäftigen also z.b. ein algorithmus um etwas zu zeichnen oder zu verschlüsseln und entschlüsseln

kuhkuh · 22. Oktober 2012

Ersetz mal

Code

Environment.Exit(0);

durch

Code

this.Close();

Environment.Exit() beendet den gesamten Prozess. Das ist zwar nicht schlimm und letztendlich auch das Ziel, aber ich finde this.Close() ästhetischer.

Kuh :>

Erleon · 22. Oktober 2012

Danke Kuh , merk ich mir.
Manta:
Hast du eventuell eine Seite wo ich mich zu den Algorithmen in C# einlesen kann?

**dmix** · 22. Oktober 2012

Bevor du mit Algorithmen und Verschlüsselung Anfängst (Wozu bei einem Taschenrechner?), verfeinre mal deinen Code mit Exception Handling. Warum? Jede Konvertierung von dir kann eine ArgumentException werfen, wenn du anstatt einer Zahl, z.B. einen String eintippst.

BlackManta · 22. Oktober 2012

Erleon schrieb:

Danke Kuh , merk ich mir.
Manta:
Hast du eventuell eine Seite wo ich mich zu den Algorithmen in C# einlesen kann?

Wir haben und damit in der letzten Zeit in der Schule beschäftigt und haben Robot Karol verwendet
Naja für mich alles nichts neues aber der rest der Klasse hat da kein Plan von

Und dmix es ging um neue einfache Programme

Erleon · 22. Oktober 2012

Gut wenn ich das jetzt mit deinen Beispiel verwirklichen will, welche Fehlermeldung kommt denn wenn ich anstatt einer Zahl einen String eingebe?
Wenn ich das "catch ( )" benutze muss ich ja in die Klammer eine Bedingung eintragen. Wie kann ich diese Bedingung ermitteln?

Edit:
Manta:
Es steht im Titel "Verbesserungs- und Codingvorschläge". Von daher

kuhkuh · 22. Oktober 2012

Code

private void button1_Click(object sender, EventArgs e)
{
if (textBox1.Text == "")
{
textBox1.Text = " Erst text eingeben";
}
else
{
try
{
eingabe1 = Convert.ToInt32(textBox1.Text);
}
catch (FormatException ex)
{
MessageBox.Show("Die Eingabe hat das falsche Format. Error: " ex.ToString());
}
textBox1.Text = "";
operation = "+";
}
}

Alles anzeigen

BlackManta · 22. Oktober 2012

Erleon schrieb:

Gut wenn ich das jetzt mit deinen Beispiel verwirklichen will, welche Fehlermeldung kommt denn wenn ich anstatt einer Zahl einen String eingebe?
Wenn ich das "catch ( )" benutze muss ich ja in die Klammer eine Bedingung eintragen. Wie kann ich diese Bedingung ermitteln?

Edit:
Manta:
Es steht im Titel "Verbesserungs- und Codingvorschläge". Von daher

hmm :

Zitat

Des Weiteren suche ich leichte Programme zum Coden.

Erleon · 22. Oktober 2012

kuhkuh schrieb:

Code

private void button1_Click(object sender, EventArgs e)

{

if (textBox1.Text == "")

{

textBox1.Text = " Erst text eingeben";

}

else

{

try

{

eingabe1 = Convert.ToInt32(textBox1.Text);

}

catch (FormatException ex)

{

MessageBox.Show("Die Eingabe hat das falsche Format. Error: " ex.ToString());

}

textBox1.Text = "";

operation = "+";

}

}

Alles anzeigen

MessageBox.Show("Die Eingabe hat das falsche Format. Error: " + ex.ToString());
Ich fühl mich gerade gut, da ich einen Fehler gefunden habe

//Edit:
Gut also wie ich sehe, muss ich nach der eingabe die Ausnahmebehandlung einbauen. Richtig?

kuhkuh · 22. Oktober 2012

Hatte das eigentlich eingefügt :'o
Aber StageTwo hat ja immer iwo iwelche Probleme Alleine die CodeTags kann man vergessen. Komplett falsch eingerückt.

Erleon · 22. Oktober 2012

Kann man da irgendwie ein Zeilenumbruch mit reingeben? Das sieht so unübersichtlich aus...

Spoiler anzeigen

//Edit ans Team:
Das bild hab ich gerade hochgeladen und bei einen Klick darauf kommt eine 404-page ( Im Expander )

/Edit2:
Gut Update:
sieht jetzt so aus:

Spoiler anzeigen

Code

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 WindowsFormsApplication1
{
public partial class Form1 : Form
{
string operation;
int eingabe1;
int eingabe2;
public Form1()
{
InitializeComponent();
}
private void beendenToolStripMenuItem_Click(object sender, EventArgs e)
{
this.Close();
}
private void button1_Click(object sender, EventArgs e)
{
if (textBox1.Text == "")
{
textBox1.Text = " Erst text eingeben";
}
else
{
try
{
eingabe1 = Convert.ToInt32(textBox1.Text);
}
catch (FormatException ex)
{
MessageBox.Show("Die Eingabe hat das falsche Format. Error: "+ ex.ToString());
}
textBox1.Text = "";
operation = "+";
}
}
private void button4_Click(object sender, EventArgs e)
{
int ergebnis = 0;
eingabe2 = Convert.ToInt32(textBox1.Text);
switch (operation)
{
case "+":
ergebnis = eingabe1 + eingabe2;
textBox1.Text =Convert.ToString(ergebnis);
break;
case "-":
ergebnis = eingabe1 - eingabe2;
textBox1.Text = Convert.ToString(ergebnis);
break;
case "*":
ergebnis = eingabe1 * eingabe2;
textBox1.Text = Convert.ToString(ergebnis);
break;
case "/":
try
{
ergebnis = eingabe1 / eingabe2;
}
catch (DivideByZeroException ex)
{
MessageBox.Show("Fehler! Eventuell durch Null dividiert? Error:" + ex.ToString());
}
textBox1.Text = Convert.ToString(ergebnis);
break;
}
}
private void button2_Click(object sender, EventArgs e)
{
try
{
eingabe1 = Convert.ToInt32(textBox1.Text);
}
catch (FormatException ex)
{
MessageBox.Show("Bitte erst eine Zahl eintippen."+"Error:"+ex.ToString());
}
textBox1.Text = "";
operation = "-";
}
private void button3_Click(object sender, EventArgs e)
{
try
{
eingabe1 = Convert.ToInt32(textBox1.Text);
}
catch (FormatException ex)
{
MessageBox.Show("Bitte erst eine Zahl eingeben. Error:" + ex.ToString());
}
textBox1.Text = "";
operation = "*";
}
private void button5_Click(object sender, EventArgs e)
{
eingabe1 = Convert.ToInt32(textBox1.Text);
textBox1.Text = "";
operation = "/";
}
}
}

Alles anzeigen

dongdong · 22. Oktober 2012

~Moved to Coding Allgemein~

Erleon · 19. März 2013

Ich muss den Thread jetzt nochmal ausgraben, ich sitze immernoch dran und hab dieses Programm zu meiner Facharbeit in Informatik gemacht.
Ich steh aber schon seit 1 Stunde auf dem Schlauch und weiß nicht weiter.

Ich habe jetzt die Button 1-9 für die Zahleneingabe in die TextBox hinzugefügt und möchte das die Zahl beim Drücken des jeweiligen Buttons in der TextBox erscheint bzw. hinzugefügt wird.
Wenn ich das über

Code

textBox1.Text = "1";

mache dann besteht die ganze textBox ja nur aus der 1. Jedoch möchte ich zum Beispiel 2x die 1 drücken. Da sollen dann beide 1sen in der Textbox erscheinen...

Ich hoffe ihr versteht was ich von euch will

Nickname09 · 19. März 2013

Code

textBox1.Text += "1";

+= heißt einfach textBox1.Text = textBox1.Text + "1", ist eine Kurzschreibweise (für faule Leute wie mich :P).

Erleon · 19. März 2013

Ok , so einfach...

Ennosuke · 19. März 2013

Hey,

zwar ist es nicht aktuell zu deiner Frage aber einfach mal ein paar Tipps/Hinweise

Niemals Informationen von Exceptions an den Anwender raus geben, außer es handelt sich um eine "unbehandelte" Exception, also eine die du nicht vorhersehen kannst. Bei allen anderen solltest du aussagekräftige Fehlermeldungen anzeigen und bei welchen die zum Absturz führen immer mit loggen, aber niemals so was wie

Code

MessageBox.Show("Die Eingabe hat das falsche Format. Error: " ex.ToString());

Es reicht zu sagen "Scheinbar haben sie keine Zahl eingeben, bitte wiederholen sie die Eingabe!" oder ähnliches, der Rest ist für den Nutzer total uninteressant und stört ihn nur.

Teilen