Blutiger Anfänger in C#

libiza · 13. Oktober 2013

Hallo Leute, ich hab mal eine Frage: Ich häng gerade bei einer Aufgabe bei der ich einen ganz ganz simplen "Taschenrechner" programmieren soll.
Ich hatte erst eine Stunde programmieren deshalb kenn ich mich noch nicht wirklich aus. Wäre nett wenn ihr es anfängerfreundlich erklären könntet was ich falsch gemacht hab.

Fehler: Verwendung der nicht zugewiesenen lokalen Variable "ergebnis".

Code

static void Main(string[] args)
{
int zahl1 = Convert.ToInt32(Console.ReadLine());
int zahl2 = Convert.ToInt32(Console.ReadLine());
string Rechenart = Console.ReadLine();
int ergebnis;
if (Rechenart == "+")
{
ergebnis = zahl1+zahl2;
Console.WriteLine(ergebnis);
}
else if (Rechenart == "-")
{
ergebnis = zahl1-zahl2;
Console.WriteLine(ergebnis);
}
else
{
Console.WriteLine("Die Rechenart ist unbekannt");
}

Alles anzeigen

fff619 · 13. Oktober 2013

einfach am Anfang

Code

int ergebnis = 0;

und das wars.
Edit: soweit ich das weiß, initialisiert C# Variablen nicht, heißt also, dass du für jede deklarierte Variable entweder den Operator new oder einen Standardwert zuweisen musst:

http://msdn.microsoft.com/de-d…ry/fa0ab757(v=vs.90).aspx

Bzw.: http://msdn.microsoft.com/de-d…ry/83fhsxwc(v=vs.90).aspx

Zitat

Bedenken Sie, dass die Verwendung nicht initialisierter Variablen in C# unzulässig ist.

libiza · 13. Oktober 2013

Danke, ich habs grad nocheinmal mit int ergebnis; durchlaufen lassen und es hat auch geklappt...

Kannst du mir vielleicht erklären warum ich int ergebnis= 0 schreiben sollte? Hab das nicht ganz verstanden

Edit: Okay, vielen dank hab dein edit zu spät gelesen

kiddy · 17. Oktober 2013

Hey ho,

libiza schrieb:

Code

if (Rechenart == "+")

ich empfehle dir fürs Vergleichen von Strings die .Equals Methode, da wenn man diesem Thread Glauben schenken darf das "==" nur auf Objektgleichheit prüft.

Demnach würde dein Code ungefähr so aussehen:

Code

if (Rechenart.Equals("+"))

Korrigiert mich, wenn ich da falsch liege.

MfG DaS kiDDy

Dipointed · 17. Oktober 2013

Das ist bei Java so, bei C# funktioniert == auch.

libiza · 17. Oktober 2013

Ich finde programmieren schon relativ anspruchsvoll und möchte erstmal dabei bleiben wie es mir der Dozent zeigt.

Ich glaub ich werd diesen Thread noch ein wenig weiter missbrauchen für meine neuen Fragen

libiza · 20. Oktober 2013

Hallo,
wäre jemand gewillt mir kurz zu erklären wie ich alle dreistelligen zahlen von 100-999 die sich durch ihre Ziffern teilen lassen ausgeben kann?
Bsp: 126, lässt sich durch 1 2 und 6 teilen.

Spoiler anzeigen

Vielen dank

javaest · 20. Oktober 2013

Erstelle dir ein eindimensionales Array, indem du per String-Splitt-Methode, die einzelnen Chars reinpackst. Nun teilst du die ursprüngliche Zahl durch jeden Char (zahl im Array), kommt eine grade Zahl raus ist es teilbar.

Beispiel:
int Ausgangszahl = 134;
int Chars Array();
Chars = String.Split(134,1);
int Ergebnis = Ausgangszahl / Chars[0];
//Hier prüfen ob es KEINE Kommazahl ist
if (Ergebnis == Kein Komma)
Console.Writeline ("Teilbar");
else
Console.Writeline("Nicht teilbar");

Ist natürlich nur eine Skizze..

fff619 · 20. Oktober 2013

javaest schrieb:

Erstelle dir ein eindimensionales Array, indem du per String-Splitt-Methode, die einzelnen Chars reinpackst. Nun teilst du die ursprüngliche Zahl durch jeden Char (zahl im Array), kommt eine grade Zahl raus ist es teilbar.

Beispiel:
int Ausgangszahl = 134;
int Chars Array();
Chars = String.Split(134,1);
int Ergebnis = Ausgangszahl / Chars[0];
//Hier prüfen ob es KEINE Kommazahl ist
if (Ergebnis == Kein Komma)
Console.Writeline ("Teilbar");
else
Console.Writeline("Nicht teilbar");

Ist natürlich nur eine Skizze..

Alles anzeigen

Sein Quellcode steht doch schon dort...
Einfach eine For-Schleife durchlaufen lassen. In dieser Schleife setzt du dort eine Variable int i auf 100 und tust bei jedem durchlauf dann diese Variable um 1 erhöhen.

Code

int h, z, e;
//Hier < 1000 (kleiner 1000), weil wir ja 1000 nicht dabei haben wollen. Du könntest auch <= 999 sagen (kleiner gleich 999)
for (int i = 100; i < 1000; i++)
{
h = i / 100;
z = (i / 10) % 10;
e = i % 10;
if (z != 0 && e != 0 && i % h == 0 && i % z == 0 && i % e == 0)
Console.WriteLine(i);
}

Alles anzeigen

Somit werden alle Zahlen ausgegeben, die durch die einzelnen Zahlen teilbar sind:

Einführung von Schleifen siehst du ab hier: http://www.youtube.com/watch?v=HoxHRQrmJSM

libiza · 20. Oktober 2013

Vielen Dank fff619,
ich hatte das mal mit einer for-schleife probiert, jedoch hab ich sie an die falsche Stelle gesetzt (direkt vor das if)

Aber jetzt ist es mir klar geworden warum das nicht funktioniert, denn die ganzen Operationen oben müssen eine vorgegebene variable i haben, richtig?

Auch danke an dich javaest, hab morgen programmier-vorlesung und da bekommen wir erst arrays.

fff619 · 20. Oktober 2013

libiza schrieb:

Vielen Dank fff619,
ich hatte das mal mit einer for-schleife probiert, jedoch hab ich sie an die falsche Stelle gesetzt (direkt vor das if)

Wenn du die for-schleife direkt vor dem if machst, dann erhöht sich zwar die Variable i, jedoch nicht die Variable h, z, e. Somit bleiben diese immer konstant bzw. werden nicht verändert.

libiza · 20. Oktober 2013

fff619 schrieb:

Wenn du die for-schleife direkt vor dem if machst, dann erhöht sich zwar die Variable i, jedoch nicht die Variable h, z, e. Somit bleiben diese immer konstant bzw. werden nicht verändert.

Alles klar, habs geschnallt.
Danke nochmal
StageTwo ist immer da wenn man mal Hilfe braucht, gefällt mir

javaest · 20. Oktober 2013

Kein Problem. Und Arrays sind ganz einfach, keine Sorge

libiza · 6. November 2013

Benötige wiedermal Hilfe. Versuche gerade zu testen ob es sich bei einem Wort um ein Palindrom handelt. z.B ANNA, OTTO etc.
Bei diesen Wörtern funktioniert es bereits. Aber wenn ich: "Anna" eingebe sieht er es nicht als Palindrom weil er "A" mit "a" vergleicht.
Lösung wäre alles einheitlich umzuwandeln. Jedoch weiß ich nicht genau wie die "ToLower" bzw. "ToUpper" Funktion richtig arbeitet.
Könnte mir jemand kurz helfen?
Vielen dank

Code

string s = "Anna";
s.ToLower();
bool IstPalindrom = true;
for (int i = 0; i < s.Length; i++)
{
if (s[i] != s[s.Length - 1 - i])
IstPalindrom = false;
}
if (IstPalindrom)
Console.WriteLine("Ist Pali");
else
Console.WriteLine("Ist kein pali");

Alles anzeigen

dawd1dawy · 6. November 2013

http://msdn.microsoft.com/en-u…8f86at%28v=vs.110%29.aspx

Zitat

This method does not modify the value of the current instance. Instead,
it returns a new string in which all characters in the current instance
are converted to lowercase.

Gruß

libiza · 6. November 2013

Vielen dank.

Code

string s = "Anna";
string low = s.ToLower();

dawd1dawy · 6. November 2013

Kein Problem

libiza · 7. November 2013

Ich verzweifel gerade "wiedermal" an einer Aufgabe:
"Hat der eingelesene String mehr als 12 Ziffern geben Sie eine Fehlermeldung aus und beenden das Programm."
Ich hab versucht eine Variable hochzuzählen und dann mit break aus der Schleife rauszukommen, aber break bringt mich immer nur zum while und nicht ganz raus.
Der restliche Teil ist bereits gelöst, aber ich weiß nicht wie ich diese dumme Fehlermeldung hinbekomme
http://pastebin.com/yCKqEqbu

Spoiler anzeigen

Code

static void Main(string[] args)
{
int summe = 0;
int anzZeichen = 0, Zahlenanzahl = 0;
int Zahl, Prüfziffer;
bool Multiplikator3= false;
Console.WriteLine("ISBN Prüfziffer berechnen");
Console.WriteLine("=========================");
Console.WriteLine("Bitte geben Sie die ISBN ein");
string s = "978 - 3 - 898264 - 776";
do
{
for (int i = 0; i < s.Length; i++)
{
if (char.IsNumber(s[i]))
{
//Wandelt den char in eine echte Zahl um
Zahl = s[i] - '0';
if (Multiplikator3 == false)
summe = summe + Zahl;
else
summe = summe + (Zahl * 3);
// Switcht den Multiplikator, da immer abwechselnd mit 1 oder 3 Multipliziert wird.
Multiplikator3 = !Multiplikator3;
}
anzZeichen++;
}
}
while (anzZeichen < s.Length);
Prüfziffer = 10 - (summe % 10);
if (Zahlenanzahl <= 12)
Console.WriteLine(Prüfziffer);
else
Console.WriteLine("Sie haben eine ungültige Zahl eingegeben");
}
}
}

