test's People
test's Issues
修改意见
目录与路径
以后不要把项目这么放吧,就打开你这个项目的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
可以直接前往该分类的页面 - 上传后的图片会被拉伸,想个办法让其保持原有比例
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.