TImer wechselt bilder nicht aus

[Mr.XYZ]

Hi,
dieser Timer läuft eigentlich normal nur wechselt er die Bilder nicht aus und das ist blöd.

Delphi-Quellcode:

procedure TForm2.Time_GalierTimer(Sender: TObject);
begin
if(i < 6) then
begin
 Memo1.Lines[3]:='Titmer_galier on';
 galier.GetBitmap(i,Char.Picture.Bitmap);
 Memo1.Lines[4]:='galier_pic'+IntToStr(i);
 i:=i+1;
 end
 else
 Memo1.Lines[5]:='Titmer_galier off';
 i:=0;
 Time_Galier.Enabled:=false;
 end;
end;

end.

es heißt aber auch immer das ein END zu viel wäre aber warum?

[himitsu]

jetzt siehst du mal, wozu eine ordentliche Einrückung gut ist,
denn dann kommt bei dem Code dieses raus

Delphi-Quellcode:

procedure TForm2.Time_GalierTimer(Sender: TObject);
begin
  if(i < 6) then
  begin
    Memo1.Lines[3]:='Titmer_galier on';
    galier.GetBitmap(i,Char.Picture.Bitmap);
    Memo1.Lines[4]:='galier_pic'+IntToStr(i);
    i:=i+1;
  end
  else
    Memo1.Lines[5]:='Titmer_galier off';
  i:=0;
  Time_Galier.Enabled:=false;
end;
end; << END zuviel

vermmutlich meinst du es aber so

Delphi-Quellcode:

procedure TForm2.Time_GalierTimer(Sender: TObject);
begin
  if i < 6 then
  begin
    Memo1.Lines[3] := 'Titmer_galier on';
    galier.GetBitmap(i, Char.Picture.Bitmap);
    Memo1.Lines[4] := 'galier_pic' + IntToStr(i);
    i := i + 1;
  end
  else
  begin  // <<<<<<<<<<<<<<<<<<<<<<<<<<
    Memo1.Lines[5] := 'Titmer_galier off';
    i := 0;
    Time_Galier.Enabled := false;
  end;
end;

[Noedel]

guck dir doch mal den code an. Da siehst du, dass die Anzahl von "Begin" micht gleich der Anzahl der "End" ist.

Delphi-Quellcode:

procedure TForm2.Time_GalierTimer(Sender: TObject);
begin
  if(i < 6) then
  begin
    Memo1.Lines[3]:='Titmer_galier on';
    galier.GetBitmap(i,Char.Picture.Bitmap);
    Memo1.Lines[4]:='galier_pic'+IntToStr(i);
    i:=i+1;
  end
  else
  Begin //fehlt
    Memo1.Lines[5]:='Titmer_galier off';
    i:=0;
    Time_Galier.Enabled:=false;
  end;  //<- Das End ist zu viel, oder aber oben fehlt ein Begin
end;

[Mr.XYZ]

ja ich sehe es aber warum klappt das mit den Bildern nicht?

[himitsu]

Zitat von Mr.XYZ:

ja ich sehe es aber warum klappt das mit den Bildern nicht?

Dann sieh nochmal genauer hin und prüfe mal nach, was dein Code ohne dieses BEGIN macht,
denn dein Code macht quasi dieses hier:

Code:

[b]procedure[/b] TForm2.Time_GalierTimer(Sender: TObject);
[b]begin[/b]
  [b]if[/b] (i < 6) [b]then
  begin[/b]
    Memo1.Lines[3]:='Titmer_galier on';
    galier.GetBitmap(i,Char.Picture.Bitmap);
    Memo1.Lines[4]:='galier_pic'+IntToStr(i);
    i:=i+1;
  [b]end
  else[/b]
  [color=silver][b]begin[/b][/color]
    Memo1.Lines[5]:='Titmer_galier off';
  [color=silver][b]end[/b];[/color]
  i:=0;
  Time_Galier.Enabled:=false;
[b]end[/b];
[b]end[/b]; [color=green]// END zuviel[/color]

Tipp: schau mal, was die letzen 2 Befehle machen

[Mr.XYZ]

ich habe das mit dem begin schon geändert aber es geht immer noch nicht. Er wechselt die Bilder nicht!!

[Blackheart]

Delphi-Quellcode:

procedure TForm2.Time_GalierTimer(Sender: TObject);
begin
  if i < 6 then // Die Klammern brauchst du hier nicht unbedingt
  begin
    Char.Picture:=nil; //Altes Bild löschen
    Memo1.Lines[3]:='Titmer_galier on';
    galier.GetBitmap(i,Char.Picture.Bitmap);
    Memo1.Lines[4]:='galier_pic'+IntToStr(i);
    Inc(i); // Ist das selbe wie i:=i+1; nur kürzer
  end
  else
  begin
    Memo1.Lines[5]:='Titmer_galier off';
    i:=0;
    Time_Galier.Enabled:=false;
  end;
end;

[Mr.XYZ]

Ähmm.. jetzt verschwindet einfach das vorherige Bild es ändert sich aber sonst nichts.

[Jens Hartmann]

Hallo,

ich bin mir nicht ganz sicher, aber fehlt da nicht so eine Art...

Delphi-Quellcode:

galier.Refresh;

//oder

galier.Repaint;

Gruß Jens

[Mr.XYZ]

ne das fehlt nicht aber ich hatte noch ne variable vertauscht.