[c++] Vereenvoudigen code

[Emiel1984]

Op dit moment heb ik maar even de volgende code;

code:

        iRandom=(rand() %25) +1;
        sprintf(link, "E:\\Programma\\Afbeeldingen\\%d.bmp", iRandom);
        afbeelding= link;
        Image2->Picture->LoadFromFile(afbeelding);

        iRandom=(rand() %25) +1;
        sprintf(link, "E:\\Programma\\Afbeeldingen\\%d.bmp", iRandom);
        afbeelding= link;
        Image3->Picture->LoadFromFile(afbeelding);

       etc...

Ik weet dat het mogenlijk zou moeten zijn om deze code in een do while loopje te gooien;

code:

        do 
        iRandom=(rand() %25) +1;
        sprintf(link, "E:\\Programma\\Afbeeldingen\\%d.bmp", iRandom);
        afbeelding= link;
        Image(teller)->Picture->LoadFromFile(afbeelding);
        Teller++;
        while(Teller < 10)


       etc...

Mijn enige probleem is dat ik het niet voor elkaar krijg om de regel ;

Image(teller)->Picture->LoadFromFile(afbeelding);

kan realiseren. Ik heb het geprobeert met

code:

sprintf(LinkToPicture,"Image%d", Teller);
LinkToPicture->Picture->LoadFromFile(afbeelding);

Helaas werkte dit werkte dus niet. Heeft iemand een idee hoe ik dit wel kan realiseren. Bvd.

[ Voor 5% gewijzigd door Emiel1984 op 24-12-2005 17:27 ]

[Daspeed]

Ik weet niet precies wat je doet in je code, maar 't lijkt erop dat je nog niet van een array of vector (oid) gehoord hebt. misschien moet je daar eens naar op zoek

[ Voor 5% gewijzigd door Daspeed op 24-12-2005 17:34 ]

[thrax]

Maak eens een array van TImage objecten. Die kun je dan in een loop laden, processen, whateveren

code:

void main(void)
{
   TImage *image[10];
   int         counter;
   char      link[128];
   int        iRandom;
     
   for (counter = 0; counter < 10; counter++)
   {
       image[counter] = new TImage();

        iRandom=(rand() %25) +1;
        sprintf(link, "E:\\Programma\\Afbeeldingen\\%d.bmp", iRandom);

        image[counter]->Picture->LoadFromFile(link);
    }

    // use your TImages here.....

    for (counter = 0; counter < 10; counter++)
    {
       delete image[counter];
       image[counter] = NULL;
    }
}

Wat had je eigenlijk in gedachten met het volgende stukje code?

code:

sprintf(LinkToPicture,"Image%d", Teller);
LinkToPicture->Picture->LoadFromFile(afbeelding);

[ Voor 29% gewijzigd door thrax op 24-12-2005 17:41 ]

[.oisyn]

Gebruikelijke beginnersfout. De namen van variabelen in code zijn alleen een handigheid voor de programmeur (het zou wat zijn als je met pure geheugenadressen moest gaan werken ), na het compileren blijft er niets meer van over en dus kun je ook niet -terwijl je programma loopt- de juiste variabele vinden bij een string.

Daarvoor zijn hele andere methoden beschikbaar. Zo kun je een array maken en de elementen van die array in een loopje aanspreken (dus niet Image1, Image2, ...; maar Image[0], Image[1], ...). Een andere manier is om gebruik te maken van een geassocieerde structuur, zoals bijvoorbeeld een std::map<std::string, TImage *>, zodat je wél met behulp van een string de juiste TImage kunt vinden.

[Emiel1984]

Ik wil graag afbeeldingen random neerzetten uiteindelijk.

Image(teller)->Left= iRandomLeft;
Image(teller)->Top= iRandomTop;

Ik ben nu nog even aan het kijken of ik een dergelijke functie in een loopje kan gooien. In dit loopje wil in dan gaan controleren of de plek bezet is (das niet zo moeilijk).

Ik wil dus eigenlijk gewoon weten of het min of meer mogenlijk is een 'link' een variabele te gooien.


I@DrClearbottom :Ik heb wel is gehoort van een array, hoe bedoel je dan oplossen met een array ?

[Verwijderd]

Hij bedoelt dat je een array maakt van instanties van het object TImage of wat je Image3 in je voorbeeld dan ook is:

C:

TImage Image[10];
for (unsigned short i=0; i<10; i++) {
  iRandom=(rand() %25) +1;
  sprintf(link, "E:\\Programma\\Afbeeldingen\\%d.bmp", iRandom);
  afbeelding= link;
  Image[i]->Picture->LoadFromFile(afbeelding);
}

[Emiel1984]

Verwijderd schreef op zaterdag 24 december 2005 @ 20:10:
Hij bedoelt dat je een array maakt van instanties van het object TImage of wat je Image3 in je voorbeeld dan ook is:

Best nog even bezig geweest om uit te zoeken hoe jullie het nou bedoelde maar het is gelukt, thks. Ik weet dat je mensen niet telkens "groetjes bedankjes enzo" mag doen maar gezien mijn tijd(nood) ben ik jullie zeer dankbaar .