From dd0158836921e4170255dddc3da808ebe47577a5 Mon Sep 17 00:00:00 2001
From: Sunil Gulabani <sunil.gulabani1@gmail.com>
Date: Wed, 15 Feb 2017 18:07:14 +0530
Subject: [PATCH] BAEL-650: Small refactoring

---
 .../lambda/dynamodb/SavePersonHandler.java    | 23 ++++++++-----------
 .../lambda/dynamodb/bean/PersonRequest.java   |  3 ++-
 2 files changed, 12 insertions(+), 14 deletions(-)

diff --git a/aws/src/main/java/com/baeldung/lambda/dynamodb/SavePersonHandler.java b/aws/src/main/java/com/baeldung/lambda/dynamodb/SavePersonHandler.java
index 70aa0fd68f..625da62efd 100644
--- a/aws/src/main/java/com/baeldung/lambda/dynamodb/SavePersonHandler.java
+++ b/aws/src/main/java/com/baeldung/lambda/dynamodb/SavePersonHandler.java
@@ -17,35 +17,32 @@ public class SavePersonHandler implements RequestHandler<PersonRequest, PersonRe
 
     private DynamoDB dynamoDb;
 
-    private final String DYNAMODB_TABLE_NAME = "Person";
-    private final Regions REGION = Regions.US_WEST_2;
+    private String DYNAMODB_TABLE_NAME = "Person";
+    private Regions REGION = Regions.US_WEST_2;
 
     public PersonResponse handleRequest(PersonRequest personRequest, Context context) {
-        context.getLogger().log("personRequest: " + personRequest);
         this.initDynamoDbClient();
 
         persistData(personRequest);
 
         PersonResponse personResponse = new PersonResponse();
         personResponse.setMessage("Saved Successfully!!!");
-        context.getLogger().log("personResponse: " + personResponse);
         return personResponse;
     }
 
     private PutItemOutcome persistData(PersonRequest personRequest) throws ConditionalCheckFailedException {
         return this.dynamoDb.getTable(DYNAMODB_TABLE_NAME)
-                .putItem(
-                        new PutItemSpec().withItem(new Item()
-                                            .withNumber("id", personRequest.getId())
-                                            .withString("firstName", personRequest.getFirstName())
-                                            .withString("lastName", personRequest.getLastName())
-                                            .withNumber("age", personRequest.getAge())
-                                            .withString("address", personRequest.getAddress())
-                        ));
+          .putItem(
+            new PutItemSpec().withItem(new Item()
+              .withNumber("id", personRequest.getId())
+              .withString("firstName", personRequest.getFirstName())
+              .withString("lastName", personRequest.getLastName())
+              .withNumber("age", personRequest.getAge())
+              .withString("address", personRequest.getAddress())));
     }
 
     private void initDynamoDbClient() {
-        final AmazonDynamoDBClient client = new AmazonDynamoDBClient();
+        AmazonDynamoDBClient client = new AmazonDynamoDBClient();
         client.setRegion(Region.getRegion(REGION));
         this.dynamoDb = new DynamoDB(client);
     }
diff --git a/aws/src/main/java/com/baeldung/lambda/dynamodb/bean/PersonRequest.java b/aws/src/main/java/com/baeldung/lambda/dynamodb/bean/PersonRequest.java
index 6071f4448e..0c0706e676 100644
--- a/aws/src/main/java/com/baeldung/lambda/dynamodb/bean/PersonRequest.java
+++ b/aws/src/main/java/com/baeldung/lambda/dynamodb/bean/PersonRequest.java
@@ -1,6 +1,7 @@
 package com.baeldung.lambda.dynamodb.bean;
 
 import com.google.gson.Gson;
+import com.google.gson.GsonBuilder;
 
 public class PersonRequest {
     private int id;
@@ -20,7 +21,7 @@ public class PersonRequest {
     }
 
     public String toString() {
-        final Gson gson = new Gson();
+        final Gson gson = new GsonBuilder().setPrettyPrinting().create();
         return gson.toJson(this);
     }