eggjs / eggjs/egg

基础插件 egg-mysql 的代码是不是该维护了,质量堪忧

Open
#4,569 3 comments 0 reactions 0 assignees View on GitHub
Inactive
Dominant language
TypeScript
Stars
19k
Forks
1.8k
PR merge metrics
No merged PRs in 30d

Description

## What happens?
egg-mysql 的文档样例有点少,就去查了[ali-rds](https://github.com/ali-sdk/ali-rds) ,在读代码时,看到 /lib/operator.js 中的代码质量太堪忧了,瞬间 egg.js 就不香了,各种神仙代码,全读一边不可能,但如果egg的基础库都是这种代码,真是不太想用了。

## 例子:
在 [ali-rds](https://github.com/ali-sdk/ali-rds) 的 [/lib/operator.js](https://github.com/ali-sdk/ali-rds/blob/master/lib/operator.js) 的第 269 行
数组直接与字符相加,并且 255 行的变量 SQL 就没必要声明成数组,运行是能运行,但太仙了

```javascript
let SQL = [ 'UPDATE ?? SET ' ];
let VALUES = [ table ];

const TEMPLATE = [];
for (const key in SQL_CASE) {
let templateSql = ' ?? = CASE ';
VALUES.push(key);
templateSql += SQL_CASE[key].when.join(' ');
VALUES = VALUES.concat(SQL_CASE[key].then);
templateSql += ' ELSE ?? END ';
TEMPLATE.push(templateSql);
VALUES.push(key);
}
SQL += TEMPLATE.join(' , ');
```
希望开发组也能维护维护基础库吧

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.