Discussion :

[totot]

Bonjour,

Je suis entrain de faire une application qui lit un fichier xml et qui insère ensuite les données dans la base si j'arrive à aller jusque là.
Je souhaiterai associés les colis à la livraison lors de la lecture du fichier qui se présente comme ceci:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
<livraisons>
  <livraison>
    <client>Mme Lagrange</client>
    <adresse>28 rue des Rainettes 33150 Cenon</adresse>
    <colis>
      <ref>172648574</ref>
      <montant>96.00</montant>
    </colis>
    <colis>
      <ref>172648586</ref>
      <montant>45.80</montant>
    </colis>
  </livraison>
  <livraison>
    <client>M. Iribarne</client>
    <adresse>5 chemin des Dunes 33510 Andernos</adresse>
    <colis>
      <ref>172648588</ref>
      <montant>146.75</montant>
    </colis>
  </livraison>  
</livraisons>

Voilà l'affichage que j'obtient:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
Livraison : client : Mme Lagrange => adresse : 28 rue des Rainettes 33150 Cenon
Colis : référence : 172648574 => montant : 96.0
Colis : référence : 172648586 => montant : 45.8
Colis : référence : 172648588 => montant : 146.75
Livraison : client : M. Iribarne => adresse : 5 chemin des Dunes 33510 Andernos
Colis : référence : 172648574 => montant : 96.0
Colis : référence : 172648586 => montant : 45.8
Colis : référence : 172648588 => montant : 146.75

voici mes classe livraison et colis:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
public class Livraison {
 
	private String client;
	private String adresse;
 
	public Livraison(String client, String adresse) {
		this.client = client;
		this.adresse = adresse;
	}
 
	public Livraison(){
		super();
	}
 
	public String getClient(){
		return client;
	}
 
	public String getAdresse(){
		return adresse;
	}
 
	public void setClient(String client) {
		this.client = client;
	}
 
	public void setAdresse(String adresse) {
		this.adresse = adresse;
	}
 
	public String toString(){
		return "Livraison : client : "+client+" => adresse : "+adresse;
	}
}

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
public class Colis {
 
	private String ref;
	private double montant=0;
	private Livraison idLivraison;
 
	public Colis(Livraison idLivraison, double montant, String ref) {
		super();
		this.idLivraison = idLivraison;
		this.montant = montant;
		this.ref = ref;
	}
 
	public Colis(){
		super();
	}
 
	public double getMontant() {
		return montant;
	}
 
	public Livraison getIdLivraison() {
		return idLivraison;
	}
 
	public String getRef(){
		return ref;
	}
 
	public void setRef(String ref) {
		this.ref = ref;
	}
 
	public void setMontant(double montant) {
		this.montant = montant;
	}
 
	public String toString(){
		return "Colis : référence : "+ref+" => montant : "+montant;
	}
}

Ma classe LitXml:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
public class LitXml {
 
	private List<Livraison> lesLivraisons;
	private List<Colis> lesColis;
 
	public LitXml(Context context)  {
		try {
			MaSaxHandler handler = new MaSaxHandler();
			Resources resources = context.getResources();
			InputStream inputStream = resources.openRawResource(R.raw.livraisons);
			handler.parse(inputStream);
	        lesLivraisons = handler.getLesLivraisons();
	        lesColis = handler.getLesColis();
		} catch (Exception e) {
			Log.i("litxml", "erreur");
		}
	}
 
	public List<String> donneLivraisons(){
		List<String> liste = new ArrayList<String>();
		for (Livraison livraison : lesLivraisons)
			liste.add(livraison.toString());
		return liste;
	}
 
	public List<String> donneColis(){
		List<String> liste = new ArrayList<String>();
		for (Colis colis : lesColis)
			liste.add(colis.toString());
		return liste;
	}
}

