pepe paractica m2 - semi terminada

[alvarezhm]

@dragonnegro78 aca esta el pull , bezzzzameeee

[dragonnegro78]

Por lo menos esta bien hecho el PR. Ya con eso aprueba para mi

[leamarty]

@dragonnegro78 cagón, borraste el comentario

Hay otro tema más importante, que es conceptual. En muchos lugares estás preguntando si cant < 30 o cant < 35 o cosas así...

Pero qué pasaría si los clientes fueran 45? o 60? O 500? Tendrías que cambiar todo el código!

Ojo con eso, no es útil hacer cosas tan especializadas y forzadas. Hay que hacer todo pensando en lo genérico y en el uso múltiple y variable.

[dragonnegro78]

Como iba a saber yo que cant era en realidad hast

[dragonnegro78]

Deci que jQuery es buena onda, pero me parecio raro que un par de lineas arriba te mates buscando el tbody pero aca te olvides. Igualmente jQuery te los agrega en el tbody porque es capo, no se.

[dragonnegro78]

Y una cosa que me habia marcado lea a mi en su momento, era que al hacer append directamente sobre la tabla verdadera, si los datos son medio pesados de convertir en una tabla, o el chabon esta en una compu que esta usando mucho CPU o es lento, entonces el flaco va a ver como append va sumando fila por fila de informacion.
Lo ideal es hacer todo el append a una variable, y cuando termines de agregar todas las filas, meter eso de 1 en la tabla real.

[leamarty]

Sí pero ya sería demasiado lio eso. A pesar de que tenés razón, yo diría que no haga eso, y que haga más foco en el resto de las cosas.

[alvarezhm]

que quilombo, jaja mañana me voy a poner con todo , u hoy a la noche ... lo del error del getJASON , no va a tirar nunca en este caso =/ @dragonnegro78 y lo de los muhos conteiiiiners tambien me parecio pero lea la dio esa parte ... yo no tengo nada q ver ahi

[dragonnegro78]

@alvarezhm Pero mira si te olvidaste de levantar php, queres probar la pagina, pones buscar y no te tira nada la pantalla, porque getJSON esta tirando error silenciosamente y vos no mostras ningun mensaje.
Te volverias loco averiguando por que, we.
La onda pepe, es que despues de escribir el choclo del success, poner el del error son tipo 3 lineas más que no solo sirven para futuro cuando el server de clientes no este en localhost, sino que ademas te ayudaban a debuggear mientras hacias el clientes.php por si por alguna razon habia un error.
Ademas, pero en esto deberia opinar @leamarty , creo que a veces, cuando estas desarrollando un sistema como este que conecta con varios servidores, para pedir autorizacion, para bajar datos, etc., no es extraño tener un pequeño servidor que SIMULE esos servidores con una pequeña cantidad de datos. Ponele, te hosteas en localhost un pequeño server de 20 clientes simulando que tenes la base de datos de los 20 millones de clientes de celular de argentina. Es solo para probar la respuesta, que pida bien y llegue bien, y en localhost nunca va a fallar, pero nunca tenes que olvidarte de agregar el .error porque cuando la pruebes de verdad, los problemas de red si existen.

Es solo buena practica, y como son solo 3 lineas pedorras, vale la pena hacerlo ):

[leamarty]

True thing. Yo cambiaría la última carita por un 😄

[dragonnegro78]

):

[alvarezhm]

entiendo! ,gracias ale ... me sos de mas utilidad q lea 💃

[dragonnegro78]

Y la comprobacion de que no se pase de la cantidad maxima de clientes (35) ?
Que pasa si viene un l337 h4x0r, te modifica el javascript y te manda 1000000 en cantidad?
Pasa esto: el .php se queda ejecutando durante un monton de tiempo hasta que llega al limite de tiempo de ejecucion (1 minuto?) y por seguridad, php cierra el proceso. Pero durante todo ese tiempo, ese proceso de php se esta comiendo el cpu a fondo.

[dragonnegro78]

Creo que esto te quedo de cuando importabamos el .json directamente al .html. Clientes no existe en ninguna parte del .html. ¿Quizas cambiaste el nombre de la variable en otras partes del codigo pero te olvidaste de cambiarlo en esta linea? Me di cuenta porque me tiro error al toque cuando probe tu pagina y busque clientes con el boton teniendo la consola abierta. Estate atento a eso porque ayuda un monton a encontrar errores de JS.