LoadSoundMemがエラー戻り値-1を返す原因が分からない

Question

前提・実現したいこと

Visual Studio上でDXライブラリを使用してゲームを作っています。
BGM以外の枠組みは完成しており最後に画面ごと(タイトル画面、ゲーム画面、リザルト画面etc)に異なるBGMを流そうとしています。

発生している問題・エラーメッセージ

エラーメッセージは出ることなくプログラムは動きますがPlaySoundMemがBGMを流しません。
LoadSoundMemでmp3ファイルを読み込みHandle_bgmに代入しています(Handle_bgm_aは複数のファイルを読み込む為に作成しておりファイルを増やすごとにbgm_b, c, d...と作成します)がDrawFormatStringで戻り値を確認したところ-1(エラー)を返していました。

以下はDXライブラリ置き場リファレンスページ(https://dxlib.xsrv.jp/function/dxfunc_sound.html)より抜粋

宣言 int LoadSoundMem( char *FileName ) ;

概略 音ファイルをメモリに読みこむ

引数 FileName :　メモリに読みこむ音ファイルのファイル名
文字列のポインタ
戻り値 －１：エラー
－１以外：サウンドハンドル

該当のソースコード

//Main.cpp

#include "DxLib.h"
#include "Main.h"
#include "Sound.h"

extern int Handle_bgm;

char Key[256];

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
	LPSTR lpCmdLine, int nCmdShow) {
	if (ChangeWindowMode(TRUE) != 0) {
		return 0;
	}
	if (DxLib_Init() != 0) {
		return 0;
	}
	if (SetDrawScreen(DX_SCREEN_BACK) != 0) {
		return 0;
	}
	if (SetBackgroundColor(0, 0, 0) != 0) {
		return 0;
	}
	PlaySoundMem(Handle_bgm, DX_PLAYTYPE_LOOP);
	while (1) {
		if (ScreenFlip() != 0) {
			break;
		}
		if (ProcessMessage() != 0) {
			break;
		}
		if (ClearDrawScreen() != 0) {
			break;
		}
		if (GetHitKeyStateAll(Key) != 0) {
			break;
		}
		Sound();
	}
	DxLib_End();
	return 0;
}

//--------------------------------------------------

//Main.h

#ifndef DEF_MAIN_H

#define DEF_MAIN_H

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
	LPSTR lpCmdLine, int nCmdShow);

#endif

//--------------------------------------------------

//Sound.cpp

#include "DxLib.h"
#include "Main.h"
#include "Sound.h"

int Handle_bgm_a = LoadSoundMem("./Audio/testsound_a.mp3");
int Handle_bgm = Handle_bgm_a;

void Sound() {
	DrawFormatString(200, 300, GetColor(255, 255, 255), "BGM : %d", Handle_bgm);
}

//--------------------------------------------------

//Sound.h

#ifndef DEF_SOUND_H

#define DEF_SOUND_H

void Sound();

#endif

試したこと

BGMと関係ないゲーム部分のコードを削りながら上記コードのようになるまで解決方法を模索しましたが状況の改善は全くありませんでした。

補足情報（FW/ツールのバージョンなど）

Microsoft Visual C++ 2019
DXライブラリ Windows版 Visual C++用 Ver3.22c

Comments

[yumetodo]

実はDxLibではmp3を確実に再生できる保証はありません。opusに変換して試してみてください。

Answer 1

PATHに問題がありました。
指定フォルダは合っていたのですがファイルの名前を間違えていました。
修正したところ無事にBGMが流れるようになりました。
丁寧な回答ありがとうございました。

Answer 2

LoadSoundMem()を呼び出すタイミングですが、現状のコードではSound.cppの冒頭で変数の宣言と一緒に実行してしまっているため、コード全体のかたちとして以下のようになってしまいます。

