3

Tengo un proceso en el cual hago evaluaciones de los empleados, para procesar los ponches, en el formulario de X cantidad de empleados que se muestren el usuario puede seleccionar los que desea procesar. Mientras esta procesando debo mostrar el Progreso y porcentaje que va transcrurriendo el proceso.

Problema: El progressBar va aumentamento pero el labelPorcentaje no aumenta, solo muestra el 100% al finalizar el proceso. La Cantidad de datos a procesar depende de los empleados que seleccione el usuario, por lo tanto en ocaciones pueden ser pocos o muchos, quiero poder apreciar el aumento del progreso durante el proceso sin importar que sean pocos o muchos los datos a procesar.

He hecho un ejemplo sencillo para representar mi problema:

private void btnEvaluar_Click(object sender, EventArgs e)
{
      labelProgreso.Visible = true;
      progressBar1.Visible = true;

      int progreso = 0, porciento = 0, totalEmpleadosProcesar = 0;

      for (int indice = 0; indice < 85; indice++) //Ciclo que representará los empleados seleccionados
      {
          totalEmpleadosProcesar = 85;  //Total de Empleados que seleccionó el usuario

          progreso++; //Aumentando el progreso 
          porciento = Convert.ToInt16(( ( (double)progreso / (double)totalEmpleadosProcesar ) * 100.00 )); //Calculo del porcentaje

          reportarProgreso(100, porciento); //función que cargará la barra de progreso

          //MessageBox.Show("Porciento: " + porciento);
      }

      MessageBox.Show("Completo...");
      labelProgreso.Visible = false;
      progressBar1.Visible = false;
}

Función que carga el ProgressBar:

private void reportarProgreso(int valorMaximo, int valor)
{
     progressBar1.Step = 1;
     progressBar1.Style = ProgressBarStyle.Continuous;
     progressBar1.Minimum = 0;
     progressBar1.Maximum = valorMaximo;

     if (valor > valorMaximo)
     {
         labelProgreso.Text = "100%";
         progressBar1.Value = valorMaximo;
     }
     else
     {
         labelProgreso.Text = Convert.ToString(valor) + "%";
         progressBar1.Value = valor;
     }
 }

Para solucionarlo intenté usar Thread.Sleep(1000); durante el recorrido del ciclo, pero de igual forma no muestra el porcentaje en el label

introducir la descripción de la imagen aquí

No deseo utilizar un Thread.Sleep porque como expresé si son pocos empleados funcionaría bien, pero si son muchos y tengo un tiempo establecido para el Thread se tardaría un montón.

Si intervengo cada interacción del ciclo con un MessageBox de prueba si me muestra el porcentaje en el label.

introducir la descripción de la imagen aquí

¿Qué estoy haciendo mal? , Cual sería la forma correcta de representar el progreso de un proceso?

Nota: No me interesa que la interfaz siga estando activa durante el proceso, por eso no he utilizado Task o BackGroundWorker...

Entorno: Visual Studio 2010 (WindowsForms) & .NET NetFramework 4.0

