Modifier les composants de base

**lennelei** · 30/04/2012, 13h01

Bonjour à tous,

j'ai un problème un peu particulier, et je souhaiterais modifier un composant de base de Talend (DynamicUtils.java qui se trouve dans ...\plugins\org.talend.desinger.routines.tisprovider_5.0.2.r78327\resources\java\routines\system).

En fait, lorsque j'utilise un schéma dynamique, Talend sort des chaînes "null" pour toutes les valeurs null... ce qui ne m'arrange pas trop, et je voudrais lui dire de mettre plutôt des chaînes vides "". Pour le moment, le moyen le plus simple que j'ai trouvé est de modifier la fonction writeValuesToStringArray du fichier DynamicUtils.java, mais je ne sais pas si c'est faisable et si oui, comment faire ^^

Quelqu'un a déjà fait ce genre de modifs ?

Merci d'avance.

**boubsy** · 03/05/2012, 15h33

Salut,

Tu n'aurai pas plus vite fait de creer une routine qui prend en entrée la colonne dynamique et te renvois un type dynamique epuré des "null"

la type dynamique integre toutes les routines pour accéder au informations et parcourir le tableau

boubsy

**lennelei** · 03/05/2012, 15h38

Hello,

aucune idée, mais dis comme ça, ça semble plus simple ^^ sauf que je ne maîtrises pas assez Talend pour comprendre ce que tu veux dire : je n'ai notamment pas encore jeté un oeil aux routines :s

Mais je vais regarder si j'arrive à en tirer un truc (si tu as une ou deux pistes pour me guider, je suis preneur !).

Merci pour la réponse en tout cas, je commençais à désespérer :p

**haskouse** · 03/05/2012, 16h30

Pour comprendre comment utiliser les routines je te renvoies vers ce tuto :
http://haskouse.developpez.com/tutor...ation-routine/

**lennelei** · 03/05/2012, 16h49

Merci, je suis déjà passé dessus ^^

C'était plus le coup de la gestion de la colonne Dynamic qui me posait problème... mais en fait, je me suis inspiré de la partie que je voulais modifier dans DynamicUtils.java et je l'ai réécrite en version simplifiée

Ca donne un truc du genre:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
    public static void removeNull(Dynamic column) {
    	for (int i = 0; i < column.getColumnCount(); i++) {
		    DynamicMetadata metadata = column.getColumnMetadata(i);
			if ("id_String".equals(metadata.getType()) && column.getColumnValue(i) == null) {
				column.setColumnValue(i, "");
			} else if ("id_Double".equals(metadata.getType()) && column.getColumnValue(i) == null) {
				column.setColumnValue(i, 0.0);
			} else if (("id_Long".equals(metadata.getType()) || "id_Integer".equals(metadata.getType())) && column.getColumnValue(i) == null) {
				column.setColumnValue(i, 0);
			}
		}
    }

C'est sûrement pas extraordinaire, mais ça semble faire le boulot sur un petit test... je vais passer ça dans mon job qui posait problème pour vous dire !

**emenuet** · 03/05/2012, 16h50

De plus, une fois des évolution effectué tu pourras les poster pour que Talend les prennent en compte directement dans la class Dynamic pour les prochaines version :
http://jira.talendforge.org/secure/Dashboard.jspa

**lennelei** · 03/05/2012, 17h49

Rah, mais c'est des boulets chez Talend...

(désolé si quelqu'un de chez Talend lit ça, mais des fois, il y'a des trucs bizarres...)

Donc ma modification ci-dessus ne marche pas, car la classe Dynamic.java est foireuse... extrait :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
    public void setColumnValue(int index, Object value) {
        if (index < this.metadatas.size()) {
            modifyColunmValue(index, value);
        }
    }
// *** jusque là, ok
    private void modifyColunmValue(int index, Object value) {
        if (values.get(index) != null)
            values.set(index, value);
        else
            values.add(index, value);
// *** sic
    }

On voit donc que la fonction modifyColunmValue teste si la valeur à modifier est nulle ou pas. Si elle est nulle, ça ajoute (!!) la valeur à la liste au niveau de l'index donné au lieu de modifier la valeur... je suis pas un expert java, mais d'un point de vue logique, je ne comprends pas...
Dans tous les cas, d'un point de vue fonctionnel, si j'ai une liste "toto",null,"tata" et que je veux modifier le null en "titi", ben en faisant un setColumnValue(1, "titi"), je me retrouve à faire un values.add(1,"titi") et à avoir donc la liste "toto","titi",null,"tata" ce qui ne m'arrange pas, mais alors pas du tout !!

**haskouse** · 04/05/2012, 09h52

Pourquoi vouloir réutiliser le code de Talend, crées a propre routine et mets y le code qui te conviens.

