Map mit Vektor: Element hinzufügen

[Migu]

Hallo zusammen,

ich habe eine Frage bzgl Maps in Java (bin sonst eher an Maps in C++ gewöhnt)
Warum klappt folgendes nicht?

Map< String,Vector<Integer> > myMap = new HashMap<>();
for (int k=0; k<5; k++){
    myMap.get("1").addElement(k);
}

Ich will eigentlich nur dass je nachdem wie oft die Schleife durchgelaufen wird, auch der Vektor demetsprechen mit Daten gefüttert wird....

Als Fehlermeldung bekomme ich

Exception in thread "main" java.lang.NullPointerException
	at Test.main(Test.java:47)
Java Result: 1

Gruß
Migu

 

[Fab1]

myMap.get("1")

Du bekommst ja eine NullPoiner Exception, somit gibt der Aufruf nichts zurück.

 

[Camill]

Wie Fab1 bereits schrieb tritt eine NullPointerException auf da sich in der HashMap noch kein Element(Vector) mit dem angegebenen Key "1" befindet. Folgendes wäre jedoch möglich:

Map<String, Vector<Integer>> map = new HashMap<>();
map.put("aKey", new Vector<Integer>());

for (int i = 0; i < 5; i++) {
	map.get("aKey").add(i);
}

 

[fastjack]

Schreibe Dir eine Methode zum putten:

void put(String key, int value) {
    Vector<Integer> vector = myMap.get(key);
    if(vector == null) {
        vector = new Vector<Integer>();
        myMap.put(key, vector);
    }
    vector.addElement(value);
}

vorausgesetzt Du hat Zugriff auf myMap, ansonsten myMap als Parameter, vielleicht dann static method? Warum Vector und nicht List?

 

[Helgon]

Map< String,Vector<Integer> > myMap = new HashMap<>();

klappt nicht, weil es lauten müsste

Map<String, Vector<Integer>> myMap = new HashMap<String, Vector<Integer>>();

 

[Camill]

Map< String,Vector<Integer> > myMap = new HashMap<>();

klappt nicht, weil es lauten müsste

Map<String, Vector<Integer>> myMap = new HashMap<String, Vector<Integer>>();

Map< String,Vector<Integer> > myMap = new HashMap<>();

Reicht imho aus.

 

[Migu]

Schreibe Dir eine Methode zum putten:

void put(String key, int value) {
    Vector<Integer> vector = myMap.get(key);
    if(vector == null) {
        vector = new Vector<Integer>();
        myMap.put(key, vector);
    }
    vector.addElement(value);
}

vorausgesetzt Du hat Zugriff auf myMap, ansonsten myMap als Parameter, vielleicht dann static method? Warum Vector und nicht List?

Ich weiss nicht genau wie ich das auf mein Problem anpassen soll....
Am besten ist es ich poste mal mein ganzer Code. Also ich will eine txt Datei einlesen und die einzelnen Spalten als doubles in verschiedenen Vektoren/Lists speichern. Dazu dachte ich wäre ein Map gut weil über die Schlüssel kann ich dann sozusagen auf die Spalten zugreifen. Das ganze soll auch automatisiert passieren.

Mein Code:

[JAVA=42]
import java.io.File;
import java.io.FileNotFoundException;
import java.util.HashMap;
import java.util.Map;
import java.util.Scanner;
import java.util.Vector;


public class funcs {

public Map<String,Vector<Double>> ReadFile(String fileName, int numberOfColumns, int headerRows){
Map<String,Vector<Double>> outMap = new HashMap();

File file = new File(fileName);
try {
Scanner scanner = new Scanner(file).useDelimiter("\\s+");
for (int i=0; i<headerRows; i++) {scanner.nextLine();}

String key = "";
while ( scanner.hasNext() ){
for (int i=0; i<numberOfColumns; i++){
key = Int2String(i);
outMap.get(key).addElement(Double.parseDouble(scanner.next()));
}
}

} catch (FileNotFoundException ex) {
ex.printStackTrace();
}

return outMap;
}


String Int2String (int number) {
String outString = "";
Integer myInt = new Integer(number);
outString = myInt.toString();
return outString;
}

}
[/code]


Übrigens klappt das mit obigen Vorschlägen auch nicht.

Gruß
Migu

 

[xehpuk]

Einen int wandelt man übrigens über

String.valueOf(int)

