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

レビューお願い致します #11

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ngchtotti
Copy link

コードのアピールポイント

簡潔に実装するための規則探しに苦労しました.(特にromanizeメソッド)
deromanizeメソッドのロジックはromanizeメソッドと比較してシンプルにできたかなと思います.

コードを書くのにかかった時間

2~3時間

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

大学の授業含めると約6年.自主的に調べたり手動かすようになってからは約3年.

実際に解いてみた感想

Rubyの文法や独特な書き方に慣れていないことを痛感しました.
エレガントな実装を模索しましたが,パワーで解決する方法しか思いつきませんでした.悔しい,,,

伊藤さんにメッセージ

チェリー本と伊藤さんの記事には大変お世話になっております.
ボコボコにレビューしてください.

@ngchtotti ngchtotti changed the title 実装完了 レビューお願い致します Oct 19, 2024
@JunichiIto
Copy link
Collaborator

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

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