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

Tuesday, March 27, code review #2

Open
JamieMaple opened this issue Mar 27, 2018 · 1 comment
Open

Tuesday, March 27, code review #2

JamieMaple opened this issue Mar 27, 2018 · 1 comment

Comments

@JamieMaple
Copy link

Good

  1. 能用 markdown 写 Readme 了,值得肯定

  2. 爬取了数据还是不错的,看得出下了功夫

  3. 对模块化有了一定认识和了解了

  4. git log 有了规范性尝试,这个真的不错!希望继续保持,详细的规范建议是参考社区规范(比如某些开源库的写法)

Improve

先补充一下昨天忘记了的几点

  1. 首先你如果用不同的浏览器(包含手机)开发会发现页面初始的样式是不一样的,这时候我们应当对 css 进行一次初始化,google 搜索关键词: css reset, normalize.css

  2. 依然代码规范的问题,制表符一定要 2 空格!制表符一定要两空格!制表符一定要两空格!

  3. body>div 这样的父子选择器,建议中间用空格 body > div

  4. js 文件用到了 commonjs 规范,这个你会发现浏览器运行不了的,这是社区规范,详情可以看 阮老师的博客 以及 seajs 作者谈模块化 等,模块化是前端工程化非常重要的一部分!

  5. js 里面 require 的一些包,建议先使用原生 js 开发以了解刀耕火种的开发方式同时也能方便后面的框架和库的学习与使用

  6. 捕捉到的数据,其实那个 status 对于爬数据的时候其实不重要,这个是后端返回前端的一个状态码而已,暂时不用管

  7. 爬取到数据的方式或路程建议记录下来,代码也好,工具也罢都可以记录到 Readme 中

Suggestion

  1. 了解前端工程化(只做了解),重点是理解前端模块化,

  2. 上面的一条可能目前比较远,所以重点还是 js 的基础,一定要把原生 js 的基础打牢

  3. 可以尝试 google 关键词:nodejs,commonjs,amd,cmd,es6 modules,http 以及 前后端分离

总体而言,今天学到的绝不会无用,爱上这个自学的过程!

@JamieMaple
Copy link
Author

JamieMaple commented Mar 27, 2018

js 书本推荐一本:红宝书《JavaScript高级程序设计》

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

1 participant