Thread: phone book

Can you please define what you mean by 'bunch of problems'?

it doesn't compile.... if i take the array out...it works fine....i can enter one contact and print out the information of that contact....but when i start to work with the array ecc... the programs didn't work anymore

For future reference, please post all compiler errors. Right now, it seems you are trying to use variables outside their scope. You create Lists in the constructors which I am guessing should be instance variables, right now the loose scope after those methods exit so trying to reference them in another method won't compile.

sorry i didn't understand what should i do

Create list as an instance variable...click on the link (blue text) in my post above.

still....doesn't work....grgrgrgrgrg

For future reference, please use the code tags (read the signature of aussiemcgr for details on how to use the highlight tags).

Once again, even if you have, re-read the link I posted above, because it contains information in there that you are missing. The code you posted is still creating local instances which lose scope after the method exits...

public void list(Contact size){
         Contact[] array_list = new Contact[1000];
}

...creates a new Contact array, which looses scope after the method exits, with the instance variable still not instantiated...

public void list(Contact size){
         array_list = new Contact[1000];
}

...instantiates the instance variable array_list as an array of type Contact and size 1000 (the values it contains still have yet to be instantiated however). Redefining the variable in the method just creates a local variable and does not use the instance variable

still...I think i did the change u suggested but it doesn't work

ummm...I don't get it...I'm trying to follow your suggestion but it doesn't work can u be more precise

ummm...I don't get it...I'm trying to follow your suggestion but it doesn't work can u be more precise

Define 'doesn't work', and please post any updated code you have thus far.

public void printContact() 
   { 
      System.out.println("Name = " + name); 
      System.out.println("Last Name = " + lastname); 
      System.out.println("Address = " + address); 
      System.out.println("E-mail = " + email); 
      System.out.println("Phone = " + phone); 
      System.out.println("Note(s) = " + notes); 
   } 
 
   ContactList con = new ContactList();
 
   }
 
public class ContactList 
{
   public ContactList()
   {
      Contact c;
      Contact[] array_list;
   }
 
      public void List(Contact size) 
      {
         Contact[] array_list = new Contact[1000];
         }
 
   public void print() 
   {
 
      int c;
      Object array_list;
      for (c = 0; c < array_list; c++) 
      {
         System.out.println(array_list[c]);
      }
 
   }
 
   public void NewContact()
   {
 
   }
   public void sort()
   {
 
   }
 
   }

First things first: Is there a main somewhere? It would be easier to see all your code because the way its written itll just make contact c and your contact array and exit

2)

public void printContact() 
   { 
      System.out.println("Name = " + name); 
      System.out.println("Last Name = " + lastname); 
      System.out.println("Address = " + address); 
      System.out.println("E-mail = " + email); 
      System.out.println("Phone = " + phone); 
      System.out.println("Note(s) = " + notes); 
   } 
 
   ContactList con = new ContactList();
 
   }

I spy two many close brackets? it seems possible

public void List(Contact size) 
      {
         Contact[] array_list = new Contact[1000];
         }

I'm thinking you don't mean to pass a Contact object named size?? I think you want to pass an int that you would put where the 1000 is.

   public void print() 
   {
 
      int c;
      Object array_list;
      for (c = 0; c < array_list; c++) 
      {
         System.out.println(array_list[c]);
      }
 
   }

i don't think that println will work. or atleast they way you think it will. You would want to use your printContact method and pass the contact array_list[c] to it. Then do array_list[c].name etc...

If you need more help, you can pm me. i threw together a quick program that'll do what i think your trying to do and i can give you more pieces of that if you need help.