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

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

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

    iOS fly ~
  • j
    jckimi
    你如果是主管的话,责无旁贷,而且,你就这么漏了源代码,不怕有问题么。看样子是对日外包?
  • i
    icestone
    这不是源代码,这个是github上找的练习题给他们联系用的,公司内部源码我才不会漏的。。iOS fly ~
  • b
    b0207191
    买个coverity检查
  • j
    jckimi
    回复3#icestone


    那代码review,就只能是你了。
  • s
    sayhier
    同理问一下,软件外包,自己又不懂软件的话,可以找什么来把把关?
  • 2
    2pang
    就一个按序插入,写的这么复杂。。。
  • L
    LittleBlue
    这是要外包面试吧,这个行业挺有前途。
    我之前还想过外包debug的
  • y
    ywergs
    一个循环就能搞定,不用区分区分首尾。(对java不熟,不过感觉应该可以)
  • 2
    2pang
    虽然不知道具体的数据结构

    ComparableCell cell = this.list.getFirst();

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

    cell = cell.getNext();
    }

    if (cell == null) {

    } else {

    }

    但这种写法不好么
  • y
    ynr
    原list是有序的话,第一个else块内写的的确冗余
  • t
    tiens
    确实,后面两个else是累赘,从前往后遍历列表元素,取cell值与value比较,只要value<data,就insertCellBefore即可。
    如果api没有insertCellBefore,只能用after的话,就可以用getPrev从后向前遍历。
    总之代码行数和可读性都会比现在好,执行效率也不低。
  • x
    xx6412223
    按序插入,用binary search 用递归。这么写。。。。太难看 效率低。
  • i
    icestone
    这个课题就是写sortedList啊,下个课题才到二叉树 iOS fly ~
  • x
    xx6412223
    回复14#icestone

    二分查找对链表,不用搜索树
  • y
    ywergs
    如果是学生作业,写成这样可以,如果是要工作中,写成这样,换成我,我会发飙的。
  • z
    zztt168
    死电工太矫情,代码能跑业务就行了
  • w
    windflash
    炸出来这么多码农?
  • c
    conglin890
    有这时间看会电视不好,代码能跑就行 iOS fly ~
  • r
    richking
    能跑而且逻辑挺清楚,符合常人的思考习惯。还知道用一头一尾预判断来加快速度。但是用二分法查找的话代码质量和效能会上一个台阶。
  • l
    linlance2000
    我是个不称职的码农。。。
    每次写这种东西,我都需要查资料,我记不住.next 之类的方法。。
  • i
    icestone
    这是让他们自己在写api,所有方法都是自己写出来的。所以现在只能调用现有的方法。 iOS fly ~
  • m
    mephistophilis
    这一行行的波浪线 你看着不恶心吗?小尾巴~
  • 5
    51vip
    提高机器配置,上 resharper ,设置好代码质量优化规则