C# foreach w pętli foreach

Question

Nie mam zbyt dużego doświadczenia z C#, więc jeśli ktoś mógłby wskazać mi właściwy kierunek, byłbym bardzo wdzięczny. Mam pętlę foreach, która odwołuje się do zmiennej obiektu. Chciałbym utworzyć kolejną pętlę foreach wewnątrz głównej, która porównuje (lub wykonuje akcje) z bieżącą zmienną z resztą zmiennych w tablicy obiektu. Mam następujący kod:

// Integrate forces for each body. 
    foreach (RigidBodyBase body in doc.Bodies) 
    { 
     // Don't move background-anchored bodies. 
     if (body.anchored) continue; 

     // This is where we will add Each Body's gravitational force 
     // to the total force exerted on the object. 

     // For each other body, get it's point and it's mass. 

      // Find the gravitational force exterted between target body and looped body. 
       // Find distance between bodies. 
        // vector addition 
       // Force = G*mass1*mass2/distance^2 
      // Find vector of that force. 
      // Add Force to TotalGravityForce 
     // loop until there are no more bodies. 
     // Add TotalGravityForce to body.totalForce 

    } 

Answer 1

Każdorazowe wykonanie foreach (nawet podczas ich gniazdowania) wewnętrzna Enumerator powinien „nowy” w górę nowego iteratora dla ciebie, tam nie powinno to być żadnym problemem. Problemy pojawiają się podczas dodawania lub usuwania elementów z kolekcji, gdy wciąż trwa iterowanie ...

Pamiętaj, aby w wewnętrznym foreach sprawdzić, czy nie jesteś na tym samym przedmiocie co zewnętrzna każdy jest na

foreach(RigidBodyBase body in doc.Bodies) 
    foreach (RigidBodyBase otherBody in doc.Bodies) 
     if (!otherBody.Anchored && otherBody != body) // or otherBody.Id != body.Id -- whatever is required... 
       // then do the work here 

przy okazji, to najlepsze miejsce, aby umieścić ten kod byłby w nieruchomości GravityForce klasy RigidBodyBase, to można po prostu napisać:

foreach (RigidBodyBase body in doc.Bodies) 
     body.TotalForce += body.GravityForce; 

chociaż w zależności od tego, co czy robienie tutaj (przenoszenie wszystkich obiektów?) to może być e więcej możliwości refaktoryzacji ... Rozważałbym także posiadanie oddzielnej nieruchomości dla "innych" sił i czy TotalForce Property zsumuje siły grawitacji i "inne" siły?

Answer 2

w tym przypadku, to chyba lepiej użyć zwykłej pętli for z indeksem do czego elementem której się znajdujesz. Próba iteracji w tej samej kolekcji z własną pętlą foreach spowoduje problemy.

Answer 3

IMHO powinno być możliwe, chociaż naprawdę powinieneś rozważyć sugestię Kibbee. Być może można zoptymalizować w ten sposób zbyt (na przykład, jak ten :)

int l = doc.Bodies.Count; 
for (int i = 0; i < l; i++) 
    for (int j = i + 1; j < l; j++) 
     // Do stuff 

Answer 4

Nie widzę problemu z tym, o ile nie zmienisz doc.Bodies wewnątrz wewnętrznej pętli, ponieważ spowoduje to wysadzenie w powietrze. Ale teoretycznie to by działało. Jeśli chodzi o optymalizację, nie jestem pewien, czy jest najlepszy, ale jest to możliwe.

Answer 5

To jest algorytm O (n^2), ale chyba nie masz wyboru. Co powiesz na umieszczenie większej ilości logiki w innej metodzie. To sprawia, że ​​rzecz po prostu stara się bardziej czytelna.

foreach (RigidBodyBase body in doc.Bodies)   
{     
    Integrateforces(ref body, Bodies); 
} 

... 

public void Integrateforces(RigidBodyBase out body, RigidBodyBase[] Bodies) 
{ 
    //Put your integration logic here 
}