NullpointerException - mal wieder -warum?

[Daishi1002]

Ich hab mal wieder ein kleines Problem.

Ich hab das Problem auf ein einfaches Codestück eingerenzen können. Ich will eine statisches Liste machen, die halt nur einmal erzeugt wird und als Kommunikation zwischen verschiedenen Threads dienen soll. Allerdings gibts Probleme. Zunächst einmal der Code

Meine Statische Liste:

import java.util.*;

class Item{
	int i;
	Item(){i=0;}
}

public class MyList {
	static Vector myList;
	MyList(){
		Vector MyList = new Vector();
	}
	
	public void add(Item item){
		myList.add(item);
	}
}

und eine kleine Klasse, die das ganze startet:

public class Starter {

	public static void main(String[] args) {
		MyList test = new MyList();
		test.add(new Item());
	}
}

Dabei tritt eine NullPointerException auf, wenn "public void add(Item item)" auf die "MyList test" zugreift. Ich verstehe aber nicht, warum. "test" wird ja im Konstruktor definiert, aber in "add()" scheint java das Object nicht mehr zu kennen.

Ich benutze Eclipse 3.0.2 und JRE 1.5 auf Win2000.

Ich hoffe, ich hab das Problem nicht zu umständlich gepostet und mich an alle Forenrichtlinien gehalten

 

[Wildcard]

Vector MyList = new Vector();

damit wird ein Vector MyList erzeugt der nur im Konstruktor existiert. Mit myList hat der überhaupt nichts zu tun.
Das ist sowieso irgendwie Blödsinn, weil jedes Objekt das auf die add Methode zugreifen will eine Instanz von
MyList erzeugen muss, und damit deinen Vector überschreibt. Wenn das überhaupt sinnvoll ist was du da vorhast, solltest du das Singelton-Pattern verwenden.

 

[Daishi1002]

Okay, dachte eigentlich, das oben wäre schon eine "Art" Singleton Pattern.

Ich versuche jetzt fogendes zu implementieren:

public class ClassicSingleton {
   private static ClassicSingleton instance = null;

   protected ClassicSingleton() {
      // Exists only to defeat instantiation.
   }
   public static ClassicSingleton getInstance() {
      if(instance == null) {
         instance = new ClassicSingleton();
      }
      return instance;
   }
}

Wobei dann "instance" mein "MyList" Object ist. Würde das funktionieren? Schauen wir mal

Danke schon mal für den Tip, Wildcard
Gruß
Daishi

 

[mic_checker]

Guck doch auch mal in die FAQ, da steht u.a. was zum Singleton.

 

[Daishi1002]

Ich nochmal. Ich hab gestern den ganzen Tag am Singleton Pattern rumprobiert und viel gelesen.

Eine Sache ist mir jetzt noch unklar. Wenn ich einen "private constructor" benutze, damit die Klasse nicht ausserhalb sich selbst instanziiert wird, wie kann ich dann die Klasse überhaupt erschaffen?

Ich glaube, meine Singleton-Klasse ist soweit richtig aufgebaut. Allerdings wenn ich in meiner Main.Methode mein Singleton erschaffen will, dann kommt die Fehlermeldung (vom Compiler), dass der Constructor nicht sichtbar ist (klar, er ist ja auch private).

public class Starter {

	public static void main(String[] args) {
		Singleton test = new Singleton();
		test.add(new Item());
	}
}

Der Fehler tritt in Zeile 4 auf.

Die Liste, die in meiner Problemstellung global sein soll, kann man jetzt nur per add erreichen, andere (Abfrage)-Möglichkeiten fehlen hier noch. Die Initialisierung des Vectors passiert im privaten Constructor, ist dass richtig?

Meine Singleton Klasse sieht jetzt so aus:

import java.util.*;

public class Singleton
{
  static private Singleton _instance;
  private Vector myList;
  
  //privater constructor
  private Singleton() {
  	Vector myList = new Vector();
  }

  public static Singleton getInstance() {
    if (_instance == null) {
      synchronized(Singleton.class) {
        if (_instance == null)
          _instance = new Singleton();
      }
    }
    return _instance;
  }

  public void add(Item item){
  	 myList.add(item);	
  }
  
}

Vielen Dank schonmal,
Gruß
Daishi1002

 

[Daishi1002]

Bin ein Schritt weiter, allerdings immer noch nicht problemfrei.

Ich weiss jetzt, wie ich auf mein Singleton zugreife (Singleton.java ist immer noch so wie oben):

