ミニマップの実装クラスを分割する #1748
Merged
ミニマップの実装クラスを分割する #1748
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
SonarCloud Quality Gate failed. |
|
✅ Build sakura 1.0.3964 completed (commit 65695e1bae by @sanomari) |
berryzplus
approved these changes
Oct 20, 2021
| @@ -0,0 +1,36 @@ | |||
| /*! @file */ | |||
| /* | |||
| Copyright (C) 2012, Moca | |||
|
|
||
| 編集ビューのコードを流用して縮小ビューを表示する | ||
| */ | ||
| class CMiniMapView : public CEditView |
There was a problem hiding this comment.
本当はCEditViewとCMiniMapViewの最小公約数的なクラスを抽出してやるといいんですけどね。
いったんこれでよいかと思います。
| if( NULL == GetMiniMap().GetHwnd() ){ | ||
| GetMiniMap().Create( GetHwnd(), GetDocument(), -1, FALSE, true ); | ||
| if( !m_cMiniMapView.GetHwnd() ){ | ||
| m_cMiniMapView.Create( GetHwnd() ); |
There was a problem hiding this comment.
他と合わせてるんだと思いますが、Createメソッドって「なかったら作る」じゃないですかね？
呼ぶ前にif文挟むの無駄じゃね？という指摘。（対応は任意。
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR の目的
ミニマップの実装クラスを新しく定義することにより、クラスの分割に着手することが目的です。
カテゴリ
PR の背景
ウインドウ処理を見直す試みをしている中で、
ミニマップ表示のために編集ビューのコードが複雑になっていることに気付きました。
PR のメリット
PR のデメリット (トレードオフとかあれば)
仕様・動作説明
アプリの仕様・機能には影響を与えない変更だと思います。
このPRでは、定義クラスを別ファイルにするだけで、分割はしません。
PR の影響範囲
テスト内容
テスト1
手順
関連 issue, PR
参考資料