ma classe MaSaxHandler

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
ublic class MaSaxHandler extends DefaultHandler {
 
	   private Livraison livraison;
	   private List<Livraison> lesLivraisons = new ArrayList<Livraison>();
	   private Colis colis;
	   private List<Colis> lesColis = new ArrayList<Colis>();
	   private String valeur;
 
	   public void parse(InputStream is) throws ParserConfigurationException, SAXException, IOException{
		   SAXParserFactory factory = SAXParserFactory.newInstance();
		   SAXParser parser=factory.newSAXParser();
		   parser.parse(is,this);
	   }
	   public void startElement( String uri, String localName, String qName, Attributes attributes)
			   throws SAXException {
		   if (localName.equals("livraison"))  livraison = new Livraison();
		   if (localName.equals("colis")) colis = new Colis();
	   }
	   public void characters(char[] ch,int start, int length) throws SAXException{
			valeur = new String(ch,start,length);
		}
	   public void endElement( String uri, String localName,String qName) throws SAXException {
	        if (localName.equals("livraison")) {
	        	lesLivraisons.add(livraison);
	        }else if (localName.equals("client")){
	        	livraison.setClient(valeur);
	        }else if (localName.equals("adresse")){
	        	livraison.setAdresse(valeur);
	        }else if (localName.equals("colis")){
	        	lesColis.add(colis);
	        }else if (localName.equals("ref")){
	        	colis.setRef(valeur);
	        }else if (localName.equals("montant")){
	        	double monLong= Double.parseDouble(valeur);
	        	colis.setMontant(monLong);
	        }
	    }
	   public List<Livraison> getLesLivraisons() {
		   return lesLivraisons;
	   }
 
	   public List<Colis> getLesColis() {
			return lesColis;
	   }
}

Code de l'activité principal:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
		for (String ligne : litXml.donneLivraisons()){
			Log.i("litxml", ligne);
			for (String ligne1 : litXml.donneColis()){
				Log.i("litxml", ligne1);
			}
 
		}

Merci de votre aide

[thelvin]

Le plus simple ce serait que la classe Livraison contienne aussi une List<Colis> que tu peux compléter à mesure que tu lis le XML.
Et donc MaSaxHandler ne contiendrait pas de List<Colis>. À la place il faut ajouter les colis à la liste de la livraison actuelle.

