pb syntaxe switch()

Version imprimable

pb syntaxe switch()

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
public MakeConnection(String type, ArrayList<ConnecInfo> listHops)
        {
if(type.contains("SSH"))
            {
                System.out.println("MakeConnection using a list of hops : type = SSH");
                this.auditClient = new AuditSSHClient(listHops);
            }
            else
            {
                if(type.contains("TELNET"))
                {
                    System.out.println("MakeConnection using a list of hops : type = TELNET");
                    this.auditClient = new AuditTelnetClient(listHops);
                }
                else
                {
                    System.out.println("MakeConnection using a list of hops : type = NULL -> TELNET");
                    this.auditClient = new AuditTelnetClient(listHops);
                }
            }
}

Bonjour, je suis débutant en java et je tombe sur un problême de syntaxe avec un switch(). J'aimerai transformer ce qui est au dessus pour utiliser un switch puisque j'ai d'autres cas à envisager. Je tombe sur un problême : il ne veeut pas du switch(type) ?

Merci de votre aide :P

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
public MakeConnection(String type, ArrayList<ConnecInfo> listHops)
        {
 
            switch(type)
            {
                case "SSH" : 
                    System.out.println("MakeConnection using a list of hops : type = SSH");
                    this.auditClient = new AuditSSHClient(listHops);
                    break;
                case "TELNET" :
                    System.out.println("MakeConnection using a list of hops : type = TELNET");
                    this.auditClient = new AuditTelnetClient(listHops);
                    break;
                case "ATDT" :
                    System.out.println("MakeConnection using a list of hops : type = ATDT");
                    break;
                default :
                    System.out.println("MakeConnection using a list of hops : type = TELNET");
                    this.auditClient = new AuditTelnetClient(listHops);
                    break; 
            }

En Java, switch ne peut être évalué qu'avec des entiers numériques (int, char, etc...).
Tu devrais donc avoir ceci :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 public MakeConnection(String type, ArrayList<ConnecInfo> listHops) { if("SSH".equals(type) { [...] } else if("TELNET".equals(type) { [...] } else if("ATDT".equals(type)) { [...] } else { [...] } }

17/10/2007, 10h12
nolofinwe
bonjour,

a ta place et meme si c'est bourrin j'assocerai a chaque "valeur" de type un nombre entiers que je testerai avec le switch (tu stoque le nombre dans une variable int...)

tu n'auras pas de probleme avec le switch si tu test un entier ;)
Code:

1 2 3 4 5 6 7 8 9 10 int i ; if (type == "UNTEL") {i=0;} if (type == "UNTEL1") {i=1;} ... switch(i){ case 0: ... case 1: ...
mais bon c'est surement pas la meilleure solution mais c'est ce que je ferai;)

a +
17/10/2007, 10h14
Duc Lebowski

Et si tu t'intéresses aux Design Patterns, je sens gros comme une maison le Pattern Strategy !

Comme ça hop, plus de if ... elseif !

C'est un exemple idéal pour s'y entraîner.
17/10/2007, 10h34
babap1

pas mal tous les renseignements, un grand merci... :king:
Est-ce que que peux avoir qques renseignements sur le Pattern Strategy pour que je fasse mon choix parmis toutes vos solutions?

Merci, a+
17/10/2007, 10h39
nolofinwe

voila un petit lien mais j'avoue que j'ai pas compris grand chose je dois pas avoir le niveau pour ca je pense :aie:

http://pcaboche.developpez.com/artic...ge=page_2#L1.3

bonne chance pour la suite
17/10/2007, 11h06
Duc Lebowski

Cela peut être intéressant :

1) Quel est le but final de ta classe ?
2) Que fais ton "auditClient" ? Est-ce une interface ?
3) Ton "type" est choisi par l'utilisateur ? Par un fichier de conf ?
17/10/2007, 11h26
babap1

1/ Ma class créer des connections vers des serveurs telephonique avec différents protocols : telnet/ssh/atdt

2/
Connection du Software a server distant.
Disconnection du server distant.
Envoi d'une instruction sur le server distant.
Check si la connection au server fonctionnne.
Connection du Software a un bond (Telnet via SSH ou ATDT via Telnet)

3/ Mon type est choisis par un fichier de conf. client. Une feuille excel qui comprend : (protocol, port, login, mdp, prompts...)

Sinon tu peux utiliser les enums qui sont utilisables dans un switch

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
25
public MakeConnection(Type type, ArrayList<ConnecInfo> listHops) {
    System.out.println("MakeConnection using a list of hops : type = " + type);
    switch(type) {
        case SSH : {
            this.auditClient = new AuditSSHClient(listHops);
            break;
        }
        case TELNET : {
            this.auditClient = new AuditTELNETClient(listHops);
            break;
        }
        case ATDT : {
            this.auditClient = new AuditATDTClient(listHops);
            break;
        }
        default : {
            this.auditClient = new AuditTelnetClient(listHops);
            break; 
        }
    }
}
 
public enum Type {
    SSH, ATDT, TELNET;
}

Ah oui donc oublies la stratégie.

Vu que je suis parti dans les délires, tu peux faire un truc comme ça aussi :
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 25 26 // main : IAuditClient client = ConnectionFactory.makeConnection("SSH", listHops); public class ConnectionFactory { private static Map<String, Class> types = new HashMap<String, Class>(); static { types.put("SSH", AuditSSHClient.class); types.put("TELNET", AuditTELNETClient.class); types.put("ATDT", AuditATDTClient.class); } public static IAuditClient makeConnection(String type, List<ConnectInfo> listHops) { IAuditClient client = null; try { client = (IAuditClient)types.get(type).newInstance(); client.connect(listHops); } catch (InstantiationException e) { e.printStackTrace(); } catch (IllegalAccessException e) { e.printStackTrace(); } return client; } }
Enfin c'est juste pour montrer qu'il y a beaucoup de possibilités...

Le but final de ma démarche est de montrer qu'après tu peux par exemple définir un fichier properties qui dit :

auditclients.properties :
==================
SSH=AuditSSHClient.class
TELNET=AuditTELNETClient.class
...

Et le jour où tu as un AuditFTPClient à intégrer tu as juste à ajouter "FTP=AuditFTPClient.class" pour que dynamiquement il soit injecté, sans avoir à toucher la moindre ligne de code....