Add common request method - Githubissues

shimoju / metabase-ruby

Ruby client library for Metabase API

MIT License

28 stars 27 forks source link

Add common request method #5

Closed shimoju closed 6 years ago

shimoju commented 6 years ago

リクエスト・レスポンス処理を共通のメソッドにしました。

get,post,put,delete,headを定義して、内部でFaradayのそれぞれのメソッドを呼び出す
run_requestを使ってもいいけどsendにした
- get,post...だと同じインタフェースのまま第二引数のパラメータをHTTPメソッドによっていい感じにしてくれる（GETならQuery string、POSTならリクエストボディとして扱われる）ので、こちらが振り分けてやる必要がなく、行数が減る
- get head deleteとpost put patchで定義を分けているのだなあ