この書き方ってわかりづらくないですか？

1. この配列が何か不明。一度変数に入れてやれば変数名で表現できます。
2. 配列要素の想定型が不明。サフィックス省略なのでconst (&double)[n]だと思います。

解決したいのは 1 のほうで、auto使ってもいいので名前が欲しいです。

const auto xxx = {...};

意味合い的に constexpr ですね。

対応しました。
配列を使うと CodeSmell になってしまったため std::array としています。
(constexpr にしたいために std::vector ではなく std:array を選択しました)

	constexpr std::array<double, 72> aZoomFactors = {
		0.01, 0.011, 0.0125, 0.015, 0.0175, 0.02, 0.0225, 0.025, 0.0275, 0.03, 0.035, 0.04, 0.045, 0.05, 0.06, 0.07, 0.08, 0.09,

さっきの配列はinitializer_listでしたか。
うまく説明できませんが違和感があります。

initializer_list 型の引数には initializer_list しか渡せないこと、空ではない initializer_list を動的に生成することはできないことから、このコンストラクタに実行時に計算した値を指定することができなくなっています。

sanomari さんの違和感とは別件かもしれませんが、私が腑に落ちなかったのはこの点です。

開始/終了イテレータを受け取る形にしました。
※ユニットテスト用に std::initializer_list<double> を受け取るオーバーロードは残しました。

	template<class InputIter>
	ZoomSetting( InputIter iterZoomFactorsFirst, InputIter iterZoomFactorsLast, double nValueMin, double nValueMax, double nValueUnit ) :

うまく説明できませんが、コンストラクタに正当性確認があることに違和感があります。

validじゃないインスタンスは使えないんですよね？
PR差分ページを上から順番に見てるので、後続処理を意識せずに書いています。

私は正当性確認の必要性そのものに疑問を覚えました。

CViewCommander での実際の使用例を見ると、すべての引数が定数になっています。開発者が間違った値を指定していないか？という検査を行っていることになりますが、ここのコードは Release ビルドでも有効なので、全ユーザーの環境で実行されることになります。

開発時に確認する必要があるのは分かりますので、別の関数にして必要な時に呼んだ方がいいと思います。

正当性判定処理を関数化しました。今回はユニットテストから使用します。

/*!
	@brief 設定値の正当性判定
*/
bool ZoomSetting::IsValid() const

私は正当性確認の必要性そのものに疑問を覚えました。

CViewCommander での実際の使用例を見ると、すべての引数が定数になっています。開発者が間違った値を指定していないか？という検査を行っていることになりますが、ここのコードは Release ビルドでも有効なので、全ユーザーの環境で実行されることになります。

開発時に確認する必要があるのは分かりますので、別の関数にして必要な時に呼んだ方がいいと思います。

このクラス、メンバ変数を private にする意味あるかな？と思いました。コンストラクタで設定した値を公開するのが目的なら、public な const メンバ変数でも要求は満たせると思います。

対応しました。
用途や構成から struct 型が適切と思われたので見直しました。

struct 型が適切

struct型は private型を持つことができない特殊な class 型です。
この型をconst変数で宣言して使う分にはきっと問題ないですが、非const（≒書き替え可能）な変数で使うなら考慮が必要っす。（どっちのパターンかは見てない）

struct型は private型を持つことができない特殊な class 型です。

いえ、struct でも private 指定はできますよ。C++ の struct はデフォルトで public な class です。（class がデフォルトで private な struct であるとも言えます）

いえ、struct でも private 指定はできますよ。C++ の struct はデフォルトで public な class です。（class がデフォルトで private な struct であるとも言えます）

知らんかったです。
structでもprivateにできるんですね。

structとclassの違い

メモリレイアウトを公開するためにstructを使うんだと考えたら、
privateなメンバーを持たせてしまうと「何それ？」になります。
privateなメンバーは公開されないので、レイアウト公開してないじゃん、ってことになるので。

インターフェースを公開するためにclassを使うんだと考えたら、
ファンクタ（＝operator()を持つstruct）を見ると「何これ？」になります。

よくよく考えるとstructにもvirutal関数を定義できますが、
virtual関数を定義したら仮想関数テーブルがメモリレイアウトに組み込まれるので、
公開されないメンバーを追加することになりますね。

まぁ、色々ありまっせ、ということなのかな。。。

initializer_list 型の引数には initializer_list しか渡せないこと、空ではない initializer_list を動的に生成することはできないことから、このコンストラクタに実行時に計算した値を指定することができなくなっています。

sanomari さんの違和感とは別件かもしれませんが、私が腑に落ちなかったのはこの点です。

ここ多分こういうことですよね。

if( const double nZoomPercentage = m_pEditDoc->m_nCurrentZoom * 100.0;
    m_pEditDoc->m_blfCurTemp && nZoomPercentage < 5.0 ){
}else if( m_pEditDoc->m_blfCurTemp ){
}else{
    //nZoomPercentage = 100%
}

m_nCurrentZoom は一時設定の時に限り有効な値が設定されるので、else で「nZoomPercentage = 100%」とはならないんです。
CodeSmell 指摘についてはこう修正しておこうと思います。

    if( const double nZoomPercentage = m_pEditDoc->m_blfCurTemp ? (m_pEditDoc->m_nCurrentZoom * 100.0) : 100.0;
        nZoomPercentage < 5.0 ){
        auto_sprintf( szFontSize, LS( STR_STATUS_FONTZOOM_1 ), nZoomPercentage );
    }else{
        auto_sprintf( szFontSize, LS( STR_STATUS_FONTZOOM_0 ), nZoomPercentage );
    }

あと、auto_sprintf は auto_sprintf_s に差し替えておいた方がいいですね。

一晩寝て思ったんですけど

 if( const double nZoomPercentage = GetZoomPercentage(); nZoomPercentage < 5.0 ){

と書けるなら読みづらさは解消するのかもです。

m_nCurrentZoom は一時設定の時に限り有効な値が設定されるので、else で「nZoomPercentage = 100%」とはならないんです。

う～む。設計とコードがズレてるかも知れないです。

double nZoomPercentage = m_pEditDoc->m_blfCurTemp ? (m_pEditDoc->m_nCurrentZoom * 100.0) : 100.0;

m_pEditDoc->m_blfCurTemp が真(0以外)のとき、m_pEditDoc->m_nCurrentZoom の 100倍を設定。
m_pEditDoc->m_blfCurTemp が偽(0)のとき、固定値 100 を設定。

あと、コード全体をよく見れてないぼくが悪いのかも知らんですが、
zoomが5%未満のときだけ小数点以下1桁を表示するのはなんでかな？と思いました。

ほか、これは読み飛ばしてもらってかまいません。

• 拡大率って普通 ratio じゃね？ 😃
• 浮動小数点を表すハンガリアン記法って n ではなくて f とか d のような・・・。

if( const double nZoomPercentage = GetZoomPercentage(); nZoomPercentage < 5.0 ){
と書けるなら読みづらさは解消するのかもです。

確かにこれが良いですね。
m_blfCurTemp の値によって取得する値を選ぶ関数が既存で CEditWnd にいくつかあるので、ここに追加します。

zoomが5%未満のときだけ小数点以下1桁を表示するのはなんでかな？と思いました。

コードだけ見ると謎ですよね。PR 説明を更新して「仕様・動作説明」の所に理由を書きました。

浮動小数点を表すハンガリアン記法

特定のプリミティブ型に依存しない「数値」(Numeric) で n ということで良いかと思います。

m_blfCurTemp の値によって取得する値を選ぶ関数が既存で CEditWnd にいくつかあるので、ここに追加します。

これを対応するとまたコードカバレッジが 80% 切ってしまいそうですが大目に見て下さい😅

あれ？書式文字列はリソースに持っていいんですか？
既存リソースに含まれているのは置いておいて、
「書式記号を翻訳してはならない」というルールを説明できますか？

たとえば、フランス語は小数点にカンマ(,)を使うんですけど、
「'%4.1f %%'」を「'%4,1f %%'」と翻訳したらおかしなことになります。（落ちます）

安全にリソース化できないのでリソース化しない、が妥当な回避策だと思います。

心配点は理解できました。※小数点の件、確かドイツ語もそうですね
(誤って翻訳された場合) 今回のは起動したらすぐ落ちるので良い(？)ですが、たまにしか使われないリソースだと特にまずいですね。

書式文字列に対して翻訳が掛かっていないかどうかについては、何かしらチェックツールを用意して機械的に検知させるというのはどうでしょうか。

nNextIndex が範囲外なら抜けるはずなので対策不要では？

ループ毎に必ず nNextIndex が変化するため杞憂だったかもしれません。
ここは while( true ) に戻します。

ここは通常の for でインデックスを動かすよう見直しました。

ところで、この定数値の根拠ってなんでしたっけ？
（「話聞けよ！」ってなっていたらスンマセン…）

刻み方については #1491 (comment) で試作したものに近い変化率でかつ、パーセントで表示した時に切りの良い値としています。
最小・最大については、1pt から 72pt (7200%) へ、72pt から 1pt (1.38%) へとズームさせるのに十分な範囲としました。

ん？

指摘じゃありませんが、結局残存してるのを確認。

ここ拡張for（＝「要素：コレクション」と書いてコレクションに含まれる要素に繰り返し操作を行う、比較的新しい形式のfor文）に見えるので、コメントの「インデックスを動かしていく」のくだりが実装とあってない気がします。

どちらかというと、拡張forが使えるところに無限ループ書くやり方に違和感あります。
あと、コードをちゃんと見れてないですが maybe unusedが「ん？」って感じです。

ここは無限ループの書き方でなく通常の for でインデックス (nNextIndex) を進めるよう改めます。

struct 型が適切

struct型は private型を持つことができない特殊な class 型です。
この型をconst変数で宣言して使う分にはきっと問題ないですが、非const（≒書き替え可能）な変数で使うなら考慮が必要っす。（どっちのパターンかは見てない）

GetZoomedValue がズーム倍率を変更できない場合には false を返す仕様としていることから、ここは true になることがない if 文でしたので変数 nCurrentPointSize と合わせて削除しました。

ファイル末尾の行末改行、「無し」から「有り」に変更。
スルーしますけど戻してもスルーする気がします。

既存設計なのでどうしようもないですが、マジックナンバーやめたいっす。

このPRに含めなくてよいですが...

1. メンバーを変更しないので const にできます。
2. この関数の呼出し元で例外を考慮しなくてよいことを noexcept で示せます。

対応の必要はないですが👆でnoexceptを付けるならこうなるはずです。

やらんでも良いですが。

後で触る気マンマンですが、とりあえずこの対応で文句ありません。

もう1個のfiltersと同じです。

改行あるのとないのとどっちが正しいか分からないですが、これがテキストファイルだとすれば最終行にも行末記号を付けるべき（POSIX標準）らしいです。

Visual Studio で編集すると末尾改行は削除されるはずですが、その他のエディタでも編集することがあるのなら、.editorconfig に insert_final_newline = false を指定しておくのが良いかもしれません。

リベースでのコンフリクト修正時に使用したエディタ (vim) により自動的に追加されたものを見落としてしまいました。

.editorconfig に insert_final_newline = false を指定しておくのが良いかもしれません。

ありがとうございます。この設定で末尾改行を除去できるか試してみます。