Debugger Krise

[s.h.a.r.k]

Ich kenne nur ein Phenomän, wenn Delphi die for-Schleife optimiert und der Debugger nicht darauf achtet. Wobei lediglich die Darstellung falsch ist, intern aber alles korrekt abläuft.

Sehe ich das schon richtig, dass die Zeile unter dem if ausgeführt wird, ob wohl 62 nicht gleich 0 ist?!

[EWeiss]

Zitat von s.h.a.r.k:

Ich kenne nur ein Phenomän, wenn Delphi die for-Schleife optimiert und der Debugger nicht darauf achtet. Wobei lediglich die Darstellung falsch ist, intern aber alles korrekt abläuft.

Sehe ich das schon richtig, dass die Zeile unter dem if ausgeführt wird, ob wohl 62 nicht gleich 0 ist?!

Das ist mein Problem!

Noch zwei pics da kann man sehen das beide IntI und IntA den wert 1 haben.
Wenn du oben dann vergleichst wirst du sehen das er trotzdem die Note löscht obwohl vorhanden.

Was soll ich denn jetzt machen.. grrrr
Optimierung habe ich ausgeschaltet alle DCU's glöscht und komplett neu Kompiliert.
Damit ich die Values verfolgen kann.

EDIT:
Hab es mal mit Packed Record versucht.. bringt auch nix.

Hier ist die Funktion vielleicht gibt es da ein problem denke aber nicht!

Delphi-Quellcode:

procedure TMidiTracker.ImpMidiFile(MidiFile: Widestring);
var
  FileInfo: TMIDIFileInfo;
  IntI: Integer;
  IntA: Integer;

begin
  if not ReadMIDIFile(MidiFile, FileInfo) then
  begin
    MessageBox(Self.Handle, 'Not a valid MIDI file.', 'Confirm', MB_OK);
    exit;
  end;

  for IntI := 1 to FileInfo.Tracks do
  begin
    for IntA := 0 to FileInfo.NoteCounter[IntI] -1 do
    begin
      if FileInfo.MIDINotes[IntI, byte(FileInfo.MIDINotes[IntA])].NoteNo = 0 then
        Grid[IntI - 1, IntA] := -1;

      if FileInfo.MIDINotes[IntI, (byte(FileInfo.MIDINotes[IntA]))].NoteNo > 0 then
      begin
        if FileInfo.MIDINotes[IntI, (byte(FileInfo.MIDINotes[IntA]))].Length = 12 then
          Grid[(IntI - 1), IntA] := (127 - FileInfo.MIDINotes[IntI, (byte(FileInfo.MIDINotes[IntA]))].NoteNo) + basenote;

        if FileInfo.MIDINotes[IntI, byte(FileInfo.MIDINotes[IntA])].Length > 12 then
          Grid[(IntI - 1), IntA] := 1000 + (127 - FileInfo.MIDINotes[IntI, (byte(FileInfo.MIDINotes[IntA]))].NoteNo + basenote);
      end;

      InstGrid[(IntI - 1), IntA] := FileInfo.MIDINotes[IntI, (byte(FileInfo.MIDINotes[IntA]))].Instrument;
    end;
  end;
  DrawGrid(self.Handle);

end;

gruss

[EWeiss]

Ist eh keiner Online...

also hier die records..
Wenn da nichts ist weiss ich auch nicht mehr.

Delphi-Quellcode:

type
  TMIDINote = record
    StartTime: DWORD;    // Start time of note in ticks
    Length: Word;        // Length of note in ticks  (Length / Ticks : 1 => 1/4 note, 2 => 1/2 note...)
    NoteNo: byte;        // Note #
    Instrument: byte;
    Velocity: byte;
  end;

  TMIDIFileInfo = record
    FileSize: Int64;    { File size (bytes) }
    Format: Word;       {0: single-track, 1: multiple tracks, synchronous, 2: multiple tracks, asynchronous}
    Tracks: Word;
    TrackNames: array[1..MaxTracks] of AnsiString;
    TickUnit: Word;     {0: Ticks per quarter note, 1: Ticks per second}
    Ticks: Word;        {Ticks per quarter note or Ticks per second}
    Marker: AnsiString;
    Copyright: AnsiString;
    Channels: byte;
    PlayTime: single;
    NoteCounter: array[1..MaxChannels] of Word;     // The number of note per channel
    DrumChannel: array[1..MaxChannels] of Boolean;  // True if it's a drum channel
    MIDINotes: array[1..MaxChannels] of array of TMIDINote;  // The data of MIDI notes per channel
  end;

Bei IntA = 3 gibt er mir ne note von 60 anstelle von 65..
Und so geht es weiter.
Das kann es doch nicht sein.

gruss

[ULIK]

Was soll denn byte(FileInfo.MIDINotes[IntA]) sein? Du machst doch hier einen Byte-Cast eines Arrays. Bist Du sicher, daß das so in Ordnung ist?

Grüße,
Uli

[EWeiss]

Zitat von ULIK:

Was soll denn byte(FileInfo.MIDINotes[IntA]) sein? Du machst doch hier einen Byte-Cast eines Arrays. Bist Du sicher, daß das so in Ordnung ist?

