0

Quiero generar elementos aleatorios de un array pero que no se repitan.

    Integer [] sonidos_numbers = {0,1,2,3,4,5};


                int valorRandom = getRandom();

                if (valorRandom == 0 )
                {
                    soundPool.play(cabeza,1,1,0,0,1);
                    imagen_1.setOnClickListener(new View.OnClickListener()
                    {
                        @Override
                        public void onClick(View view)
                        {
                            imagen_principal_1.setImageResource(images[images_numbers[0]]);
                            imagen_1.setVisibility(View.INVISIBLE);
                            soundPool.play(correcto,1,1,0,0,1);
                            turns++;

                            checkEnd();
                        }
                    });

    private static int getRandom(Integer[] array)
    {
        Random r = new Random();
        int start = 0;
        int end = 6;
        int playRandom = r.nextInt(end-start) + start;
        return array[playRandom];
    }

En mi código los elementos se generan aleatoriamente pero muchas de las veces se repite, me gustaría que una vez que un elemento ha sido generado no se vuelva a generar.

2 respuestas 2

1

Para estos casos, yo suelo utilizar un método que consiste en:

  1. Iniciar la lista de valores a generar.
  2. Generar un número aleatorio entre 1 y la longitud de la lista.
  3. Seleccionar el elemento de la lista correspondiente a ese número
  4. Eliminar de la lista el elemento seleccionado
  5. Volver al punto 2.

Esto garantiza que no se repiten elementos de forma sencilla y a partir de cualquier tipo de lista de valores a generar, pudiendo ser enteros, caracteres, frases, imágenes, etc...

Te pongo un ejemplo para tu caso concreto en Java/Android, a partir de tu propio ejemplo:

/* debes cambiar la declaración de tu tipo de datos por ArrayList */
/* Integer [] sonidos_numbers = {0,1,2,3,4,5}; */

private static ArrayList<Integer> sonidos_numbers = new ArrayList<>(); 


/* debes iniciar ArrayList con datos para la generación aleatoria */
/* En el constructor de tu clase es un lugar apropiado */
initLista();



/*   .... EL RESTO DE TU CÓDIGO   */
int valorRandom = getRandom();
...

/* declaración nueva para el generador de aleatorios sin repetidos constenidos en el ArrayList declarado inicialmente */
private static int getRandom()
{
    int valorSeleccionado = -1;

    if (sonidos_numbers.size() > 0){
        Random r = new Random();

        /* generar el indice aleatorio a usar */
        int playRandom = r.nextInt(sonidos_numbers.size());

        /* capturar valor a devolver */
        valorSeleccionado = sonidos_numbers.get(playRandom);

        /* reducir LISTA de valores para la próxima generación */
        sonidos_numbers.remove(playRandom);
    }

    return valorSeleccionado;
}


/* Función para inicializar el vector con los datos a generar, en tu caso valores del 0 al 5 */    
private static void initLista()
{
    for (Integer i=0;i<=5;i++){
        sonidos_numbers.add(i);
    }
}
14
  • Hola que tal muchas gracias por la ayuda, al ejecutar el codigo que adjuntaste el valor de valorSeleccionado no es generado el 9 mar. 2020 a las 16:48
  • @cristianrodriguezbarba, Debes usar la función de iniciar la lista InitList antes de llamar la primera vez al getRandom. (es decir, el paso 1 de mi descripción inical) el 9 mar. 2020 a las 17:04
  • he editado el codigo de la pregunta para que puedas ver como me gustaria usar el numero aleatorio de la función getRandom, el 9 mar. 2020 a las 17:21
  • @cristianrodriguezbarba, reeditada mi respuesta para "ofrecerte" una solución a partir de tu código actual... el 9 mar. 2020 a las 17:37
  • he editado tu codigo por mi codigo actual para que tengas una mejor idea de lo que quiero obtener, me gustaria obtener como lo mencione en la pregunta que el valor randomico se genere pero sin repetir, una sola vuelta, espero me hayas entendido el 9 mar. 2020 a las 17:53
1

la solucion es sencilla.... create otro array identico ,donde iras guardando cada nuevo (no repetido) numero aleatorio que se genere, y cada vez que se genere un numero aleatorio con un metodo contains aplicado sobre este nuevo array

por ejemplo asi

 int [] sonidos_numbers = {0,1,2,3,4,5};
        Random r = new Random();

        int start = 0;
        int end = sonidos_numbers.length-1;

      int[] miarrayrepetido = new int[sonidos_numbers.length];

        int nuevonumero=0;

        while(true){

            int playRandom = r.nextInt(end-start) + start;

            if( Arrays.asList(miarrayrepetido).contains(playRandom)==false){//el numeroaleatorio no existe en el array por lo tanto no se ha generado ,no  es repetido
                miarrayrepetido[nuevonumero]=playRandom;
                nuevonumero++;
                break;


            }
        }
    }

con esto se ira generando numeros aleatorios , hasta que se genere un numero aleatorio que no este guardado en miarray, y por lo tanto que no se haya generado

2
  • Buenos dias como estas, tengo dos preguntas estoy trabajando con sonidos y no se si por esa razon la condición del if me genera un error en el .contains, y la otra es que retornaria del private static get Random, muchas gracias el 9 mar. 2020 a las 14:04
  • Actualizado el codigo de mi solucion con tu nuevo codigo, mira como funciona ahora, lo unico que deberas hacer , es agrupar o retocar este codigo para que funcione a tu gusto , según como lo tengas en la aplicación, por ejemplo, si quieres que se genere un nuevo numero aleatorio cada vez que se presiona un button
    – iqanansoft
    el 10 mar. 2020 a las 1:28

Tu Respuesta

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

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