<p>今回は自分のプルリクエストを自分でレビューをするメリットについて話をしたいと思います。</p>
<h2 id="セルフレビューとは">セルフレビューとは</h2>
<p>自分が作成したプルリクエストに対して自分でレビューをすることをここではセルフレビューと呼んでいます。</p>
<p>自分は複雑な実装や大きめのプルリクエストを作成した場合は、レビューをお願いする前に自分のプルリクエストを自分でレビューするといった事を偶に実施しています。</p>
<h2 id="セルフレビューのメリット">セルフレビューのメリット</h2>
<ul>
<li>非効率な実装、無駄な処理、消し忘れのデバッグコード、コメントを書いておくべき箇所を見つけられる</li>
<li>レビュー時に他の人が自分のコードをどう見えているか客観的に把握できる</li>
<li>レビューで細かな点を指摘される事が減り、レビュアーの負担が軽減される</li>
<li>細かい指摘が減ることでレビュー事態のコミュニケーション量が減るので、レビュイーの負担も軽減される</li>
</ul>
<p>開発に集中をしていると視野が狭くなる事があるので、意外と実装後にプルリクエストを見ながら全体を俯瞰してみると、もっと効率的な書き方や読みやすい実装などに気づくことが多々あります。開発中は知識が全て頭に入っていたので理解していたコードも後で見返すと非常に理解しにくいなと感じる実装が見つかることもあり、コメントを書いておくべき箇所が見つかることもあります。</p>
<p>客観的にセルフレビューをすることで、レビュアーの人が疑問に感じてコメントを書きそうな場所も見えてくるので、その場合は事前に説明コメントを書いておくことで、レビュアーの負担を減らし全体のレビューにおけるコミュニケーション量を軽減できるので結果として全体のレビューの負担も減らすことができます。</p>
<p>このようにセルフレビューは自分の為だけでなく、レビュアーの人の為にもなります。<br>
レビューをお願いする時は、<strong>如何にレビュアーの人の負担を減らすか</strong>を考えるのは非常に大切です。</p>
<h2 id="セルフレビュー駆動開発">セルフレビュー駆動開発</h2>
<p>最初に動くコードを実装してプルリクエストを作成し、セルフレビューを実施してコメントを残して、そのコメントに対してコードを修正して改善を進めることで、セルフレビューによる駆動開発をすることもできます。</p>
<p>個人開発など誰かにレビューお願いしたいけど難しいパターンでは、このスタイルを試してみると意外と何もやらない時よりも綺麗なコード実装ができます。</p>


今回は自分のプルリクエストを自分でレビューをするメリットについて話をしたいと思います。

プルリクエストのセルフレビューのススメ

<h2 id="問題">問題</h2>
<p>自分は普段 Emacs キーバインドを利用しているのですが、Alfred の検索結果でカーソル移動をするために Ctrl + N をタイプするとカーソル移動せずにパッケージが選択される問題に悩まされていました。</p>
<img src="/images/posts/alfred-ctrl-n/alfred-ctrl-n-bad.gif" alt="ctrl+nでパッケージ選択される" width="840" height="572" />
<h2 id="解決策">解決策</h2>
<p>Alfredの設定で Features &gt; Actions &gt; Show Actions で ctrl のチェックボックスを外せば問題は解消しました。</p>
<img src="/images/posts/alfred-ctrl-n/alfred-settings.jpeg" alt="Alfredの設定画面" width="2280" height="826" />


Alfred で Ctrl + N でパッケージ選択できない問題を解消

Alfred で Ctrl + N でカーソル移動ができない問題を解消

