incorporate few review comments for bael-603 (eugenp#1429)

core-java-9/src/modules/com.baeldung.student.client/com/baeldung/student/client/StudentClient.java

@@ -4,13 +4,13 @@
 import com.baeldung.student.service.dbimpl.StudentDbService;
 import com.baeldung.student.model.Student;
 
-public class StudentClient{
+public class StudentClient {
 
-	public static void main(String[] args) {
-		StudentService service = new StudentDbService();
-		service.create(new Student());
-		service.read("17SS0001");
-		service.update(new Student());
-		service.delete("17SS0001");
-	}
+    public static void main(String[] args) {
+        StudentService service = new StudentDbService();
+        service.create(new Student());
+        service.read("17SS0001");
+        service.update(new Student());
+        service.delete("17SS0001");
+    }
 }

core-java-9/src/modules/com.baeldung.student.client/module-info.java

@@ -1,3 +1,3 @@
 module com.baeldung.student.client{
-	requires com.baeldung.student.service.dbimpl;
+    requires com.baeldung.student.service.dbimpl;
 }

core-java-9/src/modules/com.baeldung.student.model/com/baeldung/student/model/Student.java

@@ -2,11 +2,14 @@
 
 import java.util.Date;
 
-public class Student{
-	public String registrationId;
-	public String firstName;
-	public String lastName;
-	public Date dateOfBirth;
-	public String city;
-	public String country;
+public class Student {
+    private String registrationId;
+
+    public String getRegistrationId() {
+        return registrationId;
+    }
+
+    public void setRegistrationId(String registrationId) {
+        this.registrationId = registrationId;
+    }
 }

core-java-9/src/modules/com.baeldung.student.model/module-info.java

@@ -1,3 +1,3 @@
 module com.baeldung.student.model{
-	exports com.baeldung.student.model;
+    exports com.baeldung.student.model;
 }

core-java-9/src/modules/com.baeldung.student.service.dbimpl/com/baeldung/student/service/dbimpl/StudentDbService.java

@@ -2,26 +2,29 @@
 
 import com.baeldung.student.service.StudentService;
 import com.baeldung.student.model.Student;
+import java.util.logging.*;
 
-public class StudentDbService implements StudentService{
+public class StudentDbService implements StudentService {
 
-	public String create(Student student){
-		System.out.println("Creating student in DB...");
-		return student.registrationId;
-	}
+    private static Logger logger = Logger.getLogger("StudentDbService");
 
-	public Student read(String registrationId){
-		System.out.println("Reading student from DB...");
-		return new Student();
-	}
+    public String create(Student student) {
+        logger.log(Level.INFO, "Creating student in DB...");
+        return student.getRegistrationId();
+    }
 
-	public Student update(Student student){
-		System.out.println("Updating sutdent in DB...");
-		return student;
-	}
+    public Student read(String registrationId) {
+        logger.log(Level.INFO, "Reading student from DB...");
+        return new Student();
+    }
 
-	public String delete(String registrationId){
-		System.out.println("Deleteing sutdent in DB...");
-		return registrationId;
-	}
+    public Student update(Student student) {
+        logger.log(Level.INFO, "Updating sutdent in DB...");
+        return student;
+    }
+
+    public String delete(String registrationId) {
+        logger.log(Level.INFO, "Deleteing sutdent in DB...");
+        return registrationId;
+    }
 }

core-java-9/src/modules/com.baeldung.student.service.dbimpl/module-info.java

@@ -1,4 +1,5 @@
 module com.baeldung.student.service.dbimpl{
-	requires transitive com.baeldung.student.service;
-	exports com.baeldung.student.service.dbimpl;
+    requires transitive com.baeldung.student.service;
+    exports com.baeldung.student.service.dbimpl;
+    requires java.logging;
 }

core-java-9/src/modules/com.baeldung.student.service/com/baeldung/student/service/StudentService.java

@@ -2,13 +2,13 @@
 
 import com.baeldung.student.model.Student;
 
-public interface StudentService{
+public interface StudentService {
 
-	public String create(Student student);
+    public String create(Student student);
 
-	public Student read(String registrationId);
+    public Student read(String registrationId);
 
-	public Student update(Student student);
+    public Student update(Student student);
 
-	public String delete(String registrationId);
+    public String delete(String registrationId);
 }

core-java-9/src/modules/com.baeldung.student.service/module-info.java

@@ -1,4 +1,4 @@
 module com.baeldung.student.service{
-	requires transitive com.baeldung.student.model;
-	exports com.baeldung.student.service;
+    requires transitive com.baeldung.student.model;
+    exports com.baeldung.student.service;
 }