From ede6d3866d9b449a31fea7f76e11c6f7c70596d5 Mon Sep 17 00:00:00 2001
From: Rafael De Oliveira Fantinel <rafael.fantinel@zenvia.com>
Date: Mon, 26 Jun 2023 08:41:23 -0300
Subject: [PATCH] Refactoring Owner getPets function to use lambda functions

---
 .../samples/petclinic/owner/Owner.java        | 36 ++++++++-----------
 1 file changed, 14 insertions(+), 22 deletions(-)

diff --git a/src/main/java/org/springframework/samples/petclinic/owner/Owner.java b/src/main/java/org/springframework/samples/petclinic/owner/Owner.java
index ac556459d..03e409cde 100644
--- a/src/main/java/org/springframework/samples/petclinic/owner/Owner.java
+++ b/src/main/java/org/springframework/samples/petclinic/owner/Owner.java
@@ -112,16 +112,13 @@ public class Owner extends Person {
 	 * @param id to test
 	 * @return a pet if pet id is already in use
 	 */
-	public Pet getPet(Integer id) {
-		for (Pet pet : getPets()) {
-			if (!pet.isNew()) {
-				Integer compId = pet.getId();
-				if (compId.equals(id)) {
-					return pet;
-				}
-			}
-		}
-		return null;
+	public Pet getPet(final Integer id) {
+		return getPets().stream()
+			.filter(pet -> !pet.isNew())
+			.filter(pet -> pet.getId().equals(id))
+			.findFirst()
+			.orElse(null);
+
 	}
 
 	/**
@@ -129,18 +126,13 @@ public class Owner extends Person {
 	 * @param name to test
 	 * @return a pet if pet name is already in use
 	 */
-	public Pet getPet(String name, boolean ignoreNew) {
-		name = name.toLowerCase();
-		for (Pet pet : getPets()) {
-			if (!ignoreNew || !pet.isNew()) {
-				String compName = pet.getName();
-				compName = compName == null ? "" : compName.toLowerCase();
-				if (compName.equals(name)) {
-					return pet;
-				}
-			}
-		}
-		return null;
+	public Pet getPet(final String name, final boolean ignoreNew) {
+		return getPets().stream()
+			.filter(pet -> !ignoreNew && !pet.isNew())
+			.filter(pet -> pet.getName().equals(name.toLowerCase()))
+			.findFirst()
+			.orElse(null);
+
 	}
 
 	@Override