钱骏琪
ea16ea6673
Pylint部分：
...
1. 补充module docstring，即文档顶部的简要说明 （我做的说明很简陋，只是为了通过pylint的检查而为之）
2. 优化了if-else-return，即删除了不必要的elif或else
3. 拆分了所有在同一行的模块引用（不同模块的引用需要放在不同行，以便理解和阅读）
4. 删除了部分没有被使用的引用（有的引用在文件A中被使用了，但import却写在了文件B，且文件B没有使用这些引用，而文件A引用了文件B，导致文件B的所有引用也被A引用，这似乎并不合理）
5. 修改了引用的顺序（os、random等系统引用应放在自定义模块引用的上方）
6. 修改了过长的代码，注释除外（每行限制在100个字符以内）
7. 优化了条件判断的判断符号：
is和is not 用于判断两个变量是否指向同一个位置
== 和 !=用于比较两个变量
8. 为每个py文件添加了"last new line"，删除了多余的尾部空行
Pylint提示了但是没有进行的操作：
1. 将字符串改为f-string的格式
因为f-string需要python3.6以上的版本，为了防止出现版本问题，故不做修改
2. 将所有函数内的return内容改为一致的类型
因为原有的代码逻辑已经确定，修改return的类型会导致方法的无法使用，故不做修改
3. 将标识符的格式改为"snake_case naming style"
标识符的修改内容过于庞大，且可能出现程序不可预见的错误，在pylint中重要等级为0，故不做修改
JSLint Part:
1. 提高代码的可读性：将部分超过80个字符的代码行拆解开（由于文字注释的存在，很多语句仅注释就超过了80个字符，此类型的语句行不做修改）
2. 用===与！==代替==与！=的判断
3. 将i++改为i+=1
4. 分离for循环中的let与var到其他行中，以适应JSLint在循环语句中对var和let的偏好
5. 给三元表达式外围添加圆括号，避免出现优先级的问题
6. 将正则表达式修改为JSLint要求的多行模式
疑问：
形如：
for (let i = 0; i < list.length; i+=1) {
的语句，JSLint报错： [JSLint was unable to finish] Unexpected 'let'.
但是网上搜索到的结果是ES6是支持let的使用的，暂且不做修改。
2023-05-26 22:18:11 +08:00
Hui Lan
a1955341c6
Fix bug 501 - 特殊字符&加入生词库后删除按钮失效
2023-01-31 16:39:11 +08:00
王志豪
e10dbf9d67
Bug507-WuWenZhuo ( #70 )
...
### 修复了生词簿为空时，双击文章单词无法高亮的问题
通过增加一个判断，判断生词簿为空时，不把生词簿的内容进行处理，仅处理选中单词。
### 修复了生词簿为空时，取消高亮后，导致文章格式混乱的问题
原代码中，从数据库提取文章放到网页上时，使用的是.innerText的方法，导致原文章里包含的<br>标签丢失，导致文章格式混乱的问题。
Co-authored-by: unknown <Alcatraz@qq.com>
Co-authored-by: Hui Lan <lanhui@zjnu.edu.cn>
Reviewed-on: http://121.4.94.30:3000/mrlan/EnglishPal/pulls/70
Co-authored-by: 王志豪 <1594799762@qq.com>
Co-committed-by: 王志豪 <1594799762@qq.com>
2023-01-29 12:48:52 +08:00
李雨峰
9cdc9c6f7f
Bug521-LiYuFeng-Refactor ( #72 )
...
@mrlan
蓝老师：
本次改进内容如下：
1. 对生词居中问题进行修改，现在已经不会居中了。
2. 对于单词数量基数大而导致的排序速度慢的问题，我们进行了优化，提升了排序的速度。
3. 对用户交互进行了优化，当用户点击“熟悉”或“不熟悉”之后，会自动进行排序，并会跳转到那个单词的位置，用抖动的效果来提示用户。
Co-authored-by: isaac <1141730046@qq.com>
Reviewed-on: http://121.4.94.30:3000/mrlan/EnglishPal/pulls/72
Co-authored-by: 李雨峰 <1141730046@qq.com>
Co-committed-by: 李雨峰 <1141730046@qq.com>
2023-01-29 12:01:19 +08:00
ca8c1bf8de
Merge pull request 'Bug508-CenHaotian' ( #62 ) from Bug508-CenHaotian into master
...
Reviewed-on: http://121.4.94.30:3000/mrlan/EnglishPal/pulls/62
2023-01-29 11:19:49 +08:00
岑昊天
d58dacd71c
修复Bug508,解决带有特殊字符|的单词在文章中的高亮问题
2022-12-06 14:40:50 +08:00
任榆
5c85041135
Bug 512 - 文章朗读问题
...
在fillwowrd.js中添加了stopRead()函数，将其添加给对应按钮或超链接以终止朗读。
2022-11-25 15:42:37 +08:00
ecc354bc0d
Refactor: use better function
2022-08-02 12:33:41 +08:00
1d8671c5c7
Refactor: use better function name
2022-08-02 12:30:27 +08:00
8cb34e56ba
Refactor: remove duplicate code block
2022-08-02 12:26:18 +08:00
b5dacb9ad2
Improve comments
2022-08-02 11:52:40 +08:00
47e745e774
Use better variable name (use articleContent instead of txt, and use camelCase)
2022-08-02 11:45:21 +08:00
1dfe370983
Use better variable names
2022-08-02 11:39:35 +08:00
c15746bbb2
Resolve conflicts
2022-08-02 11:00:33 +08:00
50a1093781
Remove loop.index0, as it is hard to understand.
2022-07-18 19:51:23 +08:00
陈靖毅
c21659ba7e
Bug477 重构 and 增加功能 ( #48 )
...
Co-authored-by: cjybyjk <cjybyjk@zjnu.edu.cn>
Co-authored-by: Lan Hui <1348141770@qq.com>
Reviewed-on: http://121.4.94.30:3000/mrlan/EnglishPal/pulls/48
Co-authored-by: 陈靖毅 <cjybyjk@zjnu.edu.cn>
Co-committed-by: 陈靖毅 <cjybyjk@zjnu.edu.cn>
2022-07-18 16:00:08 +08:00
邱忠辉
9d420acd8b
[REFACTOR] user_service.py: Added id to user name, word and frequency for click to execute AJAX request in JS ( #30 )
...
1. Add jquery.js and word_operation.js to the static folder.
2. Changed the return type of the familiar, unfamiliar and deleteword methods of user_service.py.
```
original: return redirect(url_for('userpage', username=username))
now: return "success"
```
3. In order to get elements in JS to make ajax requests we add ids in userpage_get.html for familiar, unfamiliar, delete buttons and so on.
4. When the user's word book was retrieved, the list returned was already sorted in descending order of frequency, so redundant code was removed in userpage_get.html.
Co-authored-by: PeterQiu <www.1392993990@qq.com>
Co-authored-by: Lan Hui <1348141770@qq.com>
Reviewed-on: http://121.4.94.30:3000/mrlan/EnglishPal/pulls/30
Co-authored-by: 邱忠辉 <1392993990@qq.com>
Co-committed-by: 邱忠辉 <1392993990@qq.com>
2022-07-18 13:29:04 +08:00
蒋乐天
041cbd97fc
更新 'app/static/js/highlight.js'
2022-06-13 21:32:49 +08:00
b53e7031e5
Bug412-JiangLetian
2022-06-13 11:40:20 +08:00
ba2ff7dee2
Remove music player js because that is not the focus of EnglishPal. If people want to listen music, they have better options than EnglishPal.
2022-01-27 11:40:58 +08:00
4d2dd2b68e
Repalce old app folder with SoftArch王炫/english-pal-master/app/
2022-01-26 21:10:09 +08:00
钱骏琪
Hui Lan
王志豪
李雨峰
mrlan
岑昊天
任榆
Lan Hui
陈靖毅
邱忠辉
蒋乐天
lin