Liste speichern. Was lässt sich verbessern?

[handshake45]

class El {
	String schema;
	JSONObject header;
	JSONObject message;

	El(String schema, JSONObject header, JSONObject message) {
		this.schema = schema;
		this.header = header;
		this.message = message;
	}
}


	private static final LinkedList<El> list = new LinkedList<El>();

	public static void saveList1() {
		try (ObjectOutputStream oos = new ObjectOutputStream(new FileOutputStream("list1data.txt"))) {
			oos.writeInt(list.size());
			for (El el : list) {
				oos.writeObject(new String[] { el.schema, el.header.toString(), el.message.toString() });
			}
		} catch (IOException e) {
			e.printStackTrace();
		}
	}

	public static void loadList1() {
		if (new File("list1data.txt").exists()) {
			try (ObjectInputStream ois = new ObjectInputStream(new FileInputStream("list1data.txt"))) {
				int size = ois.readInt();
				for (int i = 0; i < size; i++) {
					String[] s = (String[]) ois.readObject();
					list.add(new El(s[0], new JSONObject(s[1]), new JSONObject(s[2])));
				}
			} catch (IOException | ClassNotFoundException e) {
				e.printStackTrace();
			}
		}
	}

Ich denke das ist zu kompliziert, übersehe ich etwas?

 

[LimDul]

Warum nicht das ganze direkt als Json speichern anstelle ein Json zu serialiseiren.

 

[Oneixee5]

Beispiel für das Speichern als JSON:

        Artist artist = new Artist();
        artist.setId(1L);
        artist.setName("The Beatles");

        ObjectMapper mapper = new ObjectMapper();

        File file = new File("artist.json");
        try {
            // Serialize Java object info JSON file.
            mapper.writeValue(file, artist);
        } catch (IOException e) {
            e.printStackTrace();
        }

        try {
            // Deserialize JSON file into Java object.
            Artist newArtist = mapper.readValue(file, Artist.class);
            System.out.println("newArtist.getId() = " + newArtist.getId());
            System.out.println("newArtist.getName() = " + newArtist.getName());
        } catch (IOException e) {
            e.printStackTrace();
        }

Maven:

<dependency>
    <groupId>com.fasterxml.jackson.core</groupId>
    <artifactId>jackson-databind</artifactId>
    <version>LATEST</version>
</dependency>

 

[handshake45]

Möchte keine zusätzliche Abhängigkeit reinbringen

 

[LimDul]

Das geht mit Sicherheit auch zu Fuß, wenn du eh JSonObject drin hast. Fände ich immer noch besser als die elendige Serialisierung. Die ist für mich nur ein Notbehelf.

 

[tommysenf]

Möchte keine zusätzliche Abhängigkeit reinbringen

JsonReader (Java(TM) EE 7 Specification APIs)

JsonWriter (Java(TM) EE 7 Specification APIs)

Weiterhin lassen sich bei den Variablen und Klassennamen sicher noch ein paar Zeichen einsparen...

class El {
    String s;
    JSONObject h, m;

    El(String a, JSONObject b, JSONObject c) {
        s=a; h=b; m=c;
    }
}

 

[kneitzel]

Weiterhin lassen sich bei den Variablen und Klassennamen sicher noch ein paar Zeichen einsparen...

YMMD!

Aber die Klasse hatte noch zwei Zeichen ... das muss noch optimiert werden 🙂

 

[handshake45]

Weiterhin lassen sich bei den Variablen und Klassennamen sicher noch ein paar Zeichen einsparen

Das wär doch gegen deine Clean Code-Regeln... Was soll der Sarkasmus?