GreedyStar / generator

Java code generating tool.
Apache License 2.0
260 stars 122 forks source link

提几点建议或bug #9

Open wnjustdoit opened 4 years ago

wnjustdoit commented 4 years ago

首先感谢作者的慷慨开源哈,然后提几个可能需要调整的点供参考: 1、生成Mapper的时候,columnMap有误:getPropertyName() -> getColumnName(); 2、建议Mapper中的所有字段和表都加上反引号``,而不是双引号,防止与mysql关键字冲突; 3、建议默认生成的实体属性类型为包装类型,而不是基本类型,另外,tinyint/smallint/int建议都转换为java.lang.Integer,datetime/time/timestamp建议都转换为java.util.Date类型。

GreedyStar commented 4 years ago

首先感谢作者的慷慨开源哈,然后提几个可能需要调整的点供参考: 1、生成Mapper的时候,columnMap有误:getPropertyName() -> getColumnName(); 2、建议Mapper中的所有字段和表都加上反引号``,而不是双引号,防止与mysql关键字冲突; 3、建议默认生成的实体属性类型为包装类型,而不是基本类型,另外,tinyint/smallint/int建议都转换为java.lang.Integer,datetime/time/timestamp建议都转换为java.util.Date类型。

非常感谢 您的关注和建议,在后续的版本中,我会对工具进行相应的调整。

另外,对于 1 中提到的错误,可否再详细描述一下,以便于复现和定位。

wnjustdoit commented 4 years ago

ResultMap生成有误:

以下方法及其重载的方法:com.greedystar.generator.utils.GeneratorUtil#generateMapperResultMap(java.util.List)#212和214行

sb.append("<id column=\"").append(info.getPropertyName())

应为:

sb.append("<id column=\"").append(info.getColumnName())