Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

レビューお願いします! #10

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

takaho9
Copy link

@takaho9 takaho9 commented Oct 18, 2024

コードのアピールポイント(感想含む)

  • 4*10n, 9*10nを表すローマ数字をROMAN_WEIGHTハッシュに含めてしまいました。
    • ローマ数字が4000までしか表現できず、該当するものが6個しかないため
  • なるべく条件分岐が少なくなるようにしました。

コードを書くのにかかった時間(ざっくりでよいです。10分、1時間、3日、etc.)

たぶん3時間くらいかかってると思います。

だいたいのプログラミング歴

学生ですが、アルバイトで1年くらいコード書いてます。
プログラミングし始めたのは、5年くらい前です。

実際に解いてみた感想

  • ちょっと考えては諦めてを繰り返して、調子いい時に書いたらサクッとかけました。寝かすの大事ですね。
  • 特に考えずに書き始めると、意外と考慮できていないことが多くて、何度もテスト落ちました...
  • できてしまうと、簡単な規則に感じますが、できるまではとても難しく感じました。

伊藤さんにメッセージ

チェリー本とrspecの記事にはとってもお世話になりました、ありがとうございます!
最近Hotwireを少し勉強し始めたので、いつかHotwire.loveにお邪魔させていただくかもです。
伊藤さんのように役に立つ記事を書けるようになりたいなあと、日々精進してます。

@JunichiIto
Copy link
Collaborator

参加ありがとうございました!動画でレビューしたのでこちらをご覧ください〜。
https://www.youtube.com/watch?v=izQPxKc_MbI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants