-2

Есть такой код:

public class Pen {

    public Pen() { //конструктор
        Random rand = new Random();
        int i = rand.nextInt(4);
        switch(i) {
            case 0:name=("Parker Urban");break;
            case 1:name=("Parker Jotter");break;
            case 2:name=("Parker IM");break;
            case 3:name=("Parker Insignia");break;
        }
    }

    public void setInfo() {
        Random random = new Random();
        int k = random.nextInt(4);
        switch(k) {
            case 0:System.out.println(name);break;
            case 1:System.out.println(name);break;
            case 2:System.out.println(name);break;
            case 3:System.out.println(name);break;
        }
    }
}

Очевидно, что по нажатию на кнопку нужно, чтобы писалось одно из именований ручки. Но, оно выбирается только 1 раз (например, Parker IM) и в дальнейшем, сколько раз на кнопку не жми - название ручки остается таким же. Я понимаю, что код - костыль, и должно быть написано не так, но что именно ???
Помогите, пожалуйста, решить загадку.

12
  • Покажите какой код вызвается при нажатии на кнопку?
    – anber
    31 мая 2015 в 14:47
  • Ээээ... А чем отличаются случаи в setInfo?
    – VladD
    31 мая 2015 в 15:00
  • каков смысл кода в setInfo() ?
    – Nofate
    31 мая 2015 в 15:56
  • При вызове метода в консоль должно выводиться название ручки. 31 мая 2015 в 16:15
  • вы понимаете, что рандом в setInfo() не делает ничего?
    – Nofate
    31 мая 2015 в 16:20

1 ответ 1

2

Значение name устанавливается у Вас в конструкторе, потом, когда вызывается метод setInfo() он просто выводит name на экран. Независимо от сгенерированного числа в setInfo(), значение name одинаково, т.к. не видно нигде где name перезаписывается.

class Pen {

   String[] array; 
   Random random;        

   public Pen() {

       random = new Random();

       array = new string[4];
       array[0] = "Parker Urban";
       array[1] = "Parker Jotter";
       array[2] = "Parker IM";
       array[3] = "Parker Insignia";
   }


   public void setInfo() {

       System.out.println(array[random.nextInt(4)]);

    }
}
11
  • Почему это рандом генерирует одниковые числа?
    – anber
    31 мая 2015 в 14:46
  • > Но, оно выбирается только 1 раз (например, Parker IM) и в дальнейшем, сколько раз на кнопку не жми - название ручки остается таким же.
    – sp7
    31 мая 2015 в 14:49
  • random.nextInt() возвращает разные числа в независимости от инициализации.
    – anber
    31 мая 2015 в 15:01
  • Возможно, здесь я точно утверждать не могу. Но проблема генерация одинаковых чисел не нова.
    – sp7
    31 мая 2015 в 15:19
  • 1
    @Дмитрий, тогда надо взять учебники и немного их осилить: ru.stackoverflow.com/questions/416634
    – Nofate
    31 мая 2015 в 16:30

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.