fciasth/web_tasks

HHU‘s Innovation laboratory web_tasks

web_tasks's Issues

几点修改建议

1. 在提交代码时，commit信息最好是有意义的内容
2. 要写注释，就算这段代码很简单
3. 考虑使用矩阵乘法来加速运算，see: http://www.cnblogs.com/xudong-bupt/archive/2013/03/19/2966954.html
4. 没有判断输入内容的范围，没有做错误处理，没有向用户提示错误
5. 虽然使用了两种计算方法，但只封装了一个，而且第二个方法还对第一个有部分的依赖（line: 17）
6. 输出的内容中，第一个数字应当是1，不要输出多余的0
7. 虽然使用了数组来保存临时结果，但是却并没有带来任何性能上的提升，反而造成了空间的浪费。如果使用数组可以考虑运行一次后将结果作为缓存，在下一次计算时便可以减少计算量
8. 第一个方法中，由于是递归运算，效率非常低，不推荐使用

检查点评分：
A --- 1. github上存在相应项目目录并且结构正确
B --- 2. fib.rb文件在运行时会首先让用户输入一个1-100之间的数字n，然后他会输出一个1-n的斐波那契数列

额外加分项：

1. 使用了两种方法编写程序

B1评价及修改建议

建议

1. 用户没输入查询条件直接点击查询时，应该给用户返回全部记录的列表
2. 编辑功能似乎无效
3. 用户体验不太好，添加文章后应该跳转到首页去，修改后也应该是要跳转的，否则用户都不知道自己有没有提交成功了
4. 希望你能借鉴一下我给的例子，写出更加ruby风格的代码
5. 中规中矩，没有什么亮点，希望能多加入一些自己的功能，来让这个程序更好用

评分

A -- 1. get / 时，可以看到留言列表和筛选表单，留言列表按照倒序排序，且每条留言后有一个删除按钮
B -- 2. 通过输入筛选条件，可以正确筛选出所需的条目
A -- 3. 点击删除按钮时，可以正确删除
A -- 4. 当用户提交无效的信息时，会提示用户出错信息。这里需要对所有可能的用户提交内容做验证。
B -- 5. 建立了一个类来管理有关留言内容的操作，且结构满足规范
B -- 6. 留言类的每一个方法都能考虑到各种的输入情况，能正确的返回数据或错误信息

抱歉那么晚才改你这个作业，不过由于时间关系就直接给你评分了，希望你能在B2和B3项目里表现的更好一些。

总评：B