[ Foro de C# ]

Ejercicio 6.3

02-Apr-2016 11:41
jose Conejero
1 Respuestas

Buenos días:

El programa esta hecho con visual Studio 2015.

Programa que produzca numero aleatorio entre (200 y 400) cada vez que pulse el botón.el programa deberá mostrar ese numero junto con la suma y el promedio  de todos los números recibidos hasta ese momento.a medida que usted pulse el botón  repetidamente el promedio deberá llegar a 300.

Lo que tengo hasta el momento es lo siguiente:


using System;
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 Ejercicio_6._3
{
    public partial class Form1 : Form
    {
        private Random Aleatorio = new Random();
        private int Intentos = 0;
        private int Intentosx = 0;

        public Form1()
        {
            InitializeComponent();
            lbl2.Text = Convert.ToString(Aleatorio.Next(200, 400));
        }

        private void button1_Click(object sender, EventArgs e)
        {
           
            int Intentos1;
            int Intentos2;
            int n ;
            int n2;
            int sumat;
            Intentos = 0;
            Intentosx = 0;
            lbl2.Text = Convert.ToString(Aleatorio.Next(200, 400));
            Intentos1 = Convert.ToInt32(lbl2.Text);
            n = Intentos1;
            Intentos2 = Convert.ToInt32(lbl2.Text);
            Intentos2 = n;
            n2 = 0;
            n2 = Intentos2;
            sumat = n + n2;
            lbl4.Text = Convert.ToString(sumat);
        }
      
        
    }
}



Muchas gracias


03-Apr-2016 23:36
Nacho Cabanes (+83)

No entiendo por qué vuelves a dar a "intentos" el valor 0 cada vez que se pulsa el botón. Si no sabes cuántos intentos hay, no podrás calcular la media.

Tampoco queda claro qué hace la variable "intentosX" (el nombre no deja claro qué diferencia tiene con "intentos"), y lo mismo pasa con tus "label": un nombre como "lbl4" no indica qué información va a mostrar esa etiqueta; sería mejor usar nombres como "lblPromedio" o "lblNumeroAzar".

En principio, te basta con: dos propiedades, que serán la cantidad de intentos y la suma hasta ese momento. Ambas comenzarán valiendo 0. El número al azar será una variable local de la función que se lanza en el evento Click. No deberías necesitar más propiedades ni más variables locales.






(No se puede continuar esta discusión porque tiene más de dos meses de antigüedad. Si tienes dudas parecidas, abre un nuevo hilo.)