Alles anzeigen

Habs hinbekommen, hier ist meine Lösung falls es noch jemanden interessiert

Spoiler anzeigen

Code

static void Main(string[] args)
{
int summe = 0;
int anzZeichen = 0;
int Zahl, Prüfziffer;
bool Multiplikator3= false;
Console.WriteLine("ISBN Prüfziffer berechnen");
Console.WriteLine("=========================");
Console.WriteLine("Bitte geben Sie die ISBN ein");
string s = Console.ReadLine();
for (int i = 0; i < s.Length; i++)
{
if (char.IsNumber(s[i]))
{
//Wandelt den char in eine echte Zahl um
Zahl = s[i] - '0';
if (Multiplikator3 == false)
summe = summe + Zahl;
else
summe = summe + (Zahl * 3);
// Switcht den Multiplikator, da immer abwechselnd mit 1 oder 3 Multipliziert wird.
Multiplikator3 = !Multiplikator3;
anzZeichen++;
}
}
Prüfziffer = 10 - (summe % 10);
if (anzZeichen <= 12)
Console.WriteLine(Prüfziffer);
else
Console.WriteLine("Sie haben eine ungültige Zahl eingegeben");

Alles anzeigen

fff619 · 7. November 2013

libiza schrieb:

Vielen dank.

Code

string s = "Anna";

string low = s.ToLower();

Eigtl. unnötig eine neue Variable zu erstellen, außer wenn du diesen Wert später weiterhin abfragen willst.

Man kann es gleich im if-Zwei abfragen:

Zitat

if (s[i].ToString().ToLower() != s[s.Length - 1 - i].ToString().ToLower())

Xaw.4 · 10. November 2013

fff619 schrieb:

Eigtl. unnötig eine neue Variable zu erstellen, außer wenn du diesen Wert später weiterhin abfragen willst.

Man kann es gleich im if-Zwei abfragen:

hallte ich nicht für schlau.
vorschlag 1: es ist oft gut sich die eingabe zu behalten.. z.b. um dann die Ausgabe "Anna ist ein Palindrom" machen zu können.
vorschlag 2: a) warim machst du ToString() bei einem string? b) Du machst aus einem programm mit linearer Laufzeit, eins mit Quadratischer (in jedem durchlauf, durchläfst du das array wieder - nichts anderes macht ToLower). c) das macht es echt viel unleserlicher - sprich du hast aus leserlichen, relativ effizienten Code, unleserlichen ineffizienten code gemacht - und das nur, um eine variable weniger anlegen zu müssen

edit:
was dir auffallen hätte können, istdass er

Code

for (int i = 0; i < s.Length; i++)

geschrieben hat, und dabei wird der Letzte Buchstabe mit dem ersten verglichen, und später, der erste mit dem letzten... sprich die doppelte arbeit.

Code

for (int i = 0; i < s.Length /2 ; i++)

Teilen