taak probleem

[hiseiba]

Dag iedereen ik was mijn taak voor school aan het maken en ik snap echt niet wat de probleem van mijn codes zijn kan iemand me helpen

schooltaak:
Ontwerp een console toepassing waarmee je berekent hoeveel iemand voor een bioscoopkaartje moet betalen.
De volle prijs van een bioscoopticket bedraagt 13,70 EUR.
Op basis van de leeftijd wordt de prijs bepaald:
Jonger dan 5 jaar: gratis;
Tussen de 5 en de 12: halve prijs;
Tussen de 13 en de 54: vol tarief;
55+: gratis

mijn codes:
//declaratie variabelen
const double cdblVollePrijsTicket = 13.70;
double dblEindPrijsTicket;
int intLeeftijd;

dblEindPrijsTicket = 0;

//invoer
Console.Write("Hoe houd ben je: ");
intLeeftijd = Convert.ToInt32(Console.Read()); ;


if (intLeeftijd < 5)
{

//uitvoer
Console.WriteLine();
Console.Write("jouw bioscoopticket is gratis.");
}
else if (intLeeftijd >= 5 && intLeeftijd<= 12)
{
//berekening
dblEindPrijsTicket = cdblVollePrijsTicket / 2;

//uitvoer
Console.WriteLine();
Console.Write("je moet " + Math.Round(dblEindPrijsTicket, 2).ToString() + "EUR betalen.");
}
else if (intLeeftijd >= 13 && intLeeftijd <= 54)
{
//berekening
dblEindPrijsTicket = cdblVollePrijsTicket;

//uitvoer
Console.WriteLine();
Console.Write("je moet " + Math.Round(dblEindPrijsTicket, 2).ToString() + "EUR betalen.");
}
else if (intLeeftijd >= 55)
{

//uitvoer
Console.WriteLine();
Console.Write("jouw bioscoopticket is gratis.");
}
else
{
Console.WriteLine();
Console.Write("je geeft geen gegeven in");
}


Console.ReadKey();

[Josk79]

Je zou het beste je code eens tussen code-tags kunnen stoppen zodat het leesbaar is. En geef eens aan wat het probleem is.

[Umbrah]

We doen hier niet aan huiswerk problemen oplossen, maar je zou kunnen beginnen eens te omschrijven waar je probleem zit volgens een compiler, wat je verwacht, wat voor foutmeldingen je hebt, en de code leesbaar te maken door de code-tags te gebruiken (met een taal specificatie, het ziet er uit als klassiek VBasic, dit).

Wat betreft je huiswerk:

Wat zei je docent/huiswerkbegeleider toen je om hulp vroeg? Wat heb je in je aantekeningen van je lessen staan? Wat voor bronnen gebruik je om dit te leren? Het is leuk spul en een belangrijke vaardigheid om te leren, en dat willen we je natuurlijk niet ontnemen, want de denkwijze die hier achter zit heb je een hoop aan later.

[Josk79]

Umbrah schreef op zaterdag 8 oktober 2022 @ 14:39:
...het ziet er uit als klassiek VBasic, dit)...

Lijkt eerder op C#

[Mrlten]

Ik zou hier een ReadLine van maken, anders houdt het op bij negenjarigen:

code:

intLeeftijd = Convert.ToInt32(Console.Read())

[Josk79]

Tot zover het opvoeden dat TS iets beter zijn best moet doen om het probleem te omschrijven.

[DukeBox]

hiseiba schreef op zaterdag 8 oktober 2022 @ 14:30:
Tussen de 5 en de 12: halve prijs;
Tussen de 13 en de 54: vol tarief;
55+: gratis

Beetje offtopic maar zoals het er nu staat zijn er 2 gaten, namelijk als je tussen de 12 en 13 jaar bent en als je tussen de 54 en 55 bent.

[Josk79]

Het aantal integers > 12 en < 13 is gelukkig niet zo groot. Idem voor 54, 55.

[Killah_Priest]

Hungarian notation: zsm hiervan afstappen in een strong typed language aangezien je hier geen vrienden mee maakt in het bedrijfsleven.

Verder ontopic: kun je duidelijk omschrijven wat er precies misgaat? Wat is bv de uitkomst die krijgt bij verschillende inputs?

[michiel_]

Mrlten schreef op zaterdag 8 oktober 2022 @ 14:45:
Ik zou hier een ReadLine van maken, anders houdt het op bij negenjarigen:

code:

intLeeftijd = Convert.ToInt32(Console.Read())

Nog mooier: Console.Read() geeft een int (charcode) terug. Ik snap dat je hier flink op vast kan lopen.
Ik zou zeggen, kijk naar de waarde van intLeeftijd bij verschillende invoeren, pak een ASCII tabel erbij en probeer uit te zoeken wat er gebeurd

[Umbrah]

Josk79 schreef op zaterdag 8 oktober 2022 @ 14:40:
[...]


Lijkt eerder op C#

Visual Basic:

Dim affirm As String = "You're right"

[Oon]

Zoals al aangegeven: Afstappen van de Hongaarse notatie. Het boeit onder in je script geen reet of je variabele een constante is, noch of het een int of double is. C# (wat dit lijkt te zijn) is een strong typed taal, dus als je appels met peren gaat vergelijken krijg je daar meteen een hele duidelijke foutmelding van. Je variabelen hiermee vervuilen zorgt alleen voor enorme refactor commits als je ooit een type verandert.

Daarnaast is Tweakers niet de juiste plek om je huiswerk voor je te laten doen (al blijken mensen hier wel in te trappen deze keer). Je zou op z'n allerminst je code tussen [code] tags kunnen zetten zodat indents worden meegenomen en het nog enigszins leesbaar is, en (ook tussen code tags) de foutmelding die je krijgt kunnen delen.

Het feit dat je het 'mijn codes' noemt maakt wat mij betreft duidelijk dat je nog aan het begin van je schoolcarrière zit en nog echt geen kaas hebt gegeten van programmeren, dus het gaat je veel helpen als je nu alvast van de slechte gewoontes afstapt want je zult hier nog wel een keer terugkomen.

Ben wel benieuwd wat er nu daadwerkelijk fout gaat, en wat voor oplossingen je zelf hebt geprobeerd op basis van de overduidelijke foutmelding die C# hierover zou moeten geven

[RobIII]

Zoals al aangegeven door anderen: we willen best met je meedenken met je huiswerk maar we verwachten óók dat je zelf wat moeite doet. We zitten hier niet om het vóór je te maken.

Begin eens met uit te leggen wat je probleem is, wát je niet snapt, wat je geprobeerd hebt, wat je gezocht en gevonden hebt etc. En dat mag dan in een nieuw topic waarin je dan ook meteen netjes code-tags gebruikt.

[ Voor 44% gewijzigd door RobIII op 08-10-2022 17:13 ]