C # -Fehler: Verwendung von nicht zugewiesenen lokalen Variablen

Der compiler nicht weiß, dass in der Umgebung.Exit() beendet das Programm; er sieht nur Sie die Ausführung einer statischen Methode auf einer Klasse. Nur initialisieren queue auf null, wenn Sie ihn deklarieren.

Queue queue = null;

InformationsquelleAutor der Antwort tvanfosson

Ein paar verschiedene Möglichkeiten, um das problem zu lösen:

Ersetzen Sie einfach Umgebung.Beenden Sie mit return. Der compiler weiß, dass return beendet die Methode, aber nicht wissen, dass Umwelt.Beenden tut.

static void Main(string[] args) {
    if(args.Length != 0) {
       if(Byte.TryParse(args[0], out maxSize))
         queue = new Queue(){MaxSize = maxSize};
       else
         return;
    } else {
       return;   
}

Natürlich können Sie wirklich nur Weg, denn Sie sind mit " 0 " als Ihre exit-code in allen Fällen. Wirklich, sollten Sie Sie wieder ein int statt mit Umgebung.Exit. Für diesen speziellen Fall, das wäre meine bevorzugte Methode

static int Main(string[] args) {
    if(args.Length != 0) {
       if(Byte.TryParse(args[0], out maxSize))
         queue = new Queue(){MaxSize = maxSize};
       else
         return 1;
    } else {
       return 2;
    }
}

Initialisieren der Warteschlange auf null, das ist wirklich nur ein compiler-trick, der sagt "ich werde herausfinden, meine eigenen, nicht initialisierte Variablen, ich danke Ihnen sehr". Es ist ein nützlicher trick, aber ich weiß nicht wie es in diesem Fall - Sie haben zu viele, wenn die Zweige zu leicht überprüfen, dass Sie tun es richtig. Wenn Sie wirklich wollte, es zu tun auf diese Weise, so etwas klarer wäre:

static void Main(string[] args) {
  Byte maxSize;
  Queue queue = null;

  if(args.Length == 0 || !Byte.TryParse(args[0], out maxSize)) {
     Environment.Exit(0);
  }
  queue = new Queue(){MaxSize = maxSize};

  for(Byte j = 0; j < queue.MaxSize; j++)
    queue.Insert(j);
  for(Byte j = 0; j < queue.MaxSize; j++)
    Console.WriteLine(queue.Remove());
}

Fügen Sie eine return-Anweisung nach der Umgebung.Exit. Wieder, dies ist eher ein compiler-trick - aber etwas mehr legit IMO, denn er fügt Semantik für den Menschen als gut (obwohl es halten Sie aus, dass die berühmt-berüchtigte 100% code coverage)

static void Main(String[] args) {
  if(args.Length != 0) {
     if(Byte.TryParse(args[0], out maxSize)) {
        queue = new Queue(){MaxSize = maxSize};
     } else {
        Environment.Exit(0);
        return;
     }
  } else { 
     Environment.Exit(0);
     return;
  }

  for(Byte j = 0; j < queue.MaxSize; j++)
     queue.Insert(j);
  for(Byte j = 0; j < queue.MaxSize; j++)
     Console.WriteLine(queue.Remove());
}

InformationsquelleAutor der Antwort Mark Brackett