in einen String um. Allgemein brauchst du das nicht, da man Integer als Key verwenden kann. Das wiederum brauchst du hier gar nicht, da es hier schon eine Liste von Double-Arrays tut.

Probier es mal hiermit (ist jedoch ungetestet):

public List<double[]> readFile(final String fileName, final int numberOfColumns, final int headerRows) {
	final List<double[]> arrays = new ArrayList<double[]>();
	final File file = new File(fileName);
	try {
		final Scanner scanner = new Scanner(file).useDelimiter("\\s+");
		for (int i = 0; i < headerRows; i++) {
			scanner.nextLine();
		}
		while (scanner.hasNextLine()) {
			final double[] doubles = new double[numberOfColumns];
			for (int i = 0; i < numberOfColumns; i++) {
				doubles[i] = scanner.nextDouble();
			}
			arrays.add(doubles);
		}
	} catch (final FileNotFoundException ex) {
		ex.printStackTrace();
	}
	return arrays;
}

 

[Migu]

Einen int wandelt man übrigens über

String.valueOf(int)

in einen String um. Allgemein brauchst du das nicht, da man Integer als Key verwenden kann. Das wiederum brauchst du hier gar nicht, da es hier schon eine Liste von Double-Arrays tut.

Probier es mal hiermit (ist jedoch ungetestet):

public List<double[]> readFile(final String fileName, final int numberOfColumns, final int headerRows) {
	final List<double[]> arrays = new ArrayList<double[]>();
	final File file = new File(fileName);
	try {
		final Scanner scanner = new Scanner(file).useDelimiter("\\s+");
		for (int i = 0; i < headerRows; i++) {
			scanner.nextLine();
		}
		while (scanner.hasNextLine()) {
			final double[] doubles = new double[numberOfColumns];
			for (int i = 0; i < numberOfColumns; i++) {
				doubles[i] = scanner.nextDouble();
			}
			arrays.add(doubles);
		}
	} catch (final FileNotFoundException ex) {
		ex.printStackTrace();
	}
	return arrays;
}

Das funktioniert wenn man Zeile 12 umwandelt in
[JAVA=12]
doubles = Double.parseDouble(scanner.next());
[/code]

Mit funktionieren meine ich dass die datei richtig eingelesen wird.
Aber das Format ist irgendwie falsch... oder ich kann noch nicht mit eine List umgehen...
Wenn ich folgendes in der main-Funktion eingebe

funcs f = new funcs();
List<double[]> list = new ArrayList<>();
list = f.ReadFile("input.txt",2,2);

for (double d : liste.get(1)){
     System.out.println(d)
}

dann stehen da untereinander die Einträge der zweiten Zeile in der Datei.
Ich will aber auf die einzelnen Spalten zugreifen, nicht Zeilen!

 

[Camill]

Wie genau sieht denn so eine Txt-Datei aus die ausgelsen werden soll (ein Beispiel wäre gut)?

 

[Migu]

Wie genau sieht denn so eine Txt-Datei aus die ausgelsen werden soll (ein Beispiel wäre gut)?

Es ist natürlich nur eine Beispieldatei. In wirklichkeit habe ich dann ein paar tausend Zeilen mit zB 10 Spalten.
So sieht die Inputdatei aus

das ist ein header
######################
1 2.56
2 10.52
3 4.58
4 1.53
5 0.745
6 0.85
7 9.456
8 24.5
9 1.12
10 4.23

Als Ergeniss bekomme ich anstatt der zweiten Spalte folgendes

2.0
10.52

 

[Camill]

Und nun soll die erste Spalte

1
2
3
4
5
6
7
8
9
10

über den Key "1" zu erreichen sein und die zweite Spalte

2.56
10.52
4.58
1.53
0.745
0.85
9.456
24.5
1.12
4.23

über den Key "2"?

 

[Migu]

Und nun soll die erste Spalte

1
2
3
4
5
6
7
8
9
10

über den Key "1" zu erreichen sein und die zweite Spalte

2.56
10.52
4.58
1.53
0.745
0.85
9.456
24.5
1.12
4.23

über den Key "2"?

Ja genau das!
Und es soll mir automatisch ein Objekt (List/Map) erstellen mit allen Spalten, je nachdem wieviele Spalten die Datei hat. Das übergibt man ja an der Funktion.

 

[Camill]

Alles klar, werde dann mal was kleines schreiben.

 

[Migu]

Danke, das wäre sehr nett!

 

[Camill]

So, habe nun eine fertige Lösung erstellt:

