1:a programmet, feedback!

Du läser in siffror i din meny, och det är väl ok, men mer intuitivt kan vara att läsa in tecknen +, -, * och / istället och ha case '+': och så vidare i din switch.

Du slarvar väldigt med texten, exempelvis skriver du i invAlt "please reenter <1-4>" men ger 5 alternativ. Blandade felstavningar: Welocme, calculatin, wich, Substraction, choosed, multiplice, Invalied ... Vad i hela friden är "Pytogars Technic"? Det vet kanske inte du heller, eftersom det valet ger "Invalid input, pleese reenter" i din kod.

Det är bra att dela upp program i funktioner, men det ska vara väldefinierade samlade enheter. Du har till exempel delat upp instruktionstexten och koden mellan geometricsTxt() och geometricsChoice(), det gör inget lättare utan tvärtom måste man sitta och para ihop allt i huvudet och hålla reda på två platser vid eventuella ändringar.

Du läser in heltal till alla dina operationer (delvis identisk kod till mul, add etc, det borde få dig att tänka över designen). I div() gör du heltalsdivision. Du kunde lika väl använda operationer med double. Nu har du ju "pratiga" funktioner som sköter sin egen input, men annars kan man tycka att tystlåtna funktioner som tar två parametrar och ger en retur kunde vara mer passande.

Funktionen med namnet "sqarea", den tror man först ska räkna ut arean på en kvadrat. När man läser koden ser man att du frågar efter längd och höjd av en kub(!) och sedan räknar ut arean av en rektangel. Fullkomligt förvirrat. Liknande men ännu konstigare med sqvolume, vad ska det betyda; volymen av en kvadrat? Där frågar du efter längd, höjd och bredd av en kub, som egentligen har samma mått på de tre per definition, och sedan ger du volymen av ett rätblock.

Den enda funktion du har som säger sig returnera någonting, main(), gör inte det.

Skriv kommentarer när du programmerar. Det går bra i början men när programmen blir större måste du veta vad och hur funktionerna fungerar och vad den returnerar om den gör det. Såsom liknande:


Kommentarer och förklaringar kommer göra dina program mycket enklare att förstå för dig själv och andra. Inte så roligt att felsöka program på flera tusen rader utan bra kommentarer.

Du kommenterar dina C och C++ program med:

//text

eller

/* text text */




/Vet_ej

@Katalysator tack för att du tog dig tid att påpeka alla programmets brister. Jag är jätteglad att få reda på allt jag kan göra bättre och kommer ta med mig den erfarenheten framöver, uppskattar verkligen feedbacken

@Vet ej tack du med för tipset om att andvända kommentarer, märkte i slutet att det blev en hel del att hålla reda på ändå ..

Det är sjukt dålig design att lägga ett helt program i en enda metod.

Hej där,

Jag har en idé för hur du ev. kan förbättra inmatningsmetoden.

Jag har också gjort Miniräknare i C++, min senaste miniräknare blev inte som dem tidigare, som liknar din, utan den blev snäppet bättre.

Jag gjorde följande för addering:
1. Jag tog bort menyvalen(kan ersättas med en if input=help så kommer en meny upp)
2. La in inmatiningen i en sträng.
3. , nu sparades plustecknets position i "pluspos"
4.

5. Konvertera strängarna(s1 och s2) till int
6. int1+int2=summa

Detta är inte ett trivialt sätt att göra det snarare en mycket restriktiv metod, men jag gillar den ändå eftersom användaren inte behöver förbereda programmet lika myket.

Detta är endast en grund till ett program, du behöver ju även de andra räknesätten.
Kolla gärna upp
Jag hoppas att allt går bra och att du skriver ett magnifikt program som kommer att slå världen med storm!

Lycka till, Knild