Grüße,
Uli

Ohne bekomme ich den Fehler !
[DCC Fehler] uMidiTracker.pas(451): E2010 Inkompatible Typen: 'Integer' und 'Dynamic array'

NoteNo gibt ein byte zurück.
bei low kommen nicht die richtigen Daten genau so wenig bei high.

Und wenn du schaust sieht das so aus..
byte(FileInfo.MIDINotes[IntA])].NoteNo

Das ist auch seltsam warum er diese Meldung ausgibt

Zitat:

'Integer' und 'Dynamic array'

Es wird ein byte zurückgeliefert und nirgends steht was von Interger


gruss

[ChrisE]

Hallo EWeiss

Zitat von s.h.a.r.k:

Ich kenne nur ein Phenomän, wenn Delphi die for-Schleife optimiert und der Debugger nicht darauf achtet. Wobei lediglich die Darstellung falsch ist, intern aber alles korrekt abläuft.!

Hast du die Optimierung an oder aus? Und hast du nach dem "Umschalten" komplett neu erzeugt?

Gruß und guten morgen,

Chris

[EWeiss]

Zitat von ChrisE:

Hallo EWeiss

Zitat von s.h.a.r.k:

Ich kenne nur ein Phenomän, wenn Delphi die for-Schleife optimiert und der Debugger nicht darauf achtet. Wobei lediglich die Darstellung falsch ist, intern aber alles korrekt abläuft.!

Hast du die Optimierung an oder aus? Und hast du nach dem "Umschalten" komplett neu erzeugt?

Gruß und guten morgen,

Chris

Guten Morgen

Ich habe die Optimierung aus.
Vorher habe ich alle DCU's gelöscht bevor ich diese ausgeschaltet habe.

Danach komplett neu erzeugt.
Jetzt arbeite ich ja mit D2009 und mein Kumpel hat hier eine TTntFileStream deklariert über die ich aber nicht verfüge.
Wäre es möglich das hier der Wurm begraben liegt?

function GetDelta(const {$IFNDEF UNICODE} F: TTntFileStream {$ELSE} F: TFileStream{$ENDIF}; var Len: word): DWORD;
Da ich ja Unicode verwende sollte er eigentlich diesen FileStream verwenden.

Und wenn wo bekomme ich die her?
Hab da nichts gefunden beim googel.

gruss

[FredlFesl]

Der Code unleserlich und kaum wartbar, weil u.a das DRY-Prinzip verletzt wurde.

Der 2.Index wird wiederholt kompliziert ermittelt und ist, wie Du ja weißt, falsch

Delphi-Quellcode:

midiNote := byte(FileInfo.MIDINotes[IntA]); // falsch
midiNote := FileInfo.MIDINotes[EinIndex,NocheinIndex].NoteNo; // bessser

Du solltest dich mit den Grundlagen robuster Programmierung befassen (DRY, KISS, Refactoring).

PS: Natürlich zeigt dir Delphi etwas an. Du "errechnest" ja einen Index (eigentlich beschneidest Du einen Pointer) und dann ist der Index eben irgendwas zwischen 0 und 255. Da steht dann irgendwo etwas...

Wenn dein Grid die Midinoten 1:1 anzeigen soll, dann funktioniert vielleicht das hier (falls ich den Code richtig verstanden habe).

Delphi-Quellcode:

Function TMidiTracker.EncodeMidiNote (midiNote : TMIDINote) : Integer;
begin
  Result := -1;
  if midiNote.NoteNo > 0 then
    if midiNote.Length = 12 then
      Result := (127 - midiNote.NoteNo) + basenote
    else if midiNote.Length > 12 then
      Result := 1000 + (127 - midiNote.NoteNo + basenote)
    else
      Result := midiNote.NoteNo
  else
    Result := -1;
end;

procedure TMidiTracker.ImpMidiFile(MidiFile: Widestring);
var
  FileInfo: TMIDIFileInfo;
  IntI: Integer;
  IntA: Integer;
  midiNote : TMIDINote;

begin
  if not ReadMIDIFile(MidiFile, FileInfo) then
  begin
    MessageBox(Self.Handle, 'Not a valid MIDI file.', 'Confirm', MB_OK);
    exit;
  end;

  for IntI := 1 to FileInfo.Tracks do
  begin
    for IntA := 0 to FileInfo.NoteCounter[IntI] -1 do
    begin
      midiNote := FileInfo.MIDINotes[IntI,IntA];
      Grid[IntI - 1, IntA] := EncodeMidiNote(midiNote);
      InstGrid[IntI - 1, IntA] := midiNote.Instrument;
    end;
  end;
  DrawGrid(self.Handle);
end;

[EWeiss]

Danke für deine Hilfe werde mir das mal genau anschauen.

gruss

[DeddyH]

Noch ein Tipp: um Arrays komplett zu durchlaufen empfiehlt es sich, die Funktionen Low() und High() zu verwenden, da man damit niemals über die Arraygrenzen hinaus zugreift. Dabei spielt es keine Rolle, ob es sich um ein dynamisches, statisches oder offenes Array handelt und ob es 0-indiziert ist oder nicht.