Skip to content

第4回 SG Tech スキルアップ勉強会 「ローマ数字変換プログラム」#12

Closed
hana-da wants to merge 2 commits intoSonicGarden:mainfrom
hana-da:main
Closed

第4回 SG Tech スキルアップ勉強会 「ローマ数字変換プログラム」#12
hana-da wants to merge 2 commits intoSonicGarden:mainfrom
hana-da:main

Conversation

@hana-da
Copy link

@hana-da hana-da commented Oct 21, 2024

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

  • SgRomanizer#romanize も SgRomanizer#deromanize も同じような形に収められた
  • ごちゃごちゃさせずにシンプルに書けたと思う

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

30分ぐらい

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

実務経験3年ぐらい?

実際に解いてみた感想

意外とすんなり書けたので嬉しい

伊藤さんにメッセージ(こちらももしあればw)

一人開発がほとんどでコードレビューをしてもらった事がほぼないのでよろしくお願いします。

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

- SgRomanizer#romanize も SgRomanizer#deromanize も同じような形に収められた
- ごちゃごちゃさせずにシンプルに書けたと思う

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

30分ぐらい

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

実務経験3年ぐらい?

# 実際に解いてみた感想

意外とすんなり書けたので嬉しい

# 伊藤さんにメッセージ(こちらももしあればw)

一人開発がほとんどでコードレビューをしてもらった事がほぼないのでよろしくお願いします。
@JunichiIto
Copy link
Collaborator

参加ありがとうございました!動画でレビューしたのでこちらをご覧ください〜。
(勉強会の中でもレビューしましたが、動画は動画で撮っておいたのでよかったらどうぞ!)
https://www.youtube.com/watch?v=0IDZBkbraaA

@hana-da
Copy link
Author

hana-da commented Dec 10, 2024

レビュー後の修正結果 👉 https://github.com/hana-da/sg-romanizer/blob/main/lib/sg_romanizer.rb

@hana-da hana-da closed this Dec 10, 2024
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