Best Practice Code Refactoring für Methoden mit fast gleicher Aufbau

[lam_tr]

Hallo zusammen,

ich stehe wieder vor ein wahrscheinlich simples Problem, bei der ich nicht weiß wie man es schöner implementieren kann, eventuell kann mir jemand, ich will die enable%TYP%ColumnEdit Methode für mehrere Typen anbieten, bisher war es immer so dass es immer an zwei Stellen anders wie bei den anderen Methoden sind. Zum Einen die #fromString und die Stelle wo das Argument dem Consumer übergeben wird.

    public void enableLongColumnEdit(int columnIndex, Consumer<EditData<T, Long>> consumer) {
        findColumn(columnIndex).setCellFactory(TextFieldTableCell.forTableColumn(new StringConverter() {
            @Override
            public String toString(Object object) {
                return object.toString();
            }

            @Override
            public Object fromString(String string) {
                return Long.parseLong(string);
            }
        }));
        findColumnByIndex(columnIndex).setOnEditCommit(new EventHandler<TableColumn.CellEditEvent<T,Object>>() {
            @Override
            public void handle(CellEditEvent<T, Object> event) {
                Object newValue = event.getNewValue();
                T row = event.getTableView().getItems(). get(event.getTablePosition().getRow());
                consumer.accept(new EditData<>(row, Long.parseLong(newValue.toString())));
            }
        });
    }

    public void enableDoubleColumnEdit(int columnIndex, Consumer<EditData<T, Double>> consumer) {
        findColumn(columnIndex).setCellFactory(TextFieldTableCell.forTableColumn(new StringConverter() {
            @Override
            public String toString(Object object) {
                return object.toString();
            }
            
            @Override
            public Object fromString(String string) {
                return Double.parseDouble(string);
            }
        }));
        findColumnByIndex(columnIndex).setOnEditCommit(new EventHandler<TableColumn.CellEditEvent<T,Object>>() {
            @Override
            public void handle(CellEditEvent<T, Object> event) {
                Object newValue = event.getNewValue();
                T row = event.getTableView().getItems(). get(event.getTablePosition().getRow());
                consumer.accept(new EditData<>(row, Double.parseDouble(newValue.toString())));
            }
        });
    }
    
    public void enableStringColumnEdit(int columnIndex, Consumer<EditData<T, String>> consumer) {
        findColumn(columnIndex).setCellFactory(TextFieldTableCell.forTableColumn());
        findColumnByIndex(columnIndex).setOnEditCommit(new EventHandler<TableColumn.CellEditEvent<T,Object>>() {
            @Override
            public void handle(CellEditEvent<T, Object> event) {
                Object newValue = event.getNewValue();
                T row = event.getTableView().getItems(). get(event.getTablePosition().getRow());
                consumer.accept(new EditData<>(row, String.valueOf(newValue)));
            }
        });
    }
    
    public TableColumn<T, Object> findColumnByIndex(int index){
        return (TableColumn<T, Object>) tableView.getColumns().get(index);
    }

Ungefähr 5-8 Methoden gleicher Aufbau ist doch quatsch oder?

 

[mrBrown]

Als erstes die StringConverter rausziehen und ihn der Methode übergeben, dann kannst du den an der Stelle, wo's dem Consumer übergeben wird, wiederverwenden.

Passende StringConverter für StandardTypen gibts schon, zumindest für das gezeigte brauchst du keine eigenen schreiben. Wenn du viele eigene brauchst, kannst du das noch über einen generischen StringConverter lösen, dem man dann einfach zwei Funktionen (String->T, T->String) übergibt.

 

[lam_tr]

Als erstes die StringConverter rausziehen und ihn der Methode übergeben, dann kannst du den an der Stelle, wo's dem Consumer übergeben wird, wiederverwenden.

Passende StringConverter für StandardTypen gibts schon, zumindest für das gezeigte brauchst du keine eigenen schreiben. Wenn du viele eigene brauchst, kannst du das noch über einen generischen StringConverter lösen, dem man dann einfach zwei Funktionen (String->T, T->String) übergibt.

Das Problem hierbei ist das der tableView.getColumns() als Rückgabewert ObservableList<TableColumn<T, ?>>zurückliefert. Kann ich nicht irgendwie einen Type beim Holen der Column bestimmen?

Wenn ich zum Beispiel den DoubleStringConvertor benutzen will, bekomme ich einen Compile-Error für den unteren Code

