Solucionado Problema con XML DOM

[franjocc]

Hola.

Estoy haciendo una práctica con Android.
Ahora con tema de XML con DOM.

Es algo sencillo, es leer y grabar en un xml las putuaciones que se obtienen.

<?xml version="1.0" encoding="UTF-8"?>
<lista_puntuaciones>
	<puntuacion fecha="1288122023410">
		<nombre>Mi nombre</nombre>
		<puntos>45000</puntos>
	</puntuacion>
</lista_puntuaciones>

El leer el xml y grabar la primera vez funciona bien, el problema es la segunda vez que se graba, se añade otro elemento.

El resultado es algo así, con el consiguiente error al leer.

<lista_puntuaciones>
	<puntuacion fecha="1288122023410">
		<nombre>Mi nombre</nombre>
		<puntos>45000</puntos>
	</puntuacion>
</lista_puntuaciones>
<lista_puntuaciones>
	<puntuacion fecha="1288122023410">
		<nombre>Mi nombre</nombre>
		<puntos>45000</puntos>
	</puntuacion>
	<puntuacion fecha="128812202381320">
		<nombre>Otro nombre</nombre>
		<puntos>31000</puntos>
	</puntuacion>
</lista_puntuaciones>

El código de la clase que realiza esto es el siguiente:

package org.example.asteroides;

import java.io.File;
import java.io.FileInputStream;
import java.io.FileNotFoundException;
import java.io.FileOutputStream;
import java.io.InputStream;
import java.io.OutputStream;
import java.util.Vector;

import javax.xml.parsers.DocumentBuilder;
import javax.xml.parsers.DocumentBuilderFactory;
import javax.xml.transform.OutputKeys;
import javax.xml.transform.Transformer;
import javax.xml.transform.TransformerFactory;
import javax.xml.transform.dom.DOMSource;
import javax.xml.transform.stream.StreamResult;

import org.w3c.dom.Document;
import org.w3c.dom.Element;
import org.w3c.dom.Node;
import org.w3c.dom.NodeList;
import org.w3c.dom.Text;

import android.annotation.TargetApi;
import android.content.Context;
import android.os.Build;
import android.os.Environment;
import android.util.Log;
import android.widget.Toast;

@TargetApi(Build.VERSION_CODES.FROYO)
public class AlmacenPuntuacionesXML_DOM implements AlmacenPuntuaciones {

	private static String FICHERO = Environment.getExternalStorageDirectory()
			+ "/puntuaciones_dom.xml";

	private Context contexto;
	private Document documento;
	private boolean cargadoDocumento;

	public AlmacenPuntuacionesXML_DOM(Context contexto) {
		this.contexto = contexto;
		cargadoDocumento = false;

		if (ExisteDirectorioAsteroides()) {
			Log.e("Asteroides", "EXISTEDIRECTORIO Existe carpeta Asteroides");
			Log.e("Asteroides", "EXISTEDIRECTORIO " + FICHERO);
			Log.e("Asteroides",
					"EXISTEDIRECTORIO" + "IndexOf "
							+ FICHERO.indexOf("/Asteroides/"));
			if (FICHERO.indexOf("/Asteroides/") == -1) {
				FICHERO = Environment.getExternalStorageDirectory()
						+ "/Asteroides/puntuaciones_dom.xml";
			}
		} else {
			Log.e("EXISTEDIRECTORIO", "No Existe carpeta Asteroides");
		}
	}

	@Override
	public void guardarPuntuacion(int puntos, String nombre, long fecha) {
		try {
			if (!cargadoDocumento) {

				// leerXML(contexto.openFileInput(FICHERO));
				leerXML(new FileInputStream(FICHERO));
			}
		} catch (FileNotFoundException e) {
			crearXML();
		} catch (Exception e) {
			Log.e("Asteroides", e.getMessage(), e);
		}
		nuevo(puntos, nombre, fecha);
		try {
			// escribirXML(contexto.openFileOutput(FICHERO,
			// Context.MODE_PRIVATE));
			escribirXML(new FileOutputStream(FICHERO, true));
		} catch (Exception e) {
			Log.e("Asteroides", e.getMessage(), e);
		}
	}

	// Nuevo elemento de puntuacion
	public void nuevo(int puntos, String nombre, long fecha) {
		if (cargadoDocumento) {
			Element puntuacion = documento.createElement("puntuacion");
			puntuacion.setAttribute("fecha", String.valueOf(fecha));
			Element e_nombre = documento.createElement("nombre");
			Text texto = documento.createTextNode(nombre);
			e_nombre.appendChild(texto);
			puntuacion.appendChild(e_nombre);
			Element e_puntos = documento.createElement("puntos");
			texto = documento.createTextNode(String.valueOf(puntos));
			e_puntos.appendChild(texto);
			puntuacion.appendChild(e_puntos);
			Element raiz = documento.getDocumentElement();
			raiz.appendChild(puntuacion);
		}
	}

