Spara nytt värde efter felaktigt värde inmatat

Console.ReadLine() returnerar en sträng som du inte gör något med.

I förra frågan visade du koden och den är känslig för felinmatningar, t ex om något annat än ett tal returneras. Du skapar Stad-objekten och läser temperatur i samma uttryck.

En separat metod för att lösa in temperatur som inte returnerar förrän du fått in ett godkänt värde blir enklare.
Nåt i stil med:

int readTemperature(String text) {
  while (true) {  // loop until valid temperature has been read
    // print text
    // read temperature
    // check temperature
    // if correct range, return temperature
    // print error message
  }
}

Programmeraren skrev:

Console.ReadLine() returnerar en sträng som du inte gör något med.

I förra frågan visade du koden och den är känslig för felinmatningar, t ex om något annat än ett tal returneras. Du skapar Stad-objekten och läser temperatur i samma uttryck.

En separat metod för att lösa in temperatur som inte returnerar förrän du fått in ett godkänt värde blir enklare.
Nåt i stil med:

int readTemperature(String text) {
  while (true) {  // loop until valid temperature has been read
    // print text
    // read temperature
    // check temperature
    // if correct range, return temperature
    // print error message
  }
}

Som jag skrev så har jag provat att skriva Stad stad = new Stad("", int.Parse(Console.ReadLine()));  där det står Console.ReadLine();
Hoppades att den skulle lägga in ny värdet från användaren och spara det. Men icke!
Så klirar mig ännu i huvudet om hur jag kan göra så att det nya värdet sparas i stad[] städer och printas ut.

Du gör kontrollen i constructorn, där kan du läsa in värde tills det är korrekt enligt mönstret jag skrev i förra inlägget. I constructorn ska du inte göra "new Stad", den koden körs just när någon utanför gör "new Stad".

Det är inte riktigt snyggt att läsa inmatning i en constructor men det är nog inte det viktigaste just nu. Som koden är nu så läser du in temperatur på två ställen, både i koden som gör "new Stad" och inuti constructorn. Bäst vore att göra det innan du skapar Stad-objektet men för att fortsätta på din kod bör du göra en constructor som bara tar stadNamn och sen frågar efter temperatur i en while-loop som jag beskrev ovan. I constructorn blir det dock "break" istf return.

Spoiler, skippa om du vill prova själv. Ej kompilerat så kan ha fel:

public Stad(string stadNamn) {
  this.stadNamn = stadNamn;
  while (true) {
    Console.Write("Ange temperatur för " + stadNamn + ": ");
    this.stadTemp = int.Parse(Console.ReadLine());
    if (this.stadTemp >= -60 && this.stadTemp <= 60)  {
      break;
    } else {
      Console.WriteLine("Felaktig inmatning! Ange temperatur mellan -60 till +60!");
    }
  }
}

Programmeraren skrev:

Du gör kontrollen i constructorn, där kan du läsa in värde tills det är korrekt enligt mönstret jag skrev i förra inlägget. I constructorn ska du inte göra "new Stad", den koden körs just när någon utanför gör "new Stad".

Det är inte riktigt snyggt att läsa inmatning i en constructor men det är nog inte det viktigaste just nu. Som koden är nu så läser du in temperatur på två ställen, både i koden som gör "new Stad" och inuti constructorn. Bäst vore att göra det innan du skapar Stad-objektet men för att fortsätta på din kod bör du göra en constructor som bara tar stadNamn och sen frågar efter temperatur i en while-loop som jag beskrev ovan. I constructorn blir det dock "break" istf return.

Spoiler, skippa om du vill prova själv. Ej kompilerat så kan ha fel:

public Stad(string stadNamn) {
  this.stadNamn = stadNamn;
  while (true) {
    Console.Write("Ange temperatur för " + stadNamn + ": ");
    this.stadTemp = int.Parse(Console.ReadLine());
    if (this.stadTemp >= -60 && this.stadTemp <= 60)  {
      break;
    } else {
      Console.WriteLine("Felaktig inmatning! Ange temperatur mellan -60 till +60!");
    }
  }
}

this.stadTemp = int.Parse(Console.ReadLine()); löste problemet med att inte nya värdet lagrades.

Tack för feedback och hjälpande tankar! =) 

MvH

Oskar