--- nono/wx/wxstatuspanel.cpp	2026/04/29 17:05:16	1.1.1.6
+++ nono/wx/wxstatuspanel.cpp	2026/04/29 17:05:19	1.1.1.7
@@ -54,6 +54,9 @@ WXStatusPanel::WXStatusPanel(wxWindow *p
 {
 	timer.SetOwner(this);
 
+	power = GetPowerDevice();
+	syncer = GetSyncer();
+
 	// インジケータ用のパラメータを用意。
 	InitIndicators();
 
@@ -195,16 +198,149 @@ static const std::vector<int> data = {
 	0x69, 0x63, 0x20, 0x54, 0x72, 0x61, 0x6e, 0x63, 0x65, 0x72,
 };
 
+// store に value を代入。更新されていれば updated を立てる。
+// value は引数に渡す時に一度だけ評価される。
+#define MODIFY(store, value_)	({	\
+	bool mod_;	\
+	decltype(value_) value = value_;	\
+	if (store != value)	{ \
+		store = value;	\
+		mod_ = true;	\
+	} else {	\
+		mod_ = false;	\
+	}	\
+	mod_;	/* return value */	\
+})
+
+//
+// インジケータ用の情報
+//
+class Status
+{
+ public:
+	virtual ~Status() = default;
+
+	// 情報取得メソッド
+	virtual bool Poll() = 0;
+};
+
+//
+// SCSI 情報
+//
+class StatusSCSI : public Status
+{
+ public:
+	StatusSCSI(SCSIBus *scsibus_) {
+		scsibus = scsibus_;
+	}
+	virtual ~StatusSCSI() override = default;
+
+	bool Poll() override {
+		bool mod = false;
+		mod |= MODIFY(bsy, scsibus->GetBSY());
+		// 転送フェーズで DataOut の時だけ書き込みとみなす
+		bool out = (scsibus->GetPhase() == SCSI::Phase::Transfer)
+		        && (scsibus->GetXfer()  == SCSI::XferPhase::DataOut);
+		mod |= MODIFY(dataout, out);
+		return mod;
+	}
+
+	uint8 bsy {};		// BSY (ID0-7)
+	bool dataout {};	// DataOut フェーズなら true
+
+ private:
+	SCSIBus *scsibus {};
+};
+
+//
+// ネットワーク情報
+//
+class StatusNet : public Status
+{
+ public:
+	StatusNet(HostNetDevice *hostnet_) {
+		hostnet = hostnet_;
+	}
+	virtual ~StatusNet() override = default;
+
+	bool Poll() override {
+		// パケット数が変化したとき、および
+		// その変化を監視しているアクティブ状態が変化したときに
+		// 変更があったとしたいので二重になっている。
+		bool tx_mod = MODIFY(tx_pkts, hostnet->GetTXPkts());
+		bool rx_mod = MODIFY(rx_pkts, hostnet->GetRXPkts());
+
+		bool mod = false;
+		mod |= MODIFY(tx_active, tx_mod);
+		mod |= MODIFY(rx_active, rx_mod);
+		return mod;
+	}
+
+	bool enable {};
+	uint64 tx_pkts {};
+	uint64 rx_pkts {};
+	bool tx_active {};
+	bool rx_active {};
+
+ private:
+	HostNetDevice *hostnet {};
+};
+
+//
+// FDD 情報
+//
+class StatusFDD : public Status
+{
+ public:
+	StatusFDD(FDDDevice *fdd_) {
+		fdd = fdd_;
+	}
+	virtual ~StatusFDD() override = default;
+
+	bool Poll() override {
+		bool mod = false;
+		mod |= MODIFY(access_led, fdd->GetAccessLED());
+		mod |= MODIFY(eject_led,  fdd->GetEjectLED());
+		return mod;
+	}
+
+	FDDDevice::LED access_led {};	// アクセス LED の状態
+	bool eject_led {};				// イジェクト LED 点灯なら true
+ private:
+	FDDDevice *fdd {};
+};
+
+//
+// NEWS の LED 情報
+//
+class StatusNEWS : public Status
+{
+ public:
+	StatusNEWS(NewsCtlrDevice *newsctlr_) {
+		newsctlr = newsctlr_;
+	}
+	virtual ~StatusNEWS() override = default;
+
+	bool Poll() override {
+		return MODIFY(led, newsctlr->GetLED());
+	}
+
+	uint led {};
+ private:
+	NewsCtlrDevice *newsctlr {};
+};
+
+
 // インジケータの初期化
 void
 WXStatusPanel::InitIndicators()
 {
+	Status *stat;
 	Indicator *ind;
 
 	// パフォーマンスカウンタ
-	syncer = GetSyncer();
-	ind = new Indicator(Indicator::PERF, 8/* >>>9999% */,
-		&WXStatusPanel::DrawPerf);
+	ind = new Indicator(Indicator::PERF, &WXStatusPanel::DrawPerf,
+		8/* >>>9999% */, NULL);
 	ind->dclick = &WXStatusPanel::DClickPerf;
 	indicators.emplace_back(ind);
 
@@ -213,6 +349,9 @@ WXStatusPanel::InitIndicators()
 	// (ホストアダプタ2つ目のことは考えてない)
 	auto spc = gMainApp.FindObject<SPCDevice>(OBJ_SPC);
 	if (spc) {
+		stat = new StatusSCSI(spc->GetSCSIBus());
+		stats.emplace_back(stat);
+
 		const std::vector<SCSIDevice *>& conn = spc->GetConnectedDevices();
 		for (const auto& dev : conn) {
 			SCSI::DevType devtype = dev->GetDevType();
@@ -222,8 +361,8 @@ WXStatusPanel::InitIndicators()
 			int scsiid = dev->GetMyID();
 
 			// 文字列は OnSCSIMediaChanged() でセットする。
-			ind = new Indicator(Indicator::SCSI0 + scsiid, 3,
-				&WXStatusPanel::DrawSCSI);
+			ind = new Indicator(Indicator::SCSI0 + scsiid,
+				&WXStatusPanel::DrawSCSI, 3, stat);
 			ind->contextmenu = &WXStatusPanel::ContextMenuCD;
 			ind->dev = dev;
 			ind->panel = new wxPanel(this);
@@ -234,36 +373,45 @@ WXStatusPanel::InitIndicators()
 
 			indicators.emplace_back(ind);
 		}
-		// BSY 取得用
-		scsibus = spc->GetSCSIBus();
 	}
 
 	// ネットワーク
-	hostnet = gMainApp.FindObject<HostNetDevice>(OBJ_HOSTNET);
-	if (hostnet) {
-		if (hostnet->GetDriverName() == "none") {
-			net_ok = DISABLE;
-		} else {
-			net_ok = ENABLE;
+	for (int i = 0; i < 2; i++) {
+		auto hostnet = gMainApp.FindObject<HostNetDevice>(OBJ_HOSTNET(i));
+		if (hostnet) {
+			auto statnet = new StatusNet(hostnet);
+			stat = statnet;
+			stats.emplace_back(stat);
+
+			if (hostnet->GetDriverName() == "none") {
+				statnet->enable = false;
+			} else {
+				statnet->enable = true;
+			}
+			// \x1e は上向き矢印、\x1f は下向き矢印。
+			// ラベルは結局機種で変わる…
+			std::string label;
+			if (gMainApp.IsX68030()) {
+				label = string_format("LAN%d\x1e\x1f", i);
+			} else {
+				label = "LAN\x1e\x1f";
+			}
+			ind = new Indicator(Indicator::LAN0 + i, &WXStatusPanel::DrawNet,
+				label, stat);
+			indicators.emplace_back(ind);
 		}
-		// \x1e は上向き矢印、\x1f は下向き矢印
-		ind = new Indicator(Indicator::LAN, 5,
-			&WXStatusPanel::DrawNet, "LAN\x1e\x1f");
-		indicators.emplace_back(ind);
-	} else {
-		net_ok = NOT_AVAILABLE;
 	}
 
 	// FD
 	for (int unit = 0; unit < FDCDevice::MAX_DRIVE; unit++) {
 		auto fdd = gMainApp.FindObject<FDDDevice>(OBJ_FDD(unit));
 		if (fdd) {
-			// 存在するドライブのリスト
-			fdd_vector.push_back(fdd);
+			stat = new StatusFDD(fdd);
+			stats.emplace_back(stat);
 
 			// 文字列は OnFDDMediaChanged() でセットする。
-			ind = new Indicator(Indicator::FDD0 + unit, 7,
-				&WXStatusPanel::DrawFD);
+			ind = new Indicator(Indicator::FDD0 + unit,
+				&WXStatusPanel::DrawFD, 7, stat);
 
 			ind->contextmenu = &WXStatusPanel::ContextMenuFD;
 			ind->dev = fdd;
@@ -276,21 +424,23 @@ WXStatusPanel::InitIndicators()
 	}
 
 	// NEWS の LED (2->1 の順で並べる)
-	newsctlr = gMainApp.FindObject<NewsCtlrDevice>(OBJ_NEWSCTLR);
+	auto newsctlr = gMainApp.FindObject<NewsCtlrDevice>(OBJ_NEWSCTLR);
 	if (newsctlr) {
-		ind = new Indicator(Indicator::NEWSLED2, 2,
-			&WXStatusPanel::DrawNewsLED2, "LED2");
+		stat = new StatusNEWS(newsctlr);
+		stats.emplace_back(stat);
+
+		ind = new Indicator(Indicator::NEWSLED2, &WXStatusPanel::DrawNewsLED2,
+			"LED2", stat);
 		indicators.emplace_back(ind);
 
-		ind = new Indicator(Indicator::NEWSLED1, 2,
-			&WXStatusPanel::DrawNewsLED1, "LED1");
+		ind = new Indicator(Indicator::NEWSLED1, &WXStatusPanel::DrawNewsLED1,
+			"LED1", stat);
 		indicators.emplace_back(ind);
 	}
 
 	// 電源 LED
-	power = GetPowerDevice();
-	ind = new Indicator(Indicator::POWER, 5,
-		&WXStatusPanel::DrawPower, "POWER");
+	ind = new Indicator(Indicator::POWER, &WXStatusPanel::DrawPower,
+		"POWER", NULL);
 	indicators.emplace_back(ind);
 
 	// テスト用
@@ -378,58 +528,22 @@ WXStatusPanel::LayoutIndicators()
 bool
 WXStatusPanel::UpdateStat()
 {
-	bool updated = false;
+	bool mod = false;
 
-	// store に value を代入。更新されていれば updated を立てる。
-	// value は一度だけ評価される。
-	auto MODIFY = [&](auto& store, auto value) {
-		bool rv = store != value;
-		updated |= rv;
-		store = value;
-		return rv;
-	};
-
-	// 電源オンか
-	MODIFY(ispower, power->IsPower());
-	MODIFY(powerled, power->GetPowerLED());
+	// 電源オン
+	mod |= MODIFY(ispower, power->IsPower());
+	mod |= MODIFY(powerled, power->GetPowerLED());
 
 	// パフォーマンスカウンタ
-	MODIFY(perf_mode, syncer->GetRunMode());
-	MODIFY(perf_rate, syncer->GetPerfCounter());
-
-	// SCSI アクセス
-	if ((bool)scsibus) {
-		MODIFY(scsi_bsy, scsibus->GetBSY());
-		// 転送フェーズで DataOut の時だけ書き込みとみなす
-		bool out = (scsibus->GetPhase() == SCSI::Phase::Transfer)
-		        && (scsibus->GetXfer()  == SCSI::XferPhase::DataOut);
-		MODIFY(scsi_out, out);
-	}
+	mod |= MODIFY(perf_mode, syncer->GetRunMode());
+	mod |= MODIFY(perf_rate, syncer->GetPerfCounter());
 
-	// ネットワーク
-	// パケット数が変化したとき、
-	// およびその変化を監視しているアクティブ状態が変化したとき
-	// に変更があったとしたいので二重になっている
-	if (net_ok != NOT_AVAILABLE) {
-		MODIFY(net_tx_active, MODIFY(net_tx_pkts, hostnet->GetTXPkts()));
-		MODIFY(net_rx_active, MODIFY(net_rx_pkts, hostnet->GetRXPkts()));
+	// それ以外
+	for (const auto stat : stats) {
+		mod |= stat->Poll();
 	}
 
-	// FD
-	for (const auto fdd : fdd_vector) {
-		int unit = fdd->GetUnitNo();
-		MODIFY(fdstat[unit].access_led, fdd->GetAccessLED());
-		MODIFY(fdstat[unit].eject_led,  fdd->GetEjectLED());
-	}
-
-	// NEWS LED
-	if (newsctlr) {
-		MODIFY(newsled, newsctlr->GetLED());
-	}
-
-	// 電源
-
-	return updated;
+	return mod;
 }
 
 // 描画本体
@@ -510,6 +624,7 @@ WXStatusPanel::DrawPerf(const Indicator
 void
 WXStatusPanel::DrawSCSI(const Indicator *ind)
 {
+	const StatusSCSI *stat = dynamic_cast<const StatusSCSI*>(ind->stat);
 	Color fg;
 	Color bg;
 	int scsiid;
@@ -519,14 +634,14 @@ WXStatusPanel::DrawSCSI(const Indicator
 	assertmsg(scsiid >= 0, "id=%d", ind->id);
 	idmask = 1U << scsiid;
 
-	if ((scsi_loaded & idmask) != 0) {
+	if (scsi_loaded[scsiid]) {
 		fg = UD_BLACK;
 	} else {
 		fg = UD_GREY;
 	}
 
-	if ((scsi_bsy & idmask) != 0) {
-		if (scsi_out) {
+	if ((stat->bsy & idmask) != 0) {
+		if (stat->dataout) {
 			bg = UD_RED;
 		} else {
 			bg = UD_YELLOW_GREEN;
@@ -542,13 +657,14 @@ WXStatusPanel::DrawSCSI(const Indicator
 void
 WXStatusPanel::DrawNet(const Indicator *ind)
 {
+	const StatusNet *stat = dynamic_cast<const StatusNet*>(ind->stat);
+	const Rect& rect = ind->rect;
 	Color fg;
 	Color bg = BGPANEL;
-	const Rect& rect = ind->rect;
 
-	if (net_ok) {
+	if (stat->enable) {
 		fg = UD_BLACK;
-		if (net_tx_active || net_rx_active) {
+		if (stat->tx_active || stat->rx_active) {
 			bg = UD_YELLOW_GREEN;
 		}
 	} else {
@@ -566,14 +682,14 @@ WXStatusPanel::DrawNet(const Indicator *
 		auto c = ind->text[i];
 		switch (c) {
 		 case '\x1e':	// 上向き矢印
-			if (net_tx_active) {
+			if (stat->tx_active) {
 				fg = UD_BLACK;
 			} else {
 				fg = UD_GREY;
 			}
 			break;
 		 case '\x1f':	// 下向き矢印
-			if (net_rx_active) {
+			if (stat->rx_active) {
 				fg = UD_BLACK;
 			} else {
 				fg = UD_GREY;
@@ -591,20 +707,21 @@ WXStatusPanel::DrawNet(const Indicator *
 void
 WXStatusPanel::DrawFD(const Indicator *ind)
 {
+	const StatusFDD *stat = dynamic_cast<const StatusFDD*>(ind->stat);
+	const Rect& rect = ind->rect;
 	Rect iconrect;
 	Color pal[3];
+	int unit;
 	enum {
 		BG = 0,
 		FG,
 		IN,
 	};
-	int unit;
 
 	unit = ind->GetFDUnit();
-	const Rect& rect = ind->rect;
 
 	pal[BG] = BGPANEL;
-	if (fdstat[unit].loaded) {
+	if (fd_loaded[unit]) {
 		pal[FG] = UD_BLACK;
 	} else {
 		pal[FG] = UD_GREY;
@@ -621,7 +738,7 @@ WXStatusPanel::DrawFD(const Indicator *i
 	iconrect.h = font_height - 2;
 	bitmap.DrawRect(pal[FG], iconrect);
 	// ディスク挿入(取り出し可)なら黄緑点灯
-	if (fdstat[unit].eject_led) {
+	if (stat->eject_led) {
 		iconrect.x += 1;
 		iconrect.y += 1;
 		iconrect.w -= 2;
@@ -636,7 +753,7 @@ WXStatusPanel::DrawFD(const Indicator *i
 	// アクセス LED はディスク非挿入時に点滅するケースがあって、
 	// その場合、フチがグレーのまま中を黄緑で塗りつぶすのは見えづらいので
 	// 中を塗りつぶす時はフチを濃くしたほうがいい。
-	switch (fdstat[unit].access_led) {
+	switch (stat->access_led) {
 	 case FDDDevice::LED::GREEN:
 		pal[FG] = UD_BLACK;
 		pal[IN] = UD_YELLOW_GREEN;
@@ -718,9 +835,10 @@ WXStatusPanel::AccessMark24[] = {
 void
 WXStatusPanel::DrawNewsLED1(const Indicator *ind)
 {
+	const StatusNEWS *stat = dynamic_cast<const StatusNEWS*>(ind->stat);
 	Color bg;
 
-	if ((newsled & NewsCtlrDevice::LED_ORANGE) != 0) {
+	if ((stat->led & NewsCtlrDevice::LED_ORANGE) != 0) {
 		bg = UD_ORANGE;
 	} else {
 		bg = BGPANEL;
@@ -732,9 +850,10 @@ WXStatusPanel::DrawNewsLED1(const Indica
 void
 WXStatusPanel::DrawNewsLED2(const Indicator *ind)
 {
+	const StatusNEWS *stat = dynamic_cast<const StatusNEWS*>(ind->stat);
 	Color bg;
 
-	if ((newsled & NewsCtlrDevice::LED_GREEN) != 0) {
+	if ((stat->led & NewsCtlrDevice::LED_GREEN) != 0) {
 		bg = UD_GREEN;
 	} else {
 		bg = BGPANEL;
@@ -849,7 +968,7 @@ void
 WXStatusPanel::OnSCSIMediaChanged(wxCommandEvent& event)
 {
 	Indicator *ind = NULL;
-	uint8 new_loaded;
+	bool new_loaded;
 
 	// 該当の Indicator を探す (見付かるはず)
 	int scsiid = event.GetInt();
@@ -864,8 +983,7 @@ WXStatusPanel::OnSCSIMediaChanged(wxComm
 	const auto disk = dynamic_cast<const SCSIDisk *>(ind->dev);
 
 	// メディアの状態を一旦クリア
-	new_loaded = scsi_loaded;
-	new_loaded &= ~(1U << scsiid);
+	new_loaded = false;
 
 	// ツールチップを設定
 	SCSI::DevType devtype = disk->GetDevType();
@@ -877,7 +995,7 @@ WXStatusPanel::OnSCSIMediaChanged(wxComm
 		tip += "-";
 	} else {
 		tip += path;
-		new_loaded |= (1U << scsiid);
+		new_loaded = true;
 	}
 	switch (disk->GetWriteMode()) {
 	 case SCSIDisk::RW::ReadOnly:
@@ -899,7 +1017,7 @@ WXStatusPanel::OnSCSIMediaChanged(wxComm
 	// 無駄に幅を取らないためメディア挿入時のみ表示。
 	// MO のライトプロテクト表示は必要になったら考える…。
 	ind->text = string_format("%s%d", SCSI::GetDevTypeName(devtype), scsiid);
-	if ((new_loaded & (1U << scsiid))) {
+	if (new_loaded) {
 		if (disk->GetWriteMode() == SCSIDisk::RW::WriteIgnore) {
 			ind->text += "\x02\x03";
 		}
@@ -908,8 +1026,8 @@ WXStatusPanel::OnSCSIMediaChanged(wxComm
 	// メディアのロード状態が変われば (再配置して) 再描画。
 	// 通常周期のタイマーによる更新チェックではメディア状態はチェックして
 	// いないので、ここで自発的に行う。
-	if (scsi_loaded != new_loaded) {
-		scsi_loaded = new_loaded;
+	if (scsi_loaded[scsiid] != new_loaded) {
+		scsi_loaded[scsiid] = new_loaded;
 
 		LayoutIndicators();
 		Refresh();
@@ -972,8 +1090,8 @@ WXStatusPanel::OnFDDMediaChanged(wxComma
 	// メディアのロード状態が変われば (再配置して) 再描画。
 	// 通常周期のタイマーによる更新チェックではメディア状態はチェックして
 	// いないので、ここで自発的に行う。
-	if (fdstat[unit].loaded != new_loaded) {
-		fdstat[unit].loaded = new_loaded;
+	if (fd_loaded[unit] != new_loaded) {
+		fd_loaded[unit] = new_loaded;
 
 		LayoutIndicators();
 		Refresh();