sakaneko117 / HUOZI

ottohzys
106 stars 16 forks source link

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

Open hua-zhi-wan opened 1 year ago

hua-zhi-wan commented 1 year ago

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

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

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

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

DSP-8192 commented 1 year ago

我超,真大佬来了

sakaneko117 commented 1 year ago

给大佬磕头了

hua-zhi-wan commented 1 year ago

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

CwavGuy commented 1 year ago

给大佬磕头了(砰砰砰