L'amélioration de la regex pour l'analyse de YouTube / Vimeo Url

J'ai fait une fonction (en JavaScript) qui prend une URL à partir soit sur YouTube ou Vimeo. Il en déduit le fournisseur et l'ID de cette vidéo (démo: http://jsfiddle.net/csjwf/).

function parseVideoURL(url) {

    var provider = url.match(/http:\/\/(:?www.)?(\w*)/)[2],
        id;

    if(provider == "youtube") {

        id = url.match(/http:\/\/(?:www.)?(\w*).com\/.*v=(\w*)/)[2];
    } else if (provider == "vimeo") {

        id = url.match(/http:\/\/(?:www.)?(\w*).com\/(\d*)/)[2];
    } else {
        throw new Error("parseVideoURL() takes a YouTube or Vimeo URL");    
    }
    return {
        provider : provider,
        id : id
    }
}

Cela fonctionne, cependant, comme une regex Novice, je suis à la recherche de moyens de l'améliorer. L'entrée, je fais face à, ressemble généralement à ceci:

http://vimeo.com/(id)
http://youtube.com/watch?v=(id)&blahblahblah.....

1) Droit maintenant, je suis en train de faire trois matches différents, ne serait-il judicieux d'essayer de la faire tout en une seule expression? Si oui, comment?

2) les correspondances existantes à être plus concis? Sont-ils inutilement complexe? ou peut-être insuffisant?

3) existe-il des YouTube ou Vimeo URL qui serait manquer d'être analysée? J'ai essayé un peu et pour l'instant il semble fonctionner assez bien.

Pour résumer: je suis simplement à la recherche de moyens d'améliorer la fonction ci-dessus. Tout conseil est le bienvenu.

OriginalL'auteur Daniel | 2011-04-10

10

Je ne suis pas sûr au sujet de votre question 3), mais à condition que votre induction sur l'url formes sont correctes, les regexes peuvent être combinés en un seul, comme suit:
```
/http:\/\/(?:www.)?(?:(vimeo).com\/(.*)|(youtube).com\/watch\?v=(.*?)&)/
```
Vous obtiendrez le match sous différentes positions (1er et 2ème matchs si vimeo, 3ème et 4ème matchs de si youtube), donc vous avez juste besoin de gérer cela.

Ou, si vous êtes tout à fait sûr que vimeo id uniquement des chiffres, alors vous pouvez faire:
```
/http:\/\/(?:www.)?(vimeo|youtube).com\/(?:watch\?v=)?(.*?)(?:\z|&)/
```
et le fournisseur et le code d'identification de la apprear en vertu de la 1ère et de la 2ème match, respcetively.

Voir stackoverflow.com/a/5831191/208457 pour une pleine YouTube analyseur

OriginalL'auteur sawa

Voici ma tentative à l'expression régulière, qui couvre la plupart des mises à jour de cas:

function parseVideo(url) {
    //- Supported YouTube URL formats:
    //  - http://www.youtube.com/watch?v=My2FRPA3Gf8
    //  - http://youtu.be/My2FRPA3Gf8
    //  - https://youtube.googleapis.com/v/My2FRPA3Gf8
    //- Supported Vimeo URL formats:
    //  - http://vimeo.com/25451551
    //  - http://player.vimeo.com/video/25451551
    //- Also supports relative URLs:
    //  - //player.vimeo.com/video/25451551

    url.match(/(http:\/\/|https:\/\/|)(player.|www.)?(vimeo\.com|youtu(be\.com|\.be|be\.googleapis\.com))\/(video\/|embed\/|watch\?v=|v\/)?([A-Za-z0-9._%-]*)(\&\S+)?/);
    var type = null;
    if (RegExp.$3.indexOf('youtu') > -1) {
        type = 'youtube';
    } else if (RegExp.$3.indexOf('vimeo') > -1) {
        type = 'vimeo';
    }

    return {
        type: type,
        id: RegExp.$6
    };
}

juste une petite amélioration: (http:\/\//https:\/\/|) pour des choses comme vimeo.com/12143242
Merci pour la suggestion, l'ont intégré.

OriginalL'auteur Yangshun Tay

Regex est merveilleusement laconique, mais peut rapidement devenir compliqué.

http://jsfiddle.net/8nagx2sk/

function parseYouTube(str) {
    //link : //youtube.com/watch?v=Bo_deCOd1HU
    //share : //youtu.be/Bo_deCOd1HU
    //embed : //youtube.com/embed/Bo_deCOd1HU

    var re = /\/\/(?:www\.)?youtu(?:\.be|be\.com)\/(?:watch\?v=|embed\/)?([a-z0-9_\-]+)/i; 
    var matches = re.exec(str);
    return matches && matches[1];
}

function parseVimeo(str) {
    //embed & link: http://vimeo.com/86164897

    var re = /\/\/(?:www\.)?vimeo.com\/([0-9a-z\-_]+)/i;
    var matches = re.exec(str);
    return matches && matches[1];
}

Parfois simple code est plus agréable à vos collègues développeurs.

https://jsfiddle.net/1dzb5ag1/

//protocol and www neutral
function getVideoId(url, prefixes) {
  var cleaned = url.replace(/^(https?:)?\/\/(www\.)?/, '');
  for(var i = 0; i < prefixes.length; i++) {
    if (cleaned.indexOf(prefixes[i]) === 0)
      return cleaned.substr(prefixes[i].length)
  }
  return undefined;
}

function getYouTubeId(url) {
  return getVideoId(url, [
    'youtube.com/watch?v=',
    'youtu.be/',
    'youtube.com/embed/',
    'youtube.googleapis.com/v/'
  ]);
}

function getVimeoId(url) {
  return getVideoId(url, [
    'vimeo.com/',
    'player.vimeo.com/'
  ]);
}

Qui préférez-vous mettre à jour?

OriginalL'auteur Jason Sebring

7

Voici ma regex

http://jsfiddle.net/csjwf/1/

Il ne fonctionne pas pour des url de la forme http://www.youtube.com/watch?v=My2FRPA3Gf8. Je reçois Object {provider: "youtube", id: "My2FRPA3Gf"}. Avis le 8 manquants dans l'id.

OriginalL'auteur Ming-Tang
6

sur sawa réponse :

une petite mise à jour sur la deuxième regex :
```
/http:\/\/(?:www\.)?(vimeo|youtube)\.com\/(?:watch\?v=)?(.*?)(?:\z|$|&)/
```
(s'échapper les points empêche l'url correspondante de type www_vimeo_com/... et $ ajoutée...)

ici est la même idée pour la mise en correspondance de l'intégration url :
```
/http:\/\/(?:www\.|player\.)?(vimeo|youtube)\.com\/(?:embed\/|video\/)?(.*?)(?:\z|$|\?)/
```
oh ! et le lien du formulaire : /^http:\/\/youtu\.be\/(.+?)(?:|$|\?)/
cette expression n'est pas assez bon, il correspond à http://player.youtube.com/sdfsdfdf qui n'est pas valide url youtube

OriginalL'auteur Romain

Pour Vimeo, Ne pas compter sur Regex comme Vimeo tend à changer/mettre à jour leur modèle d'URL de chaque maintenant et puis. Comme de 2 octobre, en 2017, il y a au total six schémas d'URL Vimeo prend en charge.

https://vimeo.com/*
https://vimeo.com/*/*/video/*
https://vimeo.com/album/*/video/*
https://vimeo.com/channels/*/*
https://vimeo.com/groups/*/videos/*
https://vimeo.com/ondemand/*/*

Au lieu de cela, l'utilisation de leur API pour valider vimeo Url. Voici ce oembed, il (doc) de l'API qui prend une URL, vérifie sa validité et de retourner un objet avec des tas d'informations vidéo(consultez le dev de la page). Bien que n'étant pas prévu mais on peut facilement l'utiliser pour valider si une URL est de Vimeo ou pas.

Donc, avec l'ajax, il devrait ressembler à cela,

var VIMEO_BASE_URL = "https://vimeo.com/api/oembed.json?url=";
var yourTestUrl = "https://vimeo.com/23374724";


$.ajax({
  url: VIMEO_BASE_URL + yourTestUrl,
  type: 'GET',
  success: function(data) {
    if (data != null && data.video_id > 0)
      //Valid Vimeo url
    else
      //not a valid Vimeo url
  },
  error: function(data) {
    //not a valid Vimeo url
  }
});

OriginalL'auteur fluffyBatman

1

3) Votre regex ne correspond pas à l'url https. Je n'ai pas testé, mais je suppose que la "http://" serait devenu "http(s)?://". Notez que ce serait changer les positions correspondantes du fournisseur et id.

OriginalL'auteur vrijdenker

Juste au cas où voici une version de php

/*
* parseVideo
* @param (string) $url 
* mi-ca.ch 27.05.2016
* parse vimeo & youtube id
* format url for iframe embed 
* https://regex101.com/r/lA0fP4/1
*/

function parseVideo($url) {
  $re = "/(http:|https:|)\\/\\/(player.|www.)?(vimeo\\.com|youtu(be\\.com|\\.be|be\\.googleapis\\.com))\\/(video\\/|embed\\/|watch\\?v=|v\\/)?([A-Za-z0-9._%-]*)(\\&\\S+)?/"; 
preg_match($re, $url, $matches);

if(strrpos($matches[3],'youtu')>-1){
    $type='youtube';
    $src='https://www.youtube.com/embed/'.$matches[6];
}else if(strrpos($matches[3],'vimeo')>-1){
    $type="vimeo";
    $src='https://player.vimeo.com/video/'.$matches[6];
}else{
    return false;
}


return array(
         'type' =>  $type //return youtube or vimeo
        ,'id'   =>  $matches[6] //return the video id
        ,'src'  =>  $src //return the src for iframe embed
        );
}

OriginalL'auteur mica

Vous devez vous connecter pour publier un commentaire.