would anyone help?

karli hujoova

would anyone help me with my c# program if i would post it here?
the problem is that my c# calculation answer must say is it kevad,suvi,sügis or talv in my program,the program is here

int vastus1, vastus2, vastus3;

{

Console.WriteLine("Sisesta esimene arv vahemikus 10-20");

vastus1 = int.Parse(Console.ReadLine());

Console.WriteLine("sisesta teine arv vahemikus 20-32");

vastus2 = int.Parse(Console.ReadLine());

Console.WriteLine("Vastus {0}", vastus2 - vastus1);

string tekst1 = Console.ReadLine();

vastus3 = int.Parse(tekst1);

}

while ((vastus1 < 1 || vastus2 < 12));

if (vastus3 <= 3 && vastus3 <= 5)

{

Console.WriteLine("On kevad");

{

if (vastus3 <= 6 && vastus3 <= 8)

{

Console.WriteLine("on suvi");

}

if (vastus3 <= 9 && vastus3 <= 11)

{

Console.WriteLine("on sügis");

}

if (vastus3 <= 12)

if (vastus3 <= 1 && vastus3 <= 2)

{

Console.WriteLine("on talv");

}

4 Replies

Know the answer? Post it — somebody with the same question will find it here.

Vulpes13y ago
I'm still not 100% clear so I'm guessing a bit what's wanted but try this:

int vastus1, vastus2, vastus3;
do
{
do
{
Console.WriteLine("Sisesta esimene arv vahemikus 10-20");
vastus1 = int.Parse(Console.ReadLine());
}
while (vastus1 < 10 || vastus1 > 20);

do
{
Console.WriteLine("sisesta teine arv vahemikus 20-32");
vastus2 = int.Parse(Console.ReadLine());
}
while (vastus2 < 20 || vastus2 > 32);

vastus3 = vastus2 - vastus1;
}
while (vastus3 < 1 || vastus3 > 12);

Console.WriteLine("Vastus {0}", vastus3);

if (vastus3 <= 3)
{
Console.WriteLine("On kevad");
}
else if (vastus3 <= 6)
{
Console.WriteLine("on suvi");
}
else if (vastus3 <= 9)
{
Console.WriteLine("on sügis");
}
else
{
Console.WriteLine("on talv");
}

Note that you can check for invalid input (i.e. abc instead of a number) using int.TryParse rather than int.Parse but, as you may not have come across this, I've left it as it is.
karli hujoova13y ago
Ye sorry about that was so tired and posted before going to bed heres the place where debugger says the problem is http://puu.sh/2h4r6
and the object of program is that insert 1 number between 10-20 and 2nd number between 20-32 repeat calculation till answer is between 1-12 and from that calculation it must say is it kevad,talv,suvi or sügis hopes thats little more logical
Vulpes13y ago
One thing which is clear is that, if vastus1 needs to be between 10 and 20 and vastus2 needs to be between 20 and 32, then the condition in your do/while statement is incorrect.

Try instead:

do
{
Console.WriteLine("Sisesta esimene arv vahemikus 10-20");
vastus1 = int.Parse(Console.ReadLine());
Console.WriteLine("sisesta teine arv vahemikus 20-32");
vastus2 = int.Parse(Console.ReadLine());
Console.WriteLine("Vastus {0}", vastus2 - vastus1);

string tekst1 = Console.ReadLine();
vastus3 = int.Parse(tekst1);
}
while (vastus1 < 10 || vastus1 > 20 || vastus2 < 20 || vastus2 > 32);

vastus3 must then be somewhere between 0 and 22.

It's not clear what values in this range correspond to kevad, suvi, sügis or talv but it is clear that your conditions aren't quite right:

For example, consider this one:

if (vastus3 <= 3 && vastus3 <= 5)

This is equivalent to simply:

if (vastus3 <= 3)

because if a number is less than or equal to 3 then it's certainly less than 5.
Jignesh Trivedi13y ago
hi,

can you please share actual logic or requirement so that we can double check you implimentation?