<p><a href="https://github.com/alievk/avatarify" target="_blank">alievk/avatarify: Avatars for Zoom and Skype</a></p>
<p>この記事は MacOS でのセットアップ方法を記述しています。<br>
環境でセットアップ手順が異なるので、LinuxやWindowsの場合はリポジトリのREADME.mdを参照してください。</p>
<p>【環境】<br>
OS: MacOS Catalina 10.15.3<br>
端末: MacBook Pro 2017<br>
プロセッサ: 2.3 GHz デュアルコアIntel Core i5<br>
メモリ: 16 GB 2133 MHz LPDDR3<br>
グラッフィックス: Intel Iris Plus Graphics 640 1536 MB<br>
GPU: なし</p>
<h2 id="インストール">インストール</h2>
<h3 id="miniconda3 をインストール">miniconda3 をインストール</h3>
<p><a href="https://docs.conda.io/projects/conda/en/latest/user-guide/install/macos.html#installing-on-macos" target="_blank">Installing on macOS — conda 4.8.3.post14+07a113d8 documentation</a></p>
<p><a href="https://docs.conda.io/en/latest/miniconda.html#macosx-installers" target="_blank">Miniconda — Conda documentation</a> よりインストーラーをダウンロードする。</p>
<pre class="language-bash"><code class="language-bash"><span class="token comment"># 対話シェルが起動するので、内容に沿ってインストールを進める。</span>
$ <span class="token function">zsh</span> Miniconda3-latest-MacOSX-x86_64.sh
$ <span class="token builtin class-name">export</span> <span class="token assign-left variable"><span class="token environment constant">PATH</span></span><span class="token operator">=</span><span class="token string">"<span class="token environment constant">$HOME</span>/miniconda3/bin:<span class="token environment constant">$PATH</span>"</span>
</code></pre>
<h3 id="avatarify のリポジトリをクローンしてインストール">avatarify のリポジトリをクローンしてインストール</h3>
<pre class="language-bash"><code class="language-bash">$ <span class="token function">git</span> clone https://github.com/alievk/avatarify.git
$ <span class="token builtin class-name">cd</span> avatarify
$ <span class="token function">zsh</span> scripts/install_mac.sh
$ conda activate avatarify
</code></pre>
<h3 id="学習済みモデル vox\-adv\-cpk\.pth\.tar をダウンロード">学習済みモデル vox-adv-cpk.pth.tar をダウンロード</h3>
<p><a href="https://drive.google.com/file/d/1L8P-hpBhZi8Q_1vP2KlQ4N6dvlzpYBvZ/view" target="_blank">Google ドライブ</a>からダウンロードして avatarify ディレクトリ配置する。<br>
※ tarファイルは解凍せずにそのまま配置する</p>
<h3 id="CamTwist をダウンロード">CamTwist をダウンロード</h3>
<p><a href="http://camtwiststudio.com/download/" target="_blank">ここ</a>からダウンロードする</p>
<h2 id="実行">実行</h2>
<h3 id="avatify を起動">avatify を起動</h3>
<p>&quot;Cannot open camera&quot; のエラーが発生したら、scripts/settings.sh の CAMID を 0 から 1, 2, ... に変更する。<br>
自分の場合は 1 に変更したら、正常に起動しました。</p>
<pre class="language-bash"><code class="language-bash">$ <span class="token function">zsh</span> run_mac.sh
</code></pre>
<h3 id="CamTwistを起動">CamTwistを起動</h3>
<p>2.1 「Desktop+」 を選んで Select をクリック<br>
2.2 Settingsの項目内で 「Confine to Application Window」を選択してセレクトボックスで「Python(avatarify)」を選ぶ</p>
<p>アバターが表示されたウィンドウで 1, 2, ... と数字キーを押せばアバターの切替えができます。</p>
<h3 id="Zoomdでバーチャルカメラを利用できるようにする">Zoomdでバーチャルカメラを利用できるようにする</h3>
<p>Mac の Zoom v4.6.8 以降のバージョンでは仮想カメラが使えないようになっているので、ライブラリ署名を削除して仮想カメラを利用できるようにする。</p>
<pre class="language-bash"><code class="language-bash">$ codesign --remove-signature /Applications/zoom.us.app
</code></pre>
<h3 id="Zoomを起動">Zoomを起動</h3>
<p>4.1 Zoom で 設定 &gt; カメラ &gt; CamTwist を選択</p>
<img src="/images/posts/avatarify/zoom.gif" alt="zoom" width="1266" height="756" />
<p>GPUが無いと ~1FPS しかパフォーマンスが出ないので、動いてはいるのですが全く楽しめないです。😢</p>


Zoomでアインシュタインになってみた

神奈川の田舎で働いてるフロントエンドエンジニアのブログ