C++ でのアプリケーション設計について

Question

#C++でのコーディング？設計について

正直に申し上げると、これが設計の問題なのかコーディングの問題なのかあるいはそのどちらも含んだ問題であるのかがよくわかっておりません。

現在VC＋＋で簡単なアプリケーションを作っております。
今はウィンドウズアプリケーションなどでよくある「設定」や「環境設定」などを実装しようとしています。

設定ダイアログを呼び出して、各種の設定をラジオボタンやエディットボックスなどで行えるようにしたいです。

##作成したいもの

設定した情報をアプリケーションの実行中は保持されるようにしたいのですが、どのようにすればいいのでしょうか？

情報を保持する用のクラスを作成し_tWinMainでインスタンス化して......???

Comments

[takabosoft]

アプリを作る際にフレームワーク等を使用しているのであれば、それらも記述してください。WTL/MFC等。

[harpy]

回答ありがとうございます。

使用しているのはMicrosoft Visual C++　でMFCを使っているのかどうかはどうやって調べるのでしょうか？
一応自分でしらべて、ここを見ればMFCを使っているかどうかを判断できるというところは載せなおします。使っていると思います。

[takabosoft]

例えばプロジェクトのプロパティ→構成プロパティ→全般→MFCの使用　という欄が”共有DLLでMFCを使う”とかになっていれば使っているかもしれません。

[takabosoft]

ついでにVisual Studioのバージョンも記載してもらった方がいい気がしました。

[harpy]

VC＋＋　２０１０ですよろしくお願いいたします。

[takabosoft]

MFC使っていないですね〜。

[harpy]

なるほど　「標準Windowsライブラリを使用する」ではMFCを使用しないのですね。
ご指摘ありがとうございます。

Answer 1

こんにちは。

アプリケーション全体の振る舞いを設定するのであれば、グローバル変数やシングルトンなどが妥当と思います。
そのようなケースで、設定値をアクセスしたい関数やクラスへパラメータで渡す場合、実際に使わないのに呼び出し経路にある関数群にも渡して中継する必要があり、可読性が悪化することが少なくありません。

設定値を管理するクラスだけを定義するヘッダを設け、設定値を使用するソースのみが#includeすることでアクセス範囲を限定するのも有効です。

Answer 2

こんにちは。

その設定を使う、一番長生きしているクラスか何かに便乗してメンバー変数等で値を保持しておくのが楽ではあります（アプリケーションクラスやメインウィンドウクラスなど）。

（他にもグローバル変数やシングルトンといったやり方もありますが、あまり推奨されていないらしいです。）

Comments

[harpy]

いただいた回答について一つ質問があります。
＞（他にもグローバル変数やシングルトンといったやり方もありますが、あまり推奨されていないらしいです。）
とありますが、どのような理由からでしょうか？

調べたところ、アプリケーション全体で使う設定値などはSingletonで持たせることがよくあると書いてありました。（ただ、クラス間の結合が強くなってしまうから慎重に。。とは書いてありました。これが理由でしょうか？）

[takabosoft]

> どのような理由からでしょうか？

ぶっちゃけ私自身、自分のアプリではシングルトンをたまに使っていまして、そういった「singletonは害悪」的な記事を見て、なんとなく良くなさそうだな・・・程度の認識のもとに書いています(^_^;)すみません。

よく言われているのは単体テストなどの妨げになりやすいという点です。
例：
https://プログラマが知るべき97のこと.com/%E3%82%A8%E3%83%83%E3%82%BB%E3%82%A4/%E3%82%B7%E3%83%B3%E3%82%B0%E3%83%AB%E3%83%88%E3%83%B3%E3%83%91%E3%82%BF%E3%83%BC%E3%83%B3%E3%81%AE%E8%AA%98%E6%83%91%E3%81%AB%E8%B2%A0%E3%81%91%E3%81%AA%E3%81%84/

環境設定はそもそもアプリ全体の動作を変えるものでしょうからシングルトンでも良いと思いますけどね。

[stdio]

その通りです。
グローバル変数やシングルトンが推奨されていない理由は、多重定義やどこで呼ばれたのかしっかりと管理しにくい等の問題が発生しやすいからです。開発が大規模になればなるほど、やらかしてしまうと取り返しのつかない結果になってしまいます。

[harpy]

ありがとうございます。

今回は簡単なプログラムであり、大規模になる可能性はないといえますのでシングルトンで実装してみます。

回答ありがとうございました。

Answer 3

設定値は

• 都度，引数等で渡す
• 設定値を参照する人には，「設定値を取得する手段」をあらかじめ指定しておく(こういうのをDIとか呼ぶのだろうか？)

とかしておけば良いと思いますが，
そういう形が（何らかの理由で）嫌で
とりあえずどこからでもアクセスできるようにしたいだけなら
（シングルトンがどうのとか言ってインスタンスの個数まで制限する必要は無いと思うので）
単にこんな感じにしておけばどうでしょうか．

//[AppSetting.h]

#pragma once

//Appの設定値
class AppSetting
{
public:
    AppSetting() : A(0), B(2), C(7) {}

public:
    //設定値の状態が，APPの設定として妥当か否かをチェックする手段
    bool IsValid() const
    {
        if( A<-2 )return false; //設定値Aは-2以上じゃないと不正
        if( B & 0x01 )return false; //設定値Bは偶数である必要がある
        return true;
    }

public: //設定値の方々
    int A;
    unsigned int B;
    int C;
};

//現在の設定値の取得
const AppSetting &GetCurrAppSetting();

//設定値の変更
bool ChangeAppSetting( const AppSetting &NewSettingVals );

//[AppSetting.cpp]

#include "stdafx.h" //VC
#include "AppSetting.h"

namespace
{
    AppSetting CurrAppSetting;
}

//現在の設定値の取得
const AppSetting &GetCurrAppSetting(){  return CurrAppSetting;  }

//設定値の変更
bool ChangeAppSetting( const AppSetting &NewSettingVals )
{
    if( !NewSettingVals.IsValid() )return false;
    CurrAppSetting = NewSettingVals;
    return true;
}