public Map<Integer, Vector<Double>> readFile(final String fileName)
		throws NumberFormatException, IOException {

	Map<Integer, Vector<Double>> map = new HashMap<Integer, Vector<Double>>();
	BufferedReader reader = new BufferedReader(new FileReader(fileName));

	String line = null;
	while ((line = reader.readLine()) != null) {

		String[] temp = line.split(" ");
		for (int i = 0; i < temp.length; i++) {
			double d = Double.parseDouble(temp[i]);

			// put new Key with empty Vector into the map if Key not exist
			// already
			if (!map.containsKey(i)) {
				map.put(i, new Vector<Double>());
			}
			map.get(i).add(d);
		}
	}
	return map;
}

Exceptions wurden nicht berücksichtigt!
Der Header wird nicht mit einbezogen, was soll mit diesem passieren?

 

[Migu]

Ja! Jetzt klappt es, nach ein paar Änderugen.
Was Probleme verursacht hat war das in der "split" Funktion das Trennzeichen kein einfaches Leerzeichen sein darf, weil die Anzahl an leerzeichen zwischen den Spalten kann variieren.

So sieht es jetzt aus (müsste eigentlich auch genauso mit einem Scanner funktionieren)

    public Map<Integer, Vector<Double>> ReadFile(final String fileName, int headerRows){
 
        Map<Integer, Vector<Double>> map = new HashMap<>();
        try{
            BufferedReader reader = new BufferedReader(new FileReader(fileName));
           
            // Overread header
            for (int i=0; i<headerRows; i++) {reader.readLine();}        
           
            // Read file content
            String line = null;
            while ( reader.ready() == true ) {
                line = reader.readLine();
                String[] temp = line.split("\\s+");
                for (int i = 0; i < temp.length; i++) {
                    double d = Double.parseDouble(temp[i]);
                   
                    // put new Key with empty Vector into the map if Key not exist
                    // already
                    if ( map.containsKey(i) == false ) 
                    {
                        map.put(i, new Vector<Double>());
                    }
                    map.get(i).add(d);
                }
            }
            reader.close();
           
        } catch (NumberFormatException e){
            System.err.println("NumberFormatException:  " + e.getMessage());
        } catch (IOException io){
            System.err.println("IOException:  " + io.getMessage());
        }
        return map;
    }

Danke an Camill und an allen anderen!

Gruß
Migu

 

[Camill]

Gut, zum entfernen von Whitespace vor und hinter einem String kann auch

trim()

verwendet werden.

 

[bERt0r]

Mit funktionieren meine ich dass die datei richtig eingelesen wird.
Aber das Format ist irgendwie falsch... oder ich kann noch nicht mit eine List umgehen...
Wenn ich folgendes in der main-Funktion eingebe

funcs f = new funcs();
List<double[]> list = new ArrayList<>();
list = f.ReadFile("input.txt",2,2);

for (double d : liste.get(1)){
     System.out.println(d)
}

dann stehen da untereinander die Einträge der zweiten Zeile in der Datei.
Ich will aber auf die einzelnen Spalten zugreifen, nicht Zeilen!

Schau dir den Code doch mal an, du gibst ja nur liste.get(1) aus. So iteriert man über eine ArrayList:

for(double[] array : liste)
{
System.out.println(Arrays.toString(array));
}

 

[Migu]

Ah ok, danke.
Wie gesagt, bin Anfänger/Umsteiger..... muss noch etwas Praxiserfahrung sammeln.

Aber eine Frage hätte ich doch noch:
Der Code oben liest ja ein Datei und speichert die Werte der Spalten in Vektoren als doubles.
Was ist aber wenn eine Spalte das Datum ist (zB 2011-10-01). Das kann man ja nicht als double einlesen.
Man kann aber Jahr, Monat und Tag als Integer/doubles einlesen wenn man das Trennzeichen auf - setzt

Meine Frage ist nun ob man in den Regular Expressions in

[JAVA=57]
Scanner scanner = new Scanner(file).useDelimiter("\\s+");
[/code]

sagen kann dass die ersten 2 "Delimiter" eben ein - und kein Leerzeichen sein sollen???

 

[Camill]

Eine Möglichkeit wäre mithilfe von

split("-")

String der bestimmten Spalte nochmals zu teilen.

 

[bERt0r]

Es gibt in Java die Klasse Date, um ein Date aus einem String zu gewinnen gibts auch schon vorgefertigtes: Parsing a Date Using a Custom Format | Example Depot