	public Vector<String> aVectorString() {
		Vector<String> result = new Vector<String>();
		String nombre = "", puntos = "";

		if (cargadoDocumento) {
			Element raiz = documento.getDocumentElement();
			NodeList puntuaciones = raiz.getElementsByTagName("puntuacion");
			for (int i = 0; i < puntuaciones.getLength(); i++) {
				Node puntuacion = puntuaciones.item(i);
				NodeList propiedades = puntuacion.getChildNodes();
				for (int j = 0; j < propiedades.getLength(); j++) {
					Node propiedad = propiedades.item(j);
					String etiqueta = propiedad.getNodeName();
					if (etiqueta != null) {
						if (etiqueta.equals("nombre")) {
							nombre = propiedad.getFirstChild().getNodeValue();
						} else if (etiqueta.equals("puntos")) {
							puntos = propiedad.getFirstChild().getNodeValue();
						}
					}
				}
				result.add(nombre + " " + puntos);
			}
		}

		return result;
	}

	// Compatible solo a partir API 8 Android
	public void escribirXML(OutputStream salida) throws Exception {
		if (cargadoDocumento) {
			// volcamos el XML al fichero
			TransformerFactory fabrica = TransformerFactory.newInstance();
			
			// añadimos sangrado y la cabecera de XML
			Transformer transformador = fabrica.newTransformer();
			fabrica.setAttribute("indent-number", new Integer(3));
			transformador.setOutputProperty(OutputKeys.INDENT, "yes");
			transformador.setOutputProperty(OutputKeys.OMIT_XML_DECLARATION, "no");
			
			// hacemos la transformacion
			StreamResult resultado = new StreamResult(salida);
			DOMSource fuente = new DOMSource(documento);
			transformador.transform(fuente, resultado);

		}
	}

	@Override
	public Vector<String> listaPuntuaciones(int cantidad) {
		try {
			if (!cargadoDocumento) {
				// leerXML(contexto.openFileInput(FICHERO));
				leerXML(new FileInputStream(FICHERO));
			}
		} catch (FileNotFoundException e) {
			crearXML();
		} catch (Exception e) {
			Log.e("Asteroides", e.getMessage(), e);
		}

		return aVectorString();
	}

	public void crearXML() {
		try {
			DocumentBuilderFactory fabrica = DocumentBuilderFactory
					.newInstance();
			DocumentBuilder constructor = fabrica.newDocumentBuilder();
			documento = constructor.newDocument();
			Element raiz = documento.createElement("lista_puntuaciones");
			documento.appendChild(raiz);
			cargadoDocumento = true;
		} catch (Exception e) {
			Log.e("Asteroides", e.getMessage(), e);
		}
	}

	public void leerXML(InputStream entrada) throws Exception {
		DocumentBuilderFactory fabrica = DocumentBuilderFactory.newInstance();
		DocumentBuilder constructor = fabrica.newDocumentBuilder();
		documento = constructor.parse(entrada);
		cargadoDocumento = true;
	}

	private boolean ExisteDirectorioAsteroides() {
		String directorioSD = Environment.getExternalStorageDirectory()
				.toString();

		String stadoSD = Environment.getExternalStorageState();
		if (!stadoSD.equals(Environment.MEDIA_MOUNTED)) {
			Toast.makeText(contexto, "No puedo leer en la memoria externa",
					Toast.LENGTH_LONG).show();
			return false;
		}

		File f = new File(directorioSD, "Asteroides");

		if (f == null) { // No ha creado el objeto correctamente
			return false;
		}

		if (!f.exists()) { // Existe el directorio Asteroides?
			if (!f.mkdir()) { // No existe lo creamos.
				return false;
			} else {
				return true;
			}
		}

		return true;

	}

}

Remarcar, que la grabación del xml se hace en una carpeta de la tarjeta SD del móvil o tablet.

En principio estaba diseñado el ejemplo de la práctica,para hacerlo en la memoria interna del teléfono/tablet en /data/data/com.android.aplicacion/files

He hecho unas modificaciones, para que grabe en una carpeta concreta dentro de la sd el archivo xml.
Esto mismo lo he hecho con las librerias de XML SAX y no me ha dado ningún problema.