AddAfter method

[Jere Johnson]

I need help figuring out how to add an element after an existing element in an array. The isCurrent boolean checks to see if there is a current element. I almost have it. I need someone to show me what im missing. Thanks,

<font size="2">  
public void addAfter(double element)
   {
   if (isCurrent()) {
   advance();
   data[currentIndex] = element;
   manyItems = currentIndex+1;
    }
    
       else if (!isCurrent()){
       currentIndex=0;
       manyItems=1;
       data[currentIndex] = element;}
 </font>

[ October 03, 2003: Message edited by: Jere Johnson ]

[Jeff Bosch]

Hi, Jere -
I don't know what problem the code is giving you, but I would recommend checking for the end of the array to avoid the ArrayIndexOutOfBounds exception.
Also, I'm assuming "manyItems" is your element count?
I also see a mismatched number of curly braces. The method addAfter() is not closed.
Maybe if you provide a little more context, like the class this method belongs to and its method signatures, I could be of further help.

[Jeff Bosch]

Another quick point:
I would explicitly test for the boolean "true": if ( isCurrent() == true ).
Then, you don't really need "else if" because you've already done the test. The else-if can be reduced to a simple else{} block.