What is wrong with my getDialString() method?

Frage zur Aufgabe

In der Diskussion

Hello guys, I don't understand why my solution is not approved. I tested it with several phone numbers and it delivered the numbers in the right format? Thanks for your help :)

Adaptiere Customer und Contact an RowItem.
Die Adapterklasse ist DataAdapter.

Initialisiere countries, bevor das Programm ausgeführt wird. Zuordnung zwischen Ländercodes und Ländernamen:
UA Ukraine
US Vereinigte Staaten von Amerika
FR Frankreich

Anforderungen:

Die Solution-Klasse muss das Feld public static Map<String, String> countries besitzen.
Initialisiere das countries-Feld in einem static-Block der Solution-Klasse mit den in der Aufgabe angegebenen Testdaten.
Die Solution-Klasse muss ein RowItem-Interface haben.
Die Solution-Klasse muss ein Contact-Interface haben.
Die Solution-Klasse muss ein Customer-Interface haben.
Die DataAdapter-Klasse muss das RowItem-Interface implementieren.
Die DataAdapter-Klasse muss zwei private Felder enthalten: Customer customer und Contact contact.
Die DataAdapter-Klasse muss einen Konstruktor mit den Parametern (Customer customer, Contact contact) besitzen. Er muss die contact- und customer-Felder initialisieren.
Implementiere die Methoden des RowItem-Interface in der DataAdapter-Klasse unter Verwendung der Kommentare in den Interfaces.

package de.codegym.task.task19.task1905; import java.util.HashMap; import java.util.Map; /* Adapter stärken */ public class Solution { public static Map<String,String> countries = new HashMap<>(); static { countries.put("UA", "Ukraine"); countries.put("US", "Vereinigte Staaten von Amerika"); countries.put("FR", "Frankreich"); } public static void main(String[] args) { } public static class DataAdapter implements RowItem { private Customer customer; private Contact contact; public DataAdapter(Customer customer, Contact contact) { this.customer = customer; this.contact = contact; } @Override public String getCountryCode() { StringBuffer countrycode = new StringBuffer(); String name = customer.getCountryName(); for (Map.Entry<String, String> entry : countries.entrySet()){ if (entry.getValue().equals(name)){ countrycode.replace(0,3, entry.getKey()); } } return countrycode.toString(); } @Override public String getCompany() { return customer.getCompanyName(); } @Override public String getContactFirstName() { return contact.getName().substring(contact.getName().indexOf(",")+2); } @Override public String getContactLastName() { return contact.getName().substring(0, contact.getName().indexOf(",")); } @Override public String getDialString() { String diaL = contact.getPhoneNumber(); StringBuffer dial = new StringBuffer(); int o = diaL.indexOf("("); dial.append(diaL.substring(0, o)); int z = diaL.indexOf(")"); dial.append(diaL.substring(o+1, z)); int s = diaL.indexOf("-"); dial.append(diaL.substring(z+1,s)); dial.append(diaL.substring(s+1)); dial.insert(0,"anrufen://"); return dial.toString(); } } public static interface RowItem { String getCountryCode(); // Beispiel: US String getCompany(); // Beispiel: CodeGym Ltd. String getContactFirstName(); // Beispiel: John String getContactLastName(); // Beispiel: Peterson String getDialString(); // Beispiel: callto://+11112223333 } public static interface Customer { String getCompanyName(); // Beispiel: CodeGym Ltd. String getCountryName(); // Beispiel: United States } public static interface Contact { String getName(); // Beispiel: Peterson, John String getPhoneNumber(); // Beispiel: +1(111)222-3333, +3(805)0123-4567, +380(50)123-4567 usw. } }

Kommentare (2)

Beliebt
Neu
Alt

Du musst angemeldet sein, um einen Kommentar schreiben zu können

Leon

Level 22

15 November 2021, 16:40

Thanks a lot, you were right, my solution was to limited.

Lisa

Level 41

15 November 2021, 10:59

first I'd try to replace anrufen with callto (that's what CG is asking for, see comments in the code). If that isn't the sole problem then rethink your getDialString method as it is very limited and eg. can't handle a second hyphen... but as said, just in case (one could think of removing all non digit chars) and if you want to test your code...

DataAdapter da = new DataAdapter(
        new Customer() {
            @Override
            public String getCompanyName() {
                return "CodeGym Ltd.";
            }

            @Override
            public String getCountryName() {
                return "Vereinigte Staaten von Amerika";
            }
        },
        new Contact() {
            @Override
            public String getName() {
                return "Peterson, John";
            }

            @Override
            public String getPhoneNumber() {
                // +1(111)222-3333, +3(805)0123-4567, +380(50)123-4567
                return "+1(111)222-3333";
            }
        });


System.out.println(da.getCountryCode());
System.out.println(da.getCompany());
System.out.println(da.getContactFirstName());
System.out.println(da.getContactLastName());
System.out.println(da.getDialString());