Fonction replace java

Version imprimable

01/07/2013, 16h15
cisco.nat

Fonction replace java
Bonjour
je veux éviter le problème d'apostrophe sur les champs d'une requette SQL
donc pour cela j'ai utilisé une fonction pour remplacer l'apostrophe par 2
Code:

1 2 3 4 5 6 public String DoubleGuillemets(String chaine){ chaine = Replace(chaine, "'", "''"); chaine=DoubleGuillemets(chaine); return chaine ; }
l'erreur est sur la ligne de replace() il m'affiche que la fonction doit contenir 3 string et c'est çà ce j'ai mais je sais pas quoi le problème

cannot find symbol
symbol: method Replace(String,String,String)
Merci
01/07/2013, 16h26
cisco.nat
bon j'ai essayé de faire çà aucune erreur mais plein des erreurs lorsque j'exécute
Code:

1 2 3 4 5 6 public String DoubleGuillemets(String chaine){ chaine = chaine.replace("'", "''"); chaine=DoubleGuillemets(chaine); return chaine ; }
Exception in thread "AWT-EventQueue-0" java.lang.StackOverflowError
at java.lang.String.codePointAt(String.java:689)
at java.util.regex.Pattern.compile(Pattern.java:1642)
at java.util.regex.Pattern.<init>(Pattern.java:1337)
at java.util.regex.Pattern.compile(Pattern.java:1047)
at java.lang.String.replace(String.java:2180)
01/07/2013, 16h35
Momoth

Bonjour,

Ton problème vient de ta récursivité, tu fais une boucle infinis dans le sens où tu n'atteindras jamais le return de ta fonction car tu l'as rapelle elle même avant. (Désolé je suis peut etre pas clair).

Edit : D'après ce que j'ai compris tu veux remplacer toutes les occurences "quote" par "double quote".

Peut être le code suivant peut t'aider :

public String DoubleGuillemets(String chaine)
{
chaine = chaine.replaceAll("'", "''");
return chaine ;
}

Bonne soirée.

-- Momoth --
01/07/2013, 16h40
thelvin

Par ailleurs, pour éviter les problèmes d'apostrophes et les autres problèmes dans les requêtes SQL, on utilise les PreparedStatement, qui elles, au moins, sont faites pour et pensent à tout.
01/07/2013, 16h43
cisco.nat

voulez vous dire que je dois pas faire le return??
01/07/2013, 16h45
cisco.nat

@thelvin pour le preparedstatement je peux l'utiliser dans ma requette parce que je faite un insert where not exists
01/07/2013, 16h52
ElSpopo
Citation:

Envoyé par cisco.nat

voulez vous dire que je dois pas faire le return??

Non, tu dois éviter la récursivité :
Code:

1 2 3 4 5 public String DoubleGuillemets(String chaine){ chaine = chaine.replaceAll("'", "''"); return chaine ; }
ou utiliser un prepareStatement pour ne plus avoir à te préoccuper de ces détails

Oui exactement mais mois j'ai des requettes insert where not exists donc je peux pas utiliser des preparedstatement par exemple
cette requette contient un insert simple ca marche avec le preparedstatement et c'est ce que j'avais au début
Code:

1 2 3 4 5 6 7 8 9 PreparedStatement pst_ci=conn.prepareStatement("INSERT INTO ci(VLAN,client,JR,vrf,address) VALUES(?,?,?,?,?)"); pst_ci.setInt(1,srv.getvlan()); pst_ci.setString(2,convertNullToEmptyString(srv.getdesc())); pst_ci.setString(3,convertNullToEmptyString(srv.getjr())); pst_ci.setString(4,convertNullToEmptyString(srv.getvrf())); pst_ci.setString(5,convertNullToEmptyString(srv.getaddress())); pst_ci.executeUpdate(); }
Mais comment je vais faire avec des insert where not exists qui contiet des where comme çà:
Code:

1 2 3 4 "INSERT INTO tout(VLAN,client)\n" + "SELECT DISTINCT '"+srv.getvlan()+"','"+DoubleGuillemets(srv.getdesc())+"' FROM dual WHERE NOT EXISTS \n" + "(SELECT VLAN,client FROM tout WHERE VLAN = '"+srv.getvlan()+"' AND client='"+DoubleGuillemets(srv.getdesc())+"')" ;

01/07/2013, 17h00
cisco.nat