//抜粋
int Handle_bgm_a = LoadSoundMem("./Audio/testsound_a.mp3");
int Handle_bgm = Handle_bgm_a;
char Key[256];

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
    LPSTR lpCmdLine, int nCmdShow) {
    if (ChangeWindowMode(TRUE) != 0) {
        return 0;
    }
    if (DxLib_Init() != 0) {
        return 0;
    }
...

そうするとLoadSoundMem()はDxLib_Init()の実行の前、それどころかWinMain()が実行し始めるより前に実行されているかたちになってしまいます。それではいけません。DxLib_Init()を実行した後に呼ぶようにしましょう。

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
    LPSTR lpCmdLine, int nCmdShow) {
    if (ChangeWindowMode(TRUE) != 0) {
        return 0;
    }
    if (DxLib_Init() != 0) {
        return 0;
    }

    // DxLib_Init()の実行後に呼び出す。もちろん、適切に関数化してもOK。
    Handle_bgm_a = LoadSoundMem("./Audio/testsound_a.mp3");
    Handle_bgm = Handle_bgm_a;
...

上記のように直したけど依然として鳴らないようであれば、ファイルが正しく読めていない可能性があります。ファイルが存在するパス(PATH)を確認しましょう。「"C:/Users/username/yourproject/Audio/testsound_a.mp3"」のようにフルパスで指定してみてください。

"./Audio/testsound_a.mp3"での指定は「相対パス指定」と言います。プログラムが実行したときの、プログラムにとってのカレントディレクトリ(./)を基点として"Audio/testsound_a.mp3"を指定しています。この相対パス指定でファイルが開けないという問題はteratailでも頻出ですので、ご自身の場合ではどうなっているか確認してください。フルパスで指定すればその問題はとりあえず避けることができます。

Visual Studio 2019の場合、既定で「プロジェクトのディレクトリ」がプログラム実行時のカレントディレクトリになります。プロジェクトのプロパティから「構成プロパティ」→「デバッグ」→「作業ディレクトリ」と辿ると「$(ProjectDir)」がセットされていることが分かります。

尚、\マークをパス区切り文字列にするのであれば、C:\Users\username\yourproject\Audio\testsound_a.mp3のようになります。

Comments

[fkcz_yfm_carp]

返信ありがとうございます。

DXLib_Initの後(PlaySoundMemの直前)にvoid関数Sound_Initializeを作成、その中でLoadSoundMemを実行するようにしてみました。
ただ、そうするとSound.cppから
「'Handle_bgm': 定義されていない識別子です。」
とエラーメッセージが出ましたので、Main.cppのSound_Initializeを置いていた部分に直接LoadSoundMemを置きましたが
「未解決の外部シンボル "void __cdecl Sound_Draw(void)" (?Sound_Draw@@YAXXZ) が関数 _WinMain@16 で参照されました」
とエラーメッセージが出ました。
改めて自分で考えても対処法が分かりません、どこに問題が出てしまったのでしょうか。

PATHを確認してみましたがそちらは正しく指定できていました。

繰り返しの質問になり申し訳ございません。

[dodox86]

> 「未解決の外部シンボル "void __cdecl Sound_Draw(void)" (?Sound_Draw@@YAXXZ) が関数 _WinMain@16 で参照されました」

ちょっと状況が把握できませんが、Sound_Draw()と言う関数はもともとSound.cpp内で定義してあったSound()と言う関数を、質問を投稿された後にfkcz_yfm_carpさんがSound_Draw()に改名したか、新たに追加したものではありませんか？ そうなのであれば、いったん整理して、

------------------------------
//(1) Sound.h で関数プロトタイプ宣言
void Sound_Draw();
void Sound_Initialize();

------------------------------
// (2) Sound.cppで関数定義
int Handle_bgm_a;
int Handle_bgm;

void Sound_Initialize() {
Handle_bgm_a = LoadSoundMem("./Audio/testsound_a.mp3");
Handle_bgm = Handle_bgm_a;
}

// 旧Sound() ?
void Sound_Draw() {
DrawFormatString(200, 300, GetColor(255, 255, 255), "BGM : %d", Handle_bgm);
}

------------------------------

とのようにでも分けて、DxLib_Init()の実行後にSound_Initialize()を呼ぶようにすれば良いと思います。
この例では、Sound()を呼んでいるところもSound_Draw()に修正してください。

[fkcz_yfm_carp]

SoundをSound_Drawに改名していたのを書き忘れていました、すみません。