C# と VB.NET の質問掲示板

C#で、SystemParametersInfoを使ってフォームからBEEP音の設定するものを作成しているのですが
SPI_SETBEEPが正常に動かない（ウィッチウインド上で値がかわらない）ので困っているのですが
どなたかアドバイスをお願いいたします＾＾；

フォーム側のプログラム--------------------------------------------------------------

        BeepSetting BeepObj1 = new BeepSetting();

        private void button1_Click(object sender, EventArgs e)
        {
            
            if (BeepObj1.BeepProperty == 0)
            {
                MessageBox.Show("ビープ音は無効に設定されています");
            }else{
                MessageBox.Show("ビープ音は有効に設定されています");
            }
            
        }

        private void button1_Click_1(object sender, EventArgs e)
        {
            if (BeepObj1.BeepProperty == 0)
            {
                BeepObj1.BeepProperty = 1;
                MessageBox.Show("ビープ音を有効にしました");
            }
            else
            {
                BeepObj1.BeepProperty = 0;
                MessageBox.Show("ビープ音を無効にしました");
            }
        }

BeepSetting側のプログラム--------------------------------------------------------------

    public class BeepSetting
    {
        private const int SPI_GETBEEP = 1;  // ビープ音のON_OFFを取得
        private const int SPI_SETBEEP = 2;  // ビープ音のON_OFFを設定
        private const int SPIF_UPDATEINIFILE = 0x1;  // 更新する
        private const int SPIF_SENDWININICHANGE = 0x2; // 全てのアプリケーションに通知して更新する
        private const int SPIF_SENDCHANGE = 2;
        
        // システムに関するパラメータを取得/設定するAPI関数
        [DllImport("user32")]
        private extern static int SystemParametersInfo
            (int uiAction,
            int uiParam,
            int pvParam,
            int fuWinIni
            );

        public int BeepAlive = 1;

        public int BeepProperty
        {
            set
            {
                SystemParametersInfo(SPI_SETBEEP, value, 0, SPIF_SENDCHANGE);
            }
           get
            {
                SystemParametersInfo(SPI_GETBEEP, 0, BeepAlive, SPIF_SENDCHANGE);

                return this.BeepAlive;
            } 

        }
    }

> [DllImport("user32")]
> private extern static int SystemParametersInfo
> (int uiAction,
>  int uiParam,
>  int pvParam,
>  int fuWinIni
> );
SystemParametersInfo の第三引数 pvParam は PVOID 型であり、これを int で宣言するのは問題があります。
ポインタ型は IntPtr で宣言するか、あるいは参照型や参照渡しを使って値を渡すようにしなければなりません。
今回の場合、SPI_SETBEEP するときには IntPtr 型を使って IntPtr.Zero を渡し、
SPI_GETBEEP するときには out bool（または out int）を使って値を受け取れるように、
SystemParametersInfo の宣言をオーバーロードするのが簡単でしょう。

[DllImport(...)] ... SystemParametersInfo(..., ..., IntPtr pvParam, ...);
[DllImport(...)] ... SystemParametersInfo(..., ..., out bool pvParam, ...);

ほかにも、BeepAlive の意味が分かりませんし、
BeepProperty は bool 型で表現すべきではないでしょうか。
（このプロパティ名もどうかと思います）

あと、SPIF_SENDCHANGE 単独では意味がなかったような気がします。
// ユーザのシステムグローバルな設定を勝手に変えちゃっていいのかしらー？

■No52128 (Hongliang さん) に返信
> 2010/08/01(Sun) 14:34:06 編集(投稿者)
> 
Hongliang さん的確なアドバイスありがとうございました。

        //SPI_SETBEEPアクション用
        [DllImport("user32")] private extern static int SystemParametersInfo
            (int uiAction,
             int uiParam, 
             IntPtr pvParam 
            );

        //SPI_GETBEEPアクション用
        [DllImport("user32")] private extern static int SystemParametersInfo
            (int uiAction,
             int uiParam, 
             out bool pvParam
            );

        public void set_BeepProperty(int Value)
        {
            SystemParametersInfo(SPI_SETBEEP, Value, IntPtr.Zero);
        }

        public void get_BeepProperty(out bool Beep_onoff)
        {
            SystemParametersInfo(SPI_GETBEEP, 0, out Beep_onoff);
        } 

のように修正し、問題解決しました＾＾ｖ

ただ・・・プロパティ化しようと考えたのですが、

    /*   public bool BeepProperty
        {
            set
            {
 �@               SystemParametersInfo(SPI_SETBEEP, value, IntPtr.Zero);　 //  value が　bool型になってしまう。　　　
            }
           get
            {
                bool Beep_onoff = false
 �A               SystemParametersInfo(SPI_GETBEEP, 0, Beep_onoff);

                return Beepon_off;
            } 

        } */  

�@   部分でvalue がBOOL型になってしまい型変換できない
�A　 Get処理ではBeep_onoffの参照渡し、値渡しができない（引数を取れないため）

そのため、プロパティ化できませんでした・・・。

あと・・・プロパティ名の命名規則的なものがあれば、教えてください。まったくの独学なので・・
その辺わからないんです；−；

> //SPI_SETBEEPアクション用
> [DllImport("user32")] private extern static int SystemParametersInfo
> (int uiAction,
>  int uiParam, 
>  IntPtr pvParam 
>  );
第四引数が足りてませんよ。もう一つのオーバーロードの方も。

> (1) SystemParametersInfo(SPI_SETBEEP, value, IntPtr.Zero);　 //  value が　bool型になってしまう。
> (1) 部分でvalue がBOOL型になってしまい型変換できない
(a) value が true かどうかをチェックして 1 または 0 を与える
(b) SystemParametersInfo の宣言の第二引数を int の代わりに bool にする
のいずれかで解決できますね。

>     bool Beep_onoff = false
> (2) SystemParametersInfo(SPI_GETBEEP, 0, Beep_onoff);
>     return Beepon_off;
> (2) Get処理ではBeep_onoffの参照渡し、値渡しができない（引数を取れないため）
意味が分かりません。引数を取れないためってどういうことでしょう？

> あと・・・プロパティ名の命名規則的なものがあれば、教えてください。まったくの独学なので・・
> その辺わからないんです；−；
MSDN ではここです。
http://msdn.microsoft.com/ja-jp/library/ms229012.aspx
でも今回のは規則とかではなく。
このプロパティは「ビープが有効かどうか」なんだから BeepEnabled あたりにした方が分かりやすいって程度の話です。