Título: C# - No puedo reproducir la siguiente cancion Publicado por: TickTack en 12 Octubre 2017, 09:51 am Hola,
hice un listBox en donde se ponen todos los nombres de las canciones que agrego mediante el control button. Pero cuando termina la cancion no se reproduce la siguiente. Les muestro como trate de que se reproduciera la siguiente cancion cuando termine una con este codigo: Código: private void axWindowsMediaPlayer1_PlayStateChange(object sender, AxWMPLib._WMPOCXEvents_PlayStateChangeEvent e) { if (e.newState == 8) { index = index + 1; if (index - 1 < listBox1.Items.Count) { axWindowsMediaPlayer1.URL = list2[list1.IndexOf(listBox1.Items[index].ToString())]; } } } Se supone que deberia funcionar. Pero para evitar confusiones les mostrare todo el codigo: Código: using System; using System.Collections; using System.Collections.Generic; using System.ComponentModel; using System.Data; using System.Drawing; using System.Linq; using System.Text; using System.Threading.Tasks; using System.Windows.Forms; namespace Player { public partial class Form1 : Form { List<string> list1 = new List<string>(); List<string> list2 = new List<string>(); List<string> list3 = new List<string>(); Timer timer1 = new Timer(); int index = 0; public Form1() { InitializeComponent(); } private void button1_Click(object sender, EventArgs e) { System.Windows.Forms.OpenFileDialog ofd = new System.Windows.Forms.OpenFileDialog(); ofd.Multiselect = true; ofd.ShowDialog(); foreach (var file in ofd.SafeFileNames) { listBox1.Items.Add(file); list1.Add(file); listBox1.Sorted = true; } foreach (var file in ofd.FileNames) { list2.Add(file); } } private void Form1_Load(object sender, EventArgs e) { } private void listBox1_MouseDoubleClick(object sender, MouseEventArgs e) { index = listBox1.SelectedIndex; axWindowsMediaPlayer1.URL = list2[list1.IndexOf(listBox1.GetItemText(listBox1.SelectedItem))]; } private void axWindowsMediaPlayer1_PlayStateChange(object sender, AxWMPLib._WMPOCXEvents_PlayStateChangeEvent e) { if (e.newState == 8) { index = index + 1; if (index - 1 < listBox1.Items.Count) { axWindowsMediaPlayer1.URL = list2[list1.IndexOf(listBox1.Items[index].ToString())]; } } } } } Alguien me puede ayudar por favor? Gracias y saludos Título: Re: C# - No puedo reproducir la siguiente cancion Publicado por: Eleкtro en 12 Octubre 2017, 13:50 pm 1.
No estás liberando la instancia de una clase que implementa la interfaz IDisposable... Citar Código 2. Estás usando una colección genérica de tipo List para almacenar las rutas absolutas de unos archivos, y luego otra lista adicional para almacenar los nombres de archivo. Tener dos colecciones distintas para practicamente la misma finalidad (puesto que los nombres de archivo son parte de la ruta absoluta del archivo) es algo completamente innecesario con lo que solo conseguirás escribir un código espagueti (confuso y destructurado) más si les asignas nombres como "list1", "list2" y "list3"... Hay que mejorar las nomenclaturas de los miembros. 3. Debido a que estás representando una colección de elementos (nombres de archivos), en lugar de utilizar el método ListBox.Add() deberías utilizar el método ListBox.AddRange() para mejorar el rendimiento de la aplicación y la respuesta de la UI. Con cada llamada individual que haces a ListBox.Add() el control envia mensajes de ventana para redibujarse, y eso para añadir 20 canciones no pasa nada, pero si fuesen muchas más pues...acabarías bloqueando la UI por momentos. 4. Estás reasignando el mismo valor a la propiedad ListBox.Sorted en el controlador button1_Click, esto son instrucciones de más, algo innecesario. Con asignarle el valor a la propiedad una única vez al cargar el control es suficiente. 5.. No estás comprobando el resultado del diálogo cuando seleccionas archivos... ¿qué ocurre si el usuario cancela la selección?. debes comprobarlo... 6.. cuando termina la cancion no se reproduce la siguiente. Prueba a invocar la funcionalidad de reproducir... Código
7.. Citar Código
En lugar de la necesidad de declarar un contador y comprobar el estado del reproductor e ir reasignando a cada rato el valor de la propiedad URL para reproducir la siguiente canción, en lugar de todo eso algo más simple sería que añadieses los elementos del ListBox (las rutas completas de los archivos a reproducir) en una lista de reproducción ( playlist.m3u ) para que el WMP las vaya reproduciendo una tras otra y no tengas que hacer más... Tienes dos opciones para ello, puedes escribir un archivo de lista de reproducción m3u (no es complicado, es un formato muy sencillo):
O bien puedes puedes crear una lista de manera guiada para usarlo en la instancia actual del WMP mediante la interfáz IWMPPlaylist
8.. Citar Código: if (e.newState == 8) ¿Has verificado que el valor de la propiedad NewState sea realmente 8 al terminar de reproducir la canción?. La documentación oficial en la MSDN no recomienda que un algoritmo se base en la "predicción" de los estados del reproductor debido a que al parecer son un poco...impredecibles. Cita de: MSDN Remarks Windows Media Player states are not guaranteed to occur in any particular order. Furthermore, not every state necessarily occurs during a sequence of events. You should not write code that relies upon state order.
Nota: En la página de la MSDN tienes un ejemplo extendido para depurar/testear el estado actual del reproductor. En general todo el código que tienes ahora mismo lo puedes mejorar y simplificar / refactorizar a algo parecido a esto: Código
Nota: Es solo un código de ejemplo para que lo adaptes a tus necesidades, falta que asocies los controladores de evento en el código, no he comprobado que funcione lo de reproducir canciones en el WMP, y tampoco he implementado la funcionalidad de la lista de reproducción ya que lo he propuesto como una mejora opcional y bueno... no voy a hacer yo todo el trabajo, jeje. Saludos. Título: Re: C# - No puedo reproducir la siguiente cancion Publicado por: TickTack en 12 Octubre 2017, 18:31 pm Hola Elektro,
1. No es lo mismo OpenFileDialog ofd = new OpenFileDialog(); que System.Windows.Forms.OpenFileDialog ofd = new System.Windows.Forms.OpenFileDialog();? 2. Es necesario que lo que haga de esta manera ya que necesito por un lado mostrar los nombres de las canciones en el listBox y por el otro lado guardar las rutas para reproducir esas canciones. Y aunque los nombres sean partes de las rutas... separarlos, segun yo, lleva mas tiempo que el codigo espagueti. 3. Bueno eso tendre que cambiarlo. Gracias por la informacion. :) 4. Yo no veo ninguna instruccion de mas. Solamente le asigne una vez una valor a ListBox.Sorted. 5. Ay si... eso me olvide... jeje 6. Ya lo probe antes de hacer este post. 7. Ok. Quiero ir por ese camino... 8. Eso tampoco lo supe. Pero no se me da la posibilidad de escribir if (e.newState == WMP_MediaEnded) { ya que me larga error con WMP_MediaEnded. Me dice que no existe. Si te diste cuenta del error por favor informame. Te estare my agradecido Gracias y saludos PD.: Tu codigo se me hizo dificil de harmonizarlo con el mio pero igual muchas gracias. (Es que tienes una forma complicada de escribir codigos, quiza por los grandes programas o codigos comerciales que ya hiciste, osea esa por esa experiencia). Título: Re: C# - No puedo reproducir la siguiente cancion Publicado por: Eleкtro en 12 Octubre 2017, 20:13 pm 1. No es lo mismo OpenFileDialog ofd = new OpenFileDialog(); que System.Windows.Forms.OpenFileDialog ofd = new System.Windows.Forms.OpenFileDialog();? Si, por supuesto, simplemente borré el nombre del namespace al citar tu código, y lo cité solamente para hacer referencia a la instrucción de tú código a la que me estaba refiriendo, pero a lo que me refería es a que debes utilizar la sentencia using para asegurarte de que el GC (Garbage Collector) libere cualquier recurso administrado y no administrado que haya sido utilizado por "X" instancia IDisposable... y así tener (o al menos intentarlo) una aplicación libre de las indeseadas fugas de consumo RAM. Es decir: Código
...o en su defecto puedes usar el método IDisposable.Dispose() ( OpenFileDialog.Dispose() ).
2. Es necesario que lo que haga de esta manera ya que necesito por un lado mostrar los nombres de las canciones en el listBox y por el otro lado guardar las rutas para reproducir esas canciones. Y aunque los nombres sean partes de las rutas... separarlos, segun yo, lleva mas tiempo que el codigo espagueti. No, no es necesario ni tampoco lleva más tiempo en términos de esfuerzo. En mi última respuesta te mostré un código en el que utilizo una colección genérica de tipo SortedList para almacenar ambos datos... y cuyos elementos se mantienen ordenados por orden alfabético según el nombre de cada archivo agregado en o eliminado de la colección... Como ves, es una mejora que simplifica tres funcionalidades del código original, en una sola. 3. Bueno eso tendre que cambiarlo. Gracias por la informacion. :) Si sigues la solución que te indiqué (vease el código que compartí en mi última respuesta) entonces no necesitas usar el método Add / AddRange ya que en su lugar se utiliza la colección SortedList como datasource. Pero de todos modos la utilización del método AddRange es una mejora significativa que siempre deberías tener en cuenta cuando necesites agregar elementos manualmente a un control de tipo Lista (ListBox, ListView, DataGridView, etc). 4. Yo no veo ninguna instruccion de mas. Solamente le asigne una vez una valor a ListBox.Sorted. Citar Código
Fíjate bien, le estás reasignando el valor en cada iteración del búcle foreach, es decir, 'listBox1.Sorted = true' por cada elemento en 'ofd.SafeFileNames'. Título: Re: C# - No puedo reproducir la siguiente cancion Publicado por: Eleкtro en 12 Octubre 2017, 20:19 pm 6. Ya lo probe antes de hacer este post. 7. Ok. Quiero ir por ese camino... De acuerdo, entonces por si te sirve de algo te muestro un algoritmo que desarrollé hace un par de años para construir listas de reproducción en formato M3U y PLS de forma sencilla. El código es un poco feo, lo escribí hace varios años, se puede refactorizar para perfeccionarlo de muchas formas para representarlo de forma más abstracta y llevar a cabo todas las modificaciones en la memoria sin realizar operaciones de lectura/escritura en el archivo, pero bueno, así lo escribí en su día cuando todavía no dominaba muchas cosas en la progamación .NET, y el caso es que funciona, que es lo importante... NOTA INFORMATIVA: --- EL SIGUIENTE CÓDIGO HA SIDO EXTRAIDO Y OFRECIDO DE FORMA GRATUITA A PARTIR DE MI FRAMEWORK COMERCIAL ELEKTROKIT FRAMEWORK , EL CUAL CONTIENE UNA INFINIDAD DE UTILIDADES ENFOCADAS A UNA AMPLIA VARIEDAD DE TEMÁTICAS Y ESCENARIOS EN LA PROGRAMACIÓN .NET, COMO ÉSTE. SI QUIEREN CONOCER MÁS ACERCA DEL PRODUCTO, MIREN MI FIRMA DE USUARIO EN EL FORO. --- --- ESTE CÓDIGO SE PUEDE USAR DE FORMA LIBRE COMO DESEEN. Nota: Debido a que el foro tiene un límite muy reducido de caracteres, me he visto obligado a eliminar gran parte de la documentación XML ( descripciones de parámetros, etc). Código
Código
Código
El código de arriba puedes tomarlo como ejemplo y nada más para saber como sería una posible manera de poder hacerlo, o bien puedes copiar y pegarlo en una nueva clase de VB.NET para compilarlo en una dll y usarlo en C#, o también puedes convertir directamente el código a C# por ejemplo usando mi conversor de código gratuito ( todos los créditos y copyright para Telerik xD ):
Modo de empleo: Código
O bien: Código
Saludos Título: Re: C# - No puedo reproducir la siguiente cancion Publicado por: TickTack en 13 Octubre 2017, 11:16 am Hola Elektro,
1. Muchas gracias por tu consejo. Ya lo hize. Pero me gustaria saber como es que estas siempre informado de que esto produce fugas de consume RAM y lo otro envia mensajes de ventana para redibujarse y que este otro produce esto? Es increible tu conocimiento!! 2. Ahh si. De esto queria hablar. Es que tu codigo que mostraste se complica con mi proyecto porque para empezar me da problemas con public souled class. Me dice que no se puede dejarlo asi porque hay otra clase que ya esta asi. Ni idea? Luego me dice que no se puede acceder a files por motivos de seguridad y son todas cosas con las que nunca me tope por la falta de experiencia..... 4. Noooooooooooooooo. Que idiota que soy. Ni me di cuenta. Perdon por lo bruto que soy.. jejej. 5. Pero entonces cual me dices que es mas recomendable usar? Tu codigo 1 o tu codigo 2 (posteaste dos codigos tuyos aca). Porque si me dices codigo 1 entonces vere que puedo hacer con el problema de public souled class y lo demas. Pero si me dices codigo 2 entonces podre pegar el codigo en una nueva clase de VB.NET para compilarlo en una dll y posteriormente usarlo en C#. Finalmente si me dices que investigue los dos codigos lo hare (por supuesto, solamente si me lo recomiendas). Para terminar te muestro como quedo el codigo hasta ahora segun tu: Código: using System; using System.Collections; using System.Collections.Generic; using System.ComponentModel; using System.Data; using System.Drawing; using System.Linq; using System.Text; using System.Threading.Tasks; using System.Windows.Forms; using WMPLib; namespace Player { public partial class Form1 : Form { List<string> list1 = new List<string>(); List<string> list2 = new List<string>(); List<string> list3 = new List<string>(); Timer timer1 = new Timer(); int index = 0; public Form1() { InitializeComponent(); } private void button1_Click(object sender, EventArgs e) { using (OpenFileDialog ofd = new OpenFileDialog()) { ofd.Multiselect = true; if (ofd.ShowDialog() == DialogResult.OK) { listBox1.Sorted = true; foreach (var file in ofd.SafeFileNames) { listBox1.Items.Add(file); list1.Add(file); } foreach (var file in ofd.FileNames) { list2.Add(file); } } } } private void Form1_Load(object sender, EventArgs e) { } private void listBox1_MouseDoubleClick(object sender, MouseEventArgs e) { index = listBox1.SelectedIndex; axWindowsMediaPlayer1.URL = list2[list1.IndexOf(listBox1.GetItemText(listBox1.SelectedItem))]; } private void axWindowsMediaPlayer1_PlayStateChange(object sender, AxWMPLib._WMPOCXEvents_PlayStateChangeEvent e) { if (e.newState == 8) { index = index + 1; if (index - 1 < listBox1.Items.Count) { axWindowsMediaPlayer1.URL = list2[list1.IndexOf(listBox1.Items[index].ToString())]; } } } } } Gracias y saludos Título: Re: C# - No puedo reproducir la siguiente cancion Publicado por: Eleкtro en 13 Octubre 2017, 12:13 pm 1. me gustaria saber como es que estas siempre informado de que esto produce fugas de consume RAM y lo otro envia mensajes de ventana para redibujarse y que este otro produce esto? Es increible tu conocimiento!! Te agradezco el reconocimiento pero en realidad en mi opinión no son cosas avanzadas que se merezcan " "admiración" " hacia alguien, o al menos yo creo que deberían ser cosas básicas que se deben aprender estudiando el lenguaje, documentándose sobre como se administran los recursos, o como trabaja el sistema de mensajería, GDI/GDI+, WinForms, etc... Siempre he sostenido que no hay mejor manera de aprender .NET que leyendo la MSDN (aunque sea un coñazo), por que en los libros suelen explicar los conceptos más "internos" de forma superficial, y luego pasa lo que pasa, que uno aprende a manejar C#/VB.NET pero eso no es suficiente ya que no aprende como funciona el sistema operativo (Windows) ni la tecnología que esté usando (WinForms, WPF, etc). Te dejo unos enlaces de interés: Administración de recursos:
Mensajería
Painting
ListBox
Cita de: MSDN Remarks The preferred way to add multiple items to the ListBox is to use the AddRange method of the ListBox.ObjectCollection class (through the Items property of the ListBox). This enables you to add an array of items to the list in a single operation. However, if you want to add items one at a time using the Add method of the ListBox.ObjectCollection class, you can use the BeginUpdate method to prevent the control from repainting the ListBox each time an item is added to the list. Once you have completed the task of adding items to the list, call the EndUpdate method to enable the ListBox to repaint. This way of adding items can prevent flickered drawing of the ListBox when a large number of items are being added to the list. En fin, si llegas a leer todos los enlaces que voy poniendo a la MSDN y docs.microsoft.com (no me refiero solo a los que he compartido ahora, sino en general), aprenderías mucho sobre .NET a "nivel gurú" o casi xD. :P
Analizar el código fuente de .NET Framework también ayuda mucho a comprender como funciona a nivel "interno":
2. tu codigo que mostraste se complica con mi proyecto porque para empezar me da problemas con public souled class. Me dice que no se puede dejarlo asi porque hay otra clase que ya esta asi. Ni idea? En general todo el código que tienes ahora mismo lo puedes mejorar y simplificar / refactorizar a algo parecido a esto: Código
Lo de "public sealed class Form1 : Form" es así por que yo escribí el código en un nuevo proyecto, pero te dije que lo tienes que adaptar, no tienes que copiar los modificadores (sealed) ni tampoco el nombre de la clase (Form1), sino el resto del código (código relevante) que está definido dentro de esa clase. En tu código imagino que lo tendrás escrito así: Código ...no necesitas cambiarlo. 5. Pero entonces cual me dices que es mas recomendable usar? Tu codigo 1 o tu codigo 2 (posteaste dos codigos tuyos aca). Porque si me dices codigo 1 entonces vere que puedo hacer con el problema de public souled class y lo demas. Pero si me dices codigo 2 entonces podre pegar el codigo en una nueva clase de VB.NET para compilarlo en una dll y posteriormente usarlo en C#. Finalmente si me dices que investigue los dos codigos lo hare (por supuesto, solamente si me lo recomiendas). Son códigos para cosas distintas. El primer código que puse en C# es una refactorización del primer código que tu compartiste, es un fix por así decirlo que corrige varios malos hábitos de programación en .NET ( los que te estuve explicando y enumerando 1., 2., 3. etc... ) y que en teoría debería servirte para corregir el problema de no poder reproducir la siguiente canción (solo en teoría, ya que como dije no probé lo del WMP). La idea era que reemplazases (todo) tú código por el que yo te mostré y decir si con esos cambios arreglaste por fin el problema. Pero sin lo de "public sealed class Form1 : Form", eso déjalo como lo tengas escrito en tu clase Form1. El segundo código en VB.NET es una implementación para administrar listas de reproducción en formato m38 o pls, eso lo puedes usar si quieres o no usarlo xD, puedes hacer basicamente lo mismo con la interfáz COM que te dije de WMP (IWMPPlaylist). Ese código te lo puse por que tú dijiste que "querías ir por ese camino" y pues... con ese código te estaba dando "el camino hecho". saludos |