5

正如在这个问题中所描述的，我正在研究一种方法，该方法从一个返回的元素List<FileInfo>在另一个中不存在List<FileInfo>。我已经实现了 Nawfal 的解决方案，如下所示：

public List<FileInfo> SourceNotInDest(List<FileInfo> SourceFiles, List<FileInfo> DestFiles)
{
 var notInDest = SourceFiles.Where(c => !DestFiles.Any(p => p.Name == c.Name)).ToList();
 return notInDest;
}

我的 SourceFiles 数据集是：

u:\folder1\a.txt
u:\folder1\b.txt
u:\folder1\c.txt
u:\folder1\d.txt

目标文件是：

u:\folder2\a.txt
u:\folder2\b.txt
u:\folder2\c.txt

当我单步执行代码并检查列表的值时，这似乎返回了预期的结果。但单元测试失败，代码如下：

public void SourceNotInDestTest()
    {
        //arrange
        FileListComparer flc = new FileListComparer(); //class that has the list compare method
        FolderReader fr = new FolderReader(); //class for getting FileInfo from folder
        List<FileInfo> expectedResult = new List<FileInfo>();
        expectedResult.Add(new FileInfo(@"U:\folder1\d.txt"));
        List<FileInfo> SourceFiles = fr.fileList(@"U:\folder1");  //gets the FileInfo for each file in the folder
        List<FileInfo> DestFiles = fr.fileList(@"U:\folder2");


        //act
        List<FileInfo> result = flc.SourceNotInDest(FTPFiles, LocalFiles);

        //assert
        CollectionAssert.AreEquivalent(result, expectedResult);
    }

即使result和expectedResult具有相同的内容——两个列表都有一个元素具有相同的文件路径和相同的其他属性——测试失败并显示以下消息：

CollectionAssert.AreEquivalent failed. The expected collection contains 1 occurrence(s)
of <U:\folder1\d.txt>. The actual collection contains 0 occurrence(s).

expectedResult但是，确实发生了U:\folder1\d.txt。我在想问题可能是我正在比较两个对象的内存地址而不是这些对象的内容，但我认为这AreEquivalent()是在比较属性。不是这样吗？

编辑：根据关于比较属性而不是地址的建议，我改用了这个 Assert，它允许测试通过：

foreach (FileInfo fi1 in result)
    {
     Assert.IsNotNull(expectedResult.Find(fi2 => fi2.FullName == fi1.FullName));
    }
foreach (FileInfo fi1 in expectedResult)
    {
     Assert.IsNotNull(result.Find(fi2 => fi2.FullName == fi1.FullName));
    }
4

3 回答 3

7

可能是因为FileInfo是引用类型，默认比较器只检查两个元素的地址是否相等。由于FileInfo是密封的，因此您不能从中派生并覆盖相等比较器。在我看来，最好的选择是编写自己的集合比较器方法（因为您不能将IEqualityComparer实例传递给CollectionAssert.AreEquivalent）。

于 2012-12-26T22:48:16.630 回答
4

测试失败，因为您的集合中有不同的对象。如果您有 2 个FileInfo引用同一文件的类实例，并且您调用instanceA.Equals(instanceB)，则结果为false.

如果您可以将代码更改为使用字符串而不是FileInfos，那么它将按您的预期工作。

于 2012-12-26T22:49:48.970 回答
1

在我看来，我可以建议 2 种比你的更好的方法 :)

  1. 选择 2 个文件名集合并比较这些集合：

    CollectionAssert.AreEquivalent(
    result.Select(fi => fi.FullName).ToArray(),
    expectedResult.Select(fi => fi.FullName).ToArray()
);
// ToArray() is added just for better output when test fails.

  2. 使用用户定义的比较器和比较FileInfo列表：

    Assert.That(
    result,
    Is
        .EquivalentTo(expectedResult)
        .Using((Comparison<FileInfo>)((fi1, fi2) => fi1.FullName.CompareTo(fi2.FullName)))
);

在以下情况下，您当前使用两个 foreach 循环的实现不会失败：

result =
    u:\folder1\a.txt
    u:\folder1\a.txt

expectedResult =
    u:\folder1\a.txt

是的，文件列表似乎不是真正的情况，但通常用两个循环替换AreEquivalent()/并不是一个好主意。Is.EquivalentTo()

于 2012-12-29T19:49:38.117 回答