We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
当前属为String类型时,会尝试使用resolvePlaceholder来进行占位符替换,对于直接使用value来指定属性值的属性不会有任何作用:
String
resolvePlaceholder
value
但是这里没有添加判断,导致重复添加PropertyValue,虽然不会影响后面bean的创建,但是属性列表有重复项也是一种冗余:
PropertyValue
增加了一定的判断之后:
出现问题的代码在: https://github.com/DerekYRC/mini-spring/blob/ef07ca1105b4c57b70d63a50425a0de6d6b5814a/src/main/java/org/springframework/beans/factory/PropertyPlaceholderConfigurer.java#L78C5-L78C88
The text was updated successfully, but these errors were encountered:
No branches or pull requests
当前属为
String
类型时,会尝试使用resolvePlaceholder
来进行占位符替换,对于直接使用value
来指定属性值的属性不会有任何作用:但是这里没有添加判断,导致重复添加
PropertyValue
,虽然不会影响后面bean的创建,但是属性列表有重复项也是一种冗余:增加了一定的判断之后:
出现问题的代码在:
https://github.com/DerekYRC/mini-spring/blob/ef07ca1105b4c57b70d63a50425a0de6d6b5814a/src/main/java/org/springframework/beans/factory/PropertyPlaceholderConfigurer.java#L78C5-L78C88
The text was updated successfully, but these errors were encountered: