技术栈不多，只有express、mongodb和redis 还有完全没必要用的docker，似乎只有一台服务器 进程管理居然还是用的forever 缺少错误收集机制，更不用说故障报警 有日志模块，但是却多个服务写在同一个文件，毫无疑问如果开多进程日志有很大的风险被覆盖 前后端没有完全分离，有少量代码使用了ejs模板，完全可以砍掉做到前后端完全分离（前后端代码放一起难以分清哪些是前端哪些是后端，混乱） 因为有两个web服务入口点，其中一个是管理后台，完全不需要nginx却也套了一层，可以砍掉 产品文档、数据库表文档、运维文档之类还算有，不过有点多余，因为那些都是基础的东西，关键部分的文档却没有 没有接触到线上环境，从代码上看没有任何async/await到处是var和.then().then()，明显node版本应该在4.x左右 又遇见了global，完全失去了IDE对代码的追踪功能 往别人对象上添加方法和属性的行为，在node后端绝对是大忌 发现有个似乎自己写的Promise模块，在删掉前搜索了一下整个项目，结果还真使用到了，多次使用了.when()方法，寡人也是第一次见啊，必需砍掉这个不标准的东西 post方法本身支持json，而用的时候却到处都是json数据里的一个json字符串，奇葩 没有任何单元测试 没有开发环境的数据库初始数据，导致部分功能完全没法在本地跑起来，所以一个项目应该要做到能在完全新环境下跑起来
今天的感悟： 1.项目还是要选择越简单的技术栈越好，这样新人可以在最短的时间接手，不要为了用某技术而用某技术，像这个项目中的docker和nginx完全是多余的 2.项目初始提交是18年却还在使用17年就已经淘汰的异步处理方法 3.一个项目最容易跑起来的一定要是开发环境，任何生产环境的配置都不能出现在版本库中
下一步要干掉全局变量，升级node版本以支持async/await，彻底拆分前后端
“往别人对象上添加方法和属性的行为，在node后端绝对是大忌”,这个我不太了解,是害怕这个修改之后,后面创建这个对象的时候构造函数都要修改,还是什么的。。大佬可以说一下有什么坏处吗。。因为我这边写的后端,好像不太需要用到面向对象的写法…自己如果做到面向对象的时候避免一下这些错误。
@shynome 这个.when还真是自己写的Promise中的东西，由于没有文档，代码写得晦涩难懂，懒得去深入研究了，工作原理基本上就是传一个数组然后回调遍历它，在最后一个元素遍历完resolve后才会到下一个then，解决的问题就是每个元素都要做相似的异步处理然后才到下一步，完全是可以用原生的promise实现的 docker仅仅是一次性使用的，根本没有dockerfile，下午拿到了正式服务器的权限进去看了一下，发现服务器居然只有一个root用户，猜测以前的人可能不知道怎么配置服务器，为了图方便所以用了docker，但却给我造成了麻烦，因为以前的公司这些都是运维干，现在我还要兼职运维的工作了 目前没用到ssl，确实如你所说是多个站点，一个面向用户，另一个对内使用，对内使用url上加端口号就行了，没必要用nginx去做转发，我不喜欢搞这么复杂，必须砍掉
@cheerApril 1。破坏了封装性，假设你看到某模块一个函数的用法不清楚想去查文档，结果找半天找不到文档，最后发现是别人自己往上加的，你会不会很气愤 2。如果往上加的东西产生冲突，导致运行出问题，你需要花很多时间去找问题
往别人对象上添加方法和属性的行为，在node后端绝对是大忌
@zengming00 赞，但这个真正能体会到还需要写一段时间的代码才能体会。 docker其实本身就需要良好自动化构建环境才能对开发无感，但只有一台服务器，这些成本确实偏高了，nginx等有前置机后也可以再考虑，这方面楼主还是很务实的。 但本地应该不需要启动docker和启动nginx来开发吧？如果要这也太尴尬了，本地开发最好的方式还是一个npm钩子就能启动才是最好的。
@zy445566 本地开发完全用不到docker和nginx，所以我才要砍掉这玩意，昨天拿到了测试服和正式服务的权限，发现坑爹的是，外面一个nginx，docker里面又一个nginx，而且两个都在用，就是说，一个请求进来，要经过两层Nginx才到node去处理 还有更坑爹的，测试服和正式服居然是混用的
我真是醉了
[root@xx server]# ls -l /usr/local/nodejs/node-v4.2.3-linux-x64/bin/node
-rwxrwxr-x 1 500 500 34799826 May 16 2018 /usr/local/nodejs/node-v4.2.3-linux-x64/bin/node
[root@xx server]# /usr/local/nodejs/node-v4.2.3-linux-x64/bin/node -v
v8.11.2
不知道为什么代码是一样的，服务器上是正常的，在我本机上跑这块内容加载不出来，浏览器控制台没报错，我不怎么写前端，搞不明白哪里的问题，用的是Vue.js v1.0.24
我还以为vue1.x能挂载到注释里呢，找了半天终于发现原来大佬用了nginx 的ssi功能，也就是< !–#include file=“menu-V.html”-- >，终于明白为什么要套上一层nginx了，终于敢砍掉这玩意了
今天上午已经砍掉了docker