2014当面は無料で提供されるらしいのでやってみて損はないかな?セキュリティホールをふせぐという謳い文句に惹かれて「SideCI」使ってみました。レビューの対象に使ったプロジェクトは前回の記事でRails4にバージョンアップしたばかりのこれです。使っている人の感想とか探せばもっと見つかると思いますが自分が参考にしたのはここです。Practiceなどのツールを組み合わせたものらしいけど、統合メニューで操作できるのは確かに有難いかも。メニュー項目がありますが自分が使ってみたものについて感想と、指摘された改善点についてざっと書きます。HREFtwitterで呟くための文字列をAPIに渡す以下のコードでセキュリティ警告が出ました。%>たしかRails3以降は自動で文字列をエスケープしてくれるはずと思って検索したらここにもそう書いてあります。Practiceのオプション設定の問題だと思います。したらGemfileに使用しているRailsのバージョンが書いてあるのだからオプション設定もそれに合わせてほしいところですが、一応以下のように修正して警告を出さないようにすることは出来ました。%>念のためhtmlタグやシングルクォートを出力して確認しましたがh()で囲まなくてもちゃんとエスケープされていました。assignmentこのサイト(Rails4はattr_accessibleが使えない)に書いてある通り、Rails4だとエラーになるからRailsをバージョンアップした時に削除したのに、そこが悪いと指摘されました。PracticeがRails4に対応していないものを使っているのか、オプション設定が厳しめになっているのかそんなところでしょう。分からないけど無視することにしました。なってくるとコードレビューしたことが混乱を引き起こすという本末転倒になってる気もしますが^^;本気でこのツールを使う気があるなら運営元に質問すればおそらく解決するでしょう。Model自分が納得したとこだけ修正しました。helpers中身が空のヘルパーファイルがありますよ。falseしておけということですね。variable納得。Rails2の頃からのソースで、よく理解していないまま書いていたコードがそのままでした。"edit"]end「newが使われていませんよ」と指摘してくれたわけです。以下のように修正しました。controllersこのプロジェクトはModel用のテストスクリプト(Model用のspecファイル)しか書いていなくて、Railsのバージョンアップをした時に以下のようなRailsが出力する雛形のコードに対するテストが不十分でしたが、バグ出しツールとして役に立ちました。saveに失敗したら’new’テンプレートをrenderするとコードでは書いていますが、newテンプレートは存在していませんでした。ファイルが存在しないことを指摘してくれたわけではありません。views部分テンプレートを使うとき:partialというオプションを使わなければならない機会っていうのは無いってことなのか?と、新たな疑問が湧きましたが、リンク先に書いてある通りに修正しました。whitespace行末のスペースを削除しろってことだけど、これは細かい!まぁチームで開発するときには一応統一しておいたほうがいいでしょうね。あと、一つのファイルで最初に見つかった行だけを指摘するみたいなので、他の場所にも存在しないかよく確認してからcommitしましょう。methodsまた使うかもしれないから残しておこうなんて思ったコードが結構残っていました。自分一人のプロジェクトなので他人を混乱させる心配はないのですが、せっかくgitでバージョン管理しているのだからバッサリ削除すべきですね。Keyを登録するのが不安なので使用するのは止めておきました。herokuではなんの警戒もせずtwitterの認証コードを環境変数にセットして使っているのに、なんで使わないのかと問われれば返答に困りますが、仕事で使う機会があれば使ってみたいと思います。