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

新人課題cat masatora #20

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

新人課題cat masatora #20

wants to merge 2 commits into from

Conversation

MasatoraAtarashi
Copy link

@MasatoraAtarashi MasatoraAtarashi commented Mar 6, 2021 •
edited
Loading

@dooooooooinggggg
お疲れ様です。新人課題をやってみたのでお手すきの際にレビュー頂けますと幸いです:pray:

実装したこと

  • catの基本機能
    • 引数で指定したファイルの中身を出力
    • 引数がない場合は標準入力から読み込んで出力
    • 引数がハイフンだけだった場合は標準入力から読み込んで出力
    • オプションだけ指定した場合は標準入力から読み込んで出力
  • オプション
    • -nオプションのみ対応しました
  • エラー処理
    • read権限がないファイルを指定するとcat: sample.txt: Permission denied
    • ディレクトリを指定するとcat: ..: Is a directory
    • 存在しないファイルを指定するとcat: sample: No such file or directory
    • 不正なオプションを指定するとillegal option -- f

MasatoraAtarashi
MasatoraAtarashi commented Mar 6, 2021
View reviewed changes
cat/masatora-cat.c
return 0;
}

#define BUFSIZE 1000
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

適当です涙

@MasatoraAtarashi MasatoraAtarashi marked this pull request as ready for review March 6, 2021 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@MasatoraAtarashi