Zugriff Map

[trashken]

Hallo an alle,

Ich habe eine Frage zu Java-Maps, etwas was ich nicht ganz verstehe:


Ich z.B habe eine Klasse mit einer Methode:

public class TestClass{



private void jButtonWriteButtonMouseClicked(java.awt.event.MouseEvent evt) {  


// eine Auswahl wird  aus einer J-List getroffen und dann als String in die AuswahlMap(Get und Set methoden) //geschrieben

AuswahlMap  TestMap = new AuswahlMap(strAuswahl);
AuswahlMap.settest(strAuswahl);


}



}


private void jButtonTestButtonMouseClicked(java.awt.event.MouseEvent evt) {

TestClass2.class.newInstance().machwas(TestMap);

}

Dies Map möchte ich dann in einer anderen Klasse in einer anderen Methode verwenden:

public class TestClass2{


public void machwas(TestMap){


System.out.println("TEST" +AuswahlMap.gettest());


}


}

Ausgelöst werden soll die Methode mit einem J-Button in TestClass1 und dann mittels newInstance() auf die Methode machwas() in TestClass2 zugreifen.Wie kann ich die Map den ordnungsgemäß übergeben?

So bekomme ich eine Null-Pointer-Exception.


Vielen Dank&Beste Grüße,

Trashken

 

[Final_Striker]

So z.B:

public class TestClass{

   AuswahlMap  testMap;

   private void jButtonWriteButtonMouseClicked(java.awt.event.MouseEvent evt) {  

      // entweder so
      testMap = new AuswahlMap(strAuswahl);

      //oder so
      testMap = new AuswahlMap();
      testMap.setTest(strAuswahl);

      // das hier macht keinen Sinn
      AuswahlMap  TestMap = new AuswahlMap(strAuswahl);
      AuswahlMap.settest(strAuswahl);
   }

   private void jButtonTestButtonMouseClicked(java.awt.event.MouseEvent evt) {
 
      TestClass2.class.newInstance().machwas(testMap);
   }
}

 

[Noctarius]

TestClass2.class.newInstance().machwas(testMap);

*schüttel*

 

[trashken]

Es funktioniert...


Wieso *schüttel*?

 

[maki]

Weil das wahrscheinlich die umständlichste und verschleierste Form (neben Reflection) für
[c]new TestClass2().machwas(testMap);[/c]
ist.

 

[Noctarius]

Weil man so (außer in Ausnahmefällen) nicht instanzieren sollte. Wenn du eine Instanz brauchst mach es richtig:

new TestClass2().machwas(...);

Abgesehen davon müsste dein newInstance Exceptions werfen und die fängst du nicht und schmeisst sie auch nicht weiter.

private void jButtonTestButtonMouseClicked(java.awt.event.MouseEvent evt) {
 
      TestClass2.class.newInstance().machwas(testMap);
   }

 

[trashken]

Hallo,

Ich habe hier mal ein weiter probiert und....folgendes Problem:

Wenn ich es so versuche:

package test;

public class TestClass{
 
   Map<String, AuswahlMap> Workmap;
   AuswahlMap  testMap;
    
   private void jButtonWriteButtonMouseClicked(java.awt.event.MouseEvent evt) {  
 
      WorkMap = new HashMap<String,AuswahlMap();
      testMap = new AuswahlMap();
      testMap.setTest(strAuswahl);
      
      
   }
private void jButtonTestButtonMouseClicked(java.awt.event.MouseEvent evt) {

new TestClass2().machwas(Workmap,testMap); 
 
}

Bekomme ich in TestClass2 folgende Meldung:

incompatible types : required java.util.Map.Entry<java.lang.String,test.Auswahlmap> found: java.lang.Object

package test;

public class TestClass2{
 

public void machwas(Map Workmap,Auswahlmap TestMap){
 
for(Map.Entry<String,AuswahlMap> e : WorkMap.entrySet()){
              Testmap  = e.getValue();

// usw
 
 
}
 
 
}

Hm...Kann mir da jemand weiterhelfen?

Beste Grüße,

Trashken

 

[xehpuk]

Bei dem Code kriegt man ja Augenkrebs. :shock:
Java naming conventions

Weißt du, was Generics sind? Du benutzt einen in Zeile 11 (wenn auch syntaktisch falsch; meckert der Compiler da nicht?).

Schon das probiert?
[JAVA=7]public void machwas(Map<String, AuswahlMap> Workmap,Auswahlmap TestMap){[/code]

 

[Landei]

Bei [c]public void machwas(Map Workmap,Auswahlmap TestMap){[/c] hast du in AuswahlMap das m klein geschrieben.

Und wie mein Vorredner schon bemerkt hat, solltest du für Map Generics benutzen. Variablen und Argumente schreibt man übrigens klein. Ehrlich gesagt finde ich deine Bezeichner insgesamt etwas konfus.