Singleton.getInstance().add(new Item());

Allerdings bekomme ich jetzt wieder eine NullPointerException (Zeile 24 von Singleton.java)

Wieso? Diesmal hab ich myList doch initialisiert im Konstruktor (Zeile 10)

Vector myList = new Vector();

Wäre klasse, wenn Ihr da nochmal draufschauen könntet.

Aber ne Menge gelernt hab ich bis hier schon - tolles Forum!

Gruß
Daishi1002

 

[bygones]

der gute alte global / lokal Fehler

public class Singleton
{
  static private Singleton _instance;
  private Vector myList;
 
  //privater constructor
  private Singleton() {
     Vector myList = new Vector();
  }

du defnierst einmal myList global, im Konstruktor aber nochmals lokal -> nicht die selben... lass das Vector im Konstruktor weg dann wird der globale angesprochen.

ich persönlich würde die getInstance Methode anders schreiben... zuviel Code

also so:

public class Singleton
{
  private static Singleton _instance = new Singleton();
  private Vector myList;
 
  //privater constructor
  private Singleton() {
     myList = new Vector();
  }

  public static Singleton getInstance() {
    return _instance;
  }

 

[TheSunToucher]

Im Eclipse gibt's unter Window -> Preferences -> Compiler -> Errors/Warnings -> Local variable hides another field or variable die Möglichkeit sich warnen zu lassen, falls eine lokale Variable ein Feld der Klasse verdeckt, also eine Warnung um genau diesen (möglichen) Fehler zu verhindern. (das gilt für 3.1M6 bei 3.0 ist aber auch irgendwo in der Gegend zu finden)

Außerdem solltest du die Methoden in einem Singleton unter umständen synchronisieren, ich habe mal ziemlich lange nach einem Fehler gesucht, weil ich die Methoden nicht synchronisiert hatte: Methoden_synchronisieren_wenn_das_Singleton-Pattern_benutzt_wird


Ist ein bischen Off-Topic, aber ich habe eine Zwischenfrage @deathbyaclown. Du hast die variable "_instance" genannt, hat das einen Grund? Gibt's eine Konvention dafür? Hab das schon mehrfach in Code gesehen, weiß aber nicht was das soll mit dem "_" vorneweg...

static private Singleton _instance;

 

[Bleiglanz]

Ich nochmal. Ich hab gestern den ganzen Tag am Singleton Pattern rumprobiert und viel gelesen.

Offenbar nicht genug: das double checked locking funktioniert in Java nicht, der Code

  public static Singleton getInstance() {
    if (_instance == null) {
      synchronized(Singleton.class) {
        if (_instance == null)
          _instance = new Singleton();
      }
    }
    return _instance;
  }

ist leider kaputt

 

[Daishi1002]

Offenbar nicht genug: das double checked locking funktioniert in Java nicht ...

Das double checked locking kaputt ist, hab ich auch irgendwo gelesen, aber dass es in meinen Code gekommen, war mir nicht bewusst.

Welcher Teil davon ist denn genau das "double checked locking"? Hat jemand einen interessanten Artikel darüber?

Und ich fürchte "_instance" kommt von mir, nicht von deathbyaclown.. Ich hab es von dieser Seite.

Ich sitze weiter dran - und mache Fortschritte! Melde mich noch

Gruß
Daishi1002

 

[Daishi1002]

Problem gelöst!
Wie bekomme ich jetzt den Haken hinter Thread?

Für interessierte, hier nochmal der Code:

import java.util.*;

public class Singleton{
  static private Singleton _instance;
  private Vector lockedPorts = new Vector();
  
  //private constructor
  private Singleton() {}

  static public Singleton getInstance() {
    if (_instance == null) 
      _instance = new Singleton();
    return _instance;
  }

  public boolean lockPort(Port port){ ... }

  
  public boolean unlockPort(Port port){ ...  }
}

und ein Testaufruf

public class Starter {

	public static void main(String[] args) {
		System.out.println("Locke Port 5");
		Singleton.getInstance().lockPort(new Port(5));
		
		System.out.println("Locke Port 5");
		boolean test = Singleton.getInstance().lockPort(new Port(5));
		System.out.println(test);
	}
}

Das "double checked locking" ist wieder raus geflogen, soweit ich das verstanden hab, ist es auch nur eine performance Verbesserung für die getInstance()-Methode.

Vielen Dank allen, die geholfen haben!
Gruß
Daishi1002