java.lang.NullPointerException beim objekt

[Guest]

Hallo, ich habe fogendes Problem. Undzwar bekomme ich das array in meinem Objekt nicht deklariert. /*1*/
funktioniert wie es soll. Aber beim initzialisieren von /*2*/ bekomme ich ein java.lang.NullPointerException.

Vieleicht könnt ihr mir helfen und einen schupser in die richtige richtung geben....

Danke, Sebastian

public class DataReader {

    class ArchiveFileType{
    	int id;   
    	String[] _dateTime; 
    	int[] val;
    	int index; 
    }

	public  ArchiveFileType[] readArchivFile(String[] pParam) throws IOException{
		
		int count;
		
		count = (int)pParam.length;

		ArchiveFileType[] data = new ArchiveFileType[count];                  /*1*/
		for (int i=0; i<count; i++){
			data[i].index = 0;                                                             /*2*/
			data[i]._dateTime = new String[this.timeCount];
			data[i].val = new int[this.timeCount];
		}

		BufferedReader f;
		String line;

		try{
			f = new BufferedReader(new FileReader(filename));
			
			while ((line = f.readLine()) != null){
				String[] row = line.split(";");
		
					//@todo ... und weiter hier
				
			}
			f.close();
		}
		catch (IOException exc){
			throw exc;
		}
		
		return data;
	}

 

[SnooP]

Problem ist, dass du zwar von ArchiveFileType nen Array erstellt und initialisiert hast - die einzelnen Elemente dieses Arrays sind aber auch ArchiveFileType-Objekte und benötigen ne entsprechende Instanzierung! ... also new bla();... am besten halt in der for-Schleife... ansonsten kannst du nicht auf ein Feld, wie .index zugreifen...

was by the way auch schlechter stil ist, da dann index offensichtlich public ist - mach private Felder und entsprechende Setter-Methoden dazu.

 

[karahead]

Du initialisierst zwar das Array:

ArchiveFileType[] data = new ArchiveFileType[count];

Damit bekommst du ein ArchieveFileType-Array mit count Objekten, jedoch ist jedes dieser count Objekte selbst nicht initialisiert.

ArchiveFileType[] data = new ArchiveFileType[count];                  /*1*/ 
      for (int i=0; i<count; i++){ 
         data[i] = new ArchiveFileType(); //Das fehlt dir noch
         data[i].index = 0;                                                             /*2*/ 
         data[i]._dateTime = new String[this.timeCount]; 
         data[i].val = new int[this.timeCount]; 
      }

--- Edit
Tja zu spät

 

[Sibob]

Stimmt.... daran hab ich nicht gedacht :-/




Danke schön,
Sebastian