Discussion :

[Sh4dow49]

(re)Bonjour,

désolé de poster aussi souvent sur ce sujet mais je ne trouve pas ce que je veux sur internet donc je sollicite votre aide pour m'aider

En fait cette fois je voudrais vérifier la validité des url saisies

ma regexp donne ceci :

Code :

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

var urlTest = /^http(s)?:\/\/[-[:alnum:]]+\.[-[:alnum:]]+\.[a-zA-Z]{2, 4}(:[0-9]+)?/;

mais le test renvoie false à chaque fois

Code :

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

1
2
3
4
if (urlTest.test(document.addApplicationForm.url.value) == false)
{
        alert("email invalide");
}

Cette fois je ne trouve rien qui m'aide.

Merci d'avance pour votre aide

[Bovino]

Ce serait pas plutôt :

Code :

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

var urlTest = /^http[s]?:\/\/[-[:alnum:]]+\.[-[:alnum:]]+\.[a-zA-Z]{2, 4}(:[0-9]+)?/;

[Sh4dow49]

Ce serait pas plutôt :

Code :

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

var urlTest = /^http[s]?:\/\/[-[:alnum:]]+\.[-[:alnum:]]+\.[a-zA-Z]{2, 4}(:[0-9]+)?/;

J'ai essayé mais ca n'a pas marché non plus (j'ai voulu modifier mais on a eu une coupure internet au même moment du coup tu as répondu avant la modification ! )

Tu n'aurais pas une autre idée dans la poche par hasard ? (ou dans la tête, au choix)

[Auteur]

Code :

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

[:alnum:]

A ma connaissance, javascript ne prend pas en charge ces mots-clefs dans les expressions régulières. Remplace le par son équivalent.

[Bovino]

Désolé, les regexp, c'est pas ma spécialité, mais j'ai trouvé ça qui semble marcher ???

Code :

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

var regexp = /(http|https):\/\/(\w+:{0,1}\w*@)?(\S+)(:[0-9]+)?(\/|\/([\w#!:.?+=&%@!\-\/]))?/

[Sh4dow49]

j'ai donc essayé

Code :

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

var urlTest = /^http[s]?:\/\/[-[a-zA-Z0-9]]+\.[-[a-zA-Z0-9]]+\.[a-zA-Z]{2, 4}(:[0-9]+)?/;

mais rien ne change j'ai toujours false avec un test sur http://www.google.fr (qui est une url valide je pense )

[Auteur]

Code :

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

/^http[s]?:\/\/[-[a-zA-Z0-9]]+\.[-[a-zA-Z0-9]]+\.[a-zA-Z]{2, 4}(:[0-9]+)?/

tu peux m'expliquer ton expression régulière ? Plusieurs points me paraissent étrange . Par exemple :

• Code :

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

  [-[a-zA-Z0-9]]+

• Code :

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

  (:[0-9]+)?

[edit]
Bovino :

Code :

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

(http|https)

attention cette expression régulière ne veut pas dire "http" ou "https" mais prend en compte :
- httpttps
- htthttps
Tu fais un ou entre le "p" de http et le "h" de https. Il aurait fallu écrire :

Code :

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

((http)|(https))

[/edit]

[Sh4dow49]

Désolé, les regexp, c'est pas ma spécialité, mais j'ai trouvé ça qui semble marcher ???

Code :

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

var regexp = /(http|https):\/\/(\w+:{0,1}\w*@)?(\S+)(:[0-9]+)?(\/|\/([\w#!:.?+=&%@!\-\/]))?/

Cette regexp marche un peu mieux mais si je fais un test sur http://arf ça passe quand même. Egalement je vois la présence d'@, je ne suis pas sur que ce symbole puisse etre compris dans une url valide


En fait j'essaye depuis ce midi de construire une regexp qui accepterai les url qui commenceraient par

http://(nomServeur.extensionDNS | @ip)(:port si besoin)

[Sh4dow49]

Code :

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

[-[a-zA-Z0-9]]+

en fait l'expression que j'ai montré était tirée d'un site et elle m'avais semblée correspondre à mes besoins je m'en suis donc servi comme base

Code :

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

(:[0-9]+)?

pour identifier le port si besoin

[Bovino]

Cette regexp marche un peu mieux mais si je fais un test sur http://arf ça passe quand même. Egalement je vois la présence d'@, je ne suis pas sur que ce symbole puisse etre compris dans une url valide

A la base, cette expression accepte les url ftp, d'où la présence du @, ceci dit, j'ai pas regardé en détail non plus...

[Sh4dow49]

J'ai modifié la rexexp elle est maintenant :

Code :

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

var urlTest = /(http|https):\/\/(\w+:{0,1}\w*)?(:[0-9]+)?(\/|\/([\w?=&\-\/]))?/;

mais http:// suivi de n'importe quoi marche.

Je ne vois vraiment pas

Avez vous d'autres idées de votre coté ?

[Matthieu2000]

J'ai modifié la rexexp elle est maintenant :

Code :

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

var urlTest = /(http|https):\/\/(\w+:{0,1}\w*)?(:[0-9]+)?(\/|\/([\w?=&\-\/]))?/;

mais http:// suivi de n'importe quoi marche.

Je ne vois vraiment pas

Avez vous d'autres idées de votre coté ?

avec les nouvelles domaines http:// est suivi par n'importe quoi!

[Sh4dow49]

Donc en fait si je fais une simple regexp du genre

Code :

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

var urlTest = /^(http|https):\/\//;

ca suffirait ? (just vérifier qu'on commence l'url correctement)

[Auteur]

[edit]

Code :

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

(http|https)

attention cette expression régulière ne veut pas dire "http" ou "https" mais prend en compte :
- httpttps
- htthttps
Tu fais un ou entre le "p" de http et le "h" de https. Il aurait fallu écrire :

Code :

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

((http)|(https))

[/edit]

[Oscar Hiboux]

Aïe, Auteur n'est pas content, il a le poil qui s'hérissonne !

[Bovino]

Oui... peut-être parce qu'il a raison...

[Sh4dow49]

oui je n'avais pas vu l'edit non plus il a bien fait de faire un rappel ^^

[Sh4dow49]

et donc pour en revenir à ma question : pensez vous qu'un test sur juste ((http)|(https)):\/\/ suffise à valider l'url ? il est vrai qu'on ne peut pas trop savori quelle tête elle aura mais on a au moins une idée (c'est tout le temps du genre http(s)://blabla.blabla(.blabla)(:port)(/bla)etcetc...)

Seulement comme je n'arrive pas a construire la regexp qui traduit ca je pose cette question quand même.

[Auteur]

quand on voit les différentes écritures...

Pour développez tu as ceci par exemple :

Code x :

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

1
2
3
4
5
6
http://www.developpez.com
http://chat.developpez.com
http://developpez.net/forums
http://www.developpez.net/forums
http://auteur.developpez.com (site des rédacteurs)

sans compter les extensions :
.com
.net
.gouv.fr
.org
etc...

donc à moins de filtrer des noms domaines particuliers, tu peux te contenter de

Code :

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

((http)|(https)):\/\/

Ceci dit quand tu tapes une url le navigateur ajoute automatiquement le http ou https.

Maintenant si je revient à ton premier post, tu réalises ce test :

Code :

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

1
2
3
4
if (urlTest.test(document.addApplicationForm.url.value) == false)
{
        alert("email invalide");
}

Tu veux tester la valider d'une adresse mail ? ou l'url d'un site ?

Je n'ai pas très bien compris le but de ton expression régulière

[Sh4dow49]

c'était bien pour tester une url, j'ai déjà réussi à trouver un test à peu près correct au niveau des emails (seule la structure est vérifiée, une adresse email du genre aaaa@aaa.aa marche quand même mais sa force à avoir la structure déjà ... )

En ce qui concerne l'url il est vrai que le naviguateur ajoute automatiquement le http/https mais ici j'enregistre l'url en base pour ensuite la réutiliser dans l'appli, je voulais donc une url complète dans la base pour ne pas prendre le risque qu'il y ai des bugs à cause du fait que l'utilisateur ne mette qu'un morceau d'url ...