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

选课模块重构计划 #33

Open
ComMouse opened this issue Nov 23, 2017 · 14 comments
Open

选课模块重构计划 #33

ComMouse opened this issue Nov 23, 2017 · 14 comments
Labels

Comments

@ComMouse
Copy link
Collaborator

ComMouse commented Nov 23, 2017

现有的选课模块代码较老,已难以维护,需要考虑重构一版。

目前选课模块的核心功能有:

  • 点击 Radio Button 快速跳转
  • 小课表
  • 教师评教得分
  • 快速查询课程安排及人数是否已满
  • 可冲抵通识课显示(仅 15 级及之前,仅根据课程名称提示)

考虑到前端技术的更迭,jQuery 在前端中使用频率逐步减少,目前有将 SPA 框架,如 Vue 等引入开发的计划,简化代码实现。但需要考虑这些框架或类库的维护门槛较 jQuery 更高一些,可能不利于后续维护。

关于功能更新和重构技术选择的问题,在此供各位讨论。

cc @ShieLian @gaocegege @laohyx

@ComMouse
Copy link
Collaborator Author

补充:由于现代浏览器中已基本包括 jQuery 的 DOM 和 AJAX 相关的 API,亦可以考虑直接基于原生 API 编写。

@at15
Copy link
Collaborator

at15 commented Nov 23, 2017

vue 的话是不是有点太多了? .....

@fztfztfztfzt
Copy link
Contributor

如果用Vue的话,是不是可以考虑把整个选课界面都重新渲染一遍。

@ComMouse
Copy link
Collaborator Author

@fztfztfztfzt 目前插件以优化现有页面为主,重渲染的话侵入程度可能太强了

@gaocegege
Copy link
Collaborator

并不需要 at 我,我对 vue 一无所知

@at15
Copy link
Collaborator

at15 commented Nov 23, 2017

@gaocegege 你就知道发狗粮

@gaocegege
Copy link
Collaborator

你行你也发

@at15
Copy link
Collaborator

at15 commented Nov 23, 2017

@gaocegege .... 向狗粮大佬低头 /w\

@ShieLian
Copy link
Collaborator

对vue和angular一无所知(

@ComMouse
Copy link
Collaborator Author

近期准备开坑,估计还是走原生了……

@at15
Copy link
Collaborator

at15 commented Dec 26, 2017

豆腿真是....

@at15
Copy link
Collaborator

at15 commented Apr 4, 2018

已经 4 个月了,鸽豆? @ComMouse

@hczhcz
Copy link

hczhcz commented Apr 4, 2018

建议原生,electsys的生命周期对于框架来说太长

// emmm…几个月前的issue了嘛,无视我好了

@ComMouse
Copy link
Collaborator Author

ComMouse commented Apr 7, 2018

@at15 鸽了,有意接手插件的可以联系~

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

No branches or pull requests

6 participants