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

可恶,好想在这里大骂一通 #9

Open
hua-zhi-wan opened this issue Nov 26, 2022 · 4 comments
Open

可恶,好想在这里大骂一通 #9

hua-zhi-wan opened this issue Nov 26, 2022 · 4 comments

Comments

@hua-zhi-wan
Copy link

阿B上看到这个项目,过来看看热闹。扫了两眼打算二次开发自己部署了。

简单描述一下看到的不妥的地方:

  1. 活字印刷类设计的不是很规范,内部的抽象较为耦合,比如对“配置项”这种适合分离设计的功能和文件系统一起耦合在类内,就比较不妥。
  2. 前端问题1:样式错误。前端部分问题相当多,我分成两点说。首先是BootStrap的使用问题,最基本的布局规则都不了解,胡乱使用样式,甚至乱加校验导致前端并没有引用到样式。使得整个项目外观非常抽象。
  3. 前端问题2:手写ajax?那你用JQuery干什么呢?
  4. 代码风格问题。这一点可能见仁见智,但……至少统一一下抽象层次吧。

综合来讲,虽然精神可嘉,但整个项目结构极度混乱,继续开发是史上雕花。Fork一下我自己搞一个新的。

@DSP-8192
Copy link
Contributor

我超,真大佬来了

@sakaneko117
Copy link
Owner

给大佬磕头了

@hua-zhi-wan
Copy link
Author

好了,我拉了一坨更干净一些的,xdm的小服务器可以歇一歇了♿。

@CwavGuy
Copy link

CwavGuy commented Jan 28, 2023

给大佬磕头了(砰砰砰

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

4 participants