PersonController - Complete

src/main/java/io/zipcoder/crudapp/PersonController.java

@@ -1,8 +1,9 @@
 package io.zipcoder.crudapp;
 
 import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.http.HttpStatus;
+import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.*;
-
 import java.util.List;
 import java.util.stream.Collectors;
 import java.util.stream.StreamSupport;
@@ -10,60 +11,43 @@ import java.util.stream.StreamSupport;
 @RestController
 public class PersonController{
 
-//    @Autowired
-//    PersonService personService;
     @Autowired
     PersonRepository personRepository;
 
     //  var URI = "http://localhost:8080/people/";
+
     @PostMapping("/people")
-    Person createPerson(@RequestBody Person person){ //(Person p)
-        personRepository.save(person);
-        return person;
+    ResponseEntity<Person> createPerson(@RequestBody Person person){
+        return new ResponseEntity<>(
+                personRepository.save(person),
+                HttpStatus.CREATED
+        );
     }
 
     @GetMapping("/people/{id}")
-    Person getPerson(@PathVariable int id){
-        int newId = id;
-        return personRepository.findOne(newId);
+    ResponseEntity<Person> getPerson(@PathVariable int id){
+        return new ResponseEntity<>(
+                personRepository.findOne(id), HttpStatus.OK
+        );
     }
 
     @GetMapping("/people")
-    List<Person> getPersonList(){
-        List<Person> list = StreamSupport
-                .stream(
-                        personRepository
-                                .findAll()
-                                .spliterator(), false
-                )
-                .collect(Collectors.toList());
-        return list;
+    ResponseEntity<Iterable<Person>> getPersonList(){
+        return new ResponseEntity<>(
+                personRepository.findAll(), HttpStatus.OK
+        );
     }
 
-    @PostMapping("/people/{id}")
-    Person updatePerson(Person p){
-        Person newPerson = personRepository.findOne(p.getId());
-        newPerson = p;
-        return p;
+    @RequestMapping(value = "/people/{id}", method = RequestMethod.PUT)
+    ResponseEntity<Person> updatePerson(Person person){
+        return new ResponseEntity<>(
+            personRepository.save(person), HttpStatus.OK
+        );
     }
 
     @DeleteMapping("/people{id}")
     void deletePerson(@PathVariable int id){
-        personRepository.delete(id);
+                personRepository.delete(id);
     }
-}
-/*
-POST /people - create a new person
-Response: 201 Created
-GET /people - get the list of all people
-Response: 200 OK
-GET /people/{id} - Get the person with id number {id}
-Response: 200 OK if found, else 404 Not Found
-PUT /people/{id} - Update the person with id number {id}
-Response: 200 OK if updated, 201 Created if a new entity was created
-DELETE /people/{id} - delete the person with id number {id}
-Response: 204 No Content
 
-# Enabling H2 Console
-spring.h2.console.enabled=true
- */
+}

src/main/java/io/zipcoder/crudapp/PersonService.java

@@ -1,59 +1,59 @@
-package io.zipcoder.crudapp;
-
-import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.stereotype.Service;
-
-import java.util.List;
-import java.util.concurrent.CopyOnWriteArrayList;
-
-@Service
-public class PersonService {
-
-    @Autowired
-    PersonRepository personRepository;
-
-    private List<Person> personList = new CopyOnWriteArrayList<Person>();
-
-    public Person addPerson(Person person){
-        personList.add(person);
-        return person;
-    }
-
-    public List<Person> getPersonList(){
-        return personList;
-    }
-
-    public Person getPerson(int id){
-        return personList
-                .stream()
-                .filter(person -> person.getId() == id)
-                .findFirst()
-                .get();
-    }
-
-    public Person updatePerson(int id, Person person){
-         personList
-                .stream()
-                .forEach(p -> {
-                    if (p.getId() == id){
-                        p.setFirstName(person.getLastName());
-                        p.setLastName(person.getLastName());
-                    }});
-         return personList
-                 .stream()
-                 .filter(person1 -> person1.getId() == id)
-                 .findFirst()
-                 .get();
-        }
-
-    public void deletePerson(int id){
-        personList
-                .stream()
-                .forEach(person -> {
-                    if (person.getId() == id){
-                        personList.remove(person);
-                    }
-                } );
-    }
-
-}
+//package io.zipcoder.crudapp;
+//
+//import org.springframework.beans.factory.annotation.Autowired;
+//import org.springframework.stereotype.Service;
+//
+//import java.util.List;
+//import java.util.concurrent.CopyOnWriteArrayList;
+//
+//@Service
+//public class PersonService {
+//
+//    @Autowired
+//    PersonRepository personRepository;
+//
+//    private List<Person> personList = new CopyOnWriteArrayList<Person>();
+//
+//    public Person addPerson(Person person){
+//        personList.add(person);
+//        return person;
+//    }
+//
+//    public List<Person> getPersonList(){
+//        return personList;
+//    }
+//
+//    public Person getPerson(int id){
+//        return personList
+//                .stream()
+//                .filter(person -> person.getId() == id)
+//                .findFirst()
+//                .get();
+//    }
+//
+//    public Person updatePerson(int id, Person person){
+//         personList
+//                .stream()
+//                .forEach(p -> {
+//                    if (p.getId() == id){
+//                        p.setFirstName(person.getLastName());
+//                        p.setLastName(person.getLastName());
+//                    }});
+//         return personList
+//                 .stream()
+//                 .filter(person1 -> person1.getId() == id)
+//                 .findFirst()
+//                 .get();
+//        }
+//
+//    public void deletePerson(int id){
+//        personList
+//                .stream()
+//                .forEach(person -> {
+//                    if (person.getId() == id){
+//                        personList.remove(person);
+//                    }
+//                } );
+//    }
+//
+//}