nybörjare behöver hjälp!!!

Du uppdaterar aldrig celcius-variabeln inuti loopen. Det första du matar in kommer gå in i samma if-sats varje gång.

Ja,precis. Jag blockerar här. Det går inte ange celsius++ för att Celsius är double.

Hur kan jag lösa det? Kan du hjälpa mig?

postitlapp skrev:

Du uppdaterar aldrig celcius-variabeln inuti loopen. Det första du matar in kommer gå in i samma if-sats varje gång.

Ja,precis. Jag blockerar här. Det går inte ange celsius++ för att Celsius är double.

Hur kan jag lösa det? Kan du hjälpa mig?

Celcius++?

Det är väl den här funktionaliteten du vill anropa igen?

 int fahrenheit = int.Parse(Console.ReadLine()); //konvertera grader som användaren angett till heltal
 double celsius = FahrToCels(fahrenheit);//konvertera fahrenheit till celsius

Edit: Problemet är att du anropar bara Console.ReadLine(), men du gör aldrig något med input:en så nästa varv är celcius oförändrad

Tack för svaret! Du är jättesnäll!! Jag förstår problem, men jag vet inte hur jag löser det. Kan du förklara mer?

postitlapp skrev:

Celcius++?

Det är väl den här funktionaliteten du vill anropa igen?

 int fahrenheit = int.Parse(Console.ReadLine()); //konvertera grader som användaren angett till heltal
 double celsius = FahrToCels(fahrenheit);//konvertera fahrenheit till celsius

Edit: Problemet är att du anropar bara Console.ReadLine(), men du gör aldrig något med input:en så nästa varv är celcius oförändrad

Det är som postitlapp skrev, du behöver upprepa den kod som frågar efter temperaturinställning.

Du har på två ställen skrivit:

Console.ReadLine();//användaren anger temperaturen.

Där behöver det istället vara:

fahrenheit = int.Parse(Console.ReadLine());
celsius = FahrToCels(fahrenheit);

Villkoret för att upprepa slingan bör vara:

while ( celsius < 73.0 || Celsius > 77.0 );

Det finns inte heller någon undantagshantering. Använd try och catch.

Det finns inte heller någon undantagshantering. Använd try och catch.

Det finns väl inga fel som kan hanteras i det programmet så varför ska hen fånga dem?

Euclid skrev:

Det finns inte heller någon undantagshantering. Använd try och catch.

Det finns väl inga fel som kan hanteras i det programmet så varför ska hen fånga dem?

int.Parse kan misslyckas. Man ska i alla fall bestämma vad som ska hända om den gör det.

Laguna skrev:

Euclid skrev:

Visa föregående citat

Det finns inte heller någon undantagshantering. Använd try och catch.

Det finns väl inga fel som kan hanteras i det programmet så varför ska hen fånga dem?

int.Parse kan misslyckas. Man ska i alla fall bestämma vad som ska hända om den gör det.

Tja, det där med felhantering är ett allmänt missförstånd bland många programmerare som trycker in try-catch block utan mening. Använda det om felet går att hantera, annars så se till att interna fel inte läcker ut externt från ditt program eller din tjänst. Int.Parse kan gå fel, men man utför inte i princip en funktion på något utan att först ha validerat input.

Jag menar inte att man ska trycka in try och catch överallt. Det är dock en del av just denna uppgift.

Euclid skrev:

Laguna skrev:

Visa föregående citat

Euclid skrev:

Det finns inte heller någon undantagshantering. Använd try och catch.

Det finns väl inga fel som kan hanteras i det programmet så varför ska hen fånga dem?

int.Parse kan misslyckas. Man ska i alla fall bestämma vad som ska hända om den gör det.

Tja, det där med felhantering är ett allmänt missförstånd bland många programmerare som trycker in try-catch block utan mening. Använda det om felet går att hantera, annars så se till att interna fel inte läcker ut externt från ditt program eller din tjänst. Int.Parse kan gå fel, men man utför inte i princip en funktion på något utan att först ha validerat input.

Varför skrev du att det inte finns några fel som behöver hanteras? 

Jag har uppdatera min kod,trycker ni att det finns problem?

class Program
{
public static double FahrToCels(int fahr)
{
double temp = (double)fahr; // tilldela fahr till double
temp = ((fahr - 32.0) * 5.0) / 9.0; // beräkna fahrenheit till celsius
return temp;//returnera värdet
}

static void Main(string[] args)
{
Console.WriteLine("Welcome! Please enter the desired temperature in Fahrenheit! "); //användaren anger grader
int fahrenheit = int.Parse(Console.ReadLine()); //konvertera grader som användaren angett till heltal
double celsius = FahrToCels(fahrenheit);//konvertera fahrenheit till celsius

do
{

if (celsius < 73.0)//villkoret är att temperaturen är lägre än 73
{
Console.WriteLine("Please increase the temperature a bit, it is too cold!");//visa till användaren
}

else if (celsius > 77.0)//villkoret är att temperaturen är högre än 77
{
Console.WriteLine(" Please lower the temperature a bit, it is too warm!");//visa till användaren

}

else
{
Console.WriteLine(" It is suitaible for sauna ! Enjoy! ");//visa till användaren
}

try//kolla att om användaren anger en siffran
{
fahrenheit = int.Parse(Console.ReadLine()); //konvertera grader som användaren angett till heltal
celsius = FahrToCels(fahrenheit);//konvertera fahrenheit till celsius

}

catch//om användaren inte anger siffran då körs innehållet nedan
{
Console.WriteLine("Please enter in a number!");//visa att användaren behöver ange en siffra
Console.ReadLine();//läs in siffran som användaren angett
}

} while (celsius < 73.0 || celsius > 77.0 );//villkoret för att slingan körs

Console.WriteLine(" It is suitaible for sauna ! Enjoy! ");//visa till användaren

Console.ReadKey();

Ser bra ut, du skulle kunna låta inmatningen ske inuti loopen, så slipper du har samma kod på flera olika ställen.

edit: keyword:et `continue` gör så att den hoppar upp till `do` och kör om loopen på nytt

using System;

class Program
{
public static double FahrToCels(int fahr)
{
return ((fahr - 32.0) * 5.0) / 9.0;
}

public static void Main(string[] args)
{
double temp = 0;
int cels = 0;

do
{
try
{
cels = int.Parse(Console.ReadLine());
}
catch (Exception e)
{
Console.WriteLine("invalid number");
continue;
}

temp = FahrToCels(cels);

if (temp < 73)
{
Console.WriteLine("Too cold, heat it up a bit");
} else if (temp > 77)
{
Console.WriteLine("Too warm, only finnish people can sauna this hot");
}

} while (temp < 73 || temp > 77);

Console.WriteLine($"Ahhh, {temp} Celsius, lovely tempature");
}
}

Hej Mali, jag har en samma uppgift som dig, jag förstår ingenting, jag undrar om du kunde hjälpa mig. Det är tre delar i uppgiften