fish-admin icon indicating copy to clipboard operation
fish-admin copied to clipboard

建议将 BaseRepository 中的 columnNames 和 columnTypes 合并

Open gowk opened this issue 5 years ago • 1 comments

public abstract String[] columnNames();
public abstract int[] columnTypes();

改成

public abstract Map<String, Integer> columns();

这样在仓储层写起来比较直观地看出每个字段和其相应的类型:

    @Override
    public Map<String, Integer> columns() {
        return ImmutableMap.<String, Integer>builder()
                .put("user_name", Types.VARCHAR)
                .put("role_id", Types.BIGINT)
                .put("encrypted_password", Types.VARCHAR)
                .put("created_at", Types.TIMESTAMP)
                .put("updated_at", Types.TIMESTAMP)
                .build();
    }

不知道是否合理,想和作者您交流一下

gowk avatar Nov 23 '19 11:11 gowk

比较合理

myliang avatar Nov 25 '19 09:11 myliang