<h2 id="%E3%81%AF%E3%81%98%E3%82%81%E3%81%AB" data-line="0" class="code-line">
<a class="header-anchor-link" href="#%E3%81%AF%E3%81%98%E3%82%81%E3%81%AB" aria-hidden="true"></a> はじめに</h2>
<p data-line="2" class="code-line">こんにちは、りゅうです！先日、会社の先輩方と「逆リファクタリングバトル」というイベントに参加しました。これは、あえてよくない設計のコードを書くことで、きれいなコードを書くことの大切さを学ぶというもです。</p>
<p data-line="4" class="code-line">イベント当日は、PCを持ってくるのを忘れるという大失態を犯し、劣悪な環境での開発を余儀なくされました。「クソコードはクソ開発環境から生まれる？」という言葉が、まさにぴったり当てはまる状況でしたね。この辺りの詳しい話は別の記事で！</p>
<span class="embed-block zenn-embedded zenn-embedded-card"><iframe id="zenn-embedded__ef8286951922e" src="https://embed.zenn.studio/card#zenn-embedded__ef8286951922e" data-content="https%3A%2F%2Fzenn.dev%2Foku_derson%2Farticles%2F42c561fff25127" frameborder="0" scrolling="no" loading="lazy"></iframe></span>
<p data-line="7" class="code-line">他の参加者の作品はこちらから！主催の方がまとめてくれています！</p>
<span class="embed-block zenn-embedded zenn-embedded-card"><iframe id="zenn-embedded__c5f8562223682" src="https://embed.zenn.studio/card#zenn-embedded__c5f8562223682" data-content="https%3A%2F%2Fzenn.dev%2Fsmartcamp%2Farticles%2Feac471f2304863" frameborder="0" scrolling="no" loading="lazy"></iframe></span>
<h2 id="%E5%83%95%E3%81%8C%E3%82%84%E3%81%A3%E3%81%9F%E9%80%86%E3%83%AA%E3%83%95%E3%82%A1%E3%82%AF%E3%82%BF%E3%83%AA%E3%83%B3%E3%82%B0" data-line="11" class="code-line">
<a class="header-anchor-link" href="#%E5%83%95%E3%81%8C%E3%82%84%E3%81%A3%E3%81%9F%E9%80%86%E3%83%AA%E3%83%95%E3%82%A1%E3%82%AF%E3%82%BF%E3%83%AA%E3%83%B3%E3%82%B0" aria-hidden="true"></a> 僕がやった逆リファクタリング</h2>
<p data-line="13" class="code-line">ざっくり、自分がやった逆リファクタリングの内容は以下の通りです。</p>
<ul data-line="15" class="code-line">
<li data-line="15" class="code-line">変数名を読みにくくする（これはデフォルト装備）
<ul data-line="16" class="code-line">
<li data-line="16" class="code-line">ローマ字を使う</li>
<li data-line="17" class="code-line">ローマ字と英語を混在させる</li>
<li data-line="18" class="code-line">一貫性のない命名方法を使う</li>
</ul>
</li>
<li data-line="19" class="code-line">無駄な例外を定義したり継承したりする
<ul data-line="20" class="code-line">
<li data-line="20" class="code-line">一貫性のない継承構造を作る</li>
</ul>
</li>
<li data-line="21" class="code-line">割り算をわざわざコンピュータが行う低級言語の演算方法に書き換える（残念ながら時間内にはできませんでした...）</li>
</ul>
<h2 id="%E5%A4%89%E6%95%B0%E5%90%8D%E3%82%92%E3%82%8F%E3%81%8B%E3%82%8A%E3%81%AB%E3%81%8F%E3%81%8F%E3%81%99%E3%82%8B" data-line="23" class="code-line">
<a class="header-anchor-link" href="#%E5%A4%89%E6%95%B0%E5%90%8D%E3%82%92%E3%82%8F%E3%81%8B%E3%82%8A%E3%81%AB%E3%81%8F%E3%81%8F%E3%81%99%E3%82%8B" aria-hidden="true"></a> 変数名をわかりにくくする</h2>
<p data-line="25" class="code-line">やはり命名規則を守ること、そして一貫した命名方法を保つことはとても大事です。開発中は自分でさえ、何を書いているかわからなくなっていました。良くないですね！</p>
<p data-line="27" class="code-line">また、クラスなのかメソッドなのかが一切わからなくなる点も、非常に読みにくくしています。</p>
<h3 id="%E6%B1%9A%E3%81%97%E3%83%9D%E3%82%A4%E3%83%B3%E3%83%88" data-line="29" class="code-line">
<a class="header-anchor-link" href="#%E6%B1%9A%E3%81%97%E3%83%9D%E3%82%A4%E3%83%B3%E3%83%88" aria-hidden="true"></a> 汚しポイント</h3>
<ul data-line="31" class="code-line">
<li data-line="31" class="code-line">ローマ字で書いたり</li>
<li data-line="32" class="code-line">例外をローマ字で <code>reigai</code> と書いたかと思えば、次は英語で <code>exception</code> と書いてみたり</li>
</ul>
<p data-line="34" class="code-line"><span class="embed-block zenn-embedded zenn-embedded-github"><iframe id="zenn-embedded__8ff4cea063db5" src="https://embed.zenn.studio/github#zenn-embedded__8ff4cea063db5" data-content="https%3A%2F%2Fgithub.com%2Fsalvage0707%2Frefactoring-reversal-battle%2Fblob%2Fe3e555619c4620ea51ede7df1f3c9c251cc35877%2Fsubmissions%2Fmeteor-submission%2Fruby%2Fsrc%2Ffizz_buzz.rb%23L15-L18" frameborder="0" scrolling="no" loading="lazy"></iframe></span><a href="https://github.com/salvage0707/refactoring-reversal-battle/blob/e3e555619c4620ea51ede7df1f3c9c251cc35877/submissions/meteor-submission/ruby/src/fizz_buzz.rb#L15-L18" style="display:none" target="_blank" rel="nofollow noopener noreferrer">https://github.com/salvage0707/refactoring-reversal-battle/blob/e3e555619c4620ea51ede7df1f3c9c251cc35877/submissions/meteor-submission/ruby/src/fizz_buzz.rb#L15-L18</a></p>
<h3 id="%E6%84%8F%E5%A4%96%E3%81%AA%E7%99%BA%E8%A6%8B" data-line="36" class="code-line">
<a class="header-anchor-link" href="#%E6%84%8F%E5%A4%96%E3%81%AA%E7%99%BA%E8%A6%8B" aria-hidden="true"></a> 意外な発見</h3>
<ul data-line="38" class="code-line">
<li data-line="38" class="code-line">僕たちエンジニアは、結構無意識に命名をわかりやすくしているらしい
<ul data-line="39" class="code-line">
<li data-line="39" class="code-line">キャメルケースやスネークケースを混在させれば、さらにわかりにくくできたはずですが、実装しているときはそんな発想がありませんでした</li>
</ul>
</li>
</ul>
<h2 id="%E7%84%A1%E9%A7%84%E3%81%AB%E4%BE%8B%E5%A4%96%E3%82%AF%E3%83%A9%E3%82%B9%E3%82%92%E5%AE%9A%E7%BE%A9%E3%81%97%E3%80%81%E7%B6%99%E6%89%BF%E3%81%99%E3%82%8B" data-line="41" class="code-line">
<a class="header-anchor-link" href="#%E7%84%A1%E9%A7%84%E3%81%AB%E4%BE%8B%E5%A4%96%E3%82%AF%E3%83%A9%E3%82%B9%E3%82%92%E5%AE%9A%E7%BE%A9%E3%81%97%E3%80%81%E7%B6%99%E6%89%BF%E3%81%99%E3%82%8B" aria-hidden="true"></a> 無駄に例外クラスを定義し、継承する</h2>
<p data-line="43" class="code-line">次に、こちらをご覧ください。最終的にはすべて <code>ArgumentError</code> を発生させるにもかかわらず、わざわざカスタム例外クラスを作成し、個別に例外をキャッチしています。</p>
<p data-line="45" class="code-line">さらに、カスタム例外を最適化するわけでもなく、あるものはベースの例外クラスを使い、あるものは派生後の例外クラスを使う。かなりの愚行ですし、無駄に読みにくいだけですね。</p>
<p data-line="47" class="code-line"><img src="https://storage.googleapis.com/zenn-user-upload/72b8ff09b276-20251122.png" alt="" class="md-img" loading="lazy"><br style="display:none">
<span class="embed-block zenn-embedded zenn-embedded-github"><iframe id="zenn-embedded__d11e1bce58a97" src="https://embed.zenn.studio/github#zenn-embedded__d11e1bce58a97" data-content="https%3A%2F%2Fgithub.com%2Fsalvage0707%2Frefactoring-reversal-battle%2Fblob%2Fe3e555619c4620ea51ede7df1f3c9c251cc35877%2Fsubmissions%2Fmeteor-submission%2Fruby%2Fsrc%2Ffizz_buzz.rb%23L13-L31" frameborder="0" scrolling="no" loading="lazy"></iframe></span><a href="https://github.com/salvage0707/refactoring-reversal-battle/blob/e3e555619c4620ea51ede7df1f3c9c251cc35877/submissions/meteor-submission/ruby/src/fizz_buzz.rb#L13-L31" style="display:none" target="_blank" rel="nofollow noopener noreferrer">https://github.com/salvage0707/refactoring-reversal-battle/blob/e3e555619c4620ea51ede7df1f3c9c251cc35877/submissions/meteor-submission/ruby/src/fizz_buzz.rb#L13-L31</a></p>
<h3 id="%E6%B1%9A%E3%81%97%E3%83%9D%E3%82%A4%E3%83%B3%E3%83%88-1" data-line="50" class="code-line">
<a class="header-anchor-link" href="#%E6%B1%9A%E3%81%97%E3%83%9D%E3%82%A4%E3%83%B3%E3%83%88-1" aria-hidden="true"></a> 汚しポイント</h3>
<ul data-line="52" class="code-line">
<li data-line="52" class="code-line">無駄なカスタム例外クラス（上の画像参照）</li>
<li data-line="53" class="code-line">一貫性のない継承クラス
<ul data-line="54" class="code-line">
<li data-line="54" class="code-line">「整数」という括りで <code>n_1ijou</code> は <code>n_seisu</code> を継承していますが、この考えでいくと、上位クラスになるべき <code>suuti_no_exception</code> と <code>n_seisu</code> に親子関係がないのです。命名が分かりにくいせいで説明が大変ですが、継承関係を持つのであれば、以下のようになるべきです（正直、継承自体が必要ないのですが...）</li>
<li data-line="55" class="code-line">
<div class="code-block-container"><pre><code class="code-line" data-line="55">"kihon_no_reigai" &lt;- "suuti_no_exception" &lt;- "n_seisu" &lt;- "n_1ijou"
</code></pre></div>
</li>
</ul>
</li>
</ul>
<p data-line="58" class="code-line"><span class="embed-block zenn-embedded zenn-embedded-github"><iframe id="zenn-embedded__9e654b811f635" src="https://embed.zenn.studio/github#zenn-embedded__9e654b811f635" data-content="https%3A%2F%2Fgithub.com%2Fsalvage0707%2Frefactoring-reversal-battle%2Fblob%2Fe3e555619c4620ea51ede7df1f3c9c251cc35877%2Fsubmissions%2Fmeteor-submission%2Fruby%2Fsrc%2Ffizz_buzz_exception%2Fn_1ijou%2Fn_1ijou.rb%23L1-L8" frameborder="0" scrolling="no" loading="lazy"></iframe></span><a href="https://github.com/salvage0707/refactoring-reversal-battle/blob/e3e555619c4620ea51ede7df1f3c9c251cc35877/submissions/meteor-submission/ruby/src/fizz_buzz_exception/n_1ijou/n_1ijou.rb#L1-L8" style="display:none" target="_blank" rel="nofollow noopener noreferrer">https://github.com/salvage0707/refactoring-reversal-battle/blob/e3e555619c4620ea51ede7df1f3c9c251cc35877/submissions/meteor-submission/ruby/src/fizz_buzz_exception/n_1ijou/n_1ijou.rb#L1-L8</a></p>
<ul data-line="60" class="code-line">
<li data-line="60" class="code-line">なぜ基本の例外クラスをそのまま使っている？<br>
<a href="https://github.com/salvage0707/refactoring-reversal-battle/blob/e3e555619c4620ea51ede7df1f3c9c251cc35877/submissions/meteor-submission/ruby/src/fizz_buzz.rb#L15" target="_blank" rel="nofollow noopener noreferrer">https://github.com/salvage0707/refactoring-reversal-battle/blob/e3e555619c4620ea51ede7df1f3c9c251cc35877/submissions/meteor-submission/ruby/src/fizz_buzz.rb#L15</a>
</li>
<li data-line="62" class="code-line">このメッセージ設定は本当に必要だった？<br>
<a href="https://github.com/salvage0707/refactoring-reversal-battle/blob/e3e555619c4620ea51ede7df1f3c9c251cc35877/submissions/meteor-submission/ruby/src/fizz_buzz_exception/kihon_no_reigai.rb#L1-L2" target="_blank" rel="nofollow noopener noreferrer">https://github.com/salvage0707/refactoring-reversal-battle/blob/e3e555619c4620ea51ede7df1f3c9c251cc35877/submissions/meteor-submission/ruby/src/fizz_buzz_exception/kihon_no_reigai.rb#L1-L2</a>
</li>
</ul>
<h2 id="%E5%89%B2%E3%82%8A%E7%AE%97%E3%82%92%E3%83%93%E3%83%83%E3%83%88%E6%BC%94%E7%AE%97%E3%81%A7%E3%82%84%E3%81%A3%E3%81%A6%E3%81%BF%E3%82%8B%EF%BC%88%E6%99%82%E9%96%93%E5%86%85%E3%81%AB%E5%AE%8C%E6%88%90%E3%81%97%E3%81%BE%E3%81%9B%E3%82%93%E3%81%A7%E3%81%97%E3%81%9F...%EF%BC%89" data-line="65" class="code-line">
<a class="header-anchor-link" href="#%E5%89%B2%E3%82%8A%E7%AE%97%E3%82%92%E3%83%93%E3%83%83%E3%83%88%E6%BC%94%E7%AE%97%E3%81%A7%E3%82%84%E3%81%A3%E3%81%A6%E3%81%BF%E3%82%8B%EF%BC%88%E6%99%82%E9%96%93%E5%86%85%E3%81%AB%E5%AE%8C%E6%88%90%E3%81%97%E3%81%BE%E3%81%9B%E3%82%93%E3%81%A7%E3%81%97%E3%81%9F...%EF%BC%89" aria-hidden="true"></a> 割り算をビット演算でやってみる（時間内に完成しませんでした...）</h2>
<p data-line="67" class="code-line">自分の開発環境では、例外クラス関係の汚しだけで結構な時間を使ってしまい、残り時間で実装したい汚しとして、あえてコンピュータアーキテクチャレベルの算術演算をわざわざ使うというアプローチを試みました。</p>
<p data-line="69" class="code-line">残念ながら時間内には完成しませんでした。劣悪な開発環境ではデバッグが難しく、期間内には直しきれませんでした。（後日完成させたので、別の機会に完成系は紹介します）</p>
<p data-line="71" class="code-line">流石に、プログラミング言語が抽象化しているものを手動で実装するのは愚行だなと思いつつ、知的好奇心と挑戦心でやってみました！</p>
<p data-line="73" class="code-line">ちなみに実装の参考にしたのは、世界で最も有名なコンピュータの教科書「パターソン＆ヘネシー『コンピュータの構成と設計』第5版 上巻」です。<br>
<img src="https://storage.googleapis.com/zenn-user-upload/54af06f5720f-20251122.jpg" alt="" class="md-img" loading="lazy"></p>
<h3 id="%E3%81%93%E3%81%AE%E6%99%82%E7%82%B9%E3%81%A7%E3%81%AE%E4%BD%9C%E6%88%90%E7%B5%90%E6%9E%9C" data-line="76" class="code-line">
<a class="header-anchor-link" href="#%E3%81%93%E3%81%AE%E6%99%82%E7%82%B9%E3%81%A7%E3%81%AE%E4%BD%9C%E6%88%90%E7%B5%90%E6%9E%9C" aria-hidden="true"></a> この時点での作成結果</h3>
<p data-line="78" class="code-line"><span class="embed-block zenn-embedded zenn-embedded-github"><iframe id="zenn-embedded__3b6b0ee820f27" src="https://embed.zenn.studio/github#zenn-embedded__3b6b0ee820f27" data-content="https%3A%2F%2Fgithub.com%2Fsalvage0707%2Frefactoring-reversal-battle%2Fblob%2Fe3e555619c4620ea51ede7df1f3c9c251cc35877%2Fsubmissions%2Fmeteor-submission%2Fruby%2Fsrc%2Fbit_warizan.rb%23L1-L21" frameborder="0" scrolling="no" loading="lazy"></iframe></span><a href="https://github.com/salvage0707/refactoring-reversal-battle/blob/e3e555619c4620ea51ede7df1f3c9c251cc35877/submissions/meteor-submission/ruby/src/bit_warizan.rb#L1-L21" style="display:none" target="_blank" rel="nofollow noopener noreferrer">https://github.com/salvage0707/refactoring-reversal-battle/blob/e3e555619c4620ea51ede7df1f3c9c251cc35877/submissions/meteor-submission/ruby/src/bit_warizan.rb#L1-L21</a></p>
<h2 id="%E3%81%BE%E3%81%A8%E3%82%81" data-line="80" class="code-line">
<a class="header-anchor-link" href="#%E3%81%BE%E3%81%A8%E3%82%81" aria-hidden="true"></a> まとめ</h2>
<p data-line="82" class="code-line">今回、逆リファクタリングバトルに参加して気づいたのは、持っている知識や引き出しによって「汚し方」が変わるということです。ある程度の知識がないと、効果的に悪いコードさえ書けないというのが大きな発見でした！</p>
<p data-line="84" class="code-line">アンチパターンの学習の重要性について、改めて認識できた会でした。</p>


【逆リファクタリングバトル】あえて、FizzBuzzを汚してみる

僕がやった逆リファクタリング

無駄に例外クラスを定義し、継承する

割り算をビット演算でやってみる（時間内に完成しませんでした...）

Discussion