Ich lade und es schließt

Checkmateing · 7. März 2011, 21:39

Hallo!
Ich habe hier einen kleinen Player geschrieben, doch wenn ich das File lade, egal ob über String oder gleich in Anführungszeichen, bei beiden kann er die Files nicht laden.
Ja ich habe die jeweiligen Files im Ordner.
Im Player rufe ich die Funktion in der DLL ganz normal auf.
Die sieht übrigens so aus:

Quellcode

void DLL_AUDIO_API lwtAudio::Audio::Init(const string musictitle)
{
//File Laden
if(!m.OpenFromFile(musictitle.c_str()))
exit (1);
}

Das aufrufen muss ich wohl weniger erläutern. <.<
Als Fehlermeldung kommt immer: "Failed to open "êe-" for reading."

pocky · 8. März 2011, 12:12

mach mal vorher

Quellcode

cout << musictitle.c_str();

und sag uns was da rauskommt.
Vielleicht is der String ja wirklich kaputt, weil beim einspielen des Textes ein fehler passiert ist

Checkmateing · 8. März 2011, 19:56

Scheint irgendwie kaputt zu sein. Denn es wird "¿ed" ausgegeben.
Muss der String vom Musiktitel den ich übergebe etwa auch statisch sein? Nein, stimmt ja. Habe ich auch schon versucht.

MfG
Check

pocky · 8. März 2011, 21:47

Zeig mal die Codezeile, in der du den string übergibst. Sonst würd ich mal sagen versuchs ohne const.
Irgendwie behagt es mir nicht ein const bei string zu sehen, kann aber leider nicht sagen warum... scheint intuition zu sein XD

Checkmateing · 8. März 2011, 22:08

Das const spielt kaum eine Rolle, habe es nämlich auch schon ohne probiert.

Ja, das wäre dann halt:

Quellcode

const string music ("Music.ogg");
myAudio.Init(music.c_str())

MfG
Check

Rushh0ur · 9. März 2011, 02:37

Lass mal das .c_str() mal weg, als Parameter wird ein string erwartet und kein char*, druch die interne Typenkonventierung wird das sowieso wieder in einen string umgewandelt, eventuell liegt da auch der Fehler, das ist quasi doppelt und dreifach gemoppelt.

Die Angabe const führt dazu das die Variable nicht geändert werden darf und sollte eigentlich bei Funktionsparameter entweder mit Pointer oder Referenzen verwendet werden, alles andere, ist zwar möglich, macht aber keinen Sinn.
Bei Variablen ist das ein anderes Thema.

Quellcode

const string music ("Music.ogg");
myAudio.Init(music)
...
void DLL_AUDIO_API lwtAudio::Audio::Init(const string musictitle)
{
//File Laden
if(!m.OpenFromFile(musictitle))
exit (1);
}

Alles anzeigen

Weitere Fehlerquellen wären die interne Zeichendarstellung und konventierund macht nen Fehler bzw zu mindest an einer Stelle nicht das gwünschte, oder der Parameter wird Falsch an die DLL weitergegeben.

Mfg Rushh0ur

Checkmateing · 9. März 2011, 14:01

Jetzt wird nach Änderung von dem "const" und dem "c_str();" nun das ausgegeben:
[Blockierte Grafik: http://www.abload.de/img/schalla3fi9.png]

Das hilft nicht weiter.
Ich verstehe es nicht so, ich habe die Funktion, den Aufruf der meiner Meinung nach korrekt ist, aber trotzdem kommt so ein Mist heraus. ([Blockierte Grafik: http://www.abload.de/img/schalla3fi9.png])
Danke schon mal für die Hilfen! (Ach ja, nicht wundern, ich habe den Inhalt und den Namen hier geändert, dass hat schon seine Richtigkeit)

MfG
Check

pocky · 9. März 2011, 18:41

lass die zeichen mal als hex ausgeben, und mach bitte die const weg

Checkmateing · 9. März 2011, 18:50

Die consts sind doch schon weg.
Wie lasse ich den String in HEX konvertieren?

MfG
Check

Rushh0ur · 9. März 2011, 19:31

Sieht wirklich nach nem typischen Parameterüberrgabefehler aus.

Wenn du in deiner Deklration enen Parameter hinzufügst oder entfernst aus eine Funktion oder einfach nur vertauscht, egal was, dann musst du die DLL neu compilieren.

Also DLL neu compilieren und anschliessend das Programm neu compilieren.

Mfg Rushh0ur

Checkmateing · 9. März 2011, 20:17

Ich kompiliere sie natürlich neu und schiebe die *.lib sowie die *.dll auch in den Ordner, auch logisch. Sonst ändert sich ja nichts.

MfG
Check

pocky · 9. März 2011, 21:02

schick nochmal den ganzen zugehörigen code, so wie du ihn jetzt hast, sonst wird das nichts

Checkmateing · 9. März 2011, 21:15

filedropper.com/desktop_14

MfG
Check

Rushh0ur · 12. März 2011, 03:04

- Achte darauf, dass du immer die originalen Headerdateien verwendest am besten ist dies zu realisiern in dem man relative Pfäde verwendet anstatt die Dateien bei änderung immer wieder zu kopieren.
Dies wird wohl der Parameterübergabefehler sein der dein Programm zum abstützen bringt.

Quellcode

void Init (const string musictitle);
<->
void Init (string musictitle);

-> Das sind zwei verschiedene Definitionen, was glausbt was passiert wenn du die Biblitohek kompillierst und anschliessend die Definition änderst und versuchst die Biblotheks-Funktion mit anderen Parametern aufzurufen.

- Für deine Anwendung brauchst du die Bibliotheksdaten der SFML solange nicht einzubinden bis du davon Funktionen verwendest, dein DLL verwendet zwar SFML aber die bindet diese eigenständig ein du musst nur deine Biblothek einbinden.

#pragma comment(lib, "Audio.lib")
~~#pragma comment(lib, "sfml-audio.lib")~~

#include <iostream>
#include "Audio_Header.hpp"
~~#include <SFML\Audio.hpp>~~

- Wenn du Biblotheken verwendest brauchst du die CPP Dateine nicht mit zu kopieren bzw. nicht mit zu includieren. Der Code ist ja schon in deiner Bibliothek compiliert.

- Schau dir mal noch mal Variablen/ Klassenendeklerationen an.

~~class _declspec(dllimport)~~ lwtAudio::Audio myAudio;

- Versuch die Funktionsaufrufparameter immer richtig an zugeben, zu mindest mit einer explitzieten Typenkonventierung

string musictitle;
...
m.OpenFromFile(musictitle~~.c_str()~~)

Siehe definition OpenFromFile

OpenFromFile (const std::string &filename)

Mehr Berichtigungen hab ich jetzt nicht gesehen.

Mfg Rushh0ur

Checkmateing · 12. März 2011, 12:58

Also lag es hauptsächlich daran, dass ich die Header-Datei nicht mit aktualisiert habe, also die, die im Projekt Player liegt.

Ansonsten wäre da noch der Klassen/Variablen-Deklarations-Fehler. Dies ist mal wieder einer meiner Flüchtigkeitsfehler gewesen, passiert mir in der Schule auch viel zu oft.
Langsam legt sich das alles aber und die Fehler verschwinden somit immer mehr. - Das nur, damit man nicht anfängt, falsch über mich zu denken.

Danke dir, Rushh0ur!

MfG
Check

Ich lade und es schließt

Ich lade und es schließt

Quellcode

Quellcode

Quellcode

Quellcode

Quellcode

Teilen

Benutzer online 1

Tags