Discussion :

[cisco.nat]

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 :

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

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

[cisco.nat]

bon j'ai essayé de faire çà aucune erreur mais plein des erreurs lorsque j'exécute

Code :

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

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)

[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 --

[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.

[cisco.nat]

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

[cisco.nat]

@thelvin pour le preparedstatement je peux l'utiliser dans ma requette parce que je faite un insert where not exists

[ElSpopo]

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

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

Code java :

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

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

[cisco.nat]

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 :

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

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 :

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

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())+"')" ;

[cisco.nat]

Bon le replaceAll çà marche pas

[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 :

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

1
2
3
4
5
 
public String DoubleGuillemets(String chaine){
  chaine = chaine.replace("'", "''");
return chaine ;
}

[cisco.nat]

non c'est ce que j'ai fait eu début çà marche pas malheureusement

[ElSpopo]

Peux tu nous donner un exemple d'entrée et de sortie ou ça ne fonctionne pas de la fonction telle quelle:

Code :

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

1
2
3
4
5
 
public String DoubleGuillemets(String chaine){
  chaine = chaine.replace("'", "''");
return chaine ;
}

afin de voir ou ce situe le problème.

[cisco.nat]

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 :

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

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())+"')" ;

[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.

[cisco.nat]

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

Code :

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

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 :

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

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

[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()))

[cisco.nat]

Enfin çà marche Merci bien et désolé

[thelvin]

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

Mais je maintiens qu'il fallait utiliser PreparedStatement.

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.

[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

[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é.