脳log[20120701] CEditView::IsSearchStringという無茶な関数。

最終更新: 2012-07-01T14:27+0900

♪ [SakuraEditor] CEditView::IsSearchStringという無茶な関数。

• 検索条件を thisで与え、検索対象を引数で与える。(なぜ検索対象が可変？そのせいで対象となる文字列(連続したバッファ)が必要で、将来的に複数行検索に対応できない)
• 仕様(出力)が検索オプションで変わる。
• 検索コードを呼ぶのでなく自分でマッチ判定をしてる。(コピペコード)
• 検索語ハイライトを行う CColor_Foundからしか呼ばれていない。
• (関数の目的から)一文字一文字呼ばれることを想定しているが、検索情報を蓄えたりせず(できず)、その場その場で判断している(無駄が多い)。CColor_Foundの方が、行が変わるごとに通知があり行末に向かって順次実行されることがわかっているぶん戦略的に動ける。

というわけで、CEditView::IsSearchStringからコピペコードを廃すると CColor_Foundが知りたがっている何番目の単語がマッチしたのかを知るためにさらなる検索が必要になるにもかかわらず、CEditView::IsSearchStringという変態のために仕様を決める必要は全然ない気がしてきた。仕様ってのはこれの→「sakura-unicode:1830」「複数単語検索のブックマーク・Grep対応 - ID: 3539115」。CEditView::IsSearchStringには消えてもらって CColor_Foundと CSearchAgent::SearchWordの組み合わせで何とかすべき問題ではないかと。SearchWordだと検索結果が改行をまたぐようになってもそのままで適応する利点もある(CColor_Foundと CColorStrategyの方には対処が必要だが)。