根據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的方式。這沒有經過測試，但你應該得到它。