Discussion :

[arno974]

Bonjour à tous,

Je débute avec Java et j'avoue avoir quelques difficultés. Je vous explique mon problème. Je souhaite effectuer une connexion à une base de données postgres.

J'ai donc pour cela télécharger le fichier jar disponible sur le site jdbc.postgresql. J'ai ensuite importé celui-ci via Eclipse en utilisant build path > add external Archives. Le point positif est que cela fonctionne correctement et que la connexion à ma base se fait bien. Néanmoins, je suis obligé de mettre des try/catch partout sinon eclipse refuse de compiler mon projet. J'ai donc au final plus d'une soixantaine de lignes de code pour un simple appel.

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
import java.sql.*;
public class vesselDistance {
 
    /**
     * @param args
     */
    public static void main(String[] args) {
        String wktGeom;
        wktGeom = "LINESTRING(3 4,10 50,20 25)";
 
        try {
            Class.forName("org.postgresql.Driver");
        } catch (ClassNotFoundException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        } 
        String url = "jdbc:postgresql://localhost:5432/dcns_postgis"; 
        Connection db = null;
        try {
            db = DriverManager.getConnection(url, "postgres", "");
        } catch (SQLException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
        Statement st = null;
        try {
            st = db.createStatement();
        } catch (SQLException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
        ResultSet rs = null;
        try {
            rs = st.executeQuery("SELECT type FROM geom_line WHERE gid = 4");
        } catch (SQLException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
        try {
            while (rs.next()) {
                System.out.print("Colonne 1 renvoyée ");
                System.out.println(rs.getString(1));
                System.out.println(wktGeom);
            }
        } catch (SQLException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
        try {
            rs.close();
        } catch (SQLException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
        try {
            st.close();
        } catch (SQLException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
 
    }
 
}

J'aurais donc aimé savoir si c'est la bonne manière de faire. Est-ce normal qu'Eclipse m'oblige à mettre autant de try/catch ?

J'ai également essayé en utilisant import plutôt que add external Archives mais le résultat est le même. D'ailleurs pourriez-vous m'expliquer la différence entre les deux ?

Merci pour votre aide

Arnaud

[adiGuba]

Salut,


C'est normal que tu ais à gérer ces exceptions. Java utilise le principe des checked-exceptions qui force les développeurs à devoir traiter certaines exceptions.

Par contre tu le fais très mal !!!

Pour chaque exception tu te contentes de logger l'info, puis tu continues ton programme. C'est idiot car en cas de problème tu continues quand même. Par exemple si tu n'arrives pas à te connecter à ta base de données, tu vas quand même tenter d'exécuter la requete (qui ne marchera forcement pas), et tu vas quand même lire les données (qui n'existeront pas), etc...

Bref cela va te générer tout plein d'exception pour rien.
Cela peut paraitre anodin ici, mais non seulement cela alourdit le code, mais dans des applications plus importante cela peut aussi porter à confusion : si tu as une exception tu peux arriver rapidement à en comprendre l'origine. Si tu en as 50 cela peut être plus compliqué.


Ce que tu dois faire, c'est d'utiliser un seul try/catch pour tout ton traitement : ici il n'y a pas de raison de vouloir le continuer si une des étapes posent problème.

De plus il faudrait utiliser des try/finally pour fermer proprement les ressources (voir la FAQ pour plus de détail).

Ce qui donnerait :

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
    public static void main(String[] args) {
        try {
            String wktGeom = "LINESTRING(3 4,10 50,20 25)";
 
            Class.forName("org.postgresql.Driver");
 
            String url = "jdbc:postgresql://localhost:5432/dcns_postgis";
            Connection db = DriverManager.getConnection(url, "postgres", "");
            try {
 
                Statement st = db.createStatement();
                try {
                    ResultSet rs = st.executeQuery("SELECT type FROM geom_line WHERE gid = 4");
                    try {
                        while (rs.next()) {
                            System.out.print("Colonne 1 renvoyée ");
                            System.out.println(rs.getString(1));
                            System.out.println(wktGeom);
                        }
                    } finally {
                        rs.close();
                    }
                } finally {
                    st.close();
                }
            } finally {
                db.close();
            }
 
        } catch (ClassNotFoundException e) {
            e.printStackTrace();
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }

A noter que l'on peut également laisser remonter l'exception, en déclarant par exemple throws SQLException dans la définition de la méthode...

J'ai également essayé en utilisant import plutôt que add external Archives mais le résultat est le même. D'ailleurs pourriez-vous m'expliquer la différence entre les deux ?

Là je ne comprend pas trop la question !? La différence entre quoi et quoi ?


a++

[arno974]

Ah merci beaucoup ! C'est beaucoup plus propre comme ça effectivement.
En fait j'avais mis autant de try/catch parce qu'Eclipse me les avait suggérée.

Concernant maintenant "Import" et "Add External Archives", j'aurais souhaité savoir la différence entre ces deux notions pour Eclipse.

Merci en tout cas pour le coup de main.

Arnaud

[adiGuba]

En fait j'avais mis autant de try/catch parce qu'Eclipse me les avait suggérée.

eclipse ne propose qu'une solution à un problème précis, mais il n'a pas une vision globale de ton application... bref il ne faut pas l'écouter aveuglément

Concernant maintenant "Import" et "Add External Archives", j'aurais souhaité savoir la différence entre ces deux notions pour Eclipse.

Le import c'est du Java et cela consiste simplement à indiquer la classe que tu veux utiliser.
Le "Add External Archives" d'eclipse sert simplement à référencer un Jar dans le classpath, afin qu'il soit accessible à la compilation.


a++

[arno974]

eclipse ne propose qu'une solution à un problème précis, mais il n'a pas une vision globale de ton application... bref il ne faut pas l'écouter aveuglément

Avec ce que tu m'as expliqué, je veux bien te croire

Concernant toujours l'Import et l'Add external archives, je pense que l'on ne parle pas de la même chose. En effet, quand je fais click droit sur mon projet, le menu déroulant qui s'affiche alors me permet de :

• soit de faire un import, où je suis alors invité à choisir entre par exemple des Archive file ou encore Project from cvs
• sinon de passer par build path puis add external jar

Si mon intuition est bonne (ce qui n'est pas gagné) add external jar travaille avec des fichiers distants contrairement à Import qui les ajoute directement dans le workspace. Ai-je tort ?

Arnaud

[adiGuba]

Ah ok c'est de la bidouille d'eclipse tout ca

Le "import" te permet d'importer des fichiers directement dans ton projet (pour les modifier par exemple), tandis qu'avec les "external jar" tu te contentes de les utiliser...

a++

[arno974]

Merci pour toutes ces infos !

@+


Arnaud