darkxex / RipJKAnimeNX

App de JKAnime para Nintendo Switch
44 stars 15 forks source link

Correccion de bugs #91

Closed Kronos2308 closed 3 years ago

darkxex commented 3 years ago

disculpa, podrías explicarme el cambio que hiciste de fembed? que no entendí el error q encontraste.

Kronos2308 commented 3 years ago

Si como sabes fembed tiene por lo general dos resoluciones en el vector 0 480 y en el 1 720p El problema es que estaba revisando fijo por el vector 1 y no caí en cuenta que puede ser que no tenga En cuello caso solo abría el navegador Pero es mejor el link directo o sea fallaba si no tenía más q una resolución disponible l json estaba en algunos casos devolviendo solo 480 y no lo cojia

Kronos2308 commented 3 years ago

Con el nuevo cambio si no se encuentra el link en un lado prueba el siguiente antes de fallar y abrir el navegador

darkxex commented 3 years ago

Entiendo, gracias.

Kronos2308 commented 3 years ago

Lo ideal que que revisara uno por uno para ver cuál es el de 720p y coja ese pero eso necesita un pedazo de código extra

Kronos2308 commented 3 years ago

Ya que las apis por lo general usan json es más fácil usar la librería de json de cpp

Kronos2308 commented 3 years ago

Cualquier cosa que no entiendas en el código me avisas

darkxex commented 3 years ago

sí no hay problema, es que me parecía raro lo de encontrar un error en fembed ya que en mi caso cuando escribí el script, usé un LastIndexOf para que busque el 720p y si no existe se salte al 480p, pero veo lo que lo modificaste y los pusiste en un arreglo. en fin eso era todo man. la verdad es que ya ni sé que cosas se necesitan para compilar la app como va hoy en día pero mientras los server funcionen no creo que deba meter mano a menos que salga un servidor nuevo.

Kronos2308 commented 3 years ago

Si pero como era un json mejor convertirlos std::map que usarlo como string para generalizar el uso de json más q otra cosa Para compilar la app es muy sencillo lo puse en el readme solo necesitas ejecutar dos comandos para instalar la librería de nsp que usa el compilador

darkxex commented 3 years ago

Gracias man, ahí le daré una mirada.

Kronos2308 commented 3 years ago

Quiero ver si logro q añadan la librería al dev kit pero de momento solo hace falta instalarla una vez

Kronos2308 commented 3 years ago

No creo q valla hacer cambios notorios en la librería de nsp