Skip to main content
se eliminaron 793 caracteres en el cuerpo
Origen Enlace
Pablo Lozano
  • 49.9k
  • 8
  • 55
  • 92

SePor un lado se me ocurre que podrías agrupar en una estructura los datos y recorrerla buscando coincidencias. De ese modo, si te aparecen más excepciones, sólo tendrías que modificar la estructura (un documento JSON, por ejemplo):

const DATOS= [
  {
    currentSong: 'Walk Away',
    currentArtist:'Alle Farben & James Blunt',
    currentCleanSong = 'Walk Away (Feat. James Blunt)';
    currentCleanArtist = 'Alle Farben & James Blunt';
    
  },
  {
    currentSong: 'On My Way',
    currentArtist:'Alan Walker, Sabrina Carpenter & Farruko',
    currentCleanSong = 'On My Way (With Sabrina Carpenter & Farruko)';
    currentCleanArtist = 'Alan Walker';
  }
];

function normalize(song, artist) {
  let clean = DATA.find(elem => elem.currentSong === song && elem.currentArtist ===artist);
  if (clean) {
    return [clean.currentCleanSong, clean.currentCleanArtist]
  } else {
    return [song,artist];
}

const refreshLyric = (currentSong, currentArtist) => {
  var vagalume_api = 'https://api.vagalume.com.br/search.php?';
  
  const [song, artist] = normalize(currentSong,cleanText(currentArtist));
        
  const xhttp = new XMLHttpRequest();
  xhttp.onreadystatechange = function() {
    if (this.readyState === 4) {
      if (this.status === 200) {
        const data = JSON.parse(this.responseText);
        if (data.type === 'exact' || data.type === 'aprox') {
          var lyric = normalizeText(data, artist);
          document.getElementById('lyric').innerHTML = lyric.replace(/\n/g, '<br />'); 
        } else {
                    //more stuff
                    //
                    //
                    //
        }
      } else {
        const page = new Page();
        page.refreshLyric2(currentSong, artist);
      }
    }
  }
  xhttp.open('GET', vagalume_api + '&apikey=' + API_KEY + '&art=' + urlencode(artist) + '&mus=' + urlencode(currentSong.toLowerCase()), true);
  xhttp.send()
}

Por otro lado, me pregunto si no intentas hilar demasiado fino: La mayoría de las búsquedas deberían funcionar con parámetros más generales: prueba a buscas uno de los artistas y el título de la canción, y luego en los resultados intenta comprobar si en los datos están los otros artistas en el título o en la lista de artistas, por ejemplo.

Se me ocurre que podrías agrupar en una estructura los datos y recorrerla buscando coincidencias. De ese modo, si te aparecen más excepciones, sólo tendrías que modificar la estructura (un documento JSON, por ejemplo):

const DATOS= [
  {
    currentSong: 'Walk Away',
    currentArtist:'Alle Farben & James Blunt',
    currentCleanSong = 'Walk Away (Feat. James Blunt)';
    currentCleanArtist = 'Alle Farben & James Blunt';
    
  },
  {
    currentSong: 'On My Way',
    currentArtist:'Alan Walker, Sabrina Carpenter & Farruko',
    currentCleanSong = 'On My Way (With Sabrina Carpenter & Farruko)';
    currentCleanArtist = 'Alan Walker';
  }
];

function normalize(song, artist) {
  let clean = DATA.find(elem => elem.currentSong === song && elem.currentArtist ===artist);
  if (clean) {
    return [clean.currentCleanSong, clean.currentCleanArtist]
  } else {
    return [song,artist];
}

const refreshLyric = (currentSong, currentArtist) => {
  var vagalume_api = 'https://api.vagalume.com.br/search.php?';
  
  const [song, artist] = normalize(currentSong,cleanText(currentArtist));
        
  const xhttp = new XMLHttpRequest();
  xhttp.onreadystatechange = function() {
    if (this.readyState === 4) {
      if (this.status === 200) {
        const data = JSON.parse(this.responseText);
        if (data.type === 'exact' || data.type === 'aprox') {
          var lyric = normalizeText(data, artist);
          document.getElementById('lyric').innerHTML = lyric.replace(/\n/g, '<br />'); 
        } else {
                    //more stuff
                    //
                    //
                    //
        }
      } else {
        const page = new Page();
        page.refreshLyric2(currentSong, artist);
      }
    }
  }
  xhttp.open('GET', vagalume_api + '&apikey=' + API_KEY + '&art=' + urlencode(artist) + '&mus=' + urlencode(currentSong.toLowerCase()), true);
  xhttp.send()
}

Por un lado se me ocurre que podrías agrupar en una estructura los datos y recorrerla buscando coincidencias. De ese modo, si te aparecen más excepciones, sólo tendrías que modificar la estructura (un documento JSON, por ejemplo):

const DATOS= [
  {
    currentSong: 'Walk Away',
    currentArtist:'Alle Farben & James Blunt',
    currentCleanSong = 'Walk Away (Feat. James Blunt)';
    currentCleanArtist = 'Alle Farben & James Blunt';
    
  },
  {
    currentSong: 'On My Way',
    currentArtist:'Alan Walker, Sabrina Carpenter & Farruko',
    currentCleanSong = 'On My Way (With Sabrina Carpenter & Farruko)';
    currentCleanArtist = 'Alan Walker';
  }
];

function normalize(song, artist) {
  let clean = DATA.find(elem => elem.currentSong === song && elem.currentArtist ===artist);
  if (clean) {
    return [clean.currentCleanSong, clean.currentCleanArtist]
  } else {
    return [song,artist];
}

Por otro lado, me pregunto si no intentas hilar demasiado fino: La mayoría de las búsquedas deberían funcionar con parámetros más generales: prueba a buscas uno de los artistas y el título de la canción, y luego en los resultados intenta comprobar si en los datos están los otros artistas en el título o en la lista de artistas, por ejemplo.

Origen Enlace
Pablo Lozano
  • 49.9k
  • 8
  • 55
  • 92

Se me ocurre que podrías agrupar en una estructura los datos y recorrerla buscando coincidencias. De ese modo, si te aparecen más excepciones, sólo tendrías que modificar la estructura (un documento JSON, por ejemplo):

const DATOS= [
  {
    currentSong: 'Walk Away',
    currentArtist:'Alle Farben & James Blunt',
    currentCleanSong = 'Walk Away (Feat. James Blunt)';
    currentCleanArtist = 'Alle Farben & James Blunt';
    
  },
  {
    currentSong: 'On My Way',
    currentArtist:'Alan Walker, Sabrina Carpenter & Farruko',
    currentCleanSong = 'On My Way (With Sabrina Carpenter & Farruko)';
    currentCleanArtist = 'Alan Walker';
  }
];

function normalize(song, artist) {
  let clean = DATA.find(elem => elem.currentSong === song && elem.currentArtist ===artist);
  if (clean) {
    return [clean.currentCleanSong, clean.currentCleanArtist]
  } else {
    return [song,artist];
}

const refreshLyric = (currentSong, currentArtist) => {
  var vagalume_api = 'https://api.vagalume.com.br/search.php?';
  
  const [song, artist] = normalize(currentSong,cleanText(currentArtist));
        
  const xhttp = new XMLHttpRequest();
  xhttp.onreadystatechange = function() {
    if (this.readyState === 4) {
      if (this.status === 200) {
        const data = JSON.parse(this.responseText);
        if (data.type === 'exact' || data.type === 'aprox') {
          var lyric = normalizeText(data, artist);
          document.getElementById('lyric').innerHTML = lyric.replace(/\n/g, '<br />'); 
        } else {
                    //more stuff
                    //
                    //
                    //
        }
      } else {
        const page = new Page();
        page.refreshLyric2(currentSong, artist);
      }
    }
  }
  xhttp.open('GET', vagalume_api + '&apikey=' + API_KEY + '&art=' + urlencode(artist) + '&mus=' + urlencode(currentSong.toLowerCase()), true);
  xhttp.send()
}