Discussion :

[charfi-dev]

Bonjour,

Je veux savoir comment je peux modifier l'expression régulière ci-dessous pour valider une url base de donnée de cette forme: jdbc:mysql://host:port/database ou jdbc:mysql://host/database

Code :

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

1
2
 
var regexp = new RegExp("^(jdbc:mysql)\://[a-zA-Z0-9\-\.]+\.[a-zA-Z]{2,3}(:[a-zA-Z0-9]*)?/?([a-zA-Z0-9\-\._\?\,\'/\\\+&%\$#\=~])*$");

Merci d'avance

[SpaceFrog]

sans plus de détails ...

Code :

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

^(jdbc:mysql)://([a-zA-Z0-9.:-]+/?)*

[iakou]

Bonjour.

Que voulez-vous faire au juste ?... ("Un problème bien exposé c'est la moitié du travail !")

Cdlt

[charfi-dev]

Merci pour vos réponses.

Le code ci-dessous m'affiche: Not a valid Database URL

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
 
function validateDatabaseUrl() {
   // var regexp = new RegExp("^(jdbc:mysql)://([a-zA-Z0-9.:-]+/?)*");
    var regexp = new RegExp("^(jdbc:postgresql)\://[a-zA-Z0-9\-\.]+\.[a-zA-Z]{2,3}(:[a-zA-Z0-9]*)?/?([a-zA-Z0-9\-\._\?\,\'/\\\+&%\$#\=~])*$");
    var url = "jdbc:mysql://128.123.44.14:8080/database_name";
    var error = "";
 
    if (!regexp.test(url)) {
        error = "Not a valid Database URL";
    } else {
        error = "Valid Database URL";
    }
    return error;
}
var x = validateDatabaseUrl();
alert(x);

Si l'expression régulière est bien formulé et matche les deux formats jdbc:mysql://host:port/database ou jdbc:mysql://host/database, normalement il doit afficher Valid Database URL
J'ai besoin de corriger mon expression pour prendre en compte les deux formats ci-dessus.

[Watilin]

Il y a un problème avec les backslashes :

Code :

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

var regexp = new RegExp("^(jdbc:postgresql)\://[a-zA-Z0-9\-\.]+\.[a-zA-Z]{2,3}(:[a-zA-Z0-9]*)?/?([a-zA-Z0-9\-\._\?\,\'/\\\+&%\$#\=~])*$");

C'est un problème classique : la regexp est déclarée indirectement, avec une chaîne. Lors de l'analyse de la chaîne, les séquences échappées sont remplacées, même celles qui n'ont pas besoin d'être échappées. Par exemple "\:" devient ":".

Quand il n'y a pas de partie dynamique dans le motif, je conseille fortement d'utiliser la forme littérale var regexp = /motif/flags;.

Dans l'exemple 128.123.44.14, la regexp ne valide pas à cause de ce passage : \.[a-zA-Z]{2,3} qui n'autorise pas de chiffres après le dernier point. D'ailleurs, aujourd'hui les tld peuvent contenir plus de 3 caractères, par exemple .museum (d'autres exemples) (à toi de juger si le cas peut se présenter dans ton application).

Pour simplifier le motif, on a le raccourci \w qui correspond à la classe [a-zA-Z0-9_]. On peut donc remplacer [a-zA-Z0-9\-\.]+ par [\w.-]+ (je pense que l'ajout du dash (_) ne fait pas de mal). Note que le point n'a pas besoin d'être échappé dans une classe, ni le tiret quand il est au bord. En fait, il y a pas mal d'échappements qu'on peut enlever puisque les deux seules choses qu'un doit obligatoirement échapper dans une classe, c'est les crochets ([, ]) et le backslash lui-même.

Le numéro de port est forcément un nombre, donc la partie correspondante devrait être (:\d+)?.

En tirant parti du fait que les parenthèses capturent par défaut, tu peux utiliser exec pour isoler les différents éléments de l'URI. J'ai repris ton motif et l'ai simplifié, essaye ça :

Code :

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

1
2
3
4
5
6
7
8
var regexp = /^jdbc:(postgresql|mysql):\/\/([\w.-]+\.\w+)(:\d+)?\/([-\w.?,'/\\+&%$#=~]+)$/;
//                  (       1        )     (      2     )( 3  )   (         4          )
// 1: moteur
// 2: nom d'hôte
// 3: port
// 4: nom de la base
 
console.log(regexp.exec("jdbc:mysql://128.123.44.14:8080/database_name"));

[iakou]

Hum !... Essayez :

Code :

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

var regexp = new RegExp("^(jdbc:mysql://)([0-9]{1,3}\.){3}[0-9]{1,3}(:8080)?/[a-z]+$") ;

Le port 8080, c'est du dur ?... Sinon vous faites (:\d+)? comme Watilin le recommande.

Idem pour le database_name, c'est quoi les contraintes ?... J'ai mis une chaine de caractères minuscules (à voir).

P.S : Attention j'ai pas eu le temps de tester ! (laisse aussi passer des IP syntaxiquement correctes mais erronées...)

[SpaceFrog]

http://jsfiddle.net/fR4av/1/

[iakou]

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
<html>
<head>
<script type="text/javascript">
     function validateDatabaseUrl()
          {
     var url = "jdbc:mysql://128.123.44.14:8080/database_name" ;
 
     var regex = new RegExp("^(jdbc:mysql://)([0-9]{1,3}\.){3}[0-9]{1,3}(:8080)?/[a-z_]+$") ;
 
     if ( ! regex.test(url) )
          alert(url + " isn't a valid Database URL") ;
     else
          alert(url + " is a valid Database URL") ;
          }
</script>
</head>
<body onload="validateDatabaseUrl()">
</body>
</html>

P.S : J'ai rajouté le "_" dans la règle pour database_name.