10
  • 3
    "No me interesa que la interfaz siga estando activa durante el proceso, por eso no he utilizado Task o BackGroundWorker..." justamente, ese es el problema.. la interfaz no esta activa, con lo cual el label no se va a refrescar. el progress si porque tiene vida propia.
    – gbianchi
    el 8 mar. 2018 a las 15:29
  • En todos los casos donde deba utilizar progressBar y un label para representar el porcentaje, debo utilizar eso BackGroundWorker entonces? :( Que otra opción tengo? el 8 mar. 2018 a las 15:31
  • no. backgroundworker no. Usa await y task. y la otra opcion es no refrescar el label :p.. c# no tiene un doevents (cosa que si tenia VB)
    – gbianchi
    el 8 mar. 2018 a las 15:32
  • @gbianchi es que aún no logro engancharme al funcionamiento del Await y Task no logro entenderlo es complicado para mi. el 8 mar. 2018 a las 15:34
  • Si se puede refrescar un label cuando este no es creado en el hilo que haces ese cambio. el 8 mar. 2018 a las 15:36

2 respuestas 2

5

El problema es que tu bucle bloquea el hilo principal de ejecución,haciendo que no se redibuje el label. Yo para solucionarlo recomiendo usar un BackgroundWorker.

Primero, define el BackgroundWorker en tu formulario:

BackgroundWorker bg = new BackgroundWorker();

Luego en el evento del boton, configuras los eventos ProgressChanged, DoWork y RunWorkerCompletedy llamas a su ejecución:

private void btnEvaluar_Click(object sender, EventArgs e)
{
    bg.WorkerReportsProgress = true;
    bg.ProgressChanged += bg_ProgressChanged;
    bg.DoWork += bg_DoWork;
    bg.RunWorkerCompleted += bg_RunWorkerCompleted;
    bg.RunWorkerAsync();
    labelProgreso.Visible = true;
    progressBar1.Visible = true;
}

Evento DoWork (he añadido un Sleep para que se aprecie el progreso) :

private void bg_DoWork(object sender, DoWorkEventArgs e)
{
    int progreso = 0, porciento = 0, totalEmpleadosProcesar = 0;

    for (int indice = 0; indice < 85; indice++) //Ciclo que representará los empleados seleccionados
    {
        totalEmpleadosProcesar = 85;  //Total de Empleados que seleccionó el usuario

        progreso++; //Aumentando el progreso 
        porciento = Convert.ToInt16((((double)progreso / (double)totalEmpleadosProcesar) * 100.00)); //Calculo del porcentaje
        System.Threading.Thread.Sleep(500);
        bg.ReportProgress(porciento);
    }
}

El evento ProgressChanged donde actualizamos tanto el Progressbar como el Label:

private void bg_ProgressChanged(object sender, ProgressChangedEventArgs e)
{
    // Change the value of the ProgressBar to the BackgroundWorker progress.
    progressBar1.Value = e.ProgressPercentage;
    progressBar1.Step = 1;
    progressBar1.Style = ProgressBarStyle.Continuous;
    progressBar1.Minimum = 0;
    progressBar1.Maximum = 100;

    if (e.ProgressPercentage > 100)
    {
        labelProgreso.Text = "100%";
        progressBar1.Value = progressBar1.Maximum;
    }
    else
    {
        labelProgreso.Text = Convert.ToString(e.ProgressPercentage) + "%";
        progressBar1.Value = e.ProgressPercentage;
    }
}

Por último, el evento RunWorkerCompleted que se ejecuta cuando el BackgroundWorker ha finalizado:

private void bg_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
{
    MessageBox.Show("Completo...");
    labelProgreso.Visible = false;
    progressBar1.Visible = false;
}
13
  • He probado tengo un problema, y un inconveniente, problema: al finalizar el proceso luego del MessageBox da error porque se esta tratando de cambiar la visibilidad del label y progress desde otro hilo (Pero esto se como resolverlo), Inconveniente: Si se fija detenidamente el mensaje de "Completo..." lo muestra antes de que la barra de progreso este cargada en su totalidad, Hay como resolver eso? aunque no es un problema grave pero al ojo del usuario no debería verse de esta forma. Debería completar la barra y luego mostrar el mensaje. el 8 mar. 2018 a las 16:15
  • @J.Rodríguez fallo mio (por no probar el código). Eso debes implementarlo en el evento RunWorkerCompleted. Ahora edito la respuesta
    – Pikoh
    el 8 mar. 2018 a las 16:17
  • @J.Rodríguez ya está. Fijate que en el evento del boton enlazo el evento bg.RunWorkerCompleted += bg_RunWorkerCompleted; y luego en el manejador del evento pongo las lineas problematicas y las quito del DoWork
    – Pikoh
    el 8 mar. 2018 a las 16:21
  • @Pikoh, una sugerencia también puede ser: en lugar de usar progressBar1.Value = 100; se pude usar: progressBar1.Value = progressBar1.Maximun;. el 8 mar. 2018 a las 16:22
  • 1
    @MauricioAriasOlave bien visto, aunque la propiedad es Maximum y no MaximumValue :)
    – Pikoh
    el 8 mar. 2018 a las 16:23
1

Lo que a mi me resultó y lo escribo por si alguien como yo viene buscando ayuda al respecto.

EL bucle anda congelando el Hilo principal de la actividad. Entonces lo hay que hacer es enviar el tema del bucle a un nuevo hilo con Thread. EL ejemplo es al subir una foto de perfil a Storage de Firebase. El while comprobaba si ya había link para enviar al imageView, como el subir la imagen tomaba unos segundos, EL bucle while congela todo. De la siguiente forma me funcionó. Aclaro que solo estoy mostrando la parte del código que puse dentro del Thread.

new Thread(
                () -> {
                    while (!uriTask.isSuccessful());

                    if(uriTask.isSuccessful()){
                        uriTask.addOnSuccessListener(uri1 -> {
                            String url = uri1.toString();
                            String id = uidTaxista.getText().toString();
                            cargarUrlFoto(url, id);
                        });
                    }
                    Log.d("FileManager", "uploadImg OK!");
                }
        ).start();

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.