Verwenden Sie "versuchen und fangen" mit dem Erstellen eines neuen Objekts

Question

Also im Grunde habe ich eine Klasse mit einem Konstruktor, insbesondere sagen, ein Konstruktor für Staaten. Ich kann nur einen neuen Status erstellen, wenn ich im Konstruktor eine Stadttypregion habe (State hat 2 Konstruktoren, beide akzeptieren Name und Region).

Der Code sieht wie folgt aus:

public State(String Name, int Population, Region Capital) throws IllegalArgumentException { 
    super(Name, Population); 
    if(!(Capital.getRegionType() == RegionType.CITY)){ 
     System.out.println("ERROR"); 
     throw new java.lang.IllegalArgumentException(); 
    } 
    this.Capital = Capital; 
} 

Der Typ der Region definiert ist, eine Enum-Klasse. Das Problem ist, wenn ich eine neue Funktion in der Testklasse (Hauptklasse) erstellen

 Region3 = new Region("BedRock", 23423,RegionType.VILLAGE); // 
    try{ 
     State s2 = new State("Khemed", Region3); // This should not be possible because Region3 is a VILLAGE instead of a CITY 
    }catch(IllegalArgumentException e){ 
    } 
    s2.addRegion(Region2);//Doesn´t work - Error 
    s2.addRegion(Region1);//Doesn´t work - Error 

Das ist im Grunde meines Problem, die zwei letzten Zeilen gibt mir eine Fehlermeldung, dass ich habe die Variable s2 intialized.

Ich habe versucht, den Code ohne "versuchen und fangen" zu sehen, ob die if-Anweisung, die ich im Konstruktor habe funktionieren würde, aber es nicht funktioniert.

Die Methode "add" ist eine Methode, die in der Super-State-Klasse (Contry) definiert ist, die es einfach erlaubt, States hinzuzufügen.

Hoffe, du kannst helfen, weil ich nicht sehen kann, wie ich das lösen kann.

ADDED (Angeforderte meine Frage zu beantworten)

Klasse Region:

public class Region { 
private String name; 
private int population; 
private RegionType regionType; 

public Region(String name, int population, RegionType regionType){ 
    this.name = name; 
    this.population = population; 
    this.regionType = regionType; 
} 

//get Region Type 
public RegionType getRegionType(){ 
    return regionType; 
} 

Enum mit Region Art:

public enum RegionType { 
CITY, 
VILLAGE, 
TOWN, 

}

Answer 1

Im zweiten Codeblock , s2 hat nur scop e innerhalb des try Blocks. Erklären Sie stattdessen s2 außerhalb der try.

Bearbeiten: Das ist, was ich für das Posting zu schnell bekommen. s2 Anweisungen sollten in der try sein.

Region3 = new Region("BedRock", 23423,RegionType.VILLAGE); 
State s2 = null; 
try{ 
    s2 = new State("Khemed", Region3); 
    s2.addRegion(Region2); 
    s2.addRegion(Region1); 
}catch(EmptyStackException e){ //You should log this or something 
} 

Answer 2

Ihre Variable s2 wird nur in Ihrem try/catch Block definiert ist, versuchen Sie, die zwei Linien im try/catch Block zu setzen.

Aber der beste Ansatz hier ist es, die Ausnahme IllegalArgumentException nicht zu fangen.

Answer 3

Region3 = new Region("BedRock", 23423,RegionType.VILLAGE); // 
try{ 
    State s2 = new State("Khemed", Region3); 
    s2.addRegion(Region2); 
    s2.addRegion(Region1); 
}catch(IllegalArgumentException e){ 
} 

Zuerst müssen Sie 2 Zeilen eingeben, um Block zu versuchen/zu fangen. Und Ihr "State" -Konstruktor hat 3 Parameter. Aber Sie verwenden einen anderen Konstruktor (2 params):

State s2 = new State("Khemed", Region3); // 2 params 
public State(String Name, int Population, Region Capital)// 3 params 

Verwendung mit Bevölkerung param wie dies in Try-Block:

Region3 = new Region("BedRock", 23423,RegionType.VILLAGE); // 
try{ 
    State s2 = new State("Khemed", 123,Region3); 
    s2.addRegion(Region2); 
    s2.addRegion(Region1); 
}catch(IllegalArgumentException e){ 
}