{"comment":{"url":"https://api.github.com/repos/aromualdo/tic-tac-toe/comments/16277359","html_url":"https://github.com/aromualdo/tic-tac-toe/commit/8b10e95ec8aae029dd931290d6fc1966c6ee427f#commitcomment-16277359","id":16277359,"user":{"login":"aromualdo","id":8933931,"avatar_url":"https://avatars.githubusercontent.com/u/8933931?v=3","gravatar_id":"","url":"https://api.github.com/users/aromualdo","html_url":"https://github.com/aromualdo","followers_url":"https://api.github.com/users/aromualdo/followers","following_url":"https://api.github.com/users/aromualdo/following{/other_user}","gists_url":"https://api.github.com/users/aromualdo/gists{/gist_id}","starred_url":"https://api.github.com/users/aromualdo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aromualdo/subscriptions","organizations_url":"https://api.github.com/users/aromualdo/orgs","repos_url":"https://api.github.com/users/aromualdo/repos","events_url":"https://api.github.com/users/aromualdo/events{/privacy}","received_events_url":"https://api.github.com/users/aromualdo/received_events","type":"User","site_admin":false},"position":26,"line":26,"path":"application/src/main/java/com/example/helloworld/HelloWorldResource.java","commit_id":"8b10e95ec8aae029dd931290d6fc1966c6ee427f","created_at":"2016-02-23T19:28:16Z","updated_at":"2016-02-23T19:28:16Z","body":"I don't like this forloop being in your constructor. I'm not going to go much into it, but this makes me wonder. Are you using the best List type for this problem? Are you constructing it in the correct way? In general this is a red flag to me that the design should be looked at more. Again, since this is a test project I'm not going to dig into it, but something to think about. "}}