Публичный класс DiscoLight help - PullRequest
0 голосов
/ 07 апреля 2010

Если кто-то может указать мне правильное направление для этого кода для моего назначения, я был бы очень признателен.

Я вставил весь код, который мне нужно завершить, но мне нужна помощь со следующим методом public void changeColour(Circle aCircle), который предназначен для случайного изменения цвета круга, если 0 приходит, свет круга измените на красный, 1 на зеленый и 2 на фиолетовый.

public class DiscoLight
{
   /* instance variables */
   private Circle light;   // simulates a circular disco light in the Shapes window
   private Random randomNumberGenerator;

   /**
    * Default constructor for objects of class DiscoLight
    */
   public DiscoLight()
   {
      super();
      this.randomNumberGenerator = new Random();           
   }


   /**
    * Returns a randomly generated int between 0 (inclusive) 
    * and number (exclusive). For example if number is 6,
    * the method will return one of 0, 1, 2, 3, 4, or 5.
    */
   public int getRandomInt(int number)
   {
      return this.randomNumberGenerator.nextInt(number);
   }


   /** 
    * student to write code and comment here for setLight(Circle) for Q4(i)
    */
   public void setLight(Circle aCircle)
   {
       this.light = aCircle;

   }


   /**
    * student to write code and comment here for getLight() for Q4(i)
    */
   public Circle getLight()
   {
       return this.light;
   }

   /** 
    * Sets the argument to have a diameter of 50, an xPos 
    * of 122, a yPos of 162 and the colour GREEN.
    * The method then sets the receiver's instance variable
    * light, to the argument aCircle.
    */ 
   public void addLight(Circle aCircle)
   {
      //Student to write code here, Q4(ii)
      this.light = aCircle;
      this.light.setDiameter(50);
      this.light.setXPos(122);
      this.light.setYPos(162);
      this.light.setColour(OUColour.GREEN);
   }


   /** 
    * Randomly sets the colour of the instance variable 
    * light to red, green, or purple.
    */  
   public void changeColour(Circle aCircle)
   {
       //student to write code here, Q4(iii)
       if (getRandomInt() == 0)
       {
           this.light.setColour(OUColour.RED);
       }
       if (this.getRandomInt().equals(1))
       {
           this.light.setColour(OUColour.GREEN);
       }
       else
       if (this.getRandomInt().equals(2))
       {
            this.light.setColour(OUColour.PURPLE);
       }
   }


   /** 
    * Grows the diameter of the circle referenced by the 
    * receiver's instance variable light, to the argument size.  
    * The diameter is incremented in steps of 2,
    * the xPos and yPos are decremented in steps of 1 until the
    * diameter reaches the value given by size. 
    * Between each step there is a random colour change.  The message 
    * delay(anInt) is used to slow down the graphical interface, as required.
    */   
   public void grow(int size)
   {   
       //student to write code here, Q4(iv) 
   }


   /** 
    * Shrinks the diameter of the circle referenced by the 
    * receiver's instance variable light, to the argument size.  
    * The diameter is decremented in steps of 2,
    * the xPos and yPos are incremented in steps of 1 until the
    * diameter reaches the value given by size. 
    * Between each step there is a random colour change.  The message 
    * delay(anInt) is used to slow down the graphical interface, as required.
    */     
   public void shrink(int size)
   {   
       //student to write code here, Q4(v)
   }


   /** 
    * Expands the diameter of the light by the amount given by
    * sizeIncrease (changing colour as it grows).
    * 
    * The method then contracts the light until it reaches its
    * original size (changing colour as it shrinks).
    */     
   public void lightCycle(int sizeIncrease)
   {
      //student to write code here, Q4(vi)
   }


   /** 
    * Prompts the user for number of growing and shrinking
    * cycles. Then prompts the user for the number of units
    * by which to increase the diameter of light.
    * Method then performs the requested growing and 
    * shrinking cycles.
    */     
   public void runLight()
   {  
    //student to write code here, Q4(vii)
   }  


   /**
    * Causes execution to pause by time number of milliseconds
    */
   private void delay(int time)
   {
      try
      {
         Thread.sleep(time); 
      }
      catch (Exception e)
      {
         System.out.println(e);
      } 
   }

}

Ответы [ 4 ]

1 голос
/ 07 апреля 2010

Вы забыли передать параметр при вызове getRandomInt() в самой первой строке ниже //student to write code here. Ваш компилятор должен уже указать на это.

Комментарий документации выше метода getRandomInt() сообщает вам, что он ожидает в качестве параметра, и что вы можете ожидать в качестве возвращаемого значения.

Кроме того, если вы хотите равных шансов, что лампа красного, зеленого или фиолетового цвета, вы можете сделать это за один вызов getRandomInt(). Сохраните значение в переменной и используйте инструкцию switch для включения правильного освещения:

int randomValue = getRandomInt(/* I am not telling you what to put here */);

switch (randomValue) {
    case 0: light.setColour(OUColour.RED); break;
    case 1: light.setColour(OUColour.GREEN); break;
    case 2: light.setColour(OUColour.PURPLE); break;
}
0 голосов
/ 07 апреля 2010
 /** 
    * Randomly sets the colour of the instance variable 
    * light to red, green, or purple.
    */  
   public void changeColour(Circle aCircle)
   {
       int i = getRandomInt(3);
       if (i == 0)
       {
           this.light.setColour(OUColour.RED);
       }
       else if (i == 1)
       {
           this.light.setColour(OUColour.GREEN);
       }
       else
       {
            this.light.setColour(OUColour.PURPLE);
       }
   }
0 голосов
/ 07 апреля 2010

вы вызываете getRandomInt несколько раз, каждый раз возвращается новое (случайное) значение. Вы должны вызвать его один раз в начале метода, а затем проверить, равно ли оно 0,1 или 2.

С уважением Гийом

0 голосов
/ 07 апреля 2010

метод getRandomInt возвращает int, поэтому вы не можете использовать equals для сравнения. Применение:

if (getRandomInt(3) == 1) {
  ...
}

и вам просто нужно позвонить один раз. сохраните случайное целое число в переменной и сравните его значение с теми, которые вам нужны.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...