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

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

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

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Visual Studio

Microsoft Visual StudioはMicrosoftによる統合開発環境(IDE)です。多種多様なプログラミング言語に対応しています。

Windows

Windowsは、マイクロソフト社が開発したオペレーティングシステムです。当初は、MS-DOSに変わるOSとして開発されました。 GUIを採用し、主にインテル系のCPUを搭載したコンピューターで動作します。Windows系OSのシェアは、90%を超えるといわれています。 パソコン用以外に、POSシステムやスマートフォンなどの携帯端末用、サーバ用のOSもあります。

WPF

Windows Presentation Foundation (WPF) は、魅力的な外観のユーザー エクスペリエンスを持つ Windows クライアント アプリケーションを作成するための次世代プレゼンテーション システムです

Q&A

解決済

1回答

5735閲覧

[WPF, C#]Pageクラスをシングルトンパターンで実装した時の懸念点について

bbdd

総合スコア43

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Visual Studio

Microsoft Visual StudioはMicrosoftによる統合開発環境(IDE)です。多種多様なプログラミング言語に対応しています。

Windows

Windowsは、マイクロソフト社が開発したオペレーティングシステムです。当初は、MS-DOSに変わるOSとして開発されました。 GUIを採用し、主にインテル系のCPUを搭載したコンピューターで動作します。Windows系OSのシェアは、90%を超えるといわれています。 パソコン用以外に、POSシステムやスマートフォンなどの携帯端末用、サーバ用のOSもあります。

WPF

Windows Presentation Foundation (WPF) は、魅力的な外観のユーザー エクスペリエンスを持つ Windows クライアント アプリケーションを作成するための次世代プレゼンテーション システムです

0グッド

0クリップ

投稿2018/04/11 06:39

前提

・WPFアプリ
・Visual Studio
・C#

お聞きしたいこと

static 変数とメモリに関してお聞きしたいです。
現状、static で各ページのインスタンスを生成(シングルトンパターンでPageクラスを実装)しておき、GetInstance()メソッドにて遷移先のPageクラスのインスタンスを取得し、NavigationService.Navigate()の引数に渡して画面遷移をする、という方法をとっています。
・ページ数は12ページ程
・ページ1 → ページ2 → ページ1 → ページ5
といったような遷移も考えられます
(この際に再度ページ1のインスタンスを生成するより、1回だけインスタンス生成されればいいやん、と思いstatic でシングルトンパターンにしました。)

⬇︎(例)ログインページクラス

public partial class LoginPage : Page { private static LoginPage _instance = new LoginPage(); public static LoginPage GetInstance() { return _instance; } public LoginPage() { InitializeComponent(); } (省略) }

・アプリ使用中のメモリ使用量は120MBほどであり、staticな変数に常に値を追加していくような処理もなく、メモリリークはしないだろうと考えているので、このまま(各ページはstaticなシングルトンパターンで実装)で良いだろうと考えているのですが、この認識はあっていますでしょうか?
それともこのままだとメモリリークの可能性があったりするのでしょうか?

(メモリリークを気にしたのは以下のためです)
static な変数はGCの対象にならない(参考:メモリリーク例について、(対象になることもあるが、基本ならない感じの認識です))のでstatic で宣言しておくと、その分のメモリを食ってしまう、と思っています。

C#、WPF共に初めてでして、もしアドバイス等あればお聞きしたいです。

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

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

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

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

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

guest

回答1

0

ベストアンサー

それともこのままだとメモリリークの可能性があったりするのでしょうか?

あります。
シングルトンなオブジェクトは、そのクラスのstatic変数から参照し続けられるために明示的に開放しない限りGCに回収されません。
よって、シングルトンなオブジェクトのフィールドやプロパティは全てstaticな変数と同じくメモリリークの可能性があります。

シングルトンなオブジェクトのフィールドとしてコレクションを持って要素を追加するような場合が特に危険です。

投稿2018/04/11 07:02

編集2018/04/11 07:16
YamakawaJunichi

総合スコア630

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

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

Zuishin

2018/04/11 07:41

可能性があるか無いかで言えば無いとは言い切れないでしょうが、メモリリークというのは水道から水が漏れ続けるように使用メモリが徐々に徐々に増えていく現象です。 一度確保されたメモリがアプリ終了まで解放されないのはメモリリークとは言いません。 この場合で言うならシングルトンオブジェクトのフィールドにコレクションを追加しても、それが無限に追加され続けるのでない限りメモリリークはありませんし、その場合はいつかコレクションが破綻して例外を出します。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問