質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

DLL

DLL(Dynamic Link Library)とは、他のモジュールからも使用する事が出来る、関数とデータが格納されているモジュールのことです。

C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

Q&A

解決済

2回答

10112閲覧

DLL内にあるグローバル変数をマルチスレッド間で共有しないようにするには?

KureteRubyLua

総合スコア206

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

DLL

DLL(Dynamic Link Library)とは、他のモジュールからも使用する事が出来る、関数とデータが格納されているモジュールのことです。

C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

1グッド

1クリップ

投稿2016/05/26 13:52

編集2016/05/26 14:00

Java上でOggをJavaからDLLを呼び出すJNA経由で再生するために、Oggファイルを再生するDLLを作ろうと、C++とC言語を組み合わせてDLLを作ったのですが、マルチスレッドだとバグが起き、原因がグローバル変数を用いており、そのグローバル変数がマルチスレッドだと同じ内容が共有されているからだと分かりました。

このグローバル変数をスレッドローカルにしようとしたのですが、コンストラクタとデストラクタがあるクラスでは、スレッドローカルの変数にする事が出来ません

何か良い解決策はありますでしょうか?C++の部分を完全にC言語に書きなおすしかないのでしょうか?

コードは以下の物となります。OggDecorderクラスなどのソースは下記のサイトに有ります。
http://marupeke296.com/OGG_main.html
この質問は下記の質問に関連したものです
https://teratail.com/questions/35915

C++

1#include <memory.h> 2#include <string> 3#include <iostream> 4#include "vorbis/vorbisfile.h" 5#include "OggDecoder.h" 6#include "OggVorbisMemory.h" 7#include "OggVorbisFile.h" 8#include "PCMPlayer.h" 9#include "DixSmartPtr.h" 10#include "DixComPtr.h" 11#include "OggMain.h" 12 13using namespace std; 14 15Dix::sp< Dix::OggVorbisMemory > spOggResource; 16Dix::sp< Dix::OggDecoder > spOggDecoder = Dix::sp< Dix::OggDecoder>(new Dix::OggDecoder(spOggResource)); 17int check = -1; 18 19extern "C" int OggDll::open(char* filePath) 20{ 21 check = -1; 22 23 spOggResource = Dix::sp< Dix::OggVorbisMemory > (new Dix::OggVorbisMemory); 24 if (spOggResource->createBuffer(filePath) == false) 25 { 26 check = -1; 27 return -1; 28 } 29 else 30 { 31 check = 0; 32 spOggDecoder = Dix::sp< Dix::OggDecoder>(new Dix::OggDecoder(spOggResource)); 33 spOggDecoder->getSampleRate(); 34 return 0; 35 } 36} 37 38extern "C" void OggDll::close() 39{ 40} 41 42 43// 指定サイズでPCM音声バッファを埋める関数 44extern "C" int OggDll::getPCMBuffer(char* buffer,int bufferSize) 45{ 46 bool a = true; 47 unsigned int b = 100; 48 if(check==-1) 49 { 50 return -1; 51 } 52 else 53 { 54 spOggDecoder->getSegment(buffer, bufferSize, &b, &a); 55 return 0; 56 } 57} 58 59// 指定サイズでPCM音声バッファを埋める関数 60extern "C" int OggDll::getSampleRate() 61{ 62 if (check == -1) 63 { 64 return -1; 65 } 66 else 67 { 68 return spOggDecoder->getSampleRate(); 69 } 70} 71 72// 指定サイズでPCM音声バッファを埋める関数 73extern "C" int OggDll::getChannel() 74{ 75 if (check == -1) 76 { 77 return -1; 78 } 79 else 80 { 81 return spOggDecoder->getChannel(); 82 } 83}
argius👍を押しています

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

どのような使い方を想定しているのか判りませんが、あるスレッドでopenしつつ、別のスレッドでもopenしたいということでしょうか。

その場合は、スレッドローカル変数で無理矢理切り分けようとするのではなく、呼び出し側にインスタンスのポインタを返して、そのポインタ経由で各種操作を行うような仕組みにすべきだと思います。C/C++のプログラムと何ら変わりません。

ソースを見る限り、オブジェクトはDix::OggVorbisMemoryDix::OggDecoderの二つなので、その二つを構造体でパックして、その構造体をnewしたポインタをJava側に返し、そのポインタ経由で操作するようにすれば良いと思います。そうすればグローバル変数を使う必要はなくなります。

コード例

C++

1struct OggHandle 2{ 3 Dix::sp<Dix::OggVorbisMemory> spOggResource; 4 Dix::sp<Dix::OggDecoder> spOggDecoder; 5}; 6 7extern "C" OggHandle* OggDll::open(char* filePath) 8{ 9 OggHandle *handle = new OggHandle; 10 handle->spOggResource = Dix::sp<Dix::OggVorbisMemory>(new Dix::OggVorbisMemory); 11 handle->spOggDecoder = Dix::sp<Dix::OggDecoder>(new Dix::OggDecoder(handle->spOggResource)); 12 handle->spOggDecoder->getSampleRate(); 13 return handle; 14} 15 16extern "C" void OggDll::close(OggHandle* handle) 17{ 18 delete handle; 19} 20 21extern "C" int OggDll::getPCMBuffer(OggHandle* handle, char* buffer,int bufferSize) 22{ 23 bool a = true; 24 unsigned int b = 100; 25 handle->spOggDecoder->getSegment(buffer, bufferSize, &b, &a); 26 return 0; 27} 28 29// 以下略

投稿2016/05/26 23:42

編集2016/05/27 05:28
catsforepaw

総合スコア5938

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

KureteRubyLua

2016/05/28 09:41

この方法でやったらうまく行きました。有難うございました
guest

0

こんにちは。

このグローバル変数をスレッドローカルにしようとしたのですが、コンストラクタとデストラクタがあるクラスでは、スレッドローカルの変数にする事が出来ません

これはどのような方法を試みられたのでしょうか? もし、thread_local指定したのではないのでしたら、このキーワードを付けてみるのも手と思います。C11, C++11以降で使える機能です。

ただ、JNA経由で呼ばれた時もちゃんと動くのか、あまり確信はもてません。外れていたらごめんなさい。


【追記】

C++

1#include<iostream> 2 3struct Foo 4{ 5 Foo() 6 { 7 std::cout << "Foo()\n"; 8 } 9 ~Foo() 10 { 11 std::cout << "~Foo()\n"; 12 } 13}; 14 15__declspec( thread ) Foo gFoo; 16 17// https://msdn.microsoft.com/ja-jp/library/9zxe5aww.aspx 18__declspec(thread) struct A { 19 A(){} 20 ~A(){} 21} aa; // C2483 error 22 23__declspec(thread) struct B {} b; // OK 24 25int main() 26{ 27 return 0; 28}

コマンドラインで、cl ソース.cpp /EHscにてコンパイルしたところ、エラーにならず、正常に実行できました。
MinGW(gcc)でも、問題ありませんでした。

コンパイラのバージョンをあげればもしかすると通るかも知れないです。

投稿2016/05/26 14:41

編集2016/05/26 15:29
Chironian

総合スコア23272

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Chironian

2016/05/26 15:24

ドキュメントには確かにエラーになると書かれてますね。 そのコードも入れてみたのですが、VC++ 2015ではエラーになりませんでした。 回答の方に実際にやってみたソースを上げます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問