Домашнее задание Java Array

Question

Назначение требует, чтобы пользователь вводил 3 радиуса и 3 записи высоты, которые я собираю в массиве, а затем определяю объем для каждого. Я застрял в массиве. По какой-то причине я получаю ArrayIndexOutOfBoundsException.

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 4 
(CylinderTest.java:19) 

Я получаю сообщение об ошибке на последнем (шестое или третье место). Я не понимаю, что я делаю неправильно. Мне трудно понять логику, это моя самая большая проблема.

Вот CylinderTest (основной)

import javax.swing.*; 

//Driver class 
public class CylinderTest 
{ 

    public static void main(String[] args) 
    { 

     Cylinder[] volume = new Cylinder[3]; 

     for (int counter = 0; counter < 6; counter++) 
     { 
      double radius = Double.parseDouble(JOptionPane 
        .showInputDialog("Enter the radius")); 
      double height = Double.parseDouble(JOptionPane 
        .showInputDialog("Enter the height")); 
      volume[counter++] = new Cylinder(radius, height); 
     } 

     String display = "Radius\tHeight\n"; 
     for (Cylinder i : volume) 
     { 
      if (i != null) 
       display += i.toString() + "\n"; 
     } 
     JOptionPane.showMessageDialog(null, display); 
    } 
} 

и здесь класс Cylinder

public class Cylinder 
{ 
    // variables 
    public static final double PI = 3.14159; 
    private double radius, height, volume; 

    // constructor 
    public Cylinder(double radius, double height) 
    { 
     this.radius = radius; 
     this.height = height; 
    } 

    // default constructor 
    public Cylinder() 
    {this(0, 0);} 

    // accessors and mutators (getters and setters) 
    public double getRadius() 
    {return radius;} 

    public void setRadius(double radius) 
    {this.radius = radius;} 

    public double getHeight() 
    {return height;} 

    public void setHeight(double height) 
    {this.height = height;} 

    public double getVolume() 
    {return volume;} 

    public void setVolume(double volume) 
    {this.volume = volume;} 

    // Volume method to compute the volume of the cylinder 
    public double volume() 
    {return PI * radius * radius * height;} 

    public String toString() 
    {return volume + "\t" + radius + "\t" + height; } 

} 

Answer 1

Ваш массив имеет размер 3, определяется этой линии:

Cylinder[] volume = new Cylinder[3]; 

Тогда вы начинаете от 0 до 5:

for (int counter = 0; counter < 6; counter++) 

И затем вы пытаетесь получить доступ к одному из этих показателей здесь:

volume[counter++] = new Cylinder(radius, height); 

Поскольку массив имеет длину 3, он имеет только индексы 0, 1 и 2. И все же вы пытаетесь индексы доступа выше два.

В качестве побочного примечания я предлагаю вам изменить инструкцию на volume[counter] = new Cylinder(radius, height);, иначе вы увеличиваете переменную for-loop counter дважды на каждой итерации.

Хорошая практика, когда цикл через индексы в массиве, чтобы использовать длину массива в состоянии:

for (int counter = 0; counter < volume.length; counter++) 

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

Answer 2

Вы объявляете массив цилиндров 3 размера, и вы пытаетесь прочитать 6 из них.

Cylinder[] volume = new Cylinder[3]; // 3 size array 

     for (int counter = 0; counter < 6; counter++) // loop 6 times 
     { 
      double radius = Double.parseDouble(JOptionPane 
        .showInputDialog("Enter the radius")); 
      double height = Double.parseDouble(JOptionPane 
        .showInputDialog("Enter the height")); 
      volume[counter++] = new Cylinder(radius, height); // read 0, 1, 2, 3, 4, 5... 
     } 

Это должно быть:

Cylinder[] volume = new Cylinder[3]; // 3 size array 

     for (int counter = 0; counter < volume.length; counter++) // loop 6 times 
     { 
      double radius = Double.parseDouble(JOptionPane 
        .showInputDialog("Enter the radius")); 
      double height = Double.parseDouble(JOptionPane 
        .showInputDialog("Enter the height")); 
      volume[counter] = new Cylinder(radius, height); 
     } 

Обратите внимание на volume.length вместо 6 и ++ удалены в объеме [счетчик ++]

Answer 3

Прежде всего, ваш массив был объявлен с размером три. Но в вашем цикле for вы получаете доступ как минимум к 6 элементам массива. Таким образом, вы будете иметь увеличить размер вашего массива по крайней мере 6. И вы должны изменить код:

volume[counter++] = new Cylinder(radius, height); 

Для

volume[counter] = new Cylinder(radius, height); 

Answer 4

Ваш Cylinder массив имеет только три элемента. Cylinder[] volume = new Cylinder[3];

Ваш for loop пытается получить доступ к элементам после элемента 2 в этом массиве. Эти элементы не существуют.

Answer 5

Уверены ли вы в этом?

Cylinder[] volume = new Cylinder[3]; 

    for (int counter = 0; counter < 6; counter++) 
    { 
     double radius = Double.parseDouble(JOptionPane 
       .showInputDialog("Enter the radius")); 
     double height = Double.parseDouble(JOptionPane 
       .showInputDialog("Enter the height")); 
     volume[counter++] = new Cylinder(radius, height); 
     //counter will count up to 5 (Array out of Bounds exception for sure..) 
     //also: why are you incrementing counter by yourself? 
    }