joy joe
# 先膜拜一下妹子 # 提几点建议 1. 代码应该精简 既然都用到构造函数了，那就应该把draw和cache里面绘制圆点的代码封装成一个原型链上的函数。 绘制多点之间连线的代码也可以通过循环来精简。另外在”点“对象代码中，有一些冗余的context属性设置，比如alpha等等。还可以考虑将Star和Dot的关系整合成父子继承关系，优化代码结构和提高性能。 2. 配置项可以扩展 可以多设置些配置项，这样能够更好体现个性化。比如：颜色，多点连线的条数。 颜色值，可以放入配置中，这样不用局限于默认的白色。多点之间连线的条数，可以方便用循环代码来实现。不用像现在这样，写出毫无扩展性的代码。 3. 扩展思维 在重绘函数drawIfMouseMoving中，是否可以考虑通过判断用户鼠标移动方向来设置Variable这样的偏移值方向，而不是目前这种随机方向。 ## 下面把我个人优化后的代码贴出来，有错误欢迎指正。
**react-and-redux/chapter-05/todo_perf/src/todos/views/todoList.js** 在这个文件中 ``` const TodoList = ({todos, onClickTodo }) => { }; ``` 定义的TodoList组件为什么会有 onClickTodo这个prop？这个prop属性没有使用过呀？ 是不是写错了？
113页第4段 原文： > 所以这个函数要用某种方法 **_区分什么TodoItem回调这个函数_** ，区分的方法只能通过函数参数。 这个地方 **_区分什么_** 这一块 好像不通顺呀。
第19页第3段第4行有错别字 原文： > 函数类型的prop等于让父组件交给了子组件一个回调函数，子组件在恰当的 **_实际_** 调用函数类型的prop，可以带上必要的参数，这样就可以反过来把信息传递给外部世界 这个地方的 **_实际_** 应该是 **_时机_**