probleme mit einer for schleife

[MikeSPK]

Hmm mach doch unter Table->Next() ein
PlhIndex = "";

[lisa]

habe ich gemacht im ersten datensatz sind die platzhalter vorhanden ab dem zweiten nicht mehr

[basedow]

Also ich würde sagen, da is einiges im argen:

AnsiString Artikel; 
AnsiString PlhIndex; 
int iD1=0; 
int WertIndex=iD1; 
int tIndex=Artikel.Length(); // ist hier nicht gut -> Länge immer 0
int rIndex=18-tIndex;        // dito -> ist immer 18

for(int i=0; i<Table1->RecordCount; i++)  // geht meiner Meinung nach besser mit ner while Schleife
{ 
    // hier solltest du iDl auf 0 setzten
    for(WertIndex;iD1<rIndex; ++iD1) 
    { 
        PlhIndex += "#"; 
        Refresh(); // Wozu ???
    } 
    Artikel=DBEdit2->Text;  // das gehört nach oben vor die innere for Schleife
    // hier müsste dann PlhIndex wieder auf leeren String gesetzt werden
    Memo1->Lines->Add(PlhIndex+Artikel); 
    Table1->Next(); 
}

Ich kenn jetz zwar diese Table nich aber ich versuchs mal:

AnsiString Artikel = "";
int tableIndex = 0;
// Vorher noch die Table1 auf das erste Element setzten - wie gesagt ich kenn die nicht
while(tableIndex < Table1->RecordCount)
{
    Artikel = DBEdit2->Text;
    while(Artikel.length() < 18)
    {
        Artikel += "#";
    }
    Memo1->Lines->Add(Artikel);
}

Probier mal aus

basedow

[lisa]

es klappt einfach nicht. [basedow] das mit der while Schleife geht auch nicht. Die schleife wird nicht gestoppt. ich glaube ich sehe den wald vor lauter bäumen nicht mehr

[Alexander Kempf]

basedow schrieb:

Also ich würde sagen, da is einiges im argen:

...aber ob eine unendliche Schleife soviel besser ist?

basedow schrieb:

AnsiString Artikel = "";
int tableIndex = 0;
// Vorher noch die Table1 auf das erste Element setzten - wie gesagt ich kenn die nicht
while(tableIndex < Table1->RecordCount)
{
    Artikel = DBEdit2->Text;
    while(Artikel.length() < 18)
    {
        Artikel += "#";
    }
    Memo1->Lines->Add(Artikel);
}

Probier mal aus

... vielleicht besser nicht.
Zumindest nicht, ohne tableIndex hochzuzählen. Und ich vermute irgendwo
sollte noch ein Zugriff auf Table1 erfolgen (in Abhängigkeit von tableIndex),
um durch die Datensätze zu navigieren.

Gruß,

Alexander

[1ntrud0r]

wenn du überprüfst ob 0<14 ist und das 1000mal ist es immer false wenn du es nicht inkrementierst oder ?

da war wohl jmd schneller

[PuppetMaster2k]

probier mal die Version, die kommt deiner ziemlich nahe

AnsiString Artikel;  
AnsiString PlhIndex = "";  
int rIndex = 0;        

for(int i=0; i<Table1->RecordCount; i++)  
{  
    //Hier ist die Zuweisung sinnvoller
    Artikel = DBEdit2->Text;  
    rIndex = 18 - Artikel.Length();

    //wie von basedow schon beschrieben, den Zähler hier auf 0 setzten.
    for(int iD1 = 0; iD1<rIndex; ++iD1)  
    {  
        PlhIndex += "#";  
        Refresh();
    }  

    Memo1->Lines->Add(PlhIndex+Artikel);  
    //Die Zeichen wieder löschen
    PlhIndex = "";
    Table1->Next();  
}

@basedow

guck dir deinen Code mal genauer an, du hast ne endlosschleife :p

Edit: Kommt davon wenn man(n) trödelt

[Braunstein]

naja, wenn du tableIndex innerhalb der Whileschleife nicht hochzählst kann die Schleife ja nicht stoppen.

[basedow]

Hoppla
Stimmt - da muss natürlich noch was rein.

AnsiString Artikel = ""; 
int tableIndex = 0; 
// Vorher noch die Table1 auf das erste Element setzten - wie gesagt ich kenn die nicht 
while(tableIndex < Table1->RecordCount) 
{ 
    Artikel = DBEdit2->Text; 
    while(Artikel.length() < 18) 
    { 
        Artikel += "#"; 
    } 
    Memo1->Lines->Add(Artikel); 
    tableIndex++;  // NEU
}

Jetzt sollte sie auch abbrechen.

Und wie bereits gesagt, du musst vor der äusseren while-Schleife dafür sorgen, dass deine Table1 auf den ersten Datensatz zeigt. Ich kenn nur die Table-Komponente nicht. Da musst du selber noch mal schauen.

@all: Man wird ja wohl noch ne blöde Abbruchbedingung vergessen dürfen

basedow

[lisa]

Danke PuppetMaster2k es klappt. ich wollt schon auf den balkon steigen und aus dem 3. stock springen.
super es klappt!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

[Alexander Kempf]

Habe mir mal erlaubt die while-Schleife etwas abzuändern.
Warum nicht bewährte Eigenschaften und Methoden der TTable-Komponente
verwenden.

AnsiString Artikel = ""; 
// int tableIndex = 0; wird nicht mehr benötigt
// Vorher noch die Table1 auf das erste Element setzten - wie gesagt ich kenn die nicht 
while(!Table1->Eof) 
{ 
    Artikel = DBEdit2->Text; 
    while(Artikel.length() < 18) 
    { 
        Artikel += "#"; 
    } 
    Memo1->Lines->Add(Artikel); 
    // tableIndex++;  // NEU ... und schon wieder alt
    Table1->Next();
}

So könnte es vielleicht auch gehen?

Gruß,

Alexander

[lisa]

ich weiss garnicht wie ich euch danken soll

[lisa]

alexander ich werde es ausprobieren. danke

[Bj]

@ lisa
nein, nicht von dem balkon springen
denn ich glaube es kommen da noch schierigere aufgaben auf dich zu, an denen du noch mehr verzweifelst, falls du weiter prgrammierst...

also bitte, kein selbstmord.

Bj

[basedow]

@Alexander: DIEEEEB!!!

basedow

[lisa]

nein das würde ich nicht machen. ich möchte mich doch weiter mit solch netten menschen wie euch unterhalten. noch mal danke ihr habt mir sehr geholfen

grus lisa

p.s selbstmord ist keine lösung!! probleme muss man sich stellen und nicht davor weglaufen

[Alexander Kempf]

basedow schrieb:

@Alexander: DIEEEEB!!!

basedow

Das war kein Diebstahl, das war Tuning

Gruß,

Alexander

[Jansen]

Offtopic-Diskussion abgetrennt.