[Resolvido]Problemas para mostrar media

import javax.swing.*;

public class exerc16
{
   public void exerc16()
   {
      int valor = 0;
      int contador = 0;
      int total = 0;
      
      while(valor !=  -1)
      {
         valor = Integer.parseInt (JOptionPane.showInputDialog(null, "Digite valores positivos. Para sair digita -1"));
         if (valor > -1)
         {
            total = total + valor;
            contador++;
         }
         
         else if(valor <= -1)
         {
            int media = total / contador;
            JOptionPane.showMessageDialog(null, "A media dos valores eh: " +media);
         }
      }
   }    
}

import javax.swing.*;  
  
public class exerc16  
{  
   public void exerc16()  
   {  
      int valor = 0;  
      int contador = 0;  
      int total = 0;  
        
      while ( (valor = Integer.parseInt (JOptionPane.showInputDialog(null, "Digite valores positivos. Para sair digita -1")) ) >  -1)  // atente-se aos parênteses nessa linha entre ( valor = ... ) > -1
      {  
         total = total + valor;  
         ++contador;  
      }
      
      if (contador > 0) {
         int media = total / contador;
         JOptionPane.showMessageDialog(null, "A media dos valores eh: " +media);
      }
   }      
}

1 - Tendo em vista que você pediu para o usuário digitar apenas valores positivos, não faz sentido o while ter valor != -1;

2 - Como o -1 é seu sentinela, você não precisa fazer um if dentro do while para testá-lo, pode fazer a atribuição a valor entre parênteses que assim que valor tiver o número irá testar se é -1 e caso positivo sairá;

3 - O ++ na frente de contador dá um pouco mais de performance ao seu algoritmo, tendo em vista que se trata de um comando isolado (apenas incrementar o contador), não afetará outras variáveis dentro do while;

4 - O else que você colocou não é necessário, pois quando o laço terminar, obrigatoriamente você terá de calcular e exibir a média;

5 - O if final serve para garantir que a média não será dividida por zero, mesmo que também seja zero;