prob mit add in einer arry list

[Stoffl]

public class Bsp3_0726406_2 {
    



static int hold = 1;
 
static String[][] addressBookHolder = new String[10000][4];
 
 
 
public static void main(String[] args){
 
Adressbuch ABCD = new Adressbuch();
 
ABCD.addEntry("Mr. ", "A", "1", "mail");
 
ABCD.addEntry("Mr. ", "A", "2", "mail");
 
ABCD.addEntry("Mr. ", "A", "0", "mail");
 
ABCD.addEntry("Mr. ", "A", "0", "mail");
 
 
 
System.out.println("Adressbuch: ");
 
ABCD.showAdressbuch();
 
 
 
System.out.println("Edited addressbook: ");
 
ABCD.editEntry(3, "Mr. ", "Duke nukem", "+00 123456", "Behind pc");
 
ABCD.showAdressbuch();
 
 
 
System.out.println("Select one person from addressbook: ");
 
ABCD.showAdressbuch(2);
 
 
 
System.out.println("Search one person from addressbook: ");
 
ABCD.searchAdressbuch("Ark");
 
ABCD.searchAdressbuch("JavafDorums");
 
}
 
 
 
public void addEntry(String anrede, String name, String telephonNummer, String mailAdresse){
 
addressBookHolder[hold][0] = anrede;
 
addressBookHolder[hold][1] = name;
 
addressBookHolder[hold][2] = telephonNummer;
 
addressBookHolder[hold][3] = mailAdresse;
 
hold++;
 
}
 
 
 
public void editEntry(int id, String anrede, String name, String telephonNummer, String mailAdresse){
 
addressBookHolder[id][0] = anrede;
 
addressBookHolder[id][1] = name;
 
addressBookHolder[id][2] = telephonNummer;
 
addressBookHolder[id][3] = mailAdresse;
 
}
 
 
 
//delete an entry from the addressbook
 
public void deletEntry(String name){}
 
 
 
public void showAdressbuch(int id){
 
System.out.println();
 
 
 
System.out.println("Aufrufzahl: " + id);
 
System.out.println( "Anrede: " + addressBookHolder[id][0] +  "\n" +
 
"Name: "+ addressBookHolder[id][1] +  "\n" +
 
"TelephonNummer: "+ addressBookHolder[id][2] +  "\n" +
 
"Mailadresse: " + addressBookHolder[id][3]);
 
System.out.println();
 
}
 
 
 
public void showAdressbuch(){
 
System.out.println();
 
for ( int i = 1; i < addressBookHolder.length; i++){
 
if(addressBookHolder[i][0] !=null){
 
System.out.println("Arufzahl: " + i);
 
System.out.println( "Anrede: " + addressBookHolder[i][0] +  "\n" +
 
"Name: "+ addressBookHolder[i][1] +  "\n" +
 
"TelephonNummer: "+ addressBookHolder[i][2] +  "\n" +
 
"Mailadresse: " + addressBookHolder[i][3]);
 
System.out.println();
 
}
 
}
 
}
 
 
 

 
public void searchAdressbuch(String name){
 
int search = 1;
 
 
 
for ( int i = 1; i < addressBookHolder.length; i++){
 
try{
 
if(addressBookHolder[i][1].equals(name)){
 
showAdressbuch(search);
 
break;
 
}
 
}
 
catch(NullPointerException NPE){}
 
search++;
 
}
 
}
 }

also das mit add hab ich net so verstanden- in anderen teilen dürften glaub ich passen -konnt es ja noch nicht ausprobieren - bei der such funktion bin ich mir net genau sicher was die tun wird- sie sollte das verfüllen zB. Kontaktname: "Franz Maier"
Gültige Suchbegriffe: "Franz", "Maier", "Mai", "ier", "anz" ...usw...
wird sie aber sicher net nicht tun- bitte um hilfe

 

[Murray]

Ich verstehe den Zusammenhang nicht - Du hast hier eine Klasse Bsp3_0726406_2, die eine andere (hier nicht gepostete) Klasse Adressbuch verwendet. In der Klasse Bsp3_0726406_2 implementierst Du allerdings einige Methoden, die offensichtlich dem API der Klasse Adressbuch entsprechen - die werden in diesem Code allerdings niemals aufgerufen.

