Click here to Skip to main content
Rate this: bad
good
Please Sign up or sign in to vote.
Hello.
 
So I am currently working on a project and now I feel the need to try and speed up the execution. I have a method with a foreach loop in it that I wish to parallelize as efficiently as possible.
 
This is how it looks now:
public void Simulate()
{
    List<Coordinate> Locations = retrieveLocations();
 
    int min = int.MaxValue;
    int bestX = -1, bestY = -1;
 
    foreach (Coordinate currPos in Locations)
    {
        string[,] tempMap = new string[globalMap.GetLength(0), globalMap.GetLength(1)];
        matrixCopy(globalMap, tempMap);
 
        int result = processData(currPos, tempMap);
 
        if (result < min)
        {
            min = result;
            bestX = currPos.X;
            bestY = currPos.Y;
        }
    }
 
    MessageBox.Show(bestX + " " + bestY + " " + min);
}
 
I would like to change the foreach loop to use Parallel.ForEach or perhaps use PLINQ. My concerns are how I can find the minimal value with as little locking as possible. Also I want to preserve some other values than just the minimum.
 
Another issue is the globalMap, is it good to copy it like I do now? All threads can't work on the original at the same time because they change the data. But perhaps I should do the copy in my processData()-method? Any thoughts?
Posted 10-Oct-11 18:36pm

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

  Print Answers RSS
0 OriginalGriff 304
1 Sergey Alexandrovich Kryukov 295
2 Shweta N Mishra 216
3 Maciej Los 210
4 PIEBALDconsult 184
0 OriginalGriff 7,630
1 Sergey Alexandrovich Kryukov 7,022
2 DamithSL 5,586
3 Manas Bhardwaj 4,946
4 Maciej Los 4,525


Advertise | Privacy | Mobile
Web03 | 2.8.1411023.1 | Last Updated 10 Oct 2011
Copyright © CodeProject, 1999-2014
All Rights Reserved. Terms of Service
Layout: fixed | fluid

CodeProject, 503-250 Ferrand Drive Toronto Ontario, M3C 3G8 Canada +1 416-849-8900 x 100