test,ddloo

修改意见

目录与路径

以后不要把项目这么放吧，就打开你这个项目的github页面，根目录底下还有一个test目录。我觉得最好还是把 test目录里面所有的东西提升到 github 的根目录里来。

readme.md

为你的项目添加一个 readme.md 文件，描述一下，告诉别人如何使用你的网站

.htaccess

你的 .htaccess 文件的关于导航到image.html的那一块应该改成:

RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^.*$ image.html

这样的话，有助于你访问 / 时也能达到你网站的根目录，如果你有其他定制需求，那就除外

script 标签

无论是否为第三方的js文件，你都应当尝试放在 <body> 的最后面。

善用 jquery

看到你引入了 jquery 之后却在代码中出现了大量的 document.getElementById ，并且绑定事件也用的是 xxx.onclick，你可以发挥出 jquery 的作用，能够用 jquery 的方法尽可能用 jquery 呀~

闭包执行

你所有的 js 代码都是暴露在全局环境下的，你应该将所有的代码封装于一个立即执行函数中，类似于

(function(){
    console.log("hello");
})();

关于 model 的认知错误

我发现在你的 link 文件夹底下，应该就是一些 model 吧，你将 user 分为 loginUser 和 registerUser。其实这样是不对的，model本身指的就是一种对应于显示的实体，这个实体可以拥有若干行为。例如，一个 User model 对应现实中的一个人，这个人可以注册、也可以登录。所以实际上你应该将这两个文件进行合并。

你在对图片进行分类的做法是完全错误的

totally 错误的，试着想象一下，如果你以后想加多一个分类，你得完成多少事情？

因为目前而言，所有和分类有关的东西几乎都已经写死了！

建议先从这个点改起，太重要了。重构会花费你很多时间。

新建一个名为 category 的数据表，这个数据表中有至少2个字段：category_id、title。这个表格用于记录你到底有哪些分类。
前端页面中的分类不应该写死，而是从数据库中读取到底有哪些分类并渲染出来
你的 image 表中应当多一个字段 category_id，用来记录你的图片到底属于哪个分类。

这个够你忙的了哈哈哈

关于存储图片

你不应该直接拿用户的图片文件名作为最终的文件名，而是应该在拿到用户图片之后，为这个图片重新随机命一个名字，然后再记录到数据库中。
你大可以以当天时间的时间戳混合上一串随机字符串作为文件名。
不然的话，现在无法上传同名文件。

也许你有困惑，那我怎么判断两张图片是否一模一样。我们一般不判断。何况我们现在也可以通过改变本地图片名字来上传两张一样的图片。

一些优化建议

在一些弹窗中应该让用户有退出弹窗的手段，例如点击弹窗外面的部分;
为你的弹窗统一样式和一些可以共用的脚本，因为你的弹窗的逻辑非常像，你可以把一些共用的逻辑抽象一下让每个弹窗都可以用而不是每个弹窗都重新写一遍;
在 url 中附上 ?category=design 可以直接前往该分类的页面
上传后的图片会被拉伸，想个办法让其保持原有比例

ddloo / test Goto Github PK

test's People

Contributors

Watchers

test's Issues

修改意见

目录与路径

readme.md

.htaccess

script 标签

善用 jquery

闭包执行

关于 model 的认知错误

你在对图片进行分类的做法是完全错误的

关于存储图片

一些优化建议

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent