[SOLUCIONADO] Serie de numeros aleatorios en un TextBox

[jmfmagnum]

Tal como dice el titulo, en el proyecto de loteria que tengo de tarea por nota final, pide la opcion de crear una serie de 14 numeros al azar que no se repitan, lo primero fue relativamente facil

Código C:

private void button29_Click(object sender, EventArgs e)
        {
            Random r = new Random();
            int tal = r.Next(1, 25);
            int tal1 = r.Next(1, 25);
            int tal2 = r.Next(1, 25);
            int tal3 = r.Next(1, 25);
            int tal4 = r.Next(1, 25);
            int tal5 = r.Next(1, 25);
            int tal6 = r.Next(1, 25);
            int tal7 = r.Next(1, 25);
            int tal8 = r.Next(1, 25);
            int tal9 = r.Next(1, 25);
            int tal10 = r.Next(1, 25);
            int tal11 = r.Next(1, 25);
            int tal12 = r.Next(1, 25);
            int tal13 = r.Next(1, 25);
           TxTNums.Text = TxTNums.Text + tal.ToString()+" ";
           TxTNums.Text = TxTNums.Text + tal1.ToString()+" ";
           TxTNums.Text = TxTNums.Text + tal2.ToString()+" ";
           TxTNums.Text = TxTNums.Text + tal3.ToString()+" ";
           TxTNums.Text = TxTNums.Text + tal4.ToString()+" ";
           TxTNums.Text = TxTNums.Text + tal5.ToString()+" ";
           TxTNums.Text = TxTNums.Text + tal6.ToString()+" ";
           TxTNums.Text = TxTNums.Text + tal7.ToString()+" ";
           TxTNums.Text = TxTNums.Text + tal8.ToString()+" ";
           TxTNums.Text = TxTNums.Text + tal9.ToString()+" ";
           TxTNums.Text = TxTNums.Text + tal10.ToString()+" ";
           TxTNums.Text = TxTNums.Text + tal11.ToString()+" ";
           TxTNums.Text = TxTNums.Text + tal12.ToString()+" ";
           TxTNums.Text = TxTNums.Text + tal13.ToString();
        }
 

Pero para evitar que se repitan, es otra historia.......

Investigando encontre el uso de arraylist

Código C:

public ArrayList NumerosRandom(int max)          
          {      
              // Crea un ArrayList para almacenar los números      
              ArrayList lstNumeros = new ArrayList();              
              // Ésto va a generar los números        
              Random rnd = new Random();                
              // Generar un numero del 1 a su máximo        
              int number = rnd.Next(1,25);          
              // Añadir el primero numero a la lista      
              lstNumeros.Add(number);          
              // ´Settear´ el numero donde comenzará a contar (para verificar si existe un numero igual)          
              int count = 0;                
              do                
              {            
                  // Generar un numero del 1 a su máximo                    
                  number = rnd.Next(1,25);                  
                  // Si el nuevo numero no está en la lista pues añádelo                    
                  if (!lstNumeros.Contains(number))                    
                  {                        
                      // Añádelo                        
                      lstNumeros.Add(number);                    
                  }                    
                  count++;                
              } while (count <= 10 * max);                
 
              // Una vez terminado de procesar la lista y añadir los números nuevos, enviarlos              
              return lstNumeros;          
          }      
 
