Skip to content

Conversation

@ykkTechc
Copy link
Owner

03_php_algorithm lesson01 ~ 02

03_php_algorithm lesson01 ~ 02

やった事/レビュー観点

03_php_algorithm lesson01 ~ 02 解答
宜しくお願い致します。

@cha-han851
Copy link

変更が反映されていません。
ご確認お願いします!

@ykkTechc
Copy link
Owner Author

03_php_algorithm lesson01 ~ 02

03_php_algorithm lesson01 ~ 02

やった事/レビュー観点

03_php_algorithm lesson01 ~ 02 解答

失礼致しました。
改めてコミットしました。
宜しくお願い致します。

@cha-han851
Copy link

差分がこれだけになってしまっています。
スクリーンショット 2022-06-24 18 23 35
再度ご確認お願いします!

@ykkTechc
Copy link
Owner Author

ykkTechc commented Jun 24, 2022

03_php_algorithm lesson01 ~ 02

03_php_algorithm lesson01 ~ 02

やった事/レビュー観点

03_php_algorithm lesson01 ~ 02 解答

度々で申し訳ありません。
こちらのコミットで反映されてませんでしょうか?
もし反映されていなければ、改めて対処したいと思います。
↓下記、URL
68fde18

@tatsuki-uchiyama
Copy link

tatsuki-uchiyama commented Jun 27, 2022

お疲れ様です!
度々の対応ありがとうございました!

  1. アホわんの前の数字が消えてしまっています。
  2. 両方で割り切れる数字=15としていますが、こういったコードを実務で使う場合は中の数字が変わる可能性や可読性を考慮して二つの条件をandで並べることを推奨します。(間違っているとは言いませんが)
  3. アルゴリズム2の方ではお釣りが9000円になっていましたので修正してください。
  4. アルゴリズム2の回答が2個並んでいるのはなぜでしょうか。
  5. コーディングの規則に則っていない箇所があるようなので修正してください。

コミットが反映されていないことが何度かあったようでしたが、本来GitHubというのはバージョン管理ツールです。
誰かがファイルを更新することをバージョンアップとして捉え更新履歴を記録していくものです。

  • 誰がいつどの箇所をどんなふうに修正したかが確認できる。
  • 何か不具合があった場合に元に戻すことで動いていた当時の状態に戻せる。
  • 同時に複数人が同じファイルを編集していてもお互いの修正箇所を見ながらあるべき姿にできる

といったような目的で使用されます。

基本的には最初にファイルをcloneしたあとの何も動かしていない状態が記録されているべきものと思いますので、今後は注意するようにして下さい。

@ghost
Copy link

ghost commented Jun 28, 2022

03_php_algorithm lesson01 ~ 02

03_php_algorithm lesson01 ~ 02

やった事/レビュー観点

03_php_algorithm lesson01 ~ 02 解答

レビュー頂きありがとうございます。
修正致しました。

GitHubの件、解説して頂きありがとうございます。
かしこまりました。
最初の段階に注意します。

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.

4 participants