根据N值中的哪一个返回不同的结果

Question

不知道如何使标题更具描述性，所以我只是从一个例子开始。我正在使用下面的代码位来选择一个枚举的方向，这取决于与给定方向相比，四个轴中的哪一个形成了最小的角度。

static Direction VectorToDirection(Vector2 direction) 
{ 
    double upDiff = System.Math.Acos(Vector2.Dot(direction, -Vector2.UnitY)); 
    double downDiff = System.Math.Acos(Vector2.Dot(direction, Vector2.UnitY)); 
    double leftDiff = System.Math.Acos(Vector2.Dot(direction, -Vector2.UnitX)); 
    double rightDiff = System.Math.Acos(Vector2.Dot(direction, Vector2.UnitX)); 

    double smallest = System.Math.Min(System.Math.Min(upDiff, downDiff), System.Math.Min(leftDiff, rightDiff)); 

    // This is the part I'm unsure about i.e. 
    // Comparing smallest with each value in turn 
    // To find out which of the four was "selected" 
    if (smallest == upDiff) return Direction.Up; 
    if (smallest == downDiff) return Direction.Down; 
    if (smallest == leftDiff) return Direction.Left; 
    return Direction.Right; 
} 

但我在最后得到关于浮点相等的resharper警告。我猜这不应该是一个问题，因为执行Min，但想知道是否有更好的方式来解决这种问题除了比较smallest与每个原始值。

Answer 1

我会把所有的选择放在一个数组中，并找到最小指数。对于4种选择，排序可能是过度的。如果此代码的性能很重要 - 请确保为不同的变体测量时间。下面

非编译代码：

static Direction VectorToDirection(Vector2 direction) 
{ 
    var directions = new Direction[]{ 
    Direction.Up, Direction.Down, Direction.Right, Direction.Left }; 
    var unit = new Vector2[] { 
    -Vector2.UnitY, Vector2.UnitY, Vector2.UnitX,-Vector2.UnitY}; 

    var minAngle = 10; 
    var minIndex = -1; 
    for(var index = 0; index < directions.length; index++) 
    { 
    double diff = System.Math.Acos(Vector2.Dot(direction, unit[index])); 
    if (diff < minAngle) 
    { 
     minAngle = diff; 
     minIndex = index; 
    } 

    return directions[minIndex]; 
} 

Answer 2

集不准确

if ((Math.Abs(smallest - upDiff) < 0.00001) return Direction.Up; 

Answer 3

你可以做的<double ,Direction>sort the dictionary一本字典，并用正确的枚举返回获得的最小值。

Answer 4

随着你在做，我不认为会有任何错误。但是，如果将来重构或更改代码，则可能会遇到一些问题。

为了安全起见，请执行resharper建议的操作。

Answer 5

这段代码应该会得到你想要的结果。

if ((Math.Abs(direction.x) >= Math.Abs(direction.y)) 
     return direction.x >= 0 ? Direction.Right : Direction.Left; 
    return direction.y >= 0 ? Direction.Up : Direction.Down; 

Answer 6

您可以定义一个包含diff和与之相关的值的类。 然后，您从这些对象中创建一个集合，并通过diff对它们进行排序。之后，您将返回与第一个元素关联的值。

但是，我不会去那里你的情况，代码是明确的，因为它是。如果可能值的数量大得多（或者事先不知道），那么我才会去寻找更通用的解决方案。

Answer 7

我通常使用switch而不是if-else（至少3个子句） - 有点简洁和快速。

switch(smallest) { 
    case upDiff: return Direction.Up; 
    case downDiff: return Direction.Down; 
    case leftDiff: return Direction.Left; 
    default: return Direction.Right; 
} 

Answer 8

你能写一些if语句吗？

if (upDiff < leftDiff && upDiff < downDiff && upDiff < rightDiff) return Direction.Up; 
if (leftDiff < upDiff && leftDiff < downDiff && leftDiff < rightDiff) return Direction.Left; 
if (rightDiff < leftDiff && rightDiff < upDiff && rightDiff < downDiff) return Direction.Right; 
return Direction.Down; 

也许它可以进一步清理，但这似乎很简单。

Answer 9

static Direction VectorToDirection(Vector2 direction) 
{ 
    var mappings = new[] 
    { 
     new { Direction = Direction.Up, Axis = -Vector2.UnitY }, 
     new { Direction = Direction.Down, Axis = Vector2.UnitY }, 
     new { Direction = Direction.Left, Axis = -Vector2.UnitX }, 
     new { Direction = Direction.Right, Axis = Vector2.UnitX } 
    }; 
    return mappings.OrderBy(m => Math.Acos(Vector2.Dot(direction, m.Axis))).Select(m => m.Direction).First(); 
} 

LINQ的方式。这没有经过测试，但你应该得到它。