int main()
{
int i;
cout << "Die Quadratzahlen von 0 - 144 sind:\r " << endl;
for(i=0;i<13;i++)
{
cout << QUADRAT;
cout << ",";

}
return 0;

}

das isch mein Programm etwas ganz simples... Es sollte alle Quadratzahlen ausgeben, aber mir hat jemand gesagt es wäre nicht optimal programmiert...
Was ist falsch und wie würde es dann besser aussehen

#define QUADRAT (i*i)

...warum macht man denn sowas?

P.S.: Bitte benutze Code-Tags, damit man den Code besser lesen kann.

Mach eine Funktion, der man einen Wert übergeben kann:

int Quadrat(int Wert)
{
    return Wert*Wert;
}

int main()
{
    // ...
    Quadrat(i);
    // ....
}