如何重构此语句？

Question

重构代码的一种方法

var foot = foot.transform.position;
var hand = hand.transform.position;
if (distance > 0.5)
        {
          foot = hand;
          foot.transform.Translate(x , y  0);
        }
        else
        {
          foot.transform.Translate(x , y , 0);
         }

这是像下面的thie一样

var foot = foot.transform.position;
var hand = hand.transform.position;
distance > 0.5 ? {
          foot = hand,
          foot.transform.Translate(x , y  0)
                 } 
       : foot.transform.Translate(x , y  0);

，还是更多干净的代码/。

Answer 1

这可能是一个选择：

var instance = distance > 0.5 ? hand.transform.position : foot.transform.position;
instance.transform.Translate(x, y, 0);

Answer 2

1. 没有理由在IF及其他方面都具有相同的代码行，可以在该逻辑控制之后移动。
2. 没有理由拥有其他理由，因为我们删除了唯一的线后，就没有剩下的行。
3. 来自在进行重新分配时，语义清楚。

var limb = foot.transform.position;

if (distance > 0.5)
{
    limb = hand.transform.position;
}

limb.transform.Translate(x, y, 0);

也不需要三元操作员。

Answer 3

我可能会接受：

var foot = foot.transform.position;
var hand = hand.transform.position;
if (distance > 0.5)
{
  foot = hand;
  foot.transform.Translate(x , y  0);
}
else
{
  foot.transform.Translate(x , y , 0);
}

并将其重构为这样的东西：

var p = ( distance > 0.5 ? hand : foot ).transform.position ;
p.transform.Translate( x, y, 0 ) ;

因为表达的简洁性都具有全部效用。

您甚至可以使其成为“单线”，但它读起来很奇怪：

( distance > 0.5 ? hand : foot ).transform
                                .position
                                .transform
                                .Translate( x, y, 0 )
                                ;

前者读得更好