(Je pars du principe que tu utilises SAX au lieu de StAX, DOM ou JAXB dans le but d'économiser de la mémoire et des bibliothèques externes. Auquel cas, on ne peut pas simplifier beaucoup.)

[totot]

Oui,

J'utilise SAXParser

Et à un client, j'associe donc la liste des colis livré?

[totot]

J'ai refait ma classe comme ceci:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
	private String client;
	private String adresse;
	private List<Colis> lesColis;
 
	public Livraison(String client, String adresse) {
		this.client = client;
		this.adresse = adresse;
		lesColis = new ArrayList<Colis>();
	}
 
	public Livraison(){
		super();
	}
 
	public String getClient(){
		return client;
	}
 
	public String getAdresse(){
		return adresse;
	}
 
	public void setClient(String client) {
		this.client = client;
	}
 
	public void setAdresse(String adresse) {
		this.adresse = adresse;
	}
 
	public void setLesColis(List<Colis> lesColis){
		this.lesColis = lesColis;
	}
 
	public String toString(){
		return "Livraison : client : "+client+" => adresse : "+adresse;
	}

Mais ici je bloque:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
if (localName.equals("colis")){
	        	livraison.setLesColis(Colis);

[totot]

Personne ?

[thelvin]

Mais... C'est pas un set qu'il faut, c'est un get -_-°.

Ou alors juste un addColis()

[totot]

Tu le fait comment ton addColis()

[totot]

Voilà ma classe livraison donc:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
public Livraison(String client, String adresse) {
		this.client = client;
		this.adresse = adresse;
		lesColis = new ArrayList<Colis>();
	}
 
	public Livraison(){
		super();
	}
 
	public String getClient(){
		return client;
	}
 
	public String getAdresse(){
		return adresse;
	}
 
	public List<Colis> getLesColis(){
		return lesColis;
	}
 
	public void setClient(String client) {
		this.client = client;
	}
 
	public void setAdresse(String adresse) {
		this.adresse = adresse;
	}
 
	public String toString(){
		return "Livraison : client : "+client+" => adresse : "+adresse+" les colis: "+lesColis;
	}

et ma classe MaSaxHandler:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
public class MaSaxHandler extends DefaultHandler {
 
	   private Livraison livraison;
	   private List<Livraison> lesLivraisons = new ArrayList<Livraison>();
	   private Colis colis;
	   private String valeur;
 
	   public void parse(InputStream is) throws ParserConfigurationException, SAXException, IOException{
		   SAXParserFactory factory = SAXParserFactory.newInstance();
		   SAXParser parser=factory.newSAXParser();
		   parser.parse(is,this);
	   }
	   public void startElement( String uri, String localName, String qName, Attributes attributes)
			   throws SAXException {
		   if (localName.equals("livraison"))  livraison = new Livraison();
		   if (localName.equals("colis")) colis = new Colis();
	   }
	   public void characters(char[] ch,int start, int length) throws SAXException{
			valeur = new String(ch,start,length);
		}
	   public void endElement( String uri, String localName,String qName) throws SAXException {
	        if (localName.equals("livraison")) {
	        	lesLivraisons.add(livraison);
	        }else if (localName.equals("client")){
	        	livraison.setClient(valeur);
	        }else if (localName.equals("adresse")){
	        	livraison.setAdresse(valeur);
	        }else if (localName.equals("colis")){
	        	livraison.getLesColis();
	        }else if (localName.equals("ref")){
	        	colis.setRef(valeur);
	        }else if (localName.equals("montant")){
	        	double monLong= Double.parseDouble(valeur);
	        	colis.setMontant(monLong);
	        }
	    }
	   public List<Livraison> getLesLivraisons() {
		   return lesLivraisons;
	   }
 
}

Mais sa me retourne pas mes colis, sa me retourne juste sa:
Livraison : client : Mme Lagrange => adresse : 28 rue des Rainettes 33150 Cenon
Livraison : client : M. Iribarne => adresse : 5 chemin des Dunes 33510 Andernos

[thelvin]

Tu t'es forcément planté dans ton test. Là tu me dis que t'as une String " les colis: " et qu'elle s'affiche pas.

Cette remarque étant faite, c'est bien beau de récupérer la liste de colis avec getColis(), mais peut-être faudrait-il songer à y ajouter le nouveau colis ? Une idée comme ça.
C'est pour ça qu'une simple méthode addColis() qui ajoute un colis à la Livraison, serait peut-être plus simple.

[totot]

J'ai ajouté cette méthode là dans la classe Livraison:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
public void addColis (Colis unColis)
{
		lesColis.add(unColis);
}

Mais ici je bloque;

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
	        }else if (localName.equals("colis")){
	        	livraison.addColis(valeur);

[totot]

J'ai essayer pas mal de chose mais rien à faire, je n'arrive pas à ajouter de colis à la livraison

[thelvin]

Tu n'aurais pas par hasard une variable nommée colis, de type Colis, que tu pourrais lui passer en paramètre ?

Et sinon ça fait combien de temps que tu as pas dormi ?

[totot]

J'ai réussi à ajouter le colis mais mon appli plante
Classe Livraison:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
public class Livraison {
 
	private String client;
	private String adresse;
	private List<Colis> lesColis;
 
	public Livraison(String client, String adresse) {
		this.client = client;
		this.adresse = adresse;
		lesColis = new ArrayList<Colis>();
	}
 
	public Livraison(){
		super();
	}
 
	public String getClient(){
		return client;
	}
 
	public String getAdresse(){
		return adresse;
	}
 
	public void setClient(String client) {
		this.client = client;
	}
 
	public void setAdresse(String adresse) {
		this.adresse = adresse;
	}
 
	public void addColis (Colis unColis)
	{
		lesColis.add(unColis);
	}
 
	public String toString(){
		String chaine = "Livraison : client : "+client+" => adresse : "+adresse+" les colis: ";
		for (Colis colis : lesColis)
			chaine = chaine + "Référence: "+colis.getRef()+" =>montant : "+colis.getMontant();
		return chaine;
	}

Classe MaSaxHandler:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
public class MaSaxHandler extends DefaultHandler {
 
	   private Livraison livraison;
	   private List<Livraison> lesLivraisons = new ArrayList<Livraison>();
	   private Colis colis;
	   private String valeur;
 
	   public void parse(InputStream is) throws ParserConfigurationException, SAXException, IOException{
		   SAXParserFactory factory = SAXParserFactory.newInstance();
		   SAXParser parser=factory.newSAXParser();
		   parser.parse(is,this);
	   }
	   public void startElement( String uri, String localName, String qName, Attributes attributes)
			   throws SAXException {
		   if (localName.equals("livraison"))  livraison = new Livraison();
		   if (localName.equals("colis")) colis = new Colis();
	   }
	   public void characters(char[] ch,int start, int length) throws SAXException{
			valeur = new String(ch,start,length);
		}
	   public void endElement( String uri, String localName,String qName) throws SAXException {
	        if (localName.equals("livraison")) {
	        	lesLivraisons.add(livraison);
	        }else if (localName.equals("client")){
	        	livraison.setClient(valeur);
	        }else if (localName.equals("adresse")){
	        	livraison.setAdresse(valeur);
	        }else if (localName.equals("colis")){
	        	livraison.addColis(colis);
	        }else if (localName.equals("ref")){
	        	colis.setRef(valeur);
	        }else if (localName.equals("montant")){
	        	double monLong= Double.parseDouble(valeur);
	        	colis.setMontant(monLong);
	        }
	    }
	   public List<Livraison> getLesLivraisons() {
		   return lesLivraisons;
	   }
 
}

Classe LitXml:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
public class LitXml {
 
	private List<Livraison> lesLivraisons;
 
	public LitXml(Context context)  {
		try {
			MaSaxHandler handler = new MaSaxHandler();
			Resources resources = context.getResources();
			InputStream inputStream = resources.openRawResource(R.raw.livraisons);
			handler.parse(inputStream);
	        lesLivraisons = handler.getLesLivraisons();
		} catch (Exception e) {
			Log.i("litxml", "erreur");
		}
	}
 
	public List<String> donneLivraisons(){
		List<String> liste = new ArrayList<String>();
		for (Livraison livraison : lesLivraisons)
			liste.add(livraison.toString());
		return liste;
	}
}

Sa plante ici:
for (Livraison livraison : lesLivraisons)

[thelvin]

Alors déjà ce serait sympathique d'utiliser un vocabulaire plus élaboré que "ça plante," et à la place de nous donner le message d'erreur associé. Tant qu'à faire, lire les messages d'erreur et essayer de les comprendre, avant, ne serait pas franchement du luxe. En fait tu es censé le faire, avant de venir nous demander

Ensuite, par corollaire et au vu également de tout le reste de la discussion, tu ne crois pas que c'est un peu tôt pour toi, d'essayer de faire des applis Android, avec du SAX en plus ? Non parce que ce n'est pas exactement facile, si tu veux, alors sans même connaître Java d'abord, ça va pas franchement avancer vite.

Y a pas de raison bien précise pour qu'une erreur arrive ici, ça dépend sûrement d'autres erreurs que tu aurais eues avant et qu'il ne fallait évidemment pas ignorer, ou du moment où tu appelles cette méthode.

[totot]

Je vais abandonner alors ce que je voulais faire pour la lecture
Ensuite j'avais prévu d'enregistrement ses infos dans une base de données et c'est pour cela que je n'avais pas faire de liste colis au début avec une référence à livraison dans colis

[Mishulyna]

Bonjour,

Tu pourrais essayer de trier la collection de Colis par colis.livraison.client.idClient dans une Map<Integer, List<Colis>> comme dans la première réponse ici.
Ou peut-être trier par colis.livraison.client dans une Map<String, List<Colis>>, vue que le Client est apparemment une variable de type String.

[totot]

Je bloque toujours pour l'affichage mais j'ai l'impression que mes livraison ne s'ajoutent plus à ma liste livraison depuis que j'ai fait une référence avec colis
Pour l'instant, je souhaite afficher les données dans une console LogCat