2015-05-11 3 views
0

Я пытаюсь выяснить код для проекта, но я не совсем понимаю его. Цель состоит в том, чтобы написать программу, которая генерирует набор из 10 случайных чисел и хранит их в массиве (правильно отображаются 10 номеров в текстовом поле). Он должен иметь кнопку, которая будет вычислять минимальные, максимальные и средние значения массива. Кажется, я не могу получить значения max и min из текстового поля. Я знаю, что люди не публикуют ответы, и я не ищу этого, но если кто-то может сказать мне, почему то, что я делаю для max/min, неверно, или указать мне в правильном направлении, я был бы признателен. Спасибо!Получение верхнего и нижнего из текстового поля

Private Sub Button1_Click(sender As Object, e As EventArgs) Handles Button1.Click 


    Dim randomArray As New Random() 
    Dim randomNumber As Integer 

    For i = 1 To 10 
     randomNumber = randomArray.Next(1, 101) 
     displaynumbersTextBox1.AppendText(randomNumber & " ") 
     displaynumbersTextBox1.Text = Convert.ToString(randomNumber) 
    Next 
End Sub 

Private Sub Button2_Click(sender As Object, e As EventArgs) Handles Button2.Click 
    Dim randomArray As New Random() 
    Dim randomNumber As Integer 

    For i = 1 To 10 
     randomNumber = randomArray.Next(1, 101) 
     displaynumbersTextBox1.AppendText(randomNumber & " ") 
    Next 

    Dim min As Integer 
    min = displaynumbersTextBox1.Text.GetLowerBound() 
    displayminTextBox2.Text = min 
End Sub 
+0

Вам нужно получить логику за пределами обработчиков кнопок (и назовите эти кнопки для любви ко всему, что хорошо!), Так что * Button2 * может знать, что * Button1 * сделал. –

+1

'люди не публикуют ответы * * что ???? * SO все ответы. Когда вы публикуете свои номера в TextBox, они становятся текстом, который не является средним (что среднее из «foo» и «red»?). Во-вторых, текстовое поле не будет знать, что вы разместили массив. UI COntrols создают ужасные переменные. Используйте массив, выполните математику и покажите пользователю результаты в элементах управления пользовательским интерфейсом. – Plutonix

ответ

0

Для начала, я вижу этот код в Button1:

TextBox1.AppendText(randomNumber & " ") 
TextBox1.Text = Convert.ToString(randomNumber) 

2-я линия отрицает первое. Просто удалите его. Вместо этого очистите текстовое поле в начале обработчика нажатия кнопки.

Далее, в кнопке 2 код проходит и воссоздает новый набор случайных чисел вместо использования набора, созданного с помощью первой кнопки. Более того, цифры никогда не сохраняются в массиве. Вам нужен массив, объявленный вне любой из кнопок, в коде для Button1 задаются элементы массива, поэтому вы можете использовать эти числа снова в Button2 более легко.