Структуры, С++

[Eddi_Brown]

Задание: написать программу, в которой будут объявлены структуры Points и Ellipse, создан динамический массив объектов Ellipse и нарисованы все объекты, занесенные в массив.
Вот код:

struct Points
{
  int x,y;
};
struct Elipse
{
  Points p1,p2;
  TColor color;
  void draw(TCanvas *cn);
};
void Elipse::draw(TCanvas *cn)
{
cn->Pen->Color=color;
cn->Pen->Width=3;
cn->Brush->Style=bsSolid;
cn->Brush->Color=random(0xffffff);
cn->Ellipse(p1.x,p1.y,p2.x,p2.y);
}
void DrawAll(int m,struct Elipse **el,TCanvas *cn)
{
int i;
for(i=0;i<m;i++)
{
 el[i]->draw(cn);
 el[i]->p1.x=random(600);
 el[i]->p1.y=random(480);
 el[i]->p2.x=random(600);
 el[i]->p2.y=random(480);
 }
}
void __fastcall TForm1::Button1Click(TObject *Sender)
{
Elipse **ell;
int n=StrToInt(Edit1->Text);
int i;
ell=new Elipse *[n];
for (i=0;i<n;i++)
ell[i]=new Elipse;
DrawAll(n,ell,Canvas);
for (i=0;i<n;i++)
delete ell[i];
delete []ell;
}

Точки должны быть произвольными, а на деле получается вот что:

То есть, эллипсы вроде есть, но очень вытянутые. И еще, если я ввел, например, 10, должно рисоваться 10 эллипсов, но получается как-то не так. Что не так?

[p51x]

Вас не смущает, что вы сначала рисуете, а только потом задаете значение точек?

[Eddi_Brown]

Как можно быть таким идиотом? Спасибо, вы бы не сказали, я бы и не заметил.