Question

我正在尝试将普通函数转换为并行函数。

该函数检查给定的坐标是否足够接近以考虑它们是否重复，在完整的坐标列表中逐一检查。

我需要访问3个局部变量：

要测试的坐标（poi_data）
坐标列表（pois）
一个应用程序变量，指示要被视为重复的距离（Config.duplicatedRange）

一旦检测到重复，该功能应停止检查并返回true。

这是我当前的代码。

class Test
{           
    public static Boolean CheckDuplicated (POIData poi_data, 
                Dictionary<string, POIData> pois)
    {
        foreach (KeyValuePair<string, POIData> item in pois)
        {
            Double distance = CalculateDistance (poi_data.latitude,
                        poi_data.longitude,
                        item.Value.latitude,
                        item.Value.longitude);

            if (distance < Config.duplicatedRange) {
                return true;
            }
        }
        return false;
    }
}

public class POIData
{
    public Double longitude { set; get; }

    public Double latitude { set; get; }
}

这是我尝试并行完成的。

public static Boolean CheckDuplicated (POIData poi_data,
        Dictionary<string, POIData> pois)
{
    Boolean result = false;
    CancellationTokenSource cts = new CancellationTokenSource (); 
    ParallelOptions options = new ParallelOptions 
                { CancellationToken = cts.Token }; 

    Parallel.ForEach (pois,
            options,
            () => false,
            (item, loopState, localCount) => {
                cts.Token.ThrowIfCancellationRequested ();
                Double distance = CalculateDistance (poi_data.latitude,
                        poi_data.longitude,
                        item.Value.latitude,
                        item.Value.longitude);
                if (distance < Config.duplicatedRange) {
                    cts.Cancel ();
                    return true;
                }
                return false;
            },
            (tempResult) => {
            if (tempResult == true) {
                Interlocked.Exchange (ref result, tempResult);
            }
        });

    return result;
}

我收到错误

错误CS0452：类型bool必须是引用类型才能在泛型类型或方法T
中将其用作类型参数System.Threading.Interlocked.Exchange<T>(ref T, T)

我做错了什么？这是最好的方法吗？还是有更简单的方法？

编辑：

感谢您的帮助，这里有更新和正常工作的功能：）

public static Boolean CheckDuplicated (POIData poi_data, 
                                       Dictionary<string, POIData> pois)
{
    int result = 0;
    Parallel.ForEach (pois,
        () => 0,
        (item, loopState, tempResult) => {
            Double distance = CalculateDistance (poi_data.latitude,
                                  poi_data.longitude,
                                  item.Value.latitude,
                                  item.Value.longitude);

            if (distance < Config.duplicatedRange) {
                loopState.Stop ();
                return 1;
            }
            return 0;
        },
        (tempResult) => {
            if (tempResult == 1) {
                Interlocked.Exchange (ref result, tempResult);
            }
        });
    return Convert.ToBoolean (result);
}

Answer 1

基本上：Interlocked.Exchange()方法没有超载会接受bool作为参数类型。显式类型的方法重载都不包含bool，因此编译器尝试匹配泛型方法重载，但由于类型参数的泛型约束而失败。

最简单的替代方法是，不要将bool用作result变量的类型。将其设为int，将其初始化为0，如果找到您要查找的项目，则将其设置为1。

（我没有仔细查看代码中的整体实现。乍一看似乎很好，而且您的问题似乎是关于错误消息的，所以我只讨论上面的内容。）

Parallel.ForEach with Dictionary和C＃中的多个局部变量

1 个答案: