Skip to content
New issue

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

老哥,这个组件以后bug哦 #3

Open
wuzhixaun opened this issue Jun 30, 2023 · 8 comments
Open

老哥,这个组件以后bug哦 #3

wuzhixaun opened this issue Jun 30, 2023 · 8 comments

Comments

@wuzhixaun
Copy link

No description provided.

@caixunshi
Copy link
Owner

什么bug?

@wuzhixaun
Copy link
Author

wuzhixaun commented Jun 30, 2023

什么bug?

假如我需要查一个全表的数据 select * from notice 后面可以接条件也可以不接条件,
不接条件是不需要拼接的是 user_code = ${userCode}
接条件是不需要拼接的是 and user_code = ${userCode}
就是需不需要and

@wuzhixaun
Copy link
Author

wuzhixaun commented Jun 30, 2023

什么bug?

还有就是 如果user_code 是数字还是字符串,如果是字符串直接这样拼接是不行的吧

@caixunshi
Copy link
Owner

什么bug?

假如我需要查一个全表的数据 select * from notice 后面可以接条件也可以不接条件, 不接条件是不需要拼接的是 user_code = ${userCode} 接条件是不需要拼接的是 and user_code = ${userCode} 就是需不需要and

这个and user_code = ${userCode}本来就是使用者去配置的,加不加and本来就是使用者自己去决定的,不过这里可以优化一下,兼容一下这种场景

@caixunshi
Copy link
Owner

什么bug?

还有就是 如果user_code 是数字还是字符串,如果是字符串直接这样拼接是不行的吧

这个我记得字符串也是可以的,你感兴趣的话可以试一下

@wuzhixaun
Copy link
Author

我试过了,不行的,如果是number 类型,mysql会触发隐式转换,这个有时候还会导致索引失效

@wuzhixaun
Copy link
Author

什么bug?

假如我需要查一个全表的数据 select * from notice 后面可以接条件也可以不接条件, 不接条件是不需要拼接的是 user_code = ${userCode} 接条件是不需要拼接的是 and user_code = ${userCode} 就是需不需要and

这个and user_code = ${userCode}本来就是使用者去配置的,加不加and本来就是使用者自己去决定的,不过这里可以优化一下,兼容一下这种场景

针对这种的话,优化肯定是有必要的

@caixunshi
Copy link
Owner

你是咋试的,这个项目在之前公司生产环境一直用的,不会用不了,数字类型会导致索引失效确实会有这个问题,之前公司都是用创建人工号,是一个字符串,所以就没有处理这种场景

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants