Discussion :

[fabou3377]

Bonjour, au vu du code ci-dessous le système de me générer oui1 selon la condition... Le paramètre est bien récupérer mais la condition n'est pas respectée selon le système ... ça paraît extrêment simple et je ne sais pas ce que je fais de faux....

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
 
  <form  name="order" method="post" >
                    <tr>
                        <td>Order No.
                        <% 
                            String orderBy = (String) request.getParameter("orderBy");
                           
                            if(orderBy=="1")
                            {
                            out.println("oui");
                            out.println(orderBy);
                            }
                            else
                            {
                            out.println("non");
                             out.println(orderBy);
                            }    
                        
                            
                        %>
                    <input type="image" src="images/icone_up.png" onclick="document.order.orderBy.value='1'"/>
 
 
                    </td>
                        <td>frac.</td>
                        <td>Search Key</td>
                        <td>Qty</td>
                        <td>Product</td>
                        <td>Color</td>
                        <td>Due Date</td>
                        <td>Update
                    <input id="orderBy" type="hidden" name="orderBy" value="<%=request.getParameter("orderBy")%>">
                    <input id="orderType" type="hidden" name="orderType" value="<%=request.getParameter("orderType")%>">
                    </td>
                    </tr>
                    </form>

[kore62]

essai Orderby.equals("1") les string se compare avec String.equals(String) si tu fait orderby == 1 tu compare les references ce sera toujours false

[kore62]

Si c'est bon marque le topic resolu

[mixouille]

Exact.

Je conseillerais ceci dit plutôt l'inverse :

Code :

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

"1".equals(orderBy)

car, si "orderBy" est nul, ca te renverra "false" (ce qui est normal), alors que orderBy.equals("1") te renverra ... une NullPointerException (car orderBy est nul !).
Tandis que "1" n'est, jusqu'à preuve du contraire, jamais nul...

[kore62]

dsl mixouille je croyais bien faire..

[mixouille]

Ola, pas ce genre de post avec moi, ta réponse était très bien, je n'ai fait qu'ajouter une précision d'ordre général (mais visiblement utile pour ce cas).

Si tu crois que je n'ai dit que des choses vraies sur ce forum (ou d'autres), fais une recherche (ceci dit j'ai réussi jusqu'ici à me tenir sur ce forum-ci )

Pour conclure :
Le plus explicite reste de coder :

Code :

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

if (orderBy != null && orderBy.equals("1"))

C'est plus lourd, mais on comprend mieux le "vrai" test.

[fabou3377]

Merci Les gars!! ça marche nickel... Ah les finesses de Java quand on débute...

[kore62]

y a pas de malaise

[mixouille]

Ah les finesses de Java quand on débute...

Ceci dit, je préfère les finesses des String java aux char* du C++ ...