Bon le replaceAll çà marche pas :(
01/07/2013, 17h12
ElSpopo
Hmm, après vérification de la javadoc, le replace remplace déjà toutes les occurrences et le replaceAll servirait à traiter des regex.

donc ça devrait fonctionner avec replace et sans appel récursif:
Code:

1 2 3 4 5 public String DoubleGuillemets(String chaine){ chaine = chaine.replace("'", "''"); return chaine ; }
01/07/2013, 17h15
cisco.nat

non c'est ce que j'ai fait eu début çà marche pas malheureusement :(
01/07/2013, 17h22
ElSpopo
Peux tu nous donner un exemple d'entrée et de sortie ou ça ne fonctionne pas de la fonction telle quelle:
Code:

1 2 3 4 5 public String DoubleGuillemets(String chaine){ chaine = chaine.replace("'", "''"); return chaine ; }
afin de voir ou ce situe le problème.

INSERT INTO tout(VLAN,client)
SELECT DISTINCT '7',' : M_CPE
' FROM dual WHERE NOT EXISTS
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at swing.bdd.DoubleGuillemets(bdd.java:30)
at swing.bdd.jButton2ActionPerformed(bdd.java:376)
at swing.bdd.access$300(bdd.java:26)
at swing.bdd$4.actionPerformed(bdd.java:125)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2018)
at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2341)
at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:402)
at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:252)
at java.awt.Component.processMouseEvent(Component.java:6505)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3321)
at java.awt.Component.processEvent(Component.java:6270)
at java.awt.Container.processEvent(Container.java:2229)
at java.awt.Component.dispatchEventImpl(Component.java:4861)
at java.awt.Container.dispatchEventImpl(Container.java:2287)
at java.awt.Component.dispatchEvent(Component.java:4687)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4832)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4492)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4422)
at java.awt.Container.dispatchEventImpl(Container.java:2273)
at java.awt.Window.dispatchEventImpl(Window.java:2719)
at java.awt.Component.dispatchEvent(Component.java:4687)
at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:723)
at java.awt.EventQueue.access$200(EventQueue.java:103)
at java.awt.EventQueue$3.run(EventQueue.java:682)
at java.awt.EventQueue$3.run(EventQueue.java:680)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:87)
at java.awt.EventQueue$4.run(EventQueue.java:696)
at java.awt.EventQueue$4.run(EventQueue.java:694)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:693)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:244)
(SELECT VLAN,client FROM tout WHERE VLAN = '7' AND client=' : M_CPE

voilà la requette :
Code:

1 2 3 4 "INSERT INTO tout(VLAN,client)\n" + "SELECT DISTINCT '"+srv.getvlan()+"','"+DoubleGuillemets(srv.getdesc())+"' FROM dual WHERE NOT EXISTS \n" + "(SELECT VLAN,client FROM tout WHERE VLAN = '"+srv.getvlan()+"' AND client='"+DoubleGuillemets(srv.getdesc())+"')" ;

01/07/2013, 17h32
ElSpopo

Une nullPointerexception.

C'est que tu as probablement passé une valeur nulle à ta fonction DoubleGuillemets.

Ton problème se situe donc dans l'appel de cette fonction.
Il est probable que cet appel : srv.getdesc() te renvoie parfois null.

Bon j'ai fait une fonction qui convertis le null to string

Code:

1
2
3
4
5
6
7
8
9
 
public static String convertNullToEmptyString(String data){
	    if(data!=null){
			return data;
		}
		else{
			return "";
		}
	}

et j'ajoute dans la requette ceci:

Code:

1
2
3
4
 
"INSERT INTO tout(VLAN,client)\n" +
"SELECT DISTINCT '"+srv.getvlan()+"','"+convertNullToEmptyString(DoubleGuillemets(srv.getdesc()))+"' FROM dual WHERE NOT EXISTS \n" +
"(SELECT VLAN,client FROM tout WHERE VLAN = '"+srv.getvlan()+"' AND client='"+convertNullToEmptyString(DoubleGuillemets(srv.getdesc()))+"')" ;

Mais encore les mêmes erreurs :s

01/07/2013, 17h50
ElSpopo

C'est normal, tu appelle ta fonction pour convertir null en string après la fonction doubleGuillemet alors que tu devrais l'appeler avant.

fait

DoubleGuillemets(convertNullToEmptyString(srv.getdesc()))

au lieu de

convertNullToEmptyString(DoubleGuillemets(srv.getdesc()))
01/07/2013, 17h56
cisco.nat

Enfin çà marche Merci bien et désolé :D
01/07/2013, 23h16
thelvin

Cool que tu aies enfin réussi à faire marcher String.replace(). Comme quoi tout arrive.

Mais je maintiens qu'il fallait utiliser PreparedStatement.

Citation:

Envoyé par cisco.nat

Mais comment je vais faire avec des insert where not exists qui contiet des where comme çà:

Exactement de la même manière qu'on utilise toujours PreparedStatement. Il n'existe aucun problème.

Tu dis toujours "ça marche pas" mais tu le prouves jamais. Et en l'occurrence tu risques pas de le prouver puisque ça marche parfaitement.
02/07/2013, 15h18
cisco.nat

si çà marche dans les requettes simples INSERT UPDATE
Mais mois j'ai des Where not exists
Mais je vais essayé le maximum parce que c'est professionnel et empeche d'avoire des erreurs
:D
02/07/2013, 17h04
thelvin

Ça marche partout. Aucune différence.

Et je t'ai déjà dit de montrer pourquoi tu penses le contraire. Normalement un informaticien devrait faire ça sans que je lui demande. Ce n'est tout de même pas compliqué.

Le voilà :D

Code:

1
2
3
4
5
6
 
PreparedStatement pst_ci=conn.prepareStatement("INSERT INTO tout(VLAN,client) "
        + "SELECT DISTINCT '"+srv.getvlan()+"','"+DoubleGuillemets(convertNullToEmptyString(srv.getdesc()))+"' FROM dual WHERE NOT EXISTS " 
+" (SELECT VLAN,client FROM tout WHERE VLAN = '"+srv.getvlan()+"')");
 
	pst_ci.executeUpdate();

02/07/2013, 18h12
thelvin

Et y a-t-il quelque chose, là-dedans, qui te faisais penser que les PreparedStatement ça marcherait pas ?

(Et est-ce que je pourrais arrêter un peu de te tenir la main ? Pourquoi est-ce que tu ne fais tout ça qu'après que je te le demande ?)

non je l'ai fait juste j'étais conne j'avais ajouté même cettes lignes:
Code:

1 2 3 4 5 6 pst.setInt(1,srv.getvlan()); pst.setString(2,convertNullToEmptyString(srv.getdesc())); pst.setString(3,convertNullToEmptyString(srv.getjr())); pst.setString(4,convertNullToEmptyString(srv.getvrf())); pst.setString(5,convertNullToEmptyString(srv.getaddress()));
Mais c'est bien de nous prendre la main de temps en temps on est là pour apprendre :)

Mais vraiment je suis spécialiste networking alors que j'ai ce projet en java avec telnet donc j'essaie en même temps de découvrir le java le jdbc et de programmer mon système
Merci Bien