哪里有人专门给做code review的啊

手下带的partner代码写的有点冗长，我一是没时间，而是技术也不够过硬。给人review也没信心。所以说不知道哪里可以有人给review.

话说这个代码能再简化嘛？list的两头进行特殊处理，这个其实没有必要吧，不过在尾部的cell
之后插入的话，getNext为null，肯定报nullpointer卡住了

iOS fly ~

你如果是主管的话，责无旁贷，而且，你就这么漏了源代码，不怕有问题么。看样子是对日外包？

这不是源代码，这个是github上找的练习题给他们联系用的，公司内部源码我才不会漏的。。iOS fly ~

买个coverity检查

回复3#icestone


那代码review，就只能是你了。

同理问一下，软件外包，自己又不懂软件的话，可以找什么来把把关？

就一个按序插入，写的这么复杂。。。

这是要外包面试吧，这个行业挺有前途。
我之前还想过外包debug的

一个循环就能搞定，不用区分区分首尾。（对java不熟，不过感觉应该可以）

虽然不知道具体的数据结构

ComparableCell cell = this.list.getFirst();

while(cell != null) {
if (cell.getData().compareTo(value) >= 0) {
break;
}

cell = cell.getNext();
}

if (cell == null) {

} else {

}

但这种写法不好么

原list是有序的话，第一个else块内写的的确冗余

确实，后面两个else是累赘，从前往后遍历列表元素，取cell值与value比较，只要value<data，就insertCellBefore即可。
如果api没有insertCellBefore，只能用after的话，就可以用getPrev从后向前遍历。
总之代码行数和可读性都会比现在好，执行效率也不低。

按序插入，用binary search 用递归。这么写。。。。太难看 效率低。

这个课题就是写sortedList啊，下个课题才到二叉树 iOS fly ~

回复14#icestone

二分查找对链表，不用搜索树

如果是学生作业，写成这样可以，如果是要工作中，写成这样，换成我，我会发飙的。

死电工太矫情，代码能跑业务就行了

炸出来这么多码农？

有这时间看会电视不好，代码能跑就行 iOS fly ~

能跑而且逻辑挺清楚，符合常人的思考习惯。还知道用一头一尾预判断来加快速度。但是用二分法查找的话代码质量和效能会上一个台阶。

我是个不称职的码农。。。
每次写这种东西，我都需要查资料，我记不住.next 之类的方法。。

这是让他们自己在写api，所有方法都是自己写出来的。所以现在只能调用现有的方法。 iOS fly ~

这一行行的波浪线 你看着不恶心吗？小尾巴~

提高机器配置，上 resharper ，设置好代码质量优化规则