WIP: Refactor-huangzirui #97

huangzirui · 2023-06-02T21:55:56+08:00

huangzirui commented

2023-06-02 21:55:56 +08:00

重构了js文件夹下部分js代码
1.重构了部分方法，代码更为简洁。
2.一些目前已弃用的属性或方法修改为支持的属性或方法。
3.修改了部分代码书写规范。
4.优化了高亮和取消高亮方法：
a.支持更多的浏览器实现高亮；
b.对性能再次优化，选中大篇文章能较快显示；
c.发现并解决了文章中若存在单词“mark”，选中“mark”以及“mark”之前的部分单词时，会导致文章发生错误的bug；
d.优化了cancelHighlighting方法，减少了时间、空间复杂度，使代码更为简洁；
5.删除了tanyanmei-fillword.js文件，测试过程中未发现其用途。

重构了js文件夹下部分js代码 1.重构了部分方法，代码更为简洁。 2.一些目前已弃用的属性或方法修改为支持的属性或方法。 3.修改了部分代码书写规范。 4.优化了高亮和取消高亮方法： a.支持更多的浏览器实现高亮； b.对性能再次优化，选中大篇文章能较快显示； c.发现并解决了文章中若存在单词“mark”，选中“mark”以及“mark”之前的部分单词时，会导致文章发生错误的bug； d.优化了cancelHighlighting方法，减少了时间、空间复杂度，使代码更为简洁； 5.删除了tanyanmei-fillword.js文件，测试过程中未发现其用途。

👍 1

huangzirui added 118 commits 2023-06-02 21:55:57 +08:00

8924166975 上传文件至 'static\js'

417dbc22f8 highlight.js: fix Bug 522.

c76d4f21ec SPM2022F-CONTRIBUTORS-XieQiuHan

3bce450620 黄子睿: 修复 'Otherwise,' 这种情况无法高亮的问题，即 Otherwise 后面跟了个逗号

74eccfbebd 重复文章刷新

0209548896 提交作业

43bd0bd09d update requirement.txt

93390374ad 标注之前队伍的改动

cb0132fd31 pull master代码，并成功运行

944c931c9b 完成了对bug509的修复，以及重构项目（去掉了业务中的前端脚本）

90db6534ab fix: merge latest remote master branch code

376ef9bcbc feat: add pong orm requirement

df82748518 feat: create classes necessary for orm operations

e2b165ada8 fix: add url 'admin' into banned url list

13d8977636 fix: set specific management displays for admin

ade10e5843 feat: add admin_service blueprint

b80fbc936c fix: fix admin_name

70d44fcf5c Login.py: fix SyntaxWrning: 'is not' with a literal.

b34f260d98 requirements.txt: use an older version of Flask to avoid deployment ERROR: No matching distribution found for Flask==2.1.0.

cabf6702a7 fix: add one way to set article level & rename some functions and vars

1015704e23 fix: fix the way to check article level

48de496caa fix: Remove 'Assignment Expresions' & Fix annotation words

44db2218c1 fix: Use better conditional judgment methods

691c5b0d43 build.sh: Install requirements first.

9075fe9eea Resolve merge conflicts.

2909b4d973 admin_service.py: 管理员页面显示的用户名按照名字排序。

6df25c58b4 查漏，业务中的两处前端标签不做修改，因为不被使用了

c9bfa08658 注释flash的使用，因为其对页面会有影响

5cffa1fada fix: use single quotation mark 'admin'

52025d55bc fix: add a blankspace

bdda754af6 fix: check current user is admin

ec6a2249ae fix: fix the pagination

13ccbaf25c fix: use select to choose article level

82896de336 update .gitignore

3e35679a91 refactor: refactor the model

2cf65123e9 feat: admin can manage articles and users without interfering with each other

7941e5d1eb fix: fix the way to show article title

e27985127a admin_manage_article.html: 文章最高难度等级是4.

b97210a9e0 admin_manage_article.html: 更为详细的 placeholder 内容.

7eb276937a admin_manage_user.html: 将默认过期时间设为365天以后.

a220450b03 mainpage_get.html: 首页不显示管理链接（可能会安全点）.

99aa4e0990 fix: fix article title show

ce28a5bf65 feat: auto select expiry_date when select user

f17995a35c fix: using new pagination mod func

c6010ccbbd admin_manage_user.html: 不再需要，所以删除.

fb6d0b23ce admin_manage_user.html: 修改 JavaScript 函数名.

3bc61a602f 添加文章成功后，修改用户信息成功后，页面显示成功信息（flash messages）。

8d8b9197b6 手动输入的文字最高难度等级是4

