Discussion :

[sal.gass]

Bonjour pour tous,

j'ai implémenté une méthode qui fait un traitement spécifique, remplissage d'un tableau à partir d'une liste,
initPartition( list<Article> articleList, String [] tab)

la méthode tourne bien, et le tableau est remplie dans la méthode,
Mais lorsque je fait un appel à cette méthode, et je donne en parapétre une liste remplie de valeur et un tableau à remplir, le tableau reste toujours null, il ne contient pas de valeurs
par contre, si je suivit la méthode par le débogage le tableau est bien remplie

code

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
private String[]partitionArticleTab ; 

public void methode(){
partitionArticleTab = new String[size];

initPartition(articlesList, partitionArticleTab );

//ici partitionArticleTab est toujours vide 
}
 
public void initPartition( list<Article> articleList, String [] tab) {
//traitement 
}

Est ce qu'il y a quelq'un qui a une solution ou me dire c'est quoi qui ne marche pas

Merci d'avance

[eric39]

Bonjour,

Qu'est ce que tu fais dans la méthode 'initPartition'??
Ca doit être là qu'est le problème.

a+

[sal.gass]

salut, Merci pour votre réponse,

non le problème n'est pas dans la méthode initPartition() comme j'ai dit,

j'ai dit que l'orsque je fait un débogage l'objet locale dans la méthode initPartition( list<Article> articleList, String [] tab) est bien remplie,

mais l'orque je fait un appel à initPartition(articleList, articleTab) par exemple

articleTab est toujours vide,

si vous vouler je vous envoie le code de la méthode initPartition().

Merci

[Astartee]

non le problème n'est pas dans la méthode initPartition() comme j'ai dit

Sauf que toi tu n'arrives pas à trouver ton problème et que nous nous te disons que c'est là qu'il est Donc vas-y, fais-nous confiance, et donne le code !

Tiens, allez, je parie même par avance que tu fais un "tab = new String[...]" dans initPartition...

[eric39]

Peut être que je me trompe mais j'ai quand même l'impression que le problème est dans la méthode 'initPartition'.

Parce que si la liste n'est pas vide, que les éléments sont bien stockées dans le tableau, alors le tableau n'est pas vide!

en fait tu réinstancies surement ton tableau...
Tu n'as d'ailleur pas besoin de le passer en paramètre, il semble être une variable globale.

[sal.gass]

Sauf que toi tu n'arrives pas à trouver ton problème et que nous nous te disons que c'est là qu'il est Donc vas-y, fais-nous confiance, et donne le code !

Voici la méthode:

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
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
 
/**
         * Calculate Partition
         */
	public void initPartition(List<ArticleCMO> articleList,
			String partitionTab[]) {
		if (LOG.isInfoEnabled()) {
			LOG.info("initPartition() --> Start");
		}
		Integer nombreDivisions = new Integer(0);
 
		List<ParametreCMO> listParam = new ArrayList<ParametreCMO>();
		Long idElement = getIdElementSelected();
		try {
			if (idElement != null) {
				listParam.add(CMOUtilities.getNewEqualParametreCMO(
						StrategieAssortimentCMOConstant.FIELD_ID, idElement));
 
				strategiePrixCMO = serviceRepartitionArticlesConsumer
						.getStrategiePrixBusinessObject(listParam);
 
				if (strategiePrixCMO != null
						&& strategiePrixCMO.getElementNomenclature() != null
						&& strategiePrixCMO.getElementNomenclature()
								.getStructureNomenclature() != null
						&& strategiePrixCMO.getElementNomenclature()
								.getStructureNomenclature().getNomenclature() != null
						&& strategiePrixCMO.getElementNomenclature()
								.getStructureNomenclature().getNomenclature()
								.getTypeGestionMarketing() != null) {
					nombreDivisions = strategiePrixCMO.getElementNomenclature()
							.getStructureNomenclature().getNomenclature()
							.getTypeGestionMarketing().getNbPartition();
 
					typeGestion = strategiePrixCMO.getElementNomenclature()
							.getStructureNomenclature().getNomenclature()
							.getTypeGestionMarketing().getLibelle();
				}
			}
 
			viewList = new ArrayList<PartitionCMO>();
			viewList = strategiePrixCMO.getPartitionPrix();
 
			partitions = divisionPartitions(nombreDivisions, strategiePrixCMO
					.getPartitionPrix());
 
			int nbArticles = articleList.size();
			partitionTab = new String[nbArticles];
 
			int i = 0;
			// check if there are one view
			if (viewList.size() == 1) {
				for (ArticleCMO article : articleList) {
					double prixVenteArticle = article.getPrixVente()
							.getMontant();
					int j = 1;
 
					for (PartitionCMO partition : partitions) {
						double min = partition.getIntervalle().getMinimus();
						double max = partition.getIntervalle().getMaximus();
 
						if (prixVenteArticle >= min
								&& article.getPrixVente().getMontant() < max) {
 
							StringBuffer partitionLabel = new StringBuffer();
							if (typeGestion.equalsIgnoreCase("Tiertile")) {
								partitionLabel.append("T");
							} else {
								partitionLabel.append("Q");
							}
 
							partitionLabel.append(j);
							partitionTab[i] = new String(partitionLabel);
						}
						j++;
					}
					i++;
				}
			}
 
			else {
				// get partitions on the first view
				StringBuffer partitionFirstViewString = new StringBuffer();
 
				if (typeGestion.equalsIgnoreCase("Tiertile")) {
					partitionFirstViewString.append("T");
				} else {
					partitionFirstViewString.append("Q");
				}
				List<PartitionCMO> partitionsFirstView = new ArrayList<PartitionCMO>();
				List<PartitionCMO> partionview = new ArrayList<PartitionCMO>();
				partionview.add(viewList.get(0));
				partitionsFirstView = divisionPartitions(nombreDivisions,
						partionview);
 
				partitionFirstViewString.append(partitionsFirstView.size());
 
				for (ArticleCMO article : articleList) {
 
					double prixVenteArticle = article.getPrixVente()
							.getMontant();
					int j = 1;
 
					for (PartitionCMO partition : partitions) {
						double min = partition.getIntervalle().getMinimus();
						double max = partition.getIntervalle().getMaximus();
 
						if (prixVenteArticle >= min && prixVenteArticle < max) {
 
							// Check if Article price belongs to first view
							if (prixVenteArticle < viewList.get(0)
									.getIntervalle().getMaximus()
									&& j <= partitionsFirstView.size()) {
 
								StringBuffer partitionLabel = new StringBuffer();
								if (typeGestion.equalsIgnoreCase("Tiertile")) {
									partitionLabel.append("T");
								} else {
									partitionLabel.append("Q");
								}
 
								partitionLabel.append(j);
								partitionTab[i] = new String(partitionLabel);
							} else if (max >= viewList.get(0).getIntervalle()
									.getMaximus()) {
 
								// browse the list of view from the second
								// element
								boolean flag = false;
								for (int indexView = 1; indexView < viewList
										.size(); indexView++) {
									StringBuffer indexViewString = new StringBuffer(
											"V");
									indexViewString.append(indexView + 1);
 
									// Check if Article price belongs to current
									// view
 
									if (prixVenteArticle < viewList.get(
											indexView).getIntervalle()
											.getMaximus()
											&& flag == false) {
										flag = true;
										StringBuffer partitionLabel = new StringBuffer();
										partitionLabel
												.append(partitionFirstViewString);
										partitionLabel.append(indexViewString);
 
										partitionTab[i] = new String(
												partitionLabel);
 
									}
								}
							}
						}
						j++;
					}
					i++;
				}
			}
 
		} catch (Exception e) {
			e.printStackTrace();
			IhmUtilities.showError(e);
		}
 
		if (LOG.isInfoEnabled()) {
			LOG.info("initPartition() --> Start");
		}
	}

Tiens, allez, je parie même par avance que tu fais un "tab = new String[...]" dans initPartition...

Oui j'ai fait ça :
partitionTab = new String[nbArticles];

Est ce que ça est une erreur,
si oui ? je ne doit pas faire ça ?

[sal.gass]

salut Eric,

Code :

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

Tu n'as d'ailleur pas besoin de le passer en paramètre, il semble être une variable globale.

Non, j'ai besoinde le passer en paramètre car je veux faire le meme traitement pour pour une autre liste et un autre tableau.

Merci.

[eric39]

Dans ce cas là, si ton tableau en entrée est toujours vide au début, tu peux faire une méthode qui retourne le tableau créé.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
/**
 * Calculate Partition
 */
public String[] initPartition(List<ArticleCMO> articleList) {
    // ton code
    int nbArticles = articleList.size();
    String[] partitionTab = new String[nbArticles];
    // ton code
    return partitionTab;
}

Et tu appeles la méthode:

Code :

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

1
2
 
partitionArticleTab = initPartition(articlesList);

[sal.gass]

ok, je vous remercie Eric pour votre réponse,
je vais essayé... je reste en cours.

[Astartee]

Une petite réponse plus générale pour compléter

Oui j'ai fait ça
Est ce que ça est une erreur, si oui ? je ne doit pas faire ça ?

Eh oui, c'est une erreur !

Explication :
Tu passes en paramètre un tableau String[] qui est un objet. Donc si sur ce tableau tu effectues des affectations, les modifications seront visible après la sortie de la méthode puisqu'elles auront été faites sur ce même objet.
Sauf que l'instruction new String[...] crée un nouveau tableau ! Tu n'agis donc plus sur le tableau passé en paramètre, mais sur un nouvel objet, que tu vas "jeter" à la fin de ta méthode après l'avoir rempli...


Solutions possibles (avec code simplifié qui ne compilera certainement pas et qui ne correspond pas exactement à ton traitement) :

1/ initialiser le tableau en dehors de la méthode, et le remplir dans la méthode

Code :

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

1
2
3
4
5
6
7
8
String[] tableau = new String[list.size()];
initTableau(list, tableau);
 
private void initTableau(List l, String[] t) {
   for (int i = 0 ; i < l.size() ; i++) {
      t[i] =...;
   }
}

2/ initialiser un nouveau tableau dans la méthode et le renvoyer pour pouvoir l'affecter à une variable en dehors de la méthode (c'est ce que te propose eric39)

Code :

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

1
2
3
4
5
6
7
8
9
String[] tableau = initTableau(list);
 
private String[] initTableau(List l) {
   String[] t = new String[l.size()];
   for (int i = 0 ; i < l.size() ; i++) {
      t[i] = ...;
   }
   return t;
}

[sal.gass]

Le problème est résolut, Thanks.

Explication :
Tu passes en paramètre un tableau String[] qui est un objet. Donc si sur ce tableau tu effectues des affectations, les modifications seront visible après la sortie de la méthode puisqu'elles auront été faites sur ce même objet.
Sauf que l'instruction new String[...] crée un nouveau tableau ! Tu n'agis donc plus sur le tableau passé en paramètre, mais sur un nouvel objet, que tu vas "jeter" à la fin de ta méthode après l'avoir rempli...

Je comprend bien maintenant que pour quoi le tableau est vide après l'appel de la méthode initPartition(), merci Astartee pour cette explication.

J'ai suivie la première méthode
public void initPartition(List<Article>articleList, String[]tabPartition) {}

Je vous remercie eric39 pour votre participation.

Je pense qu'il faut maitriser de plus la POO .

[Djobird]

C'est un problème que tu trouveras dans beaucoup de langages et qui n'est pas lié à la POO. C'est plus un problème d'allocation mémoire qu'autre chose

[sal.gass]

C'est un problème que tu trouveras dans beaucoup de langages et qui n'est pas lié à la POO. C'est plus un problème d'allocation mémoire qu'autre chose

+1,
Merci pour l'information.