c# - 为什么这个 Linq 不工作

Question

我有一个相当丑陋的对象(是的，我需要那里的元组 :)):

var roomToIndex = new Dictionary<RoomNode, Tuple<Int32, Dictionary<GrowDirections, Int32>>>();

我像这样初始化这个字典:

for (var i = 0; i < roomsAdjacent.Count(); i++) {
    roomToIndex.Add(roomsAdjacent.ElementAt(i), new Tuple<Int32, Dictionary<GrowDirections, Int32>>(i, new Dictionary<GrowDirections, Int32>()));
    roomToIndex.ElementAt(i).Value.Item2.Add(GrowDirections.Top, 0);
    roomToIndex.ElementAt(i).Value.Item2.Add(GrowDirections.Right, 0);
    roomToIndex.ElementAt(i).Value.Item2.Add(GrowDirections.Bottom, 0);
    roomToIndex.ElementAt(i).Value.Item2.Add(GrowDirections.Left, 0);
}

其中 roomsAdjacent 是 RoomNodes 的列表，GrowDirections 是 [Flags] 可枚举的。
在这些初始化步骤之后，我增加了“内部”字典的整数值，最后我想获得具有最大值的 GrowDirection 和 RoomNode。
我尝试通过以下方式(现在)做到这一点:

///Use the Counts to determine optimal FillDirection
var rDC = roomToIndex
    ///Select the RoomNode and the GrowDirection (incl. Count) with the highest Count for each Room
    .Select(r => new { RoomNode = r.Key, Dict = r.Value.Item2.OrderByDescending(dirCount => dirCount.Value).ToList()[0] })
    ///Order those RoomNodes and GrowDirections descending
    ///Take the first Element and used the Dict's Key (GrowthDirection)
    .OrderByDescending(rGI => rGI.Dict.Value).ToList();
var rDC0 = rDC[0];
if (rDC0.Dict.Key == GrowDirections.Top || rDC0.Dict.Key == GrowDirections.Bottom)
    fillDirection = GrowDirections.Top | GrowDirections.Bottom;
else
    fillDirection = GrowDirections.Right | GrowDirections.Left;
foreach (var rTI in roomToIndex.Where(rTI => rTI.Key != rDC0.RoomNode))
    roomCellCount[rTI.Value.Item1] = 0;

rDC 的类型为 { RoomNode, Dictionary } 并且我没有遇到任何问题。但是当我调试并进入下一行时:

var rDC0 = rDC[0];

调试器跳过该行，直接转到“if 语句”并抛出错误，告诉我我遇到了 NullReferenceException？？!!
当我查看“rDC 对象”中的值时，没有空值。

它可以是什么？感谢您的任何提示/帮助:)

Answer 1

正在检查您的代码 rDC 的类型是List<KeyValuePair<RoomNode, something very complicated> .重要的不是非常复杂的事情，而是KeyValuePair<TKey, TValue>是值类型 ( struct )。这意味着 List<KeyValuePair<TKey, TValue>>不能有 null 的元素.这意味着 rDC0不能是 null .这基本上也是您告诉我们的。

但是，如果你得到一个 NullReferenceException正如你所描述的，它必须是rDC0.Dict那是 null .然而，Dict属性不能是 null因为它已经非常清楚地被初始化为一个新的 Dictionary<GrowDirections, Int32>通过您的初始化代码。

因此，您在问题中提供的代码不应表现出您所描述的行为。您的代码有些不同，或者您获得的行为与您描述的不完全相同。您提到的调试器问题可能是调试发布版本或符号与可执行代码不同步的结果。

我建议您尝试以下一种或多种方法来解决您的问题:

• 重建解决方案以确保调试器在您调试时显示正确的源代码
• 切换到调试版本以关闭会使调试困惑的优化
• 将您的数据和代码分成更小的部分，以摆脱您拥有的复杂且难以理解的代码

最后一个建议是什么将解决(或已经解决)您的问题。让我给你一些建议:

而不是使用 Dictionary<GrowDiretions, Int32>您也许可以创建一个具有四个属性的类型，这有望使您的代码在做什么更清楚:

class GrowCounts {
  public Int32 TopCount { get; set; }
  public Int32 RightCount { get; set; }
  public Int32 BottomCount { get; set; }
  public Int32 LeftCount { get; set; }
  public GrowDirections MaxGrowDirection {
    get { // Return GrowDirections.Top if TopCount has the highest count etc. }
  }
}

而不是使用 new Tuple<T1, T2>使用 Tuple.Create让编译器推断元组的类型。

你真的需要Tuple吗？第一个元素是索引？您的一些代码将 for 循环与 Count 结合使用和 ElementAt以这种方式访问​​集合需要一个索引。但是，也许您可​​以将这些循环转换为 foreach循环，在此过程中您会发现不需要索引。如果可能的话，你可以摆脱 Tuple .