Panos,
Yeah, my change won't work if you have multi-sets of the children elements,
because you will only add
the last set to your vector. Does your children elements always follow the same
sequential order such as
<date/><value/><change/><date/><value/><change/>.... ? If this is the case, then
use the following code in
endElement():
> if (qName.compareToIgnoreCase("date") == 0) {
> date = new String(tempstring);
> } else if (qName.compareToIgnoreCase("value") == 0) {
> value = new String(tempstring);
> } else if (qName.compareToIgnoreCase("change") == 0) {
> change = new String(tempstring);
vector.add(new Data(date, value, change));
> } else if (qName.compareToIgnoreCase("data") == 0) {
> // vector.add(new Data(date, value, change));
//do nothing, since this is the end of the root.
> }
>
Let me know if this works. If the assumption of sequential orders isn't right,
let me know also, I will give you
another example.
-JJ
Panagiotis Plevrakis wrote:
> I am still getting only the last set of <date><value><change>. I forgot to
> mention that <data> is the root element and within this element there is a
> sequence of <date><value><change> elements.
>
> Panos
>
> >From: Jianjun Zhang <[EMAIL PROTECTED]>
> >Reply-To: "JDJList" <[EMAIL PROTECTED]>
> >To: "JDJList" <[EMAIL PROTECTED]>
> >Subject: [jdjlist] Re: For Charles or anyone that can help
> >Date: Thu, 16 May 2002 15:11:12 -0700
> >
> >You should reuse class variables (date, value, change, ...) in your
> >endElement(...) callback routines. Everytime this is
> >called, you are assigning new values to date, value and change (make them
> >pointing to new values). Essentially, at the
> >end, your vector contains a list of objects with the same internal variable
> >values (the last one). Use the following:
> >
> > if (qName.compareToIgnoreCase("date") == 0) {
> > date = new String(tempstring);
> > } else if (qName.compareToIgnoreCase("value") == 0) {
> > value = new String(tempstring);
> > } else if (qName.compareToIgnoreCase("change") == 0) {
> > change = new String(tempstring);
> > } else if (qName.compareToIgnoreCase("data") == 0) {
> > vector.add(new Data(date, value, change));
> > }
> >
> >This may not be the most efficient way of doing this (a string copy is
> >performed), but requires least change, a better probably
> >is to keep a stack of the values so that no copies are needed.
> >
> >This is assuming that there is only one set of date, value, change elements
> >per data element parent.
> >I haven't tested this, but hope it works.
> >
> >-JJ
> >
> >Panagiotis Plevrakis wrote:
> >
> > > In the following code I'm parsing an xml file that contains
> > > the elements <date>,<value>,<change> more than once. The parent
> > > element of these elements is <data> . I load their values in a Vector
> > > wrapped in a Data object. While I expect more than one set of data to be
> > > displayed, I get only the last one. What am I doing wrong?
> > > Thanks for your help.
> > > ***********************************************************
> > > import java.util.*;
> > > import java.io.*;
> > > import java.net.*;
> > > import javax.xml.parsers.*;
> > > import org.xml.sax.*;
> > > import org.xml.sax.helpers.*;
> > >
> > > public class Parser extends DefaultHandler {
> > > Vector vector = new Vector();
> > > String date, value, change, tempstring;
> > > Data data = null;
> > >
> > > public static void main(String[]args) {
> > > Parser p = new Parser();
> > > p.parse("database.xml");
> > > p.display();
> > > }
> > >
> > > private void display() {
> > > for (int i = 0; i < vector.size(); i++) {
> > > System.out.println(vector.size()); //debug
> > > data = (Data) vector.elementAt(i);
> > >
> > > System.out.println(data.getDate() + "\n" +
> > > data.getValue() + "\n" +
> > > data.getChange() + "\n");
> > > System.out.println();
> > > }
> > > }
> > >
> > > private void parse(String file) {
> > > try {
> > > SAXParserFactory spf = SAXParserFactory.newInstance();
> > >
> > > spf.setValidating(true);
> > > SAXParser saxParser = spf.newSAXParser();
> > > XMLReader xmlReader = saxParser.getXMLReader();
> > >
> > > xmlReader.setContentHandler(this);
> > > xmlReader.setErrorHandler(this);
> > > String urlstring = "file:" +
> > > new File (file).getAbsolutePath();
> > > URL url = new URL(urlstring);
> > > URLConnection connection = url.openConnection();
> > > InputStream inputstream = connection.getInputStream();
> > > InputSource inputsource = new InputSource(inputstream);
> > >
> > > xmlReader.parse(inputsource);
> > > } catch (Throwable err) {
> > > System.out.println(err.getMessage());
> > > }
> > > }
> > >
> > > public void startElement(String namespaceURI, String localName,
> > > String qName, Attributes atts) throws SAXException {
> > > tempstring = "";
> > > if (qName.compareToIgnoreCase("data") == 0) {
> > > date = "";
> > > value = "";
> > > change = "";
> > > }
> > > }
> > >
> > > public void endElement(String uri, String localName,
> > > String qName) throws SAXException {
> > > if (qName.compareToIgnoreCase("date") == 0) {
> > > date = tempstring;
> > > } else if (qName.compareToIgnoreCase("value") == 0) {
> > > value = tempstring;
> > > } else if (qName.compareToIgnoreCase("change") == 0) {
> > > change = tempstring;
> > > } else if (qName.compareToIgnoreCase("data") == 0) {
> > > vector.add(new Data(date, value, change));
> > > }
> > > }
> > >
> > > public void characters(char[] ch, int start,
> > > int length) throws SAXException {
> > > String charactersfound = new String(ch, start, length);
> > > tempstring = tempstring + charactersfound;
> > > }
> > >
> > > class Data {
> > >
> > > private String date = "";
> > > private String value = "";
> > > private String change = "";
> > >
> > > public Data() {}
> > >
> > > public Data(String date, String value, String change) {
> > > this.date = date;
> > > this.value = value;
> > > this.change = change;
> > > }
> > >
> > > public String getDate() {
> > > return date;
> > > }
> > >
> > > public String getValue() {
> > > return value;
> > > }
> > >
> > > public String getChange() {
> > > return change;
> > > }
> > >
> > > public void setDate(String date) {
> > > this.date = date;
> > > }
> > >
> > > public void setValue(String value) {
> > > this.value = value;
> > > }
> > >
> > > public void setChange(String change) {
> > > this.change = change;
> > > }
> > > } //end Data
> > > } //end Parser
> > > *********************************************************
> > >
> > > _________________________________________________________________
> > > MSN Photos is the easiest way to share and print your photos:
> > > http://photos.msn.com/support/worldwide.aspx
> > >
> > > To change your membership options, refer to:
> > > http://www.sys-con.com/java/list.cfm
> >
> >
> >To change your membership options, refer to:
> >http://www.sys-con.com/java/list.cfm
>
> _________________________________________________________________
> Chat with friends online, try MSN Messenger: http://messenger.msn.com
>
> To change your membership options, refer to:
> http://www.sys-con.com/java/list.cfm
To change your membership options, refer to:
http://www.sys-con.com/java/list.cfm
