yangjunhan / angular-hacker-news Goto Github PK

改进建议：

1. 加入 https://prettier.io/ 配置，统一格式化代码
2. router 的 category 使用 router params，而不是写死在data里面
3. tslint 中 增加 "no-any": true
4. service中应该通过 map 封装好 total Page， 而不是在 component 再进行计算
   

   angular-hacker-news/angular-hacker-news/src/app/components/news-list/news-list.component.ts

   Line 44 in f9ea8fe

   this.totalPage = Math.ceil(this.api.dataSize / this.pageSize);

5. 所有retry 接口的代码在 http interceptor 中进行封装 https://angular.io/api/common/http/HttpInterceptor

新的作业：

1. 所有的 Component 修改为 OnPush 模式：https://netbasal.com/a-comprehensive-guide-to-angular-onpush-change-detection-strategy-5bac493074a4
2. 对时间处理的函数修改为 date-fns

1. interceptor 与 service 拆分开
2. 搞清楚 markForCheck 、 OnPush、zone.js 与 detectChanges 之间的关系，直接在这个 issue 下面回复
   ref https://danielwiehl.github.io/edu-angular-change-detection/
   ref https://angular.io/guide/zone
3. 在 tsconfig 中增加 paths，使用路径映射 https://www.tslang.cn/docs/handbook/module-resolution.html 将引入的相对地址改为绝对地址
4. 将 page 改为按 module lazyload 加载