**lennelei** · 04/05/2012, 12h29

Ben à un moment, je suis bien obligé d'utiliser la classe Dynamic pour gérer la colonne non ?

Enfin, quand je récupère la colonne dynamique dans mon tJavaRow, c'est un objet de type Dynamic, et mon problème, c'est la fonction setColumnValue(...) de cette classe Dynamic.

Je ne vois pas comment je peux, simplement, passer outre ce point là ?

**haskouse** · 04/05/2012, 13h22

Pourquoi tu ne crées pas ta propre routine avec une fonction prenant comme attribut la classe Dynamic ?

**lennelei** · 04/05/2012, 14h14

Ben je crois que c'est ce que je fais ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
    public static void removeNull(Dynamic COLUMN) {
    	FOR (int i = 0; i < COLUMN.getColumnCount(); i++) {
		    DynamicMetadata metadata = COLUMN.getColumnMetadata(i);
			IF ("id_String".equals(metadata.getType()) && COLUMN.getColumnValue(i) == NULL) {
				COLUMN.setColumnValue(i, "");
			} else IF ("id_Double".equals(metadata.getType()) && COLUMN.getColumnValue(i) == NULL) {
				COLUMN.setColumnValue(i, 0.0);
			} else IF (("id_Long".equals(metadata.getType()) || "id_Integer".equals(metadata.getType())) && COLUMN.getColumnValue(i) == NULL) {
				COLUMN.setColumnValue(i, 0);
			}
		}
    }

Mais le problème, c'est que COLUMN.setColumnValue(i,0) ne marche pas si la valeur de la colonne i est nulle...

**haskouse** · 04/05/2012, 15h33

Tu ne peux pas modifier le code de setColumnValue().
Je n'arrive pas à retrouver cette fonction sur mon Talend.
Quel est le chemin complet (packages) de la classe Dyanmic ?

**lennelei** · 04/05/2012, 15h41

Le source :
en v5
Talend-5.0.2\studio\Talend-Studio-r78327-V5.0.2\plugins\org.talend.desinger.routines.tisprovider_5.0.2.r78327\resources\java\routines\system\Dynamic.java

en v4
C:\Program Files\Talend-4.2.3\studio\Talend-All-r67267-V4.2.3\plugins\org.talend.desinger.routines.tisprovider_4.2.3.r67267\resources\java\routines\system\Dynamic.java

La classe :
en v5
Talend-5.0.2\studio\Talend-Studio-r78327-V5.0.2\plugins\org.talend.desinger.routines.tisprovider_5.0.2.r78327\routines\system\Dynamic.class

en v4
Talend-4.2.3\studio\Talend-All-r67267-V4.2.3\plugins\org.talend.desinger.routines.tisprovider_4.2.3.r67267\routines\system\Dynamic.class

**haskouse** · 04/05/2012, 15h46

Je vois que tu utilises TIS et non pas TOS.
Je n'ai pas TOS donc je ne pourrais pas t'aider, tu devrais voir avec quelqu'un qui a le même environnement que toi sur son poste il te sera de plus grande aide.

**lennelei** · 04/05/2012, 16h05

ok, merci quand même

à tout hasard, tu ne saurais pas si je peux recompiler les classes de Talend ?

**haskouse** · 04/05/2012, 17h48

Franchement je ne sais pas.

**lennelei** · 09/05/2012, 16h58

Bon, en attendant, la classe sera corrigée en v 5.0.3

http://jira.talendforge.org/browse/TDI-21019

Sinon, j'ai "trouvé" une solution de contournement, mais c'est galère (il faut copier la liste des métadonnées - ce qui me pose un souci car je n'arrive pas à dupliquer la liste complète pour le moment, je ne connais pas bien cette partie de Java

- supprimer la colonne ajoutée lors du setColumnValue et remettre la bonne liste des métadonnées puisque sinon, les métadonnées ne correspondent plus aux colonnes...)

'fin, vivement que ça soit corrigé -_-'

**rdubois** · 10/05/2012, 10h48

Rah, mais c'est des boulets chez Talend...

Passons.

Le bug est fixé et ne nécessite plus que de la QA. Le fix sera disponible en 5.0.3.

Rémy.

**lennelei** · 11/05/2012, 16h24

Envoyé par rdubois

Passons.

Le bug est fixé et ne nécessite plus que de la QA. Le fix sera disponible en 5.0.3.

Rémy.

Bonjour,

je sais, c'est un peu raide comme commentaire, c'est pour ça que j'ai ajouté :
"(désolé si quelqu'un de chez Talend lit ça, mais des fois, il y'a des trucs bizarres...)"

je continue en mp, ça n'a rien à faire ici.