Ansonsten:
1. Deine Suchmethode findet nur dann etwas, wenn der Suchbegriff genau mit dem Nachnamen übereinstimmt. Um mit den genannten erlaubten Eingaben einen Treffer zu erzielen, müsste a) auch der Vorname geprüft werden b) nicht equals, sondern contains aufgerufen werden
2. Der Code würde durch sinnvolle Einrückungen an Lesbarkeit gewinnen
3. Von solchen Konstrukten

catch(NullPointerException NPE){}

bekomme ich Augenbluten

 

[SlaterB]

add fügt einen neuen Eintrag ein, search sucht und show gibt ihn aus,
alles besteht aus vielen einzelnen Befehlen,

ganz ganz grob gesehen in 20 Sekunden scheint es zu funktionieren

was genau ist nun die Frage?

 

[Stoffl]

das porb is das der code nicht funktioniert*seufz*
wie implemtiere ich das add edit ...?
so damit mein adress buch funktioniert

 

[Murray]

Auf den ersten Blcik sollten add und edit funktionieren - wenn sie denn in der Klasse Adressbuch stehen würden (s.o.)

 

[Stoffl]

public class Bsp3_0726406_2 {
    
class adressbuch{


int hold = 1;
 
String[][] addressBookHolder = new String[10000][4];
 
 
 
 void main(String[] args){
 
Adressbuch ABCD = new Adressbuch();
 
ABCD.addEntry("Mr. ", "A", "1", "mail");
 
ABCD.addEntry("Mr. ", "A", "2", "mail");
 
ABCD.addEntry("Mr. ", "A", "0", "mail");
 
ABCD.addEntry("Mr. ", "A", "0", "mail");
 
 
 
System.out.println("Adressbuch: ");
 
ABCD.showAdressbuch();
 
 
 
System.out.println("Edited addressbook: ");
 
ABCD.editEntry(3, "Mr. ", "A, "1", "mail");
 
ABCD.showAdressbuch();
 
 
 
System.out.println("Select one person from addressbook: ");
 
ABCD.showAdressbuch(2);
 
 
 
System.out.println("Search one person from addressbook: ");
 
ABCD.searchAdressbuch("Ark");
 
ABCD.searchAdressbuch("JavafDorums");
 
}
.....

so meinst du sicher nicht oder?

 

[Murray]

Gib mir mal 'nen Tip: was ist der Unterschied zur ersten Version? (Ich hab jetzt nicht die Zeit, das alles zu vergleichen)

 

[Landei]

@Murray: Der Unterschied ist das "class adressbuch"
@Stoffl: Du hast deine Beispielklasse, die Addressbuch benutzt und die Klasse Adressbuch, die tut, was von ihr verlangt wird. Du vermischst beide Arten von Code. Man kann zwar "Klassen in Klassen" schreiben, davon würde ich dir aber erst mal abraten. Also: Mach eine zweite Datei, nenn sie Adressbuch.java und packe den Code, der zur Funktion eines Adressbuches gehört da rein:

class Adressbuch {
  int hold = 1; 
  String[][] addressBookHolder = new String[10000][4]; 

  public void addEntry(String anrede, String name, String, telephonNummer, String mailAdresse){ 
    addressBookHolder[hold][0] = anrede; 
    addressBookHolder[hold][1] = name; 
    addressBookHolder[hold][2] = telephonNummer; 
    addressBookHolder[hold][3] = mailAdresse; 
    hold++; 
  }

//...usw

}

Die Implementierung könnte wesentlich verbessert werden, aber es geht ja erst mal um's Prinzip...

 

[Stoffl]

public class A {
    


static int hold = 1;

static String[][] addressBookHolder = new String[10000][4];

 

public static void main(String[] args){

A ABCD = new A();

System.out.println("Vorhande Konakte: ");
ABCD.addEntry("Mr. ", "Ark", "1111111111", "Engineeringserver.com");

ABCD.addEntry("Mr. ", "Javaforums", "0000000000", "Javaforums.net");

ABCD.addEntry("Mr. ", "Duke", "+00 123456", "World");

ABCD.addEntry("Mr. ", "Someone", "+0012233", "right here!");

ABCD.addEntry("Mr. ", "Someone", "+0012233", "right here!");     

ABCD.showAddressBook();


ABCD.searchAddressBook("Ark");



ABCD.deletEntry("Ark");


ABCD.searchAddressBook("Ark");

 

}

 

public void addEntry(String salutation, String name, String telephoneNumber, String address){

addressBookHolder[hold][0] = salutation;

addressBookHolder[hold][1] = name;

addressBookHolder[hold][2] = telephoneNumber;

addressBookHolder[hold][3] = address;

hold++;

}

 





public void deletEntry(String name){



}



public void showAddressBook(int id){

System.out.println();

 

System.out.println("Record number: " + id);

System.out.println( "Salutation: " + addressBookHolder[id][0] +  "\n" +

"Name: "+ addressBookHolder[id][1] +  "\n" +

"TelephoneNumber: "+ addressBookHolder[id][2] +  "\n" +

"Address: " + addressBookHolder[id][3]);

System.out.println();

}

 

public void showAddressBook(){

System.out.println();

for ( int i = 1; i < addressBookHolder.length; i++){

if(addressBookHolder[i][0] !=null){

System.out.println("Record number: " + i);

System.out.println( "Salutation: " + addressBookHolder[i][0] +  "\n" +

"Name: "+ addressBookHolder[i][1] +  "\n" +

"TelephoneNumber: "+ addressBookHolder[i][2] +  "\n" +

"Address: " + addressBookHolder[i][3]);

System.out.println();

}

}

}

 



public void searchAddressBook(String name){

int search = 1;

 

for ( int i = 1; i < addressBookHolder.length; i++){

try{

if(addressBookHolder[i][1].contains(name)){

showAddressBook(search);

break;

}

}

catch(NullPointerException NPE){}

search++;

}

}

 

}

danke für den hinweiß - hab noch ein prob, dass mit dem löschen weiß ich nicht so genau wie gehts- könnts ihr mir helfen?

 

[Murray]

Aber zwei Klassen hast Du doch immer noch nicht, oder? Jetzt heisst die Klasse, die sowohl das Adressbuch als den nutzenden Code beinhaltet, einfach nur A.

Und zum Löschen: da Du die Daten ja in einem Array hast, reicht es, die Inhalte mit null zu überschreiben. Du müsstest also - wie bei searchEntry - zunächst den passenden Satz finden und dann einfach alles auf null setzen.

 

[Murray]

Ach ja: der Nachteil an dieser einfachen Lösung ist natürlich, dass der Platz nicht wieder freigegeben wird. Man schafft es also irgendwann, die Liste zum Platzen zu bringen (nämlich dann, wenn hold die Größe des Arrays überschreitet).
Und das passiert auch dann, wenn man zwischendurch wieder Einträge löscht, denn der Platz wird ja nicht erneut benutzt. Das könnte man verbessern, indem beim Einfügen nicht immer an die Position hold geschrieben wird, sondern vielmehr der erste Eintrag genommen wird, der komplett leer ist.

 

[Landei]

Natürlich wäre eine Lösung mit Lists oder Maps besser - dafür sind sie schließlich da

 

[Stoffl]

Aber zwei Klassen hast Du doch immer noch nicht, oder? Jetzt heisst die Klasse, die sowohl das Adressbuch als den nutzenden Code beinhaltet, einfach nur A.

Und zum Löschen: da Du die Daten ja in einem Array hast, reicht es, die Inhalte mit null zu überschreiben. Du müsstest also - wie bei searchEntry - zunächst den passenden Satz finden und dann einfach alles auf null setzen.

das die array später keine platz mehr hat nemm ich in kauf da die angabe so gestellt ist- aber das mit delet schafft ich einfach nicht- kannst du mir eine ögliche lösung posten

 

[Murray]

public void deleteEntry( String name){

  for ( int i = 1; i < addressBookHolder.length; i++){
    if( addressBookHolder[i] != null) {  //--- wir vermeiden NullPointerExceptions, anstatt sie hinterher zu fangen
      if( addressBookHolder[i][1] != null) { //-- s.o.
        if(addressBookHolder[i][1].contains(name)) { //--- wir haben den zu loeschenden Eintrag gefunden
           /* Der Eintrag muss jetzt "geloescht" werden; dazu muessen alle Array-Elemente wieder */
           /* auf null gesetzt werden */
           /* Wie koennte das wohl gehen? */
        }
      }
    }
  }
}