Question

I am developing a small program, and to generate the coordenates I am using this code:

public double[] GenerateNewCircle(int maxSize, int numberOfBalls, double centerx, double centery)
        {

            double[] position = null;

            double angle = 360 / 6;
            double angleRad = DegreeToRadian(incrementAngle);
            position = CalculatePosition(radiusX, radiusY, centerx, centery, angleRad);

            return position;
        }

public double[] CalculatePosition(double radiusX, double radiusY, double centerX, double centerY, double angle)
        {
            double[] position = new double[2];

            position[0] = Math.Cos(angle) * (radiusX + centerX)*1.5;
            position[1] = Math.Sin(angle)  * (radiusY + centerY)*1.5;

            return position;
        }

To check if not overlapping or if is in the area I used this

bool circleIsAllowed(List<Circle> circles, Circle newCircle)
        {
            if (newCircle.x < 10 || newCircle.x > pictureBox1.Width)
                return false;
            if (newCircle.y < 9 || newCircle.y > pictureBox1.Height)
                return false;

            foreach (Circle it in circles)
            {
                double aX = Math.Pow(it.x - newCircle.x, 2);
                double aY = Math.Pow(it.y - newCircle.y, 2);
                double Dif = Math.Abs(aX - aY);
                double ra1 = it.r / 2;
                double ra2 = it.r / 2;

                double raDif = Math.Pow(ra1 + ra2, 2);
                if ((raDif + 1) > Dif) return false;             
            }
            return true; // no existing circle overlaps
        }

I have two problems right now

1. It blocks because is also trying to generate a circle around it, when is a space near.

2. How can I increase the distance between the circles? They are too close.

This circles is to draw around a circle (around it)

This is the image:

Answer 1

There is mistake in second code block. Try this:

double aX = Math.Pow(it.x - newCircle.x, 2);
double aY = Math.Pow(it.y - newCircle.y, 2);
double distance = Math.Pow(aX + aY, 0.5); // distance between circles centers

double ra1 = it.r / 2;
double ra2 = it.r / 2;
// is it.r diameter? if so, then ok, else div by 2 is useless

double circlesDistance = 10; // distance between circles (not centers)

if (distance < ra1 + ra2 + circlesDistance)
{
    return false;
}