Feda /dev/ - No Javascript allowed

He creado un script para sacar invis de forocoches

https://pastebin.com/k5A3HV41

Las invitaciones solo salen en horas exactas es decir 16:00 - 17:00 ....

Instrucciones
Es recomendable poner el script a y 59 a funcionar por si se parase, junto con el captcha(s) de https://forocoches.com/codigo que expira en approx un minuto.

Si habéis puesto bien la hora sonará POLEEE y tendréis el código en el clipboard, pegais en el input y ya tenéis la tan deseada invitación

AVISO: El comando (linea 51) para copiar al clipboard y el del sonido depende del SO qe uses, en este caso está para MacOS y dice POLEEEE

En las PR os suelen mirar la complejidad ciclomatica? Me han rechazado una PR por poner un for dentro de un else que hay dentro de otro for, no quieren que ponga breaks ni returns en medio del método. El tio que pasa las revisiones es un grandisimo gilipollas pero hoy se ha superado con estas virguerias xd

un for dentro de un else que hay dentro de otro for

A ver xD

#41109 #41110
https://imgur.com/a/49Fgnq2

No sé, se lo he pasado a mis compañeros y a nadie le parece mal pero este tio se lo ha tomado como si fuera una grandisima mierda muy mal optimizada y que no sé programar. Lo he probado a hacer con recursividad con una flag y tampoco le ha gustado. Dice que así no está bien pero no dice cómo hacerlo bien asi que welp

#41112 A mi ese codigo me parece un truño y soy de la opinión de hacer las cosas explícitas pero es que es horrible.

const hasFirstPage = (r) => this.hasRol(r) || r.submenus.find(this.hasRol)
const firstPage = RUTAS.find(hasFirstPage)
if (firstPage) this.router.navigate(firstPage.url)

private findFirstAuthorizedPage(routes: RutaMenu[]): RutaMenu {
  for (let route of routes) {
    if (this.hasRol(route)) {
      return route;
    } else if (route.submenus && route.submenus.length > 0) {
      const authorizedSubroute = this.findFirstAuthorizedPage(route.submenus);
      if (authorizedSubroute) {
        return authorizedSubroute;
      }
    }
  }
  return null;
}

private getFirstPageByRole(): void {
  const firstPage = this.findFirstAuthorizedPage(DICCIONARIO_RUTAS_MENU);
  if (firstPage) {
    this.router.navigate([firstPage.url]);
  }
}

private getFirstPageByRole(): void {
  let authorizedRoutes: RutaMenu[] = [];
  for (let route of DICCIONARIO_RUTAS_MENU) {
    if (this.hasRol(route)) {
      authorizedRoutes.push(route);
    } else if (route.submenus && route.submenus.length > 0) {
      for (let subroute of route.submenus) {
        if (this.hasRol(subroute)) {
          authorizedRoutes.push(subroute);
        }
      }
    }
  }
  if (authorizedRoutes.length > 0) {
    this.router.navigate([authorizedRoutes[0].url]);
  }
}

private getFirstPageByRole(): void {
  let queue: RutaMenu[] = DICCIONARIO_RUTAS_MENU;
  let firstPage: RutaMenu = null;
  while (queue.length > 0 && !firstPage) {
    const route = queue.shift();
    if (this.hasRol(route)) {
      firstPage = route;
    } else if (route.submenus && route.submenus.length > 0) {
      queue = queue.concat(route.submenus);
    }
  }
  if (firstPage) {
    this.router.navigate([firstPage.url]);
  }
}

#41125 El metodo no tiene que filtrar las rutas a las que tiene acceso el usuario sino obtener la ruta inicial del usuario por rol, osea lo primero que verá tras logarse, que es dinámico según el rol del usuario.

El código de hasRole es sólo esto:

private hasRol(menu: RutaMenu): boolean {
return menu.primera_ruta?.length > 0 && menu.primera_ruta.includes(this.usuarioLogado.rol);
}

#41127 el tema entiendo que es porqué no aporta ninguna explicación más allá del "esta mal", a mi ahora con el cambio de curro me tiran muuuchas cosas atrás, pero siempre con comentarios sobre porqué no deberian estar así