• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Poprawność metody - java

VPS Starter Arubacloud
0 głosów
400 wizyt
pytanie zadane 9 listopada 2022 w Java przez stefan96 Nowicjusz (120 p.)

Witajcie,

przychodzę do Was z następującym pytaniem: czy stosowanie metody, która znajduje się poniżej jest "złą praktyką"? Jeśli tak, to można w niej zmienić, żeby była ona lepsza?

 

    @SuppressWarnings("unchecked")
    public <T> T getFieldValue(String field, String id, Class<T> type) {
        try {
            PreparedStatement preparedStatement = connection.prepareStatement("SELECT " + field + " FROM data WHERE id = ?;");
            preparedStatement.setString(1, id);
            ResultSet resultSet = preparedStatement.executeQuery();
            if (resultSet.next()) {
                Object object = resultSet.getObject(field);
                preparedStatement.close();
                if (type.isInstance(object)) return (T) object;
            }
        } catch (SQLException exception) {
            exception.printStackTrace();
        }
        return null;
    }

 

Pozdrawiam, Stefan.

1 odpowiedź

+1 głos
odpowiedź 10 listopada 2022 przez Wiciorny Ekspert (269,120 p.)

generalnie zależy co masz na myśli "poprawki" wszystko zależy od wymagań biznesowych

Metoda jest "dirty" generalnie źle napisana sam fakt tego już wskazuje dla programisty  zastosowanie 
 

@SuppressWarnings("unchecked")

wyraźna informacja, że zachodzi tutaj u Ciebie rzutowanie, które może być błędne, dodatkowo nie ma obsługi rzutowania

 if (type.isInstance(object)) return (T) object;
        }

Czyli jeśli nie jest to typ a także uwaga PODTYP.. bo jeśli typ jest częścią interfejsu np. String jest częścią Object więc instanceOf... zwróci true. 
Więc takie rzutowanie raz, że nie ma warunku " co jeśli nie zachodzi" to dodatkowo  może generować mase  problemów. 

Dodatkowo zabawne bo sprawdzasz czy "typ jest" instancją pobranego elementu typu Object

Object object = resultSet.getObject(field)

więc z natury Javy to praktycznie zawsze jest prawdziwe 

Assert.assertTrue(jakisTyp instanceof Object);

bo w Javie wszystko jest obiektem

Idąc dalej... return null-> to jest najgorsze co można zrobić i w Javie nigdy nie powinieneś zwracać nula, szczególnie że jeśli statement zwróci błąd to metoda wyrzuci błąd a nie zwróci nula 
 

1
komentarz 10 listopada 2022 przez Wiciorny Ekspert (269,120 p.)
Jest jeszcze kilka błędów dotyczących result set, zamykania statement i tego w którym momencie i w jakim bloku...

Podobne pytania

+1 głos
1 odpowiedź 186 wizyt
pytanie zadane 17 listopada 2021 w Java przez ptomeccc Użytkownik (800 p.)
0 głosów
0 odpowiedzi 262 wizyt
pytanie zadane 22 stycznia 2021 w Java przez stronek1899 Nowicjusz (160 p.)
–1 głos
0 odpowiedzi 298 wizyt
pytanie zadane 16 lipca 2020 w Java przez T100 Obywatel (1,450 p.)

92,453 zapytań

141,262 odpowiedzi

319,088 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...