Skip to content

レビュー後の修正 - #1

Open
Haru-53 wants to merge 2 commits into
mainfrom
task-answer2026
Open

レビュー後の修正#1
Haru-53 wants to merge 2 commits into
mainfrom
task-answer2026

Conversation

@Haru-53

@Haru-53 Haru-53 commented Aug 14, 2026

Copy link
Copy Markdown
Owner

Q4

比較演算子を使用するときはデータ型まで見ないとバグの混入に繋がるので、必ず厳密比較(===)を使用するようにしてください!
処理にelseifを使用していない点は守られていますが、if文のネスト(入れ子)になっており、orの条件である「windowsまたはmacの場合」という要件が正しく表現されていません。論理演算子(|| または or)を使って、「windowsの場合、またはmacの場合」という条件式で記載すると1つの if文とelseの中で完結させられますので修正お願いします!

→厳格比較(===)に修正し、論理演算子を使用して要件を表現しました。

Q12

比較演算子を使用するときはデータ型まで見ないとバグの混入に繋がるので、必ず厳密比較(===)を使用するようにしてください!

→厳格比較(===)に修正しました

応用Q3

[ask]&$personと参照渡しの形式になっているますが、参照渡しにした意図を教えてください!

→配列に含まれているもとのデータも直接操作して年齢を追加することができるので参照(&)にしました。

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.

1 participant