Kadai2-hisamura #15
Open
Kadai2-hisamura #15
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
hisamura333
force-pushed
the
kadai2-hisamura
branch
from
b190222 to
de46855
Compare
hisamura333
force-pushed
the
kadai2-hisamura
branch
from
de46855 to
14db38c
Compare
hisamura333
force-pushed
the
kadai2-hisamura
branch
from
38cbcf3 to
ce65e82
Compare
tenntenn
reviewed
Jul 31, 2019
| return "", err | ||
| } | ||
| convertedName = dstFile.Name() | ||
| defer dstFile.Close() |
Member
There was a problem hiding this comment.
deferで指定した関数呼び出しは関数終了時に実行されるのでforの中で呼び出さない。
関数に分けることで避けることができる。
tenntenn
reviewed
Jul 31, 2019
| } | ||
|
|
||
| //指定したディレクトリ配下のファイルを取得 | ||
| func dirwalk(dir string) []string { |
tenntenn
reviewed
Jul 31, 2019
| func dirwalk(dir string) []string { | ||
| files, err := ioutil.ReadDir(dir) | ||
| if err != nil { | ||
| panic(err) |
Member
There was a problem hiding this comment.
panicは使わない。
プロダクトを開発している際に使うことは殆どない。
tenntenn
reviewed
Jul 31, 2019
| panic(err) | ||
| } | ||
|
|
||
| var paths []string |
tenntenn
reviewed
Jul 31, 2019
| @@ -0,0 +1,70 @@ | |||
| package convert | |||
tenntenn
reviewed
Jul 31, 2019
| package main | ||
|
|
||
| import ( | ||
| "dojo6/kadai2/hisamura/convert" |
Member
There was a problem hiding this comment.
標準パッケージ以外は空行を分けることが多い。
その際、サードパティパッケージが下になる。
tenntenn
reviewed
Jul 31, 2019
| convertFile, err := convert.Convert(flagOps) | ||
|
|
||
| if err != nil { | ||
| fmt.Println(err) |
tenntenn
reviewed
Jul 31, 2019
| } | ||
|
|
||
| func validation(flagOps convert.FlagOps) bool { | ||
| if flagOps.Dest != "png" && flagOps.Dest != "jpg" && flagOps.Dest != "jpeg" && flagOps.Dest != "gif" { |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
標準パッケージでどのように使われているか
fmt.Fprintでは以下のように記述されている。
同じファイル内のFprint内での呼び出しは以下。
io.writerのインターフェースはwriteをメソッドに持っている型なら引数に設定できるので、os.Stdoutを引数に指定し、呼び出している。
io.Readerとio.Writerがあることでどういう利点があるのか具体例を挙げて考えてみる
コードが簡潔にできるために使用することができる。
interfaceがないと、それぞれの型を引数に指定し、複数の関数を作らなくてはいけない。
それがinterfaceを設定できることで、interfaceの関数は使用できることが約束される。
io.Readerの型を満たしているということはReadメソッドが使用できることが約束されており、引数にio.Readerを指定することで関数内でその引数の型の違いを気にすることなく、Readメソッドを使うことができ、シンプルにコードを記述することができる。