Debugger Krise

[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

[ChrisE]

Hallo,

ich glaube das was ULIK versucht hat klar zu machen ist, dass du oft diesen Code hier hast:byte(FileInfo.MIDINotes[IntA]) .
Wenn man deiner Record-Deklaration vertrauen darf, dann ist aber MIDINotes ein array[..] of array of TMIDINote (hier ja ohne s). Wenn du dieses Record auf ein Byte "reduzierst", dann siehst du wahrscheinlich nur das erste Byte von TMIDINote.

Gruß, Chris

[EWeiss]

Zitat von ChrisE:

Hallo,

ich glaube das was ULIK versucht hat klar zu machen ist, dass du oft diesen Code hier hast:byte(FileInfo.MIDINotes[IntA]) .
Wenn man deiner Record-Deklaration vertrauen darf, dann ist aber MIDINotes ein array[..] of array of TMIDINote (hier ja ohne s). Wenn du dieses Record auf ein Byte "reduzierst", dann siehst du wahrscheinlich nur das erste Byte von TMIDINote.

Gruß, Chris

Versteh aber was soll ich übergeben ?
Eine Idee?

Denn wie du gelesen hast

Zitat:

[DCC Fehler] uMidiTracker.pas(451): E2010 Inkompatible Typen: 'Integer' und 'Dynamic array'

bekomme ich einen Fehler!
Bei low Irrationale werte und bei high stürzt die Anwendung ab.

EDIT:
Schau mal oben im Thread "Bild 3" dann kannst du sehen was ich vom Debugger geliefert bekomme
Und die werte stimmen alle werden aber falsch übergeben/ausgelesen vom Compiler.


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.