From bd9e5f3882fe6b56e71633e37529fbc95dc762ab Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=EA=B9=80=EA=B2=BD=EC=A7=84?= <김경진@DESKTOP-6URC5U8>
Date: Wed, 27 May 2020 10:45:15 +0900
Subject: [PATCH] Refactoring

Replace Magic Number with Symbolic Constant
drawFunctionPatterns()
Not to use magic number
---
 .../main/java/io/nayuki/qrcodegen/QrCode.java   | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/java/src/main/java/io/nayuki/qrcodegen/QrCode.java b/java/src/main/java/io/nayuki/qrcodegen/QrCode.java
index 512e314..7013ed7 100644
--- a/java/src/main/java/io/nayuki/qrcodegen/QrCode.java
+++ b/java/src/main/java/io/nayuki/qrcodegen/QrCode.java
@@ -52,6 +52,12 @@ import java.util.Objects;
  */
 public final class QrCode {
 	
+	private static final int FINDER_SIZE = 3;
+
+	private static final int TIMING_COORDINATE = 6;
+
+
+
 	/*---- Static factory functions (high level) ----*/
 	
 	/**
@@ -354,15 +360,14 @@ public final class QrCode {
 	private void drawFunctionPatterns() {
 		// Draw horizontal and vertical timing patterns
 		for (int i = 0; i < size; i++) {
-			final boolean isBlack = i % 2 == 0;
-			setFunctionModule(6, i, isBlack);
-			setFunctionModule(i, 6, isBlack);
+			setFunctionModule(TIMING_COORDINATE, i, i % 2 == 0);
+			setFunctionModule(i, TIMING_COORDINATE, i % 2 == 0);
 		}
 		
 		// Draw 3 finder patterns (all corners except bottom right; overwrites some timing modules)
-		drawFinderPattern(3, 3);
-		drawFinderPattern(size - 4, 3);
-		drawFinderPattern(3, size - 4);
+		drawFinderPattern(FINDER_SIZE, FINDER_SIZE);
+		drawFinderPattern(size - 1 - FINDER_SIZE, FINDER_SIZE);
+		drawFinderPattern(FINDER_SIZE, size - 1 - FINDER_SIZE);
 		
 		// Draw numerous alignment patterns
 		int[] alignPatPos = getAlignmentPatternPositions();