java-women / ChatToolForBeginners

初心者向けチャットツールコンテンツ
5 stars 3 forks source link

参加者演習用プロジェクト指摘修正 #54

Closed zoosm3 closed 8 years ago

zoosm3 commented 8 years ago

インデントが崩れているので再フォーマットしたほうが良いかも。

念のためフォーマットしました。

pom.xmlのwebsocketは削除しておいたほうが良いと思います。

動作に必要なはずです。そのままにします。

以下のクラスは初期プロジェクトではなしで、演習で作ってもらったほうが良いと思います。

  • App.java

App.javaはSpring Bootプロジェクトを作成すると最初に生成されます。 ゼロの状態から手で作るものではないのでそのままにします。 それ以外は削除しました。

WebSecurityConfig.javaのconfigureGlobalメソッドの中身はからにして欲しいです。 空にしました。

javajo.jsは穴埋めにしてほしいです。 穴埋めにしましたが、初期化と接続は難しめなので実装済みにしています。

erk5 commented 8 years ago

pom.xmlのwebsocketは削除しておいたほうが良いと思います。

動作に必要なはずです。そのままにします。

演習者には事前にchat-tutrialをビルドしてもらうのでwebsocketはローカルリポジトリにダウンロード済みなので、 動作には問題ないため演習で記述してもらうようにしたいと思いますので削除します。

erk5 commented 8 years ago

App.javaはSpring Bootプロジェクトを作成すると最初に生成されます。 ゼロの状態から手で作るものではないのでそのままにします。 それ以外は削除しました。

説明資料にApp.javaの説明がなかったので作ってもらったほうが良いと思いましたが、 説明資料に追加してもらうことにしたのでこのままでOKです。