4d99405bfa 简化管理员页面信息。删除退出登录按钮，可以返回到前一页后再退出，不影响使用体验。删除'管理员您好'欢迎词，没啥意义。

b3b154a24f 简化管理文章与管理用户页面信息。

0e25737381 管理文章页面的文章列表中，每篇文章不再在内容部分重新显示标题。

9eb5210d3f Level与Date的冒号后面加个空格，使得后面的信息更加看得清楚。

d4ac709385 将删除按钮移到第一行，避免因为文章的标题过长跨行导致删除按钮形状改变。

0ce1c6eb6e 文章管理页面：每篇文章中保留换行，方便查看。

5b2f5199a8 1. 取消userpage_get.html中提示删除单词信息的代码和取消user_service.userpage中render_template的flashed_messages参数。因为删除单词操作已经是异步了，而提示信息的出现是同步执行，所以就注释了代码且没有产生太大影响。

2. 修改取消user_service.deleteword中对注释flash代码的注释，根据上一步进行了重新解释。

0c16a4dc6f 判断文章是否已经出现的语句写错位置了，改正下

4417cf7017 Article.py: remove debug statement.

6f1dd13419 测试的print忘删了

7107f634c2 Merge branch 'test' into dev-fixBug509-reconstruction

1f150fc847 refactor: use ajax to get expiry_date

688ed72473 Correct grammar。

15bb925024 将记录阅读过文章的数据结果改为字典，以及修改了flag的问题

f3d609c92b Merge Wang Ziming's work and Wu Yuhan's work.

5d20e92061 Merge branch 'Bug522-HuangZirui' of http://121.4.94.30:3000/mrlan/EnglishPal into DevLocal

cc8ca47f8c refactor: remove sql sentences

11ae093fd7 Merge branch 'Alpha' into Bug522-HuangZirui

ce2e1f2978 Merge branch 'DevLocal' into Bug522-HuangZirui

16de0a7fd9 修改变量命名：existing_articles → had_read_articles

84affaeb69 修改 /<username> 路由存在的问题（每次调用别的路由他都会被调用），新路由为 /<username>/userpage;同时因为修改了路由导致访问userpage_get的时候会导致静态文件路径生成错误，这里修改了\static\config.yml中的静态资源路径，修改后也都可以正常访问到的

da13e5bbd5 修复Bug（没找到文章后立即上一篇会回到上上篇文章） & 标签添加id方便测试

8f132ed87b 添加了阅读完所有文章的提示

70917df47b 删除测试代码

03145b57d9 修复边界值问题（当刚开始就没有找到文章或者就根本被没有文章的时候，会出现上一篇按钮）

fc3e27488b 给标签添加id，方便测试

3eca9234a9 Merge branch 'Bug393-TanYanMei' of http://121.4.94.30:3000/mrlan/EnglishPal into Bug393-TanYanMei

320a99d479 Move Tan Yanmei's fillword.js to app/static/js/tanyanmei-fillword.js. Delete the strange folder static js.

e9e2bd3d23 Remove static\js

d6bd24ee1c Merge branch 'Alpha' of http://121.4.94.30:3000/mrlan/EnglishPal into Bug393-TanYanMei

merge alpha

58d7349afe Change from bug359-zhangkeli

21a77ef2df Merge branch 'Alpha' of http://121.4.94.30:3000/mrlan/EnglishPal into Alpha

ef786795e2 Resolve merge conflict

6be035f282 修复当没有找到文章或者文章读完时，直接刷新页面或者session不关闭重新进入页面，导致的错误；

b88bc8f36b Merge branch 'Bug509-XieQiuHan-WangZiming' of http://121.4.94.30:3000/mrlan/EnglishPal into Alpha-snapshot20230425

d30a434b2a 修改变量名had_read_articles->visited_articles

5654fbf9bc 修改：使用新的/<username>/userpage路由

e118d92659 Merge branch 'Alpha-snapshot20230425' of http://121.4.94.30:3000/mrlan/EnglishPal into Alpha-snapshot20230427

779dafefe8 Merge branch 'Bug509-XieQiuHan-WangZiming' of http://121.4.94.30:3000/mrlan/EnglishPal into Alpha-snapshot20230427

a80b062b87 refactor: remove variable 'count'

18ca48b422 Merge branch 'Bug522-HuangZirui' of http://121.4.94.30:3000/mrlan/EnglishPal into Bug522-HuangZirui

a4cc4fd011 Merge branch 'Bug522-HuangZirui' of http://121.4.94.30:3000/mrlan/EnglishPal into Alpha-snapshot20230506

