Removing Method

[Bgreen7887]

//********************************************************************
//  CDCollection.java       Author: Lewis/Loftus
//
//  Represents a collection of compact discs.
//********************************************************************

import java.text.NumberFormat;

public class CDCollection
{
   private CD[] collection;
   private int count;
   private double totalCost;
	private CD[] temp;

   //-----------------------------------------------------------------
   //  Constructor: Creates an initially empty collection.
   //-----------------------------------------------------------------
   public CDCollection ()
   {
      collection = new CD[100];
      count = 0;
      totalCost = 0.0;
		[B]
   }///start method
	
	public String removeCd(String art) {
	 int i =0;
	 while (!collection[i].equals(art)){
	 i++;
	 }
	 
	 int j = 0;
	 temp= new CD[collection.length-1];
	 for ( int j = 0; j< collection.length-1; j++) {
	 		collection[j]=temp[j]) 
			return System.out.println(collection[j]);
			
			[/B]
			
	 }
	 
	 
	}// Trying to make a remove method that takes deletes a cd and shifts the space(The part in bold)...

[m00nchile]

Unless the array is some sort of requirement, if you're doing something that has a lot of add, remove and lookup commands, I'd suggest using one of the List implementations, ArrayList for example. Look it up.

[Bgreen7887]

Can anyone tell me why im getting a null pointer exception plzz b4 12 a.m..

//********************************************************************
//  CDCollection.java       Author: Lewis/Loftus
//
//  Represents a collection of compact discs.
//********************************************************************

import java.text.NumberFormat;

public class CDCollection
{
   private CD[] collection;
   private int count;
   private double totalCost;
	private CD[] temp;

   //-----------------------------------------------------------------
   //  Constructor: Creates an initially empty collection.
   //-----------------------------------------------------------------
   public CDCollection ()
   {
      collection = new CD[100];
      count = 0;
      totalCost = 0.0;
		
   }///s[B]tart method
	
	public void removeCD(String title) {
	 int i =0;
	 while (!collection[i].equals(title));
	 {
				 i++;
	 for (int g=i; g<collection.length-1;g++){
	 		if (g==collection.length-1) {
				collection[collection.length -1] =null;
				 } else {
				 collection[g] = collection[g+1];
				 }
	 			}
	 int j = 0;
	 temp= new CD[collection.length-1];
	 for ( j = 0; j< collection.length-1; j++) {
	 		collection[j]=temp[j] ;
			}
	 	}
	 }
	[/B]
   //-----------------------------------------------------------------
   //  Adds a CD to the collection, increasing the size of the
   //  collection if necessary.
   //-----------------------------------------------------------------
   public void addCD (String title, String artist, double cost,
                      int tracks)
   {
      if (count == collection.length)
         increaseSize();

      collection[count] = new CD (title, artist, cost, tracks);
      totalCost += cost;
      count++;
   }


[CODE]//********************************************************************
//  Tunes.java       Author: Lewis/Loftus
//
//  Demonstrates the use of an array of objects.
//********************************************************************

public class Tunes
{
   //-----------------------------------------------------------------
   //  Creates a CDCollection object and adds some CDs to it. Prints
   //  reports on the status of the collection.
   //-----------------------------------------------------------------
   public static void main (String[] args)
   {
      CDCollection music = new CDCollection ();

      music.addCD ("Storm Front", "Billy Joel", 14.95, 10);
      music.addCD ("Come On Over", "Shania Twain", 14.95, 16);
      music.addCD ("Soundtrack", "Les Miserables", 17.95, 33);
      music.addCD ("Graceland", "Paul Simon", 13.90, 11);
	  music.removeCD("Double live");
      System.out.println (music);

      music.addCD ("Double Live", "Garth Brooks", 19.99, 26);
      music.addCD ("Greatest Hits", "Jimmy Buffet", 15.95, 13);

      System.out.println (music);
   }
}

//-----------------------------------------------------------------
// Returns a report describing the CD collection.
//-----------------------------------------------------------------
public String toString()
{
NumberFormat fmt = NumberFormat.getCurrencyInstance();

String report = "~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n ";
report += "My CD Collection\n\n";

report += "Number of CDs: " + count + "\n";
report += "Total cost: " + fmt.format(totalCost) + "\n";
report += "Average cost: " + fmt.format(totalCost/count);

report += "\n\nCD List:\n\n";

for (int cd = 0; cd < count; cd++)
report += collection[cd].toString() + "\n";

return report;
}

//-----------------------------------------------------------------
// Increases the capacity of the collection by creating a
// larger array and copying the existing collection into it.
//-----------------------------------------------------------------
private void increaseSize ()
{
CD[] temp = new CD[collection.length * 2];

for (int cd = 0; cd < collection.length; cd++)
temp[cd] = collection[cd];

collection = temp;
}
}
[/CODE]

[m00nchile]

temp= new CD[collection.length-1];
	 for ( j = 0; j< collection.length-1; j++) {
	 		collection[j]=temp[j] ;
			}
	 	}
	 }

Your temp array is one element shorter than the collection array, so the last iteration of the for loop spawns the exception.
Also, did you think about my suggestion on using a List?