findColumn(columnIndex).setCellFactory(TextFieldTableCell.forTableColumn(new DoubleStringConverter());

 

[mrBrown]

Das Problem hierbei ist das der tableView.getColumns() als Rückgabewert ObservableList<TableColumn<T, ?>>zurückliefert. Kann ich nicht irgendwie einen Type beim Holen der Column bestimmen?

Und das benutzt du in findColumn? Du kannst die Methode natürlich generisch machen, und eine TableColumn<T, S> zurück geben.

 

[lam_tr]

Und das benutzt du in findColumn? Du kannst die Methode natürlich generisch machen, und eine TableColumn<T, S> zurück geben.

Wenn ich es generisch machen will, dann bekomme ich bei der Verwendung einen Object für S zurück.

    public <S> TableColumn<T, S> findColumnByIndex3(int index) {
        TableColumn<T, S> col = (TableColumn<T, S>) tableView.getColumns().get(index);
        return col;
    }

Und wenn ich den Typ mitgebe, so würde es wahrscheinlich gehe, bin aber nicht sicher ob das schön ist.

public <S> TableColumn<T, S> findColumnByIndex2(int index, S dataType ) {
        TableColumn<T, S> col = (TableColumn<T, S>) tableView.getColumns().get(index);
        return col;
    }

Vielleich erwähne ich da an der Stelle, warum es so kompliziert gemacht werden muss. Die UI wird über FXML erstellt und ich bin dabei eine Helperklasse anzulegen die mir das Editiermodus ein bisschen erleichtert. Da ich die Columns nicht mit einer ID versehen habe, rufe ich im Helperklasse die tableView.getColumns() auf.

Nachtrag:

tableConfigure.findColumnByIndex2(2, Double.class).setCellFactory(new Callback<TableColumn<Booking,Class<Double>>, TableCell<Booking,Class<Double>>>() {
            @Override
            public TableCell<Booking, Class<Double>> call(TableColumn<Booking, Class<Double>> param) {
                return null;
            }
        });;

Ich bekomme an der Stelle nicht den Typ sondern Class<Double> zurück.

 

[mrBrown]

Grob etwa:

public void test() {
        enableColumnEdit(0, editData -> { }, new LongStringConverter());
    }

    private <T,V> void enableColumnEdit(int columnIndex, Consumer<EditData<T, V>> consumer, StringConverter<V> stringConverter) {
        final Callback<TableColumn<T, V>, TableCell<T, V>> tableColumnTableCellCallback = TextFieldTableCell.forTableColumn(stringConverter);
        final TableColumn<T, V> tableColumn = findColumnByIndex(columnIndex);
        
        tableColumn.setCellFactory(tableColumnTableCellCallback);
        tableColumn.setOnEditCommit(new EventHandler<TableColumn.CellEditEvent<T,V>>() {
            @Override
            public void handle(TableColumn.CellEditEvent<T, V> event) {
                Object newValue = event.getNewValue();
                T row = event.getTableView().getItems(). get(event.getTablePosition().getRow());
                consumer.accept(new EditData<>(row, stringConverter.fromString(newValue.toString())));
            }
        });
    }

    private <T,S> TableColumn<T, S> findColumnByIndex(int index) {
        return (TableColumn<T, S>) tableView.getColumns().get(index);
    }

 

[lam_tr]

Grob etwa:

public void test() {
        enableColumnEdit(0, editData -> { }, new LongStringConverter());
    }

    private <T,V> void enableColumnEdit(int columnIndex, Consumer<EditData<T, V>> consumer, StringConverter<V> stringConverter) {
        final Callback<TableColumn<T, V>, TableCell<T, V>> tableColumnTableCellCallback = TextFieldTableCell.forTableColumn(stringConverter);
        final TableColumn<T, V> tableColumn = findColumnByIndex(columnIndex);
       
        tableColumn.setCellFactory(tableColumnTableCellCallback);
        tableColumn.setOnEditCommit(new EventHandler<TableColumn.CellEditEvent<T,V>>() {
            @Override
            public void handle(TableColumn.CellEditEvent<T, V> event) {
                Object newValue = event.getNewValue();
                T row = event.getTableView().getItems(). get(event.getTablePosition().getRow());
                consumer.accept(new EditData<>(row, stringConverter.fromString(newValue.toString())));
            }
        });
    }

    private <T,S> TableColumn<T, S> findColumnByIndex(int index) {
        return (TableColumn<T, S>) tableView.getColumns().get(index);
    }

Vielen Dank! Endlich kann ich diesen duplizierten Code entfernen