Muchas gracias Leo por toda la ayuda brindada, por el conocimiento obtenido y el buen humor en las clases.<br>Voy a tener en cuenta las recomendaciones que nos hiciste y quiero pedirte si tenés algún ejemplo de paginador que me recomendás.<br>
Saludos para todos y muchas gracias por ayudarnos a aprender!!!<br><br><div class="gmail_quote">El 18 de agosto de 2012 00:32, Leonardo Tadei - Pegasus Tech Supply <span dir="ltr"><<a href="mailto:leonardot@pegasusnet.com.ar" target="_blank">leonardot@pegasusnet.com.ar</a>></span> escribió:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Estimados todos,<br>
<br>
Les envío la corrección de los trabajos finales.<br>
Espero que la experiencia de hacer estos trabajos haya sido instructiva<br>
y enriquecedora para todos: como una instancia más del aprendizaje, es<br>
la oportunidad de hacer sus aportes con las herramientas que obtuvieron<br>
durante el curso.<br>
Recuerden que, tal como convinimos al principio del curso, todos tienen<br>
1 punto menos en la nota final por la fecha de entrega. Les pongo además<br>
el porcentaje obtenido, para que tengan una mejor medida de las cosas<br>
que vimos y las pautas de corrección.<br>
<br>
Las calificaciones son:<br>
<br>
Alumno % Calif Total Curso<br>
---------------------------------------------------<br>
AVILES, Julio Sebastián incompleta ---- --------------<br>
BARRAUD, David 96% 8.40 7.42 curso aprobado<br>
BEROIZA, Pablo Andrés 76% 5.4 ---- --------------<br>
DI SCALA, Antonela 83% 6.45 6.64 curso aprobado<br>
MARQUEZ, Sergio incompleta ---- --------------<br>
PELIZZARO, Luciano Hugo 91% 7.65 6.04 curso aprobado<br>
<br>
<br>
Algunas cosas que vale la pena mencionar como aportes a sus trabajos:<br>
<br>
Julio: tu trabajo final está muy incompleto para que lo pueda aprobar.<br>
Lo siento. Muchas cosas no funcionan y te faltan los ABM de Modelos y<br>
separar las Ventas y Compras del propio vehículo para poder vincular un<br>
mismo propietario a más de un auto. No cumplís además con el único<br>
requisito técnico que pedimos: que los datos de la conexión a la DB<br>
estén en un solo lugar.<br>
Está bastante bien encarado, pero no te acercás a un trabajo<br>
mínimamente funcional :( Podemos vernos en la FI cuando empiecen los<br>
cursos de cuatrimestre y ver juntos como podrías seguir adelante.<br>
<br>
David: muy buen trabajo. El único error que puede ser considerado como<br>
tal que tenés, es que luego de salir no estás destruyendo bien la<br>
sesión, porque lo que presionar el botón "atrás" del navegador te vuelve<br>
a la misma y seguís logueado. Varias panallas necesitan un paginador,<br>
sobre todo la de solicitudes, porque esas cosas siempre se agregan y con<br>
los meses de uso, va a tardar demasiado en cargar la página.<br>
Como cosas a mejorar, podría las solicitudes de reparación en solapas<br>
según el estado o con un filtro de algún tipo, porque a medida que se<br>
agreguen, esa pantalla de vuelve muy poco usable. Emprolijar en la vista<br>
de las solicitudes los NULL y las fechas en cero le dará mejor aspecto<br>
para el usuario final. También sería más cómodo si mostrás en las combos<br>
los Insumos alfabéticamente, porque si no es más difícil encontrarlos.<br>
Muy buena la generación de PDF para los comprobantes. Como detalle, le<br>
podrías generar un nombre para cada tipo de comprobante y concatenarle<br>
el número, así el que los guarda no tiene un trabajo manual extra para<br>
organizarlos.<br>
<br>
Pablo: tu trabajo final está bien en general. No hay validaciones del<br>
lado del cliente, y faltan algunas validaciones del lado del servidor,<br>
como el no poder cargar dos veces el mismo taller al mismo alumno. Estás<br>
usando una plantilla para la decoración, pero todo el código que<br>
generaste no usa estilos y está decorado inline.<br>
La interfaz para cargar las asistencias necesita urgente algo para que<br>
sea difícil cargar mal los horarios, porque si no el tiempo se calcula<br>
mal. Dicho sea de paso, si ponés solo minutos sin horas, el tiempo se<br>
calcula mal también. También un mecanismo para poner la fecha más simple<br>
y menos ambiguo.<br>
El logout arriba al lado de los "agregar" es algo incómodo: podría ir<br>
perfectamente abajo del todo del menú, un poco separado del resto.<br>
<br>
Antonela: está bien estructurado y muy bien en general. Lo más flojo es<br>
la usabilidad: no es fácil encontrar las cosas, algunas pantallas no<br>
tienen vuelta atrás, buscar datos sirve solo para verlos y no para<br>
editarlos y algunas ediciones no precargan las combos.<br>
Poner en una caja de selección un dato para usar deja de ser viable y<br>
tenés, por ejemplo 2000 o 3000 datos. Para este software, si lo van a<br>
poner en producción, deberías cambiar al menos el selector de Alumnos,<br>
porque si no el tamaño de esa lista lo hará funcionar mal. Se podría<br>
reemplazar por un buscador que use AJAX y te traiga solo un subconjunto<br>
de los datos.<br>
Otra cosa que no sirve es el calendario, ya que para cargar una fecha<br>
de nacimiento tenés que clickear unas 20 veces! Tener un calendario está<br>
bueno, pero se tiene que poder llegar rápido a las fechas que más se<br>
usen.<br>
<br>
Sergio: hiciste un muy buen trabajo de base para la aplicación y está<br>
muy bien encarada. Sin embargo, la parte que funciona es demasiado poco<br>
para poder aprobarte. Lo siento.<br>
La probé incluso poniendo a PHP en un modo muy permisivo, porque<br>
trabajando en modo Estrict aparecían errores que podían estar impidiendo<br>
las correctas llamadas JS y AJAX, pero aún así no se pueden cargar<br>
clientes ni proveedores, las ediciones de los cargados no funcionan, las<br>
provincias no están implementadas, no se agregan ventas ni compras y no<br>
se pueden marcar como cobradas.<br>
Cuánto tiempo creés que te hace falta para terminarla? Con tu buena<br>
calificación y esto que me entregaste, seguro que podés conseguir con<br>
Silvio una prórroga para entregar más tarde y aprobar el curso.<br>
Además, por el trabajo que hiciste, me gustaría que hagas el curso de<br>
Diseño de Software con Objetos: te va a dar un muy buen pantallazo de<br>
otra forma de concebir y construir software.<br>
<br>
Luciano: hiciste un buen trabajo. La gráfica que usaste es amena y sirve<br>
a la usabilidad con el uso de HTML5. el mayo problema que tenés es la<br>
falta de validación del lado del servidor para manejar inconsistencias.<br>
Por ejemplo, creé una Localidad, luego un Laboratorio en esa Localidad,<br>
borré la Localidad (y pude!) y el Laboratorio desapareció del listado,<br>
si bien seguía estando para elegir de las listas desplegables.<br>
Me desorientó un poco que las opciones del menú que despliegan también<br>
lleven a pantallas. Claro que esto no baja puntos, pero significó que<br>
pasé un rato hasta encontrar el ABM de Médicos.<br>
Las funciones de Baja y Edición creo que serían más cómodas si<br>
estuvieran en la misma pantalla (con dos íconos) en vez de tener que<br>
pasar de una pantalla a la otra. Tal vez ser podría integrar más todavía<br>
con el alta y la búsqueda.<br>
Del uso de AJAX no opino demasiado, pero ya que estabas con eso,<br>
podrías haber hecho formularios de alta y modificación emergentes, y<br>
sobre todo sacar los alert() de confirmación de que se cargaron las<br>
cosas. Los listados que con el tiempo serán largos, como los de<br>
Pacientes y Médicos entre otros, van a necesitar un paginador para<br>
restringir el tiempo de carga cuando haya muchos datos.<br>
<br>
<br>
Nada más por ahora.<br>
Les envío un saludo cordial.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Leonardo Tadei<br>
<a href="mailto:leonardot@pegasusnet.com.ar">leonardot@pegasusnet.com.ar</a><br>
Web: <a href="http://leonardo.tadei.com.ar" target="_blank">http://leonardo.tadei.com.ar</a><br>
Firma pública: <a href="http://www.pegasusnet.com.ar/LeonardoTadei-public.key" target="_blank">http://www.pegasusnet.com.ar/LeonardoTadei-public.key</a><br>
<br>
_______________________________________________<br>
Php-avanzado mailing list<br>
<a href="mailto:Php-avanzado@pato2.fi.mdp.edu.ar">Php-avanzado@pato2.fi.mdp.edu.ar</a><br>
<a href="http://www3.fi.mdp.edu.ar/cgi-bin/mailman/listinfo/php-avanzado" target="_blank">http://www3.fi.mdp.edu.ar/cgi-bin/mailman/listinfo/php-avanzado</a><br>
</font></span></blockquote></div><br>