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

ASP.NET、C++/CLI、Java 何でもどうぞ

C# と VB.NET の入門サイト

Re[5]: notifyicon多重表示回避

分類:[C#]　

お世話になっていますれれれです。

早速質問なんですが

notifyiconの多重表示を防ぐ方法を探しています。
現在は
mainclass内にnotifyiconコンポーネントを貼り付けてメインフォームが表示されるとnotifyiconが表示されるようにしています。
ところがsubclass内にてmainclassのインスタンスを作成するたびにnotifyiconが表示されます。

mainclass.Designer.csファイルにnotifyiconのプロパティやイベントについての設定が書かれているためです。

subclass内
↓
mainclassインスタンス作成
↓
mainclassでpublic変数宣言やInitializeComponent();を実行//多重表示発生

となっています。

どうすれば回避できるでしょうか？

■No20829 (れれれさん) に返信

メインが沢山出来るということが、変だと思います。一つしか作らないようにすればいいのでは？
あるいは、何故沢山作るのでしょう？

■No20832 (Jitta on the way さん) に返信
> ■No20829 (れれれさん) に返信
>
> メインが沢山出来るということが、変だと思います。一つしか作らないようにすればいいのでは？
> あるいは、何故沢山作るのでしょう？

mainclassにシリアルポートコンポーネントをpublicで宣言していてsubclassがsubclassフォームを開いてシリアル通信を行うたびにmainclassのインスタンスが作られてしまうのです。
mainclass, subclassともにpublic partial classとしています。（もとからですが）

public partial class Mainclass : Form
{
・・・
Subclass sub = new Subclass();
・・・
sub.show();
}

public partial class Subclass : Form
{
・・・・
private void button1_Click(object sender, EventArgs e) //シリアル通信をおこなう
{
Mainclass main = new Mainclass(); // notifyicon多重表示
・・・
}
}

Mainclass main = new Mainclass();をprivate以外のところで実行すると無限ループに入ったので上記のようにしました。

> private void button1_Click(object sender, EventArgs e) //シリアル通信をおこなう
> {
> Mainclass main = new Mainclass(); // notifyicon多重表示
> ・・・
> }

ボタンが押されるたんびに Mainclass が生成されてますが、これが"正しい使い方"なのですか?
どこかで最初に一個だけ作っておいて、使いまわしてはいかんのでしょうか。

■No20845 (επιστημη さん) に返信
>> private void button1_Click(object sender, EventArgs e) //シリアル通信をおこなう
>> {
>> Mainclass main = new Mainclass(); // notifyicon多重表示
>> ・・・
>> }
>
> ボタンが押されるたんびに Mainclass が生成されてますが、これが"正しい使い方"なのですか?
> どこかで最初に一個だけ作っておいて、使いまわしてはいかんのでしょうか。
>
　　private void button1_Click(object sender, EventArgs e) //シリアル通信をおこなう
　　{
　　　　Mainclass main = new Mainclass(); // notifyicon多重表示
　　　　・・・
　　main.Dispose();　
　　}
としたらnotifyiconが多重表示されなくなりました（表示されてすぐ消える。）

■No20853 (れれれさん) に返信
>>ボタンが押されるたんびに Mainclass が生成されてますが、これが"正しい使い方"なのですか?
>>どこかで最初に一個だけ作っておいて、使いまわしてはいかんのでしょうか。
>>
> 　　private void button1_Click(object sender, EventArgs e) //シリアル通信をおこなう
> 　　{
> 　　　　Mainclass main = new Mainclass(); // notifyicon多重表示
> 　　　　・・・
> 　　main.Dispose();　
> 　　}
> としたらnotifyiconが多重表示されなくなりました（表示されてすぐ消える。）

それでいいんか。
本当にそれでいいんか。

■No20856 (シャノンさん) に返信
> ■No20853 (れれれさん) に返信
> >>ボタンが押されるたんびに Mainclass が生成されてますが、これが"正しい使い方"なのですか?
> >>どこかで最初に一個だけ作っておいて、使いまわしてはいかんのでしょうか。
> >>
まず「使いまわす」というのがよくわかりません。

privateメソッド内の変数はその中でしか使えませんからシリアル通信するごとに（privateメソッドを実行するごとに）インスタンスを作らないといけないと考えています。
publicだと、MainclassでSubclassのpublicインスタンスを作って、SubclassでMainclassのpublicインスタンスを作ると無限ループに入り例外になります。

http://www.atmarkit.co.jp/fdotnet/csharp_abc2/csabc2_003/cs2_003_01.html#cs0302
を参考に考えています。

ひょっとしてnamespaceに作るなんて出来るのでしょうか？（シリアルポートコンポーネントはMainclassで宣言したのでできないと思います）

2008/06/18(Wed) 16:37:38 編集(投稿者)

例えば、以下のようにするのではダメでしょうか？

> public partial class Mainclass : Form
> {
> ・・・
> Subclass sub = new Subclass();
> ・・・
sub.Owner = this;

> sub.show();
> }
>
> public partial class Subclass : Form
> {
> ・・・・
> private void button1_Click(object sender, EventArgs e) //シリアル通信をおこなう
> {
× Mainclass main = new Mainclass(); // notifyicon多重表示
○ Mainclass main = (Mainclass)this.Owner;
> ・・・
> }
> }

もしダメだとしたら何がダメでしょうか？
なぜSubclassでMainclassをnewしているのでしょう？
＃なぜMainclassをnewするの？という疑問点は既にJittaさんをはじめとして
＃指摘されていますが、再度繰り返します。

> （シリアルポートコンポーネントはMainclassで宣言したので...(略)

Mainclassに配置したシリアルポートコンポーネントというのを使用したいがためであれば、
そのシリアルポートコンポーネントをSubclassに配置すればいいように思えるのですが。

■No20861 (よねＫＥＮ さん) に返信
> 2008/06/18(Wed) 16:37:38 編集(投稿者)
> 
> 例えば、以下のようにするのではダメでしょうか？
> ～～～略
> もしダメだとしたら何がダメでしょうか？
> なぜSubclassでMainclassをnewしているのでしょう？
> ＃なぜMainclassをnewするの？という疑問点は既にJittaさんをはじめとして
> ＃指摘されていますが、再度繰り返します。

sub.show()しているMainclassのオブジェクトと
subclassのボタン押下時に生成しているMainclassのオブジェクトは
同一のもだと思われているように感じます。
もしくは、クラスをVBでいうところのモジュールと勘違い？

>>（シリアルポートコンポーネントはMainclassで宣言したので...(略)
> 
> Mainclassに配置したシリアルポートコンポーネントというのを使用したいがためであれば、
> そのシリアルポートコンポーネントをSubclassに配置すればいいように思えるのですが。
> 
私もこのほうがすっきりすると思います。

2008/06/19(Thu) 12:34:40 編集(投稿者)

■No20861 (よねＫＥＮさん) に返信

> もしダメだとしたら何がダメでしょうか？
提示してくださったコードでも動きましたし、処理が速くなったのでこちらでもいいと思います。

> なぜSubclassでMainclassをnewしているのでしょう？
クラス間で一つのコンポーネントを操作する場合（？）の方法としてインスタンスを作成する、しか知らないからです。　
Ownerというプロパティを教えてくれてありがとうございます

> Mainclassに配置したシリアルポートコンポーネントというのを使用したいがためであれば、
> そのシリアルポートコンポーネントをSubclassに配置すればいいように思えるのですが。

Mainclassでもシリアル通信を使うので出来れば一つのシリアルポートコンポーネントを使いまわしたいのです。

■No20890 (れれれさん) に返信
> 2008/06/19(Thu) 12:34:40 編集(投稿者)
>
>>なぜSubclassでMainclassをnewしているのでしょう？
> クラス間で一つのコンポーネントを操作する場合（？）の方法としてインスタンスを作成する、しか知らないからです。　
> Ownerというプロパティを教えてくれてありがとうございます

　まぁ、Tag プロパティでも、いけるんだけどね。

　Mainclass というのは、Form から派生しているとおり、Form なんですね。
で、インスタンスとクラスの違いというのは、オブジェクト指向で、必ずといってもいいほどつまずくところなのですが、
NotifyIcon がたくさん表示されてしまったように、「インスタンス」というのは「実体」です。
「クラス」は、実体にたいして「設計図」といえるでしょう。
new という演算子を使っていますが、英単語の「new」って、どういう意味ですか？「新しい」ですよね。
では、「new Mainclass()」とは、何をしているのでしょう？
設計図から、新しい実体を作っているのです。
ここで新しい実体を作っているので、NotifyIcon が多数表示されてしまいます。

　「使い回し」というのは、プログラムが作られたときに、
あなたがコード化しなくても、どこかで「new Mainclass()」が実行されているわけです。
実行されていないと、フォームが表示されるはずがありませんからね。
この、勝手に作られているフォームを使いましょう、ということです。

が、、、

>>Mainclassに配置したシリアルポートコンポーネントというのを使用したいがためであれば、
>>そのシリアルポートコンポーネントをSubclassに配置すればいいように思えるのですが。
>
> Mainclassでもシリアル通信を使うので出来れば一つのシリアルポートコンポーネントを使いまわしたいのです。

ということなら、Mainclass からシリアルポートを操作する処理を引っぺがしましょう。
そうすると、Subclass が Mainclass を知る必要がなくなります。

　っていうか。
　このスレッドを見ながら思ったのですが、普通は「これをしたい」と考え、「したい」事からコード化しますね。
オブジェクト指向ってば、それをいったん置いておいて、「したいことをさせるべきなのはなんなのか」を考えなきゃいけないんですね。
難しいわけだ。

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

ASP.NET、C++/CLI、Java 何でもどうぞ

C# と VB.NET の入門サイト

Re[5]: notifyicon多重表示回避

過去ログには書き込み不可