f64d06fbbf fix: fix Bug 531 and use ES6 grammar

01ecc83768 refactor: refactor the way to check article level

ce28b91bd5 屏幕刷新，停止阅读

df64065dcc 点击上下篇，停止阅读

becef7e343 Merge branch 'Bug502-YuGaoXiang' of http://121.4.94.30:3000/mrlan/EnglishPal into Alpha-snapshot20230506

5039f5710e AJAX载入文章数据

d9f6df7fbe AJAX载入文章数据

9f3f5b43e1 special_characters = '\_©~<=>+/[]*&$%^@.,?!:;#()"“”—‘’{}|' 用于过滤字符，我将其中的“-”删去，使连字符没有被过滤，实现录入例如fifty-six等组合词的功能。另外对于删除过滤是否会引发字符bug，答案是肯定的，但是这段代码中的过滤字符虽然多，但是并没有完全过滤掉所有字符，（过滤的只是键盘上能打出的字符，不包括输入法中能打出的特殊字符），所以字符bug本身就一直存在，我认为减少一个对“-”字符的过滤不会造成问题。

acd8db6e3e special_characters = '\_©~<=>+/[]*&$%^@.,?!:;#()"“”—‘’{}|' 用于过滤字符，我将其中的“-”删去，使连字符没有被过滤，实现录入例如fifty-six等组合词的功能。另外对于删除过滤是否会引发字符bug，答案是肯定的，但是这段代码中的过滤字符虽然多，但是并没有完全过滤掉所有字符，（过滤的只是键盘上能打出的字符，不包括输入法中能打出的特殊字符），所以字符bug本身就一直存在，我认为减少一个对“1-”字符的过滤不会造成问题。

39d96014d9 pull最新的snapshot-20230511，后更新了difficulty.py和Article.py的部分代码，提交了新的pickle文件

349488167b requirements.txt: install snowballstemmer for better computing a word's difficulty level.

030b89706e special_characters = '\_©~<=>+/[]*&$%^@.,?!:;#()"“”—‘’{}|' 用于过滤字符，我将其中的“-”删去，使连字符没有被过滤，实现录入例如fifty-six等组合词的功能。另外对于删除过滤是否会引发字符bug，答案是肯定的，但是这段代码中的过滤字符虽然多，但是并没有完全过滤掉所有字符，（过滤的只是键盘上能打出的字符，不包括输入法中能打出的特殊字符），所以字符bug本身就一直存在，我认为减少一个“-”字符对程序的过滤过程不会造成问题。

c151a0efaa 去掉了get_difficulty_level_for_user的多出的break

1295616d5b Merge branch 'Bug476-YuHuangtao' of http://121.4.94.30:3000/mrlan/EnglishPal into Alpha-snapshot20230519

a5c3564f15 difficulty.py: do not stem a word twice.

67e921ba60 difficulty.py: todo.

b41e1044bc difficulty.py: add some stop words, hoping that getting the next article can be faster.

03ccb3527a 重构前端阅读js，新增阅读器全局对象，新增生词朗读按钮

c6bf323c60 修改格式

1543b3095d Merge remote-tracking branch 'origin/Alpha-snapshot20230519' into Refactor-XunYucan

a0ddf4bdad 上传文件至 'app/static/js'

修复了 Bug492 选中问号出现多个问号的问题。
解决了选中紧跟标点符号的单词，单词能正常显示。
优化了选中较长的文章时页面容易出现卡顿的问题。

688a198768 已经与Alpha-snapshot20230525 分支同步，重新提交

38837c9c2f 合并最新的Alpha-snapshot20230529

cdf6180901 刷新屏幕，点击上下篇，加入生词库，停止阅读

d402bb45cb 刷新屏幕，点击上下篇，加入生词库，停止阅读(更改)

b22c654f0f Merge branch 'Bug529-GuHan' of http://121.4.94.30:3000/mrlan/EnglishPal into Alpha-snapshot20230601

682247bff1 refactor partial function and code writing specifications

9816596cf8 删除 'app/static/js/tanyanmei-fillword.js'

mrlan commented

2023-06-04 18:39:06 +08:00

谢谢。可以在优化的部分添加点注释吗？解释一下。

huangzirui added 1 commit 2023-06-05 08:37:32 +08:00

616d35a92f 增加了部分优化代码的注释

This pull request has changes conflicting with the target branch.

app/templates/userpage_get.html

You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.

git checkout -b refactor-huangzirui master

git pull origin refactor-huangzirui

Step 2:

Merge the changes and update on Gitea.

git checkout master

git merge --no-ff refactor-huangzirui

git push origin master

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: mrlan/EnglishPal#97