[C#] Som probleem

[Verwijderd]

Ik ben momenteel een rekenmachine aan het maken in C#.

Echter heb ik een probleem als ik tussentijds een som wil uitrekenen.

Zie code:

C#: 100

String lastSom = last;
String Tussensom = Werkgetal + lastSom + uitkfield.Text;
uitkfield.Text = Convert.ToString(EindSom);
i = 0;

Werkgetal is een double, last een String die bijvoorbeeld het teken + of - kan bevatten.
Als ik het debug, heeft hij deze waarde inderdaad bij zich, Maar als ik verder ga dan staat er op lastSom null in de som Tussensom.

De som moet dus bijvoorbeeld worden:
Tussensom = Werkgetal + waarde in uikfield.
Maar indien last een - bevat, moet het uiteraard een - som worden. Ik kan dit probleem verhelpen met een if/else if / else maar dat geeft weer veel regels onnodige code.

Hoe verhelp ik dit?

[Soundless]

Is er een reden waarom je last aan lastsom assigned?
En je weet dat strings in C# reference types zijn?

[Verwijderd]

Soundless schreef op vrijdag 26 november 2010 @ 00:28:
Is er een reden waarom je last aan lastsom assigned?
En je weet dat strings in C# reference types zijn?

Geen reden, als je ergens al een tijdje niet uit komt ga je van alles proberen.
Dat weet ik, alleen weet niet hoe ik het voor elkaar krijg om die waarde van last in de som te tonen.

[alex3305]

Zo te zien ben je met iets basaals aan het begin, dat is erg goed en ga vooral zo door. Echter is het handig om iets te leren over type conversie ook wel casten genoemd.

C# is namelijk een hard-typed taal. Concreet houdt dit in dat een object van een bepaald type nooit zomaar een ander type kan worden. Om jouw eigen code als voorbeeld te gebruiken; een double kan nooit zomaar een string worden.

Het is dus de bedoeling dat je voor je rekenmachine allereerst alle waarden gaat converteren. Dit houdt dus in dat je tijdens de verwerking van de som alleen maar getal-types wilt hebben (double, int, float, etc). Je moet hiervoor dus de invoer valideren (is het een nummer?) en omzetten naar jouw gewenst type. Daarna moet je de verwerking doen en weer terug omzetten naar een string om het aan de gebruiker te tonen. Dat laatste doe je overigens al in de derde regel van bovenstaande code.

---

Naast deze praktische hulp wil ik je ook even wijzen op een aantal programmeerstandaarden. Ik zie namelijk in jouw voorbeeld drie verschillende typen 'naming conventions' voortkomen, waarbij je er eigenlijk maar één correct gebruikt.

• Een lokale variabele (in een functie/methode) begin je altijd met een kleine letter en woorden die daarop volgen krijgen een hoofdletter. Voorbeeld: tussenSom.
• Een property (of methodenaam) begin je altijd met een hoofdletter en woorden die daarop volgen ook. Voorbeeld: EindSom.
• Een privé variabele zou ik altijd beginnen met een _ waarnaast je daarna verder gaat met een kleine letter. Voorbeeld: _beginSom.

Zo wordt je code namelijk een stuk overzichtelijker!

[Verwijderd]

Bedankt, ik ga het gelijk aanpassen! Alleen heb ik nog 1 vraag. last kan ik niet omzetten naar een cijfer, omdat het een teken + - / * ect. is. Hoe kan ik deze waarde dus in de som gebruiken ?

[NMe]

Verwijderd schreef op vrijdag 26 november 2010 @ 00:38:
Bedankt, ik ga het gelijk aanpassen! Alleen heb ik nog 1 vraag. last kan ik niet omzetten naar een cijfer, omdat het een teken + - / * ect. is. Hoe kan ik deze waarde dus in de som gebruiken ?

Kijken welk teken het is en op basis daarvan een andere if/else in gaan?

[Verwijderd]

NMe, dat geef ik inderdaad aan dat dat een mogelijkheid is in mijn beginpost, alleen geeft dit extra regels code, ik was benieuwd of het op mijn manier ook kon.

[NMe]

Verwijderd schreef op vrijdag 26 november 2010 @ 00:43:
NMe, dat geef ik inderdaad aan dat dat een mogelijkheid is in mijn beginpost, alleen geeft dit extra regels code, ik was benieuwd of het op mijn manier ook kon.

Het zal je vast verbazen, maar programmeren draait om regels code schrijven.

Er is in sommige talen een kortere variant (eval) maar gelukkig bestaat die in C# niet. En ik zeg gelukkig omdat 99 van de 100 keer dat eval gebruikt wordt, het een foute keuze is.

[Ventieldopje]

Aanvullend op alex een voorbeeldje:

C#:

using System;

namespace Voorbeeld
{
    class Boerderij
    {
        private int _animalCount = 0;
        
        public VoorbeeldClass()
        {
            
        }

        public int AnimalCount
        {
             get { return _animalCount; }
             set { _animalCount = value }
        }

        public void ShowCount()
        {
            Console.WriteLine(_animalCount.ToString());
        }
    }
}

Eventueel kun je zelfs nog met regions gaan werken zodat je je code in visual studio mooi op kan klappen en kan structureren, mocht je code erg groot worden .. VS10x Code Map kan daar bij ook een handige aanvulling zijn voor visual studio 2010.

C#:

#region Voorbeeldje

// Hier je code

#endregion

C#:

Convert.ToString(EindSom);

Kun je natuurlijk makkelijker schrijven als

C#:

EindSom.ToString();

Maargoed,

Je zou dit op kunnen lossen door switch te gebruiken ..

C#:

switch(lastSom) // + of - of *
{
    case "+":
        Console.WriteLine("Ik had hier een plus sommetje moeten doen.");
        break;
    case "-":
        Console.WriteLine("Ik had hier een min sommetje moeten doen.");
        break;
    case "*":
        Console.WriteLine("Vermenigvuldigen doen we maar in bed!");
        break;
    case "sinterklaas":
        Console.WriteLine("Shit! had ik nou dat gat in het dak al gerepareert?");
        break;
    default:
        Console.WriteLine("Lekkah niks doen jongahhh.");
        break;
}

Scheelt je een hoop code wat je normaal kwijt was geweest aan if / elsejes

@NME:
Er is wel een omweg, maar dat is via CSharpCodeProvider .. klikkerdeklik, een ietwat uitgebreide eval

[ Voor 5% gewijzigd door Ventieldopje op 26-11-2010 01:08 ]