private void button29_Click(object sender, EventArgs e)
       {
 
           Numeros nmr = new Numeros();
           const int Total = 25;
           // Aquí va el número máximo 1-9999            
           ArrayList lstNumeros =nmr.NumerosRandom(Total);            
           for (int i = 0; i < lstNumeros.Count; i++)            
           {                
               TxTNums.Text = lstNumeros[i].ToString()+“ ” ;
 
           }
 

En si el problem es: "Numeros nmr = new Numeros();" no se que funcion cumple, como no existe en el array y esta definido como clase lo encontre raro asi que lo saque el boton quedo asi

Código C:

private void button29_Click(object sender, EventArgs e)
       {
 
 
           const int Total = 25;
           // Aquí va el número máximo 1-9999            
           ArrayList lstNumeros = NumerosRandom(Total);            
           for (int i = 0; i < lstNumeros.Count; i++)            
           {                
               TxTNums.Text = lstNumeros[i].ToString()+“ ” ;
 
           }
 

Y solo muestra un numero en el textbox, ojala no sea mucha molestia el preguntar como poder escribir 14 numeros aleatorios , en el rango de 1 a 25

[Malenko]

Y si en lugar de usar 14 variables usases una lista? Además de que te ayudaría muchísimo a encontrar la solución al problema, es muy feo el como estás resolviéndolo ahora mismo, es una solución muy poco mantenible.

[Drako_18]

Buenas compañero,

Concatena los resultados.... No los guardes tal cual.


private void button29_Click(object sender, EventArgs e)
{
const int Total = 25;
// Aquí va el número máximo 1-9999
ArrayList lstNumeros = NumerosRandom(Total);
for (int i = 0; i < lstNumeros.Count; i++)
{
TxTNums.Text = lstNumeros[i].ToString()+“ ” ;
}


Un saludo!!

[Malenko]

Es más óptimo trabajar con un StringBuilder para concatenar dentro del for y al final hacer el ToString al textbox ;)

[jmfmagnum]

Iniciado por Drako_18

Buenas compañero,

Concatena los resultados.... No los guardes tal cual.


private void button29_Click(object sender, EventArgs e)
{
const int Total = 25;
// Aquí va el número máximo 1-9999
ArrayList lstNumeros = NumerosRandom(Total);
for (int i = 0; i < lstNumeros.Count; i++)
{
TxTNums.Text = lstNumeros[i].ToString()+“ ” ;
}


Un saludo!!

a Malenko: esa es la idea, cambiar el mar que hize arriba, canbaindolo por lo que puse abajo.
para Drako:
Y como concateno resultados?, pusiste lo mismos que habia puesto en el boton sin cambio alguno , lamento preguntar tanto, cuando tengo trabajos de programar me pongo nervioso

[Malenko]

Iniciado por jmfmagnum

cuando tengo trabajos de programar me pongo nervioso

Efectivamente, es tu trabajo programarlos. Ya se te han dado herramientas para que pudieras hacerlo tu mismo y esperamos que así sea. Cuando hayas aplicado cambios respecto a tu código original, subelo y entonces volveremos a examinarlo. Lo que no vamos a hacer es programar tus tareas porque entonces no vas a aprender nada.

Así pues, quedamos a la espera de que apliques los cambios.

[Drako_18]

Buenas compañero,

Estoy con Malenko, concatenar en XX lenguaje, lo vas a encontrar con una simple búsqueda en Google, te animo a hacerla!!! ;)


¡Saludos!

[jmfmagnum]

bueno ahi lo encontre

Código C++:

private void button29_Click(object sender, EventArgs e)
        {
            Random rnumero = new Random();
      
            List<int> numeros = new List<int>();
            while (numeros.Count < 14)
            {
                int num = rnumero.Next(1, 25);
                if (!numeros.Any(x => x == num))//revisa que el numero no se repita
                {
                    numeros.Add(num);
                    TxTNums.Text = TxTNums.Text + " " + num.ToString();
                }
 
            }
        }
 

lamento la molestia, ahi encontre la solucion , gracias a ambos por su tiempo, este tema esta solucionado

[Malenko]

La solución esta bastante bien, lo único que cambiaría sería la asignación a TxTNums al salir del bucle while. Es muchísimo más óptimo trabajar en el while con un StringBuilder dentro del while y al terminar asignarle el resultado al TxtNums. En un entorno profesional el cambio que te comento sería "obligatorio" por temas de rendimiento, pero siendo una solución para un ejercicio de clase, si comparas tu último código con el primero que pusiste verás que hay un abismo. Muy bien :)