Tutorium 06.05¶

Warmup¶

Welche Methode wird hier aufgerufen: Die von Dog oder die von Animal?

Animal a = new Dog();
a.makeSound();

Aufgabe 1¶

Dieser Code soll so verändert werden, dass man neue Tiere zum Zoo hinzufügen kann, ohne das dieser verändert werden muss. Anschließend sollen noch Kühe und Schweine zum Zoo hinzugefügt werden. (Der Zoo sollte sich hierfür nicht ändern)

import java.util.ArrayList;
import java.util.List;

class Dog {
    public void makeSound() {
        System.out.println("Woof");
    }
}

class Cat {
    public void makeSound() {
        System.out.println("Meow");
    }
}

class Zoo {
    public void makeAllSounds(List<Object> animals) {
        for (Object animal : animals) {
            if (animal instanceof Dog) {
                Dog dog = (Dog) animal;
                dog.makeSound();
            } else if (animal instanceof Cat) {
                Cat cat = (Cat) animal;
                cat.makeSound();
            }
        }
    }
}

public class Main {
    public static void main(String[] args) {
        List<Object> animals = new ArrayList<>();

        animals.add(new Dog());
        animals.add(new Cat());

        Zoo zoo = new Zoo();
        zoo.makeAllSounds(animals);
    }
}

Aufgabe 2¶

Dieser Code soll überarbeitet werden, damit die Wiederholungen verschwinden.

class EmailNotification {
    private String recipient;

    public EmailNotification(String recipient) {
        this.recipient = recipient;
    }

    public void send(String message) {
        if (recipient == null || recipient.isBlank()) {
            throw new IllegalArgumentException("Recipient is missing");
        }

        System.out.println("Preparing notification...");
        System.out.println("Sending EMAIL to " + recipient + ": " + message);
        System.out.println("Notification sent.");
    }
}

class SmsNotification {
    private String recipient;

    public SmsNotification(String recipient) {
        this.recipient = recipient;
    }

    public void send(String message) {
        if (recipient == null || recipient.isBlank()) {
            throw new IllegalArgumentException("Recipient is missing");
        }

        System.out.println("Preparing notification...");
        System.out.println("Sending SMS to " + recipient + ": " + message);
        System.out.println("Notification sent.");
    }
}

public class Main {
    public static void main(String[] args) {
        EmailNotification email = new EmailNotification("alice@example.com");
        SmsNotification sms = new SmsNotification("+49123456789");

        email.send("Your package was shipped.");
        sms.send("Your code is 123456.");
    }
}

Bemerkung

Sollte das mit einer abstrakten Klasse oder einem Interface gelöst werden?

Aufgabe 3¶

class Checkout {
    public void pay(String paymentType, double amount) {
        if (amount <= 0) {
            System.out.println("Invalid amount");
            return;
        }

        if (paymentType.equals("paypal")) {
            System.out.println("Paying " + amount + " using PayPal");
        } else if (paymentType.equals("creditcard")) {
            System.out.println("Paying " + amount + " using credit card");
        } else {
            System.out.println("Unknown payment type");
        }
    }
}

public class Main {
    public static void main(String[] args) {
        Checkout checkout = new Checkout();

        checkout.pay("paypal", 29.99);
        checkout.pay("creditcard", 99.99);
        checkout.pay("paypall", 10.00);
        checkout.pay("paypal", -5.00);
    }
}

Wie kann dieser Code verbessert werden?

Anforderungen:

Keine statischen Strings wie „paypal“.
Ungültige Summen sollten nicht nur eine Fehlermeldung printen.
Wenn neue Zahlungsmethoden hinzugefügt werden, sollte der Checkout nicht größer und größer werden.
Für die Zahlungsmethoden sollte ein Interface verwendet werden.
Für die Zahlungstype sollte ein Enum verwendet werden.

Aufgabe 4¶

import java.util.ArrayList;
import java.util.List;

class User {
    private final int id;
    private final String name;

    public User(int id, String name) {
        this.id = id;
        this.name = name;
    }

    public int getId() {
        return id;
    }

    public String toString() {
        return "User{id=" + id + ", name='" + name + "'}";
    }
}

class Product {
    private final int id;
    private final String title;

    public Product(int id, String title) {
        this.id = id;
        this.title = title;
    }

    public int getId() {
        return id;
    }

    public String toString() {
        return "Product{id=" + id + ", title='" + title + "'}";
    }
}

class Repository {
    private final List<Object> items = new ArrayList<>();

    public void save(Object item) {
        items.add(item);
    }

    public Object findById(int id) {
        for (Object item : items) {
            if (item instanceof User) {
                User user = (User) item;

                if (user.getId() == id) {
                    return user;
                }
            }

            if (item instanceof Product) {
                Product product = (Product) item;

                if (product.getId() == id) {
                    return product;
                }
            }
        }

        return null;
    }
}

public class Main {
    public static void main(String[] args) {
        Repository repository = new Repository();

        repository.save(new User(1, "Alice"));
        repository.save(new Product(2, "Keyboard"));

        User user = (User) repository.findById(1);
        Product product = (Product) repository.findById(2);

        System.out.println(user);
        System.out.println(product);
    }
}

Wie kann dieser Code verbessert werden?

Anforderungen:

Kein List<Object>
Keine casts in main
Keine instanceof in Repository
Das Repository sollte für Nutzer, Produkte und alle zukünftigen Klassen mit einer ID funktionieren.
Verwende Generics