M
McLovin
Gast
Hi,
ich habe nun mein Kontaktbuch mit den wichtigsten Funktionen umgesetzt. Allerdings habe ich einige Fragen zu meiner Umsetzung:
1. Das ständige umher kopieren von "phoneBook" dürfte unwahrscheinlich Performancefressend sein, gibt es da eine Möglichkeit das ohne das umherkopieren zu lösen? (Ausser das Telefonbuch statisch zu erzeugen)
2. Wo verbirgt sich der (logische) Fehler in diesem Codefragment?
3. Arbeite ich richtig mit der ArrayList?
5. Gibt es ansonsten noch Ratschläge wie ich etwas eleganter hätte lösen können oder Auffälligkeiten?
Hier meine Quelltexte:
Contact.java
PhoneBook.java
PS: Ja, bei PhoneBook war ich zu faul zu kommentieren
Gruß,
McLovin
ich habe nun mein Kontaktbuch mit den wichtigsten Funktionen umgesetzt. Allerdings habe ich einige Fragen zu meiner Umsetzung:
1. Das ständige umher kopieren von "phoneBook" dürfte unwahrscheinlich Performancefressend sein, gibt es da eine Möglichkeit das ohne das umherkopieren zu lösen? (Ausser das Telefonbuch statisch zu erzeugen)
2. Wo verbirgt sich der (logische) Fehler in diesem Codefragment?
Code:
System.out.println("Insert the new Tel.(otherwise leave it empty): ");
if ( (tel = sc.next()) != "")
phoneBook.get(index-1).setTelefonNumber(tel);
3. Arbeite ich richtig mit der ArrayList?
5. Gibt es ansonsten noch Ratschläge wie ich etwas eleganter hätte lösen können oder Auffälligkeiten?
Hier meine Quelltexte:
Contact.java
Code:
public class Contact
{
private String name;
private String telefonNumber;
Contact(String name, String telefonNumber)
{
this.name = name;
this.telefonNumber = telefonNumber;
}
/**
* Sets the (new) name of the contact.
* @param: new name of the contact
*/
public void setName(String newName)
{
this.name = newName;
}
/**
* Sets the (new) telefonnumber of the contact.
* @param: new telefonnumber of the contact
*/
public void setTelefonNumber (String newTel)
{
this.telefonNumber = newTel;
}
/**
* Returns the name of the contact.
* @return: name
*/
public String getName ()
{
return this.name;
}
/**
* Returns the telefonnumber of the contact.
* @return: telefonNumber
*/
public String getTelefonNumber()
{
return this.telefonNumber;
}
}
PhoneBook.java
Code:
import java.util.*;
public class PhoneBook
{
static Scanner sc = new Scanner(System.in); // Scanner-Object to interact with user
static void showMenu()
{
System.out.println("new contact <1>");
System.out.println("show contacts <2>");
System.out.println("edit contact <3>");
System.out.println("delete contact <4>");
System.out.println("exit <0>");
System.out.println("------------------");
}
static Contact newContact()
{
String name;
String tel;
System.out.print("Insert the name of your contact: ");
name = sc.next();
System.out.print("Insert the telefonnumber of your contact: ");
tel = sc.next();
Contact con = new Contact(name, tel);
System.out.println("Successfully created a new contact!\n");
return con;
}
static void printPhoneBook(ArrayList<Contact> phoneBook, int numberContact)
{
for (int i = 0; i<numberContact;i++)
{
System.out.print( (i+1) + ": " + phoneBook.get(i).getName()+ " ");
System.out.println("Tel.: " + phoneBook.get(i).getTelefonNumber());
}
}
static ArrayList<Contact> editContact(ArrayList<Contact> phoneBook)
{
int index;
String name, tel;
System.out.println("Which contact do you want to edit? :");
index = sc.nextInt();
System.out.println("Insert the new name(otherwise leave it empty): ");
if ( (name = sc.next()) != "")
phoneBook.get(index-1).setName(name);
System.out.println("Insert the new Tel.(otherwise leave it empty): ");
if ( (tel = sc.next()) != "")
phoneBook.get(index-1).setTelefonNumber(tel);
return phoneBook;
}
static ArrayList<Contact> deleteContact(ArrayList<Contact> phoneBook)
{
int index;
System.out.print("Which contact do you want to delete? :");
index = sc.nextInt();
System.out.print("Do you really want to delete " + phoneBook.get(index-1).getName() + "(j/n)");
if (sc.next() == "j")
{
phoneBook.remove(index-1);
}
return phoneBook;
}
public static void main(String[] args)
{
ArrayList<Contact> phoneBook = new ArrayList<Contact> (5);
int input = 1337;
int numberContacts=0;
System.out.println("Welcome to your phonebook!");
do
{
showMenu();
input = sc.nextInt();
switch(input)
{
case 1:
System.out.println("");
phoneBook.add(newContact());
numberContacts++;
break;
case 2:
System.out.println("");
printPhoneBook(phoneBook, numberContacts);
break;
case 3:
editContact(phoneBook);
break;
case 4:
System.out.println("");
phoneBook = deleteContact(phoneBook);
numberContacts--;
break;
}
} while(input !=0);
}
}
PS: Ja, bei PhoneBook war ich zu faul zu kommentieren
Gruß,
McLovin