controller methods for HOME

src/main/java/io/zipcoder/persistenceapp/controller/HomeController.java

@@ -8,14 +8,17 @@ import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.*;
 
-import javax.xml.ws.Response;
-
 @RestController
 public class HomeController {
 
-    @Autowired
+    private final
     HomeService homeService;
 
+    @Autowired
+    public HomeController(HomeService homeService) {
+        this.homeService = homeService;
+    }
+
     @PostMapping(value = "/homes")
     public ResponseEntity<Home> createHome(@RequestBody Home home) {
         homeService.addHome(home);
@@ -31,20 +34,23 @@ public class HomeController {
         return new ResponseEntity<>(homes, HttpStatus.OK);
     }
 
+    // these get methods are throwing ambiguous handler methods
+    // Illegal state exception - fixed by specifying URL's better
+
     @GetMapping(value = "/homes/{id}")
     public ResponseEntity<Home> getHomeById(@PathVariable Integer id) {
         Home h = homeService.findHomeById(id);
         return new ResponseEntity<>(h, HttpStatus.OK);
     }
 
-    @GetMapping(value = "/homes/{homeNumber}")
-    public ResponseEntity<Home> getHomeByHomeNumber(@PathVariable String homeNumer) {
-        Home h = homeService.findHomeByHomeNumber(homeNumer);
+    @GetMapping(value = "/homes/homenumber/{homeNumber}")
+    public ResponseEntity<Home> getHomeByHomeNumber(@PathVariable String homeNumber) {
+        Home h = homeService.findHomeByHomeNumber(homeNumber);
         return new ResponseEntity<>(h, HttpStatus.OK);
     }
 
     // Test this
-    @GetMapping(value = "/homes/{address}")
+    @GetMapping(value = "/homes/address/{address}")
     public ResponseEntity<Home> getHomeByAddress(@PathVariable String address) {
         Home h = homeService.findHomeByAddress(address);
         return new ResponseEntity<>(h, HttpStatus.OK);

src/main/java/io/zipcoder/persistenceapp/controller/PersonController.java

@@ -10,9 +10,14 @@ import org.springframework.web.bind.annotation.*;
 
 @RestController
 public class PersonController {
-    @Autowired
+    private final
     PersonService personService;
 
+    @Autowired
+    public PersonController(PersonService personService) {
+        this.personService = personService;
+    }
+
     @GetMapping(value = "/people")
     public ResponseEntity<Iterable<Person>> getPeople() {
         Iterable<Person> people = personService.getAllPeople();

src/main/java/io/zipcoder/persistenceapp/service/HomeService.java

@@ -14,10 +14,12 @@ import java.util.List;
 @Service
 public class HomeService {
 
-    @Autowired
+    private final
     JdbcTemplate jdbcTemplate;
 
-    public HomeService() {
+    @Autowired
+    public HomeService(JdbcTemplate jdbcTemplate) {
+        this.jdbcTemplate = jdbcTemplate;
     }
 
     public void addHome(Home home) {

src/main/java/io/zipcoder/persistenceapp/service/PersonService.java

@@ -19,10 +19,13 @@ public class PersonService {
 
     private final JDBCType[] types = {JDBCType.VARCHAR, JDBCType.VARCHAR, JDBCType.VARCHAR, JDBCType.DATE, JDBCType.SMALLINT };
 
-    @Autowired
+    private final
     JdbcTemplate jdbcTemplate;
 
-    public PersonService() {}
+    @Autowired
+    public PersonService(JdbcTemplate jdbcTemplate) {
+        this.jdbcTemplate = jdbcTemplate;
+    }
 
     public void add(Person p) {
         Object[] info = {p.getFirstName(), p.getLastName(), p.getMobile(), p.getBirthday(), p.getHomeId()};