Foro de elhacker.net

Bueno, le presento mi última aplicación en VB, que si bien no esta pulida y tiene pequeños errores, se las presento con todo el orgullo  ;D

Si bien el programa es simple y no llega a mas de las 1000 líneas de código les dejo el source para que vean y critiquen todo lo que quieran  :D

(http://www.image-upload.net/images/delz5gy1cdaw8dvjsk.jpg)

El programa no solo reproduce música, también tiene agregados como:
*Función Random
*Función PlayList (muy pobre pero funciona perfectamente)
*Busca temas en la lista
*Busca temas en internet (simple y avanzada)
*Modo máscara
*Modo minimizado, no molesta en la barra de inicio.

Todo lo que está arriba son agregados para mi, ya que este programa lo tenía que hacer como trabajo práctico, y solo tenia que reproducir musica con los comandos básicos.

Espero que les guste, y les voy a hacer sincero, desde que esta terminado es lo unico que uso para escuchar por los recursos que consume  :D

Link: http://www.megaupload.com/?d=Z4AD61W8 (http://www.megaupload.com/?d=Z4AD61W8)

PD: Muchisimas gracias a ElHacker.net por ayudarme con algunos toquesitos del programa.
Otra cosita, en los créditos del MP aparece otro pibe pero en verdad no hizo nada, lo puse para salvarle la materia porque el pobre se tenía que sacar 10  :silbar:

Enjoy

Mil disculpas por olvidar el link y gracias por avisar.

Hola!

Quizás soy muy nuevo en el Foro, pero... donde están los links de descarga?

Saludos!

y sacaron 10 ??  :P


hecho...

1 - Indentar el codigo fuente, es terrible ver en algunas pantallas el codigo fuente como esta indentado de una forma extraterrestre, parece un copy y paste de un gran profesiona , no te enojes solo digo que "parece".
2 - Usar Option Explicit si es posible.
3 - abri el formulario 2 y vi lo que yo llamo "quilombo de controles", seria mejor ordenar un poco la estetica de esa pantalla no me gusto.
4 - es un .exe de casi 1 mega de tamaño que yo creo que se puede dejar en 200 kb , debe ser por la iamgen de fondo que pones, no uses bmp...usa jpg o otro formato de menor tamaño..
5 - podes reemplazar el multimdia control , con apis directamente...
6 - todavia no entendi para que es ese control del volumen...¿para el volumen de la pc? si es asi te lo podes ahorrar y hacerlo con api's tambien...
7 - yo pensaba que se minimizaba al sistray y se minimiza arriba de la barra de tareas que queda "horrible" y al final ocupa mas espacio en pantalla que minimizado.
8 - hay codigo repetido en algunos lados, te creas funciones y lo haces mas corto, si ahora no llega a las 1000 lineas yo te digo que lo podes dejar en menos de 200 lineas...

saludos.

Je,je,je. Dijo "critiquen" no "ejecuten".  ;D

En verdad, muy buena crítica. Es consistente y realista. Me sumo a la misma...

Saludos!

Jeje yo quiero un profe como ese que me de 10 por trabajos como este , "No digo que este mal", pero no creo que este de 10 ^^ te dejo un 6 raspado ;)  :laugh:

1- Es un "defecto de fabrica" mio, en mi año como estudiante de programación nunca me enseñaron a ordenar el código. Nunca es tarde.
2- Por gustos mios no uso Option Explicit, no le veo nada bueno.
3- Es que viste el quilombo de frames que forman el menu.
4- Son todas Jpg las img.
5- Tengo que usar el multimedia control porque estaba en la consigna.
6- El volume.ocx, también tengo que usarlo porque estaba en la consigna.
7- A mi  me gusta como queda  :rolleyes:
8- EN eso no me fije, como bien dije al principio del post, "no esta pulido el código".

Gracias.

PD: todavía no se la nota  :P

Jeje, eso suena como no supieses para que sirve ;)

Si sabes para que sirve ya me diras que no le ves bueno... :laugh: :laugh:

Karcrack, era justo lo que iba a escribir :P

El tema es que si no usas la opción Explicit (osea, requerir declaración de variables) puedes llegar a tener un verdadero embrollo si el código es muy largo.

Ni hablar, si no pones la opción y usas variables con nombres "poco" expresivos...

Saludos!

Hay veces que es muy molesto usar Option Explicit ya que tienes que declarar todas las variables pero tiene algo bueno ya que si no declaras las variables pueden confundir el tipo y joderse todo el proyecto  :P

Bueno... puede ser una molestia .... pero te crea buenos habitos de programación... (cosa que VB no suele hacer :P)

Muchas veces, si utilizas variables con nombres largos puede que te equivoques al escribirlas... en eso Option Explicit te ayuda...

Yo Cuando no uso Option Explicit a las variables les pongo en el nombre la primer letra con mayuscula entonces despues cuando la voy a usar la pongo en minusucula y si al pasar a la siguiente linea no me pone la primer letra en mayuscula es por que lo escrivi mal :laugh: :laugh: :laugh:  Ya se no es lo mas apropiado... Cosas de novatos :P

P.D: A mi especialmente lo que no me gusta del programa es lo que desia seba... eso de que no se esconde en el Tray si no que arriva :S

No estoy seguro, pero creo que eso es un bug. Porque por mas que le pongas showintaskbar en true, cuando minimizas, lo manda al escritorioy no a la barra de tareas.

Ahora, para solucionarlo, uso 2 formularios (uno tipo "splash" y el otro modal). Es la unica forma que encontre de solucionar eso. Aunque quizas, exista alguna mejor...

Saludos!