Web Application, executer une fonction

Bonsoir tout le monde.

Je travaille sur un projet de création d'une web application oracle.

j'ai une fonction que je cherche à optimiser.

Code:

---

1 2 3 4 5 6 7 8 9 10 11

FUNCTION GET_ALL_BIEN_BY_CLIENT(v_numclient Client.NUMCLIENT%type, v_villebien Bien.VILLEBIEN%type) RETURN sqlcur AS pcur sqlcur; BEGIN open pcur for select bien.*, etatbien.LIBELLEETAT from BIEN right join etatbien on(bien.NUMBIEN = etatbien.NUMBIEN) where bien.NUMBIEN in (select proposer.numbien from proposer where proposer.NUMCLIENT = v_numclient) and bien.VILLEBIEN = v_villebien; return pcur; END GET_ALL_BIEN_BY_CLIENT;

---

Cette fonction me permet de retourner les biens d'un client donné d'une ville donnée

Code:

---

1 2	http://localhost:8080/projet/AFFICHAGE_LISTE?v_numclient=1&v_villebien=Paris

---

ce que je cherche à faire c'est afficher tout les bien d'un client dans toutes les villes, dois je ajouter une nouvelle fonction ou bien y a une autre alternative en utilisant la même fonction ce qui me permettra de gagner en temps et que je trouve plus propres.

Merci d'avance

Rapidement, vous pourriez modifier votre WHERE en ceci :

Code:

---

AND (bien.VILLEBIEN = v_villebien OR v_villebien IS NULL)

---

Comme ça, si vous passez null en second argument, vous aurez toutes les villes.

En terme d'optimisation :

1. Évitez le select *, nommez toutes les colonnes
2. La partie traitée dans le IN, ne peut-elle pas remonter en jointure ?
3. Vous avez fait une jointure externe, mais le côté externe disparait avec les filtres mis en places, vous pouvez donc remplacer votre RIGHT JOIN par un INNER JOIN.
4. Des alias facilitent la relecture
5. Une mise en page claire ne fait pas gagner en performance, mais vous gagnez en lisibilité et donc en maintenance, il ne faut pas le négliger

Ce n'est que mon avis, mais ça me paraît plus lisible !

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24

FUNCTION GET_ALL_BIEN_BY_CLIENT ( v_numclient Client.NUMCLIENT%type, v_villebien Bien.VILLEBIEN%type ) RETURN sqlcur AS pcur sqlcur;   BEGIN   open pcur FOR SELECT bn.*, -- à remplacer par toutes les colonnes eb.libelleetat FROM bien bn INNER JOIN etatbien eb ON bn.numbien = eb.numbien INNER JOIN proposer pr ON pr.numbien = bn.numbien WHERE pr.numclient = v_numclient AND ( bn.villebien = v_villebien OR v_villebien IS NULL );   RETURN pcur;   END GET_ALL_BIEN_BY_CLIENT;

---

En fonction de votre modèle de données, et notamment de la cardinalité entre les tables proposer et bien, cette fonction peut ou ne peut pas donner les mêmes résultats que la votre

Salam

Merci beaucoup Waldar ça marche bien, et merci pour les suggestions