[转贴]"女程序员"代码...竟然还有增强版...

最新推荐文章于 2020-05-20 22:33:55 发布

转载最新推荐文章于 2020-05-20 22:33:55 发布 · 961 阅读

文章标签：

#function #工作

杂七杂八专栏收录该内容

7 篇文章

订阅专栏

本文通过一个虚构的爱情算法示例，展示了如何精简代码并提高其可读性和效率。通过对原始算法进行重构，包括提取重复逻辑为独立函数、合并相似操作、调整线程处理方式等步骤，最终实现了更加紧凑且逻辑清晰的代码。

result love(boy, girl)
{
    if( boy.有房() and boy.有车() )
    {
        boy.set(nothing);
        return girl.嫁给(boy);
    }
    if( girl.愿意等() )
    {
    while(!(boy.赚钱 > 100,000 and girl.感情 > 8 ) )
    {
        for( day=1; day <=365; day++)
        {
            if( day == 情人节 )
            if( boy.givegirl(玫瑰) )
                girl.感情++;
            else
                girl.感情--;　　
            if( day == girl.生日 )
            if( boy.givegirl(玫瑰) )
            {
                girl.感情++;
            else
                girl.感情--;
                boy.拼命赚钱();
            }
        }
        if( boy.有房() and boy.有车() )
        {
            boy.set(nothing);
            return girl.嫁给(boy);
        }
        年龄++;
        girl.感情--;
    }
    return girl.goto( another_boy);
}

这段代码可以写得更紧凑些：

1. girl()和boy()结婚的条件重复了两次，应该用单独的函数包装。
2. 在365天的循环中，对待两个特殊日子的处理逻辑完全相同，应该合并，将对日子的处理的语句写在一起。
3. boy.拼命赚钱()是boy在单独线程里面处理的，girl只是在检查boy线程工作的状态，所以boy.拼命赚钱()语句不应该出现，而应该换线程等待语句，比如sleep(1天);
4. 最后一个return 语句有很大的语病，正确的逻辑是选择一个新的boy，然后重新递归执行：return love(selectAnotherBoy());

修改后如下：

function marry(boy, girl)
{
boy.set(nothing);
return girl.嫁给(boy);
}

function canMarry(boy, girl)
{
return boy.有房() and boy.有车();
}

function love(boy, girl)
{
if (canMarry(boy, girl) {
marry(boy, girl);
return;
}

if (girl.愿意等()) {
  for (; !(boy.年收入 > 100,000 && girl.感情 > 8); girl.感情 --) {
   for (day=1; day <=365; day++) {
    if (day == 情人节 || day == girl.生日) {
     if (boy.givegirl(玫瑰))
      girl.感情++;
     else
      girl.感情--;　
    }
    sleep(1天);
   }

   if (canMarry(boy, girl) {
    marry(boy, girl);
    return;
   }
  }
}
love(getAnotherBoy(), girl);
}