неправильно работает программа

[BLACK_RAIN]

Здравствуйте.
Нужно в цикле заполнить окно кругами так, чтобы они не накладывались друг на друга.

написал код, но круги всё-равно накладываются.

struct TBall{
    TVector Position;
    TVector Motion;
    double Radius;
    COLORREF Color;
};


struct TVector{
    double X;
    double Y;
    double Z;
};


TVector Vect(double X, double Y, double Z){
    TVector V1;
    V1.X = X;
    V1.Y = Y;
    V1.Z = Z;
    return V1;
}


double VectorsDistance(TVector V1, TVector V2){
//    return sqrt(pow(V1.X - V2.X, (double)2) + pow(V1.Y - V2.Y, (double)2) + pow(V1.Z - V2.Z, (double)2));
    return sqrt(pow(V1.X - V2.X, (double)2) + pow(V1.Y - V2.Y, (double)2));
}

VOID SetBalls(int nMaxCount){
    nMaxCount = 2000;
    if (nMaxCount > 0){
        if (Balls!=NULL)
            free(Balls);
        Balls = (TBall*)malloc(sizeof(TBall));
        BallsCount = 1;
        memset(&Balls[0], 0, sizeof(TBall));

        Balls[0].Radius = 50;
        Balls[0].Position = Vect(50,50,0);
        Balls[0].Color = RGB(rand() % 255, rand() % 255, rand() % 255);

            for (int i = 0; i < nMaxCount; i++){
                int tmpRadius = 30;
                BOOL valid = true;
                TVector tmpPos;
                for (int j = 0; j < BallsCount; j++){
                     tmpPos = Vect(rand() % 300, rand() % 300, 0);
                     double d = VectorsDistance(tmpPos, Balls[j].Position);
                     double r = tmpRadius + Balls[j].Radius;
                     valid = (valid) && (d > r);
                     if (!valid)
                        break;
                }
                if (valid){
                    Balls = (TBall*)realloc(Balls, sizeof(TBall) * (BallsCount + 1));
                    memset(&Balls[BallsCount], 0, sizeof(TBall));
                    Balls[BallsCount].Radius = tmpRadius;
                    Balls[BallsCount].Position = tmpPos;
                    BallsCount += 1;

                }
            }
    }
    return;
}

не понимаю, почему работает неправильно.

[Аватар]

Дык tmpPos позиция круга, который не пересекается с последним из Balls. А со всеми остальными как попало )

[BLACK_RAIN]

Сообщение от Аватар

Дык tmpPos позиция круга, который не пересекается с последним из Balls. А со всеми остальными как попало )

Всмысле как это? Там идет цикл проверки позиции всех кругов. Если tmpPos наложился хотябы на один из них - цикл прекращается и новый круг не создается. Затем рандомится новый tmpPos и т.д.
Разве нет? Где ошибка тогда?
p.s. Сейчас еще потестил. Такое ощущение, что проблема в функции rand(). Если ее не использовать, то всё нормально.

[Аватар]

А чего tmpPos каждый раз в цикле новый? В том, вложенном. Один же нужен для всех проверок

Если ее не использовать, то всё нормально.

Дык и все нормально, что координаты не меняются )

[Black Fregat]

Сообщение от BLACK_RAIN

Всмысле как это? Там идет цикл проверки позиции всех кругов.

И на каждом шаге этого цикла рандомится новый tmpPos

[BLACK_RAIN]

Спасибо. Осознал и исправил ошибку