Poprawność metody - java

Question

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.

Answer 1

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 
 

Comments

Jest jeszcze kilka błędów dotyczących result set, zamykania statement i tego w którym momencie i w jakim bloku...