Skip to content

Commit 78328e1

Browse files
committed
added lombok features. :))
1 parent f6ddb23 commit 78328e1

File tree

6 files changed

+33
-26
lines changed

6 files changed

+33
-26
lines changed

person-service/src/main/java/com/cevher/ms/person/domain/Person.java

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,19 @@
11
package com.cevher.ms.person.domain;
22

33

4-
import lombok.AllArgsConstructor;
5-
import lombok.Data;
6-
import lombok.NoArgsConstructor;
4+
import lombok.*;
75

86
import javax.persistence.Entity;
97
import javax.persistence.GeneratedValue;
108
import javax.persistence.GenerationType;
119
import javax.persistence.Id;
1210

1311
@Entity
14-
@Data
12+
@Getter
13+
@Setter
14+
@ToString
15+
@Builder
16+
@EqualsAndHashCode
1517
@AllArgsConstructor
1618
@NoArgsConstructor
1719
public class Person {

person-service/src/main/java/com/cevher/ms/person/repository/PersonRepository.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66

77
import java.util.Optional;
88

9-
@Repository
109
public interface PersonRepository extends JpaRepository<Person, Long> {
1110
Optional<Person> findById(Long id);
1211
}

person-service/src/main/java/com/cevher/ms/person/service/PersonService.java

Lines changed: 13 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,9 @@
44
import com.cevher.ms.person.repository.PersonRepository;
55
import com.cevher.ms.person.vm.DepartmentVM;
66
import com.cevher.ms.person.vm.ResponseTempVM;
7+
import lombok.RequiredArgsConstructor;
78
import lombok.extern.slf4j.Slf4j;
9+
import lombok.val;
810
import org.springframework.beans.factory.annotation.Autowired;
911
import org.springframework.stereotype.Service;
1012
import org.springframework.web.client.RestTemplate;
@@ -13,11 +15,10 @@
1315

1416
@Service
1517
@Slf4j
18+
@RequiredArgsConstructor
1619
public class PersonService {
17-
@Autowired
18-
private PersonRepository personRepository;
1920

20-
@Autowired
21+
private PersonRepository personRepository;
2122
private RestTemplate restTemplate;
2223

2324
public Person savePerson(Person person) {
@@ -28,18 +29,18 @@ public Person savePerson(Person person) {
2829
public ResponseTempVM getPersonWithDepartment(Long personId) {
2930
log.info("getPersonWithDepartment method of PersonService");
3031

31-
ResponseTempVM vm = new ResponseTempVM();
32-
Person person = personRepository.findById(personId).orElse(new Person());
32+
val person = personRepository.findById(personId).orElse(new Person());
3333

3434
// TODO : not for production, you should use config and common class or method.
3535
// TODO : We can use Message Broker for example Apache Kafka
36-
DepartmentVM department = restTemplate.getForObject(
37-
"http://department-service/departments/" + person.getDepartmentId()
38-
, DepartmentVM.class);
39-
40-
vm.setPerson(person);
41-
vm.setDepartment(department);
42-
return vm;
36+
val url = "http://department-service/departments/" + person.getDepartmentId();
37+
val department = restTemplate.getForObject(url, DepartmentVM.class);
38+
39+
return ResponseTempVM
40+
.builder()
41+
.person(person)
42+
.department(department)
43+
.build();
4344
}
4445

4546
public List<Person> findAllPerson() {

person-service/src/main/java/com/cevher/ms/person/vm/DepartmentVM.java

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
11
package com.cevher.ms.person.vm;
22

3-
import lombok.AllArgsConstructor;
4-
import lombok.Data;
5-
import lombok.NoArgsConstructor;
3+
import lombok.*;
64

7-
@Data
5+
@Getter
6+
@Setter
7+
@ToString
8+
@Builder
9+
@EqualsAndHashCode
810
@AllArgsConstructor
911
@NoArgsConstructor
1012
public class DepartmentVM {

person-service/src/main/java/com/cevher/ms/person/vm/ResponseTempVM.java

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,13 @@
11
package com.cevher.ms.person.vm;
22

33
import com.cevher.ms.person.domain.Person;
4-
import lombok.AllArgsConstructor;
5-
import lombok.Data;
6-
import lombok.NoArgsConstructor;
4+
import lombok.*;
75

8-
@Data
6+
@Getter
7+
@Setter
8+
@ToString
9+
@Builder
10+
@EqualsAndHashCode
911
@AllArgsConstructor
1012
@NoArgsConstructor
1113
public class ResponseTempVM {

person-service/src/main/java/com/cevher/ms/person/web/rest/PersonController.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
import com.cevher.ms.person.domain.Person;
44
import com.cevher.ms.person.service.PersonService;
55
import com.cevher.ms.person.vm.ResponseTempVM;
6+
import lombok.RequiredArgsConstructor;
67
import lombok.extern.slf4j.Slf4j;
78
import org.springframework.beans.factory.annotation.Autowired;
89
import org.springframework.web.bind.annotation.*;
@@ -12,9 +13,9 @@
1213
@RestController
1314
@RequestMapping("/people")
1415
@Slf4j
16+
@RequiredArgsConstructor
1517
public class PersonController {
1618

17-
@Autowired
1819
private PersonService personService;
1920

2021
@PostMapping("/")

0 commit comments

Comments
 (0)