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.

Sign up for GitHub

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

Jump to bottom

lsコマンド1を作成する #24

Closed
asya81 opened this issue Nov 20, 2022 · 2 comments
Closed

lsコマンド1を作成する #24

asya81 opened this issue Nov 20, 2022 · 2 comments
Milestone
05. lsコマンドを作る 1

Comments

@asya81
Copy link
Owner

asya81 commented Nov 20, 2022

lsコマンドを作る1 | FBC に沿って、lsコマンド（オプションなし）を作る。

refs #20 #21 #22 #23
@asya81 asya81 added this to the 05. lsコマンドを作る 1 milestone Nov 20, 2022
asya81 added a commit that referenced this issue Nov 20, 2022
@asya81
refs #24 lsコマンド1を実装
c8c23c3
@asya81 asya81 mentioned this issue Nov 20, 2022
Merged
@asya81
Copy link
Owner Author

asya81 commented Nov 26, 2022 •
edited
Loading

レビュー指摘事項

  • メソッド名が適切になるよう修正する（ファイルとディレクトリの両方を含む言葉を使う）
  • マジックナンバーを使わない
  • 二次元配列をループして表示用の文字列を作成する処理をよりシンプルにする
  • 出力する処理も含めてmain メソッドの中で行う
  • テストケースが1件しかないので、適切な数だけテストを追加する

@asya81
Copy link
Owner Author

asya81 commented Dec 4, 2022 •
edited
Loading

再レビュー指摘事項

  • 同様の修正についてはコミットをまとめるとよい（gitのどのコマンドを使うのか？要調査→git rebase -i）
  • 定数名に説明不要な単語は含めない
  • 特殊なパターンのテストを追加（要素数とその出力される行列数に着目して、パターンを洗い出すと良い）

@asya81 asya81 mentioned this issue Dec 14, 2022
Closed
asya81 added a commit that referenced this issue Dec 15, 2022
@asya81
Merge pull request #25 from asya81/ls_1
c18868f 
refs #24 lsコマンド1を実装
@asya81 asya81 mentioned this issue Sep 1, 2023
Closed
@asya81 asya81 closed this as completed Oct 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
05. lsコマンドを作る 1
Development

No branches or pull requests
1 participant
@asya81