diff --git a/.gitignore b/.gitignore index 00ed595..2f29f53 100644 --- a/.gitignore +++ b/.gitignore @@ -1,2 +1,4 @@ **/.idea *.iml +**/build +**/.gradle diff --git a/java/README.md b/java/README.md new file mode 100644 index 0000000..a33548b --- /dev/null +++ b/java/README.md @@ -0,0 +1,38 @@ +# Video store kata + +Based on the book 'Refactoring' by Martin Fowler. More information on this book can be found on https://martinfowler.com/books/refactoring.html + +Imagine a company of theatrical players who go out to various events performing plays. Typically, a customer will request a few plays and the company charges +them based on the size of the audience and the kind of play they perform. There are currently two kinds of plays that the company performs: tragedies and comedies. As well as providing a bill for the performance, the company gives its customers “volume credits” which they can use for discounts on future performances—think of it as a customer loyalty mechanism. + +Plays.json +```json +{ + "hamlet": {"name": "Hamlet", "type": "tragedy"}, + "as-like": {"name": "As You Like It", "type": "comedy"}, + "othello": {"name": "Othello", "type": "tragedy"} +} +``` + +Invoices.json +```json +[ + { + "customer": "BigCo", + "performances": [ + { + "playID": "hamlet", + "audience": 55 + }, + { + "playID": "as-like", + "audience": 35 + }, + { + "playID": "othello", + "audience": 40 + } + ] + } +] +``` \ No newline at end of file diff --git a/java/build.gradle b/java/build.gradle new file mode 100644 index 0000000..b91c990 --- /dev/null +++ b/java/build.gradle @@ -0,0 +1,21 @@ +apply plugin: 'java' + +group 'kata' +version '1.0-SNAPSHOT' +sourceCompatibility = 12 + +repositories { + mavenCentral() +} + +test { + useJUnitPlatform() + testLogging { + events "passed", "skipped", "failed" + } +} + +dependencies { + testCompile 'org.junit.jupiter:junit-jupiter:5.5.1' + testCompile 'org.assertj:assertj-core:3.13.2' +} diff --git a/java/gradle/wrapper/gradle-wrapper.jar b/java/gradle/wrapper/gradle-wrapper.jar new file mode 100644 index 0000000..87b738c Binary files /dev/null and b/java/gradle/wrapper/gradle-wrapper.jar differ diff --git a/java/gradle/wrapper/gradle-wrapper.properties b/java/gradle/wrapper/gradle-wrapper.properties new file mode 100644 index 0000000..44e7c4d --- /dev/null +++ b/java/gradle/wrapper/gradle-wrapper.properties @@ -0,0 +1,5 @@ +distributionBase=GRADLE_USER_HOME +distributionPath=wrapper/dists +distributionUrl=https\://services.gradle.org/distributions/gradle-5.2.1-bin.zip +zipStoreBase=GRADLE_USER_HOME +zipStorePath=wrapper/dists diff --git a/java/gradlew b/java/gradlew new file mode 100755 index 0000000..af6708f --- /dev/null +++ b/java/gradlew @@ -0,0 +1,172 @@ +#!/usr/bin/env sh + +############################################################################## +## +## Gradle start up script for UN*X +## +############################################################################## + +# Attempt to set APP_HOME +# Resolve links: $0 may be a link +PRG="$0" +# Need this for relative symlinks. +while [ -h "$PRG" ] ; do + ls=`ls -ld "$PRG"` + link=`expr "$ls" : '.*-> \(.*\)$'` + if expr "$link" : '/.*' > /dev/null; then + PRG="$link" + else + PRG=`dirname "$PRG"`"/$link" + fi +done +SAVED="`pwd`" +cd "`dirname \"$PRG\"`/" >/dev/null +APP_HOME="`pwd -P`" +cd "$SAVED" >/dev/null + +APP_NAME="Gradle" +APP_BASE_NAME=`basename "$0"` + +# Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script. +DEFAULT_JVM_OPTS='"-Xmx64m"' + +# Use the maximum available, or set MAX_FD != -1 to use that value. +MAX_FD="maximum" + +warn () { + echo "$*" +} + +die () { + echo + echo "$*" + echo + exit 1 +} + +# OS specific support (must be 'true' or 'false'). +cygwin=false +msys=false +darwin=false +nonstop=false +case "`uname`" in + CYGWIN* ) + cygwin=true + ;; + Darwin* ) + darwin=true + ;; + MINGW* ) + msys=true + ;; + NONSTOP* ) + nonstop=true + ;; +esac + +CLASSPATH=$APP_HOME/gradle/wrapper/gradle-wrapper.jar + +# Determine the Java command to use to start the JVM. +if [ -n "$JAVA_HOME" ] ; then + if [ -x "$JAVA_HOME/jre/sh/java" ] ; then + # IBM's JDK on AIX uses strange locations for the executables + JAVACMD="$JAVA_HOME/jre/sh/java" + else + JAVACMD="$JAVA_HOME/bin/java" + fi + if [ ! -x "$JAVACMD" ] ; then + die "ERROR: JAVA_HOME is set to an invalid directory: $JAVA_HOME + +Please set the JAVA_HOME variable in your environment to match the +location of your Java installation." + fi +else + JAVACMD="java" + which java >/dev/null 2>&1 || die "ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH. + +Please set the JAVA_HOME variable in your environment to match the +location of your Java installation." +fi + +# Increase the maximum file descriptors if we can. +if [ "$cygwin" = "false" -a "$darwin" = "false" -a "$nonstop" = "false" ] ; then + MAX_FD_LIMIT=`ulimit -H -n` + if [ $? -eq 0 ] ; then + if [ "$MAX_FD" = "maximum" -o "$MAX_FD" = "max" ] ; then + MAX_FD="$MAX_FD_LIMIT" + fi + ulimit -n $MAX_FD + if [ $? -ne 0 ] ; then + warn "Could not set maximum file descriptor limit: $MAX_FD" + fi + else + warn "Could not query maximum file descriptor limit: $MAX_FD_LIMIT" + fi +fi + +# For Darwin, add options to specify how the application appears in the dock +if $darwin; then + GRADLE_OPTS="$GRADLE_OPTS \"-Xdock:name=$APP_NAME\" \"-Xdock:icon=$APP_HOME/media/gradle.icns\"" +fi + +# For Cygwin, switch paths to Windows format before running java +if $cygwin ; then + APP_HOME=`cygpath --path --mixed "$APP_HOME"` + CLASSPATH=`cygpath --path --mixed "$CLASSPATH"` + JAVACMD=`cygpath --unix "$JAVACMD"` + + # We build the pattern for arguments to be converted via cygpath + ROOTDIRSRAW=`find -L / -maxdepth 1 -mindepth 1 -type d 2>/dev/null` + SEP="" + for dir in $ROOTDIRSRAW ; do + ROOTDIRS="$ROOTDIRS$SEP$dir" + SEP="|" + done + OURCYGPATTERN="(^($ROOTDIRS))" + # Add a user-defined pattern to the cygpath arguments + if [ "$GRADLE_CYGPATTERN" != "" ] ; then + OURCYGPATTERN="$OURCYGPATTERN|($GRADLE_CYGPATTERN)" + fi + # Now convert the arguments - kludge to limit ourselves to /bin/sh + i=0 + for arg in "$@" ; do + CHECK=`echo "$arg"|egrep -c "$OURCYGPATTERN" -` + CHECK2=`echo "$arg"|egrep -c "^-"` ### Determine if an option + + if [ $CHECK -ne 0 ] && [ $CHECK2 -eq 0 ] ; then ### Added a condition + eval `echo args$i`=`cygpath --path --ignore --mixed "$arg"` + else + eval `echo args$i`="\"$arg\"" + fi + i=$((i+1)) + done + case $i in + (0) set -- ;; + (1) set -- "$args0" ;; + (2) set -- "$args0" "$args1" ;; + (3) set -- "$args0" "$args1" "$args2" ;; + (4) set -- "$args0" "$args1" "$args2" "$args3" ;; + (5) set -- "$args0" "$args1" "$args2" "$args3" "$args4" ;; + (6) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" ;; + (7) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" ;; + (8) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" "$args7" ;; + (9) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" "$args7" "$args8" ;; + esac +fi + +# Escape application args +save () { + for i do printf %s\\n "$i" | sed "s/'/'\\\\''/g;1s/^/'/;\$s/\$/' \\\\/" ; done + echo " " +} +APP_ARGS=$(save "$@") + +# Collect all arguments for the java command, following the shell quoting and substitution rules +eval set -- $DEFAULT_JVM_OPTS $JAVA_OPTS $GRADLE_OPTS "\"-Dorg.gradle.appname=$APP_BASE_NAME\"" -classpath "\"$CLASSPATH\"" org.gradle.wrapper.GradleWrapperMain "$APP_ARGS" + +# by default we should be in the correct project dir, but when run from Finder on Mac, the cwd is wrong +if [ "$(uname)" = "Darwin" ] && [ "$HOME" = "$PWD" ]; then + cd "$(dirname "$0")" +fi + +exec "$JAVACMD" "$@" diff --git a/java/gradlew.bat b/java/gradlew.bat new file mode 100644 index 0000000..0f8d593 --- /dev/null +++ b/java/gradlew.bat @@ -0,0 +1,84 @@ +@if "%DEBUG%" == "" @echo off +@rem ########################################################################## +@rem +@rem Gradle startup script for Windows +@rem +@rem ########################################################################## + +@rem Set local scope for the variables with windows NT shell +if "%OS%"=="Windows_NT" setlocal + +set DIRNAME=%~dp0 +if "%DIRNAME%" == "" set DIRNAME=. +set APP_BASE_NAME=%~n0 +set APP_HOME=%DIRNAME% + +@rem Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script. +set DEFAULT_JVM_OPTS="-Xmx64m" + +@rem Find java.exe +if defined JAVA_HOME goto findJavaFromJavaHome + +set JAVA_EXE=java.exe +%JAVA_EXE% -version >NUL 2>&1 +if "%ERRORLEVEL%" == "0" goto init + +echo. +echo ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH. +echo. +echo Please set the JAVA_HOME variable in your environment to match the +echo location of your Java installation. + +goto fail + +:findJavaFromJavaHome +set JAVA_HOME=%JAVA_HOME:"=% +set JAVA_EXE=%JAVA_HOME%/bin/java.exe + +if exist "%JAVA_EXE%" goto init + +echo. +echo ERROR: JAVA_HOME is set to an invalid directory: %JAVA_HOME% +echo. +echo Please set the JAVA_HOME variable in your environment to match the +echo location of your Java installation. + +goto fail + +:init +@rem Get command-line arguments, handling Windows variants + +if not "%OS%" == "Windows_NT" goto win9xME_args + +:win9xME_args +@rem Slurp the command line arguments. +set CMD_LINE_ARGS= +set _SKIP=2 + +:win9xME_args_slurp +if "x%~1" == "x" goto execute + +set CMD_LINE_ARGS=%* + +:execute +@rem Setup the command line + +set CLASSPATH=%APP_HOME%\gradle\wrapper\gradle-wrapper.jar + +@rem Execute Gradle +"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -classpath "%CLASSPATH%" org.gradle.wrapper.GradleWrapperMain %CMD_LINE_ARGS% + +:end +@rem End local scope for the variables with windows NT shell +if "%ERRORLEVEL%"=="0" goto mainEnd + +:fail +rem Set variable GRADLE_EXIT_CONSOLE if you need the _script_ return code instead of +rem the _cmd.exe /c_ return code! +if not "" == "%GRADLE_EXIT_CONSOLE%" exit 1 +exit /b 1 + +:mainEnd +if "%OS%"=="Windows_NT" endlocal + +:omega diff --git a/java/settings.gradle b/java/settings.gradle new file mode 100644 index 0000000..7fa6465 --- /dev/null +++ b/java/settings.gradle @@ -0,0 +1,2 @@ +rootProject.name = 'theatrical-plays' + diff --git a/java/src/main/java/Invoice.java b/java/src/main/java/Invoice.java new file mode 100644 index 0000000..9a267f8 --- /dev/null +++ b/java/src/main/java/Invoice.java @@ -0,0 +1,13 @@ +import javax.swing.*; +import java.util.List; + +public class Invoice { + + public String customer; + public List performances; + + public Invoice(String customer, List performances) { + this.customer = customer; + this.performances = performances; + } +} diff --git a/java/src/main/java/Performance.java b/java/src/main/java/Performance.java new file mode 100644 index 0000000..ac7093f --- /dev/null +++ b/java/src/main/java/Performance.java @@ -0,0 +1,10 @@ +public class Performance { + + public String playID; + public int audience; + + public Performance(String playID, int audience) { + this.playID = playID; + this.audience = audience; + } +} diff --git a/java/src/main/java/Play.java b/java/src/main/java/Play.java new file mode 100644 index 0000000..59552f4 --- /dev/null +++ b/java/src/main/java/Play.java @@ -0,0 +1,10 @@ +public class Play { + + public String name; + public String type; + + public Play(String name, String type) { + this.name = name; + this.type = type; + } +} diff --git a/java/src/main/java/StatementPrinter.java b/java/src/main/java/StatementPrinter.java new file mode 100644 index 0000000..78419e9 --- /dev/null +++ b/java/src/main/java/StatementPrinter.java @@ -0,0 +1,50 @@ +import java.text.NumberFormat; +import java.util.Locale; +import java.util.Map; + +public class StatementPrinter { + + public String print(Invoice invoice, Map plays) { + var totalAmount = 0; + var volumeCredits = 0; + var result = String.format("Statement for %s\n", invoice.customer); + + NumberFormat frmt = NumberFormat.getCurrencyInstance(Locale.US); + + for (var perf : invoice.performances) { + var play = plays.get(perf.playID); + var thisAmount = 0; + + switch (play.type) { + case "tragedy": + thisAmount = 40000; + if (perf.audience > 30) { + thisAmount += 1000 * (perf.audience - 30); + } + break; + case "comedy": + thisAmount = 30000; + if (perf.audience > 20) { + thisAmount += 10000 + 500 * (perf.audience - 20); + } + thisAmount += 300 * perf.audience; + break; + default: + throw new Error("unknown type: ${play.type}"); + } + + // add volume credits + volumeCredits += Math.max(perf.audience - 30, 0); + // add extra credit for every ten comedy attendees + if ("comedy" == play.type) volumeCredits += Math.floor(perf.audience / 5); + + // print line for this order + result += String.format(" %s: %s (%s seats)\n", play.name, frmt.format(thisAmount / 100), perf.audience); + totalAmount += thisAmount; + } + result += String.format("Amount owed is %s\n", frmt.format(totalAmount / 100)); + result += String.format("You earned %s credits\n", volumeCredits); + return result; + } + +} diff --git a/java/src/test/java/StatementPrinterTests.java b/java/src/test/java/StatementPrinterTests.java new file mode 100644 index 0000000..c9d2790 --- /dev/null +++ b/java/src/test/java/StatementPrinterTests.java @@ -0,0 +1,30 @@ +import org.junit.jupiter.api.Test; + +import java.util.List; +import java.util.Map; + +import static org.assertj.core.api.Assertions.assertThat; + +public class StatementPrinterTests { + + @Test + void printsStatements() { + Map plays = Map.of("hamlet", new Play("Hamlet", "tragedy"), + "as-like", new Play("As You Like It", "comedy"), + "othello", new Play("Othello", "tragedy")); + + Invoice invoice = new Invoice("BigCo", List.of(new Performance("hamlet", 55), + new Performance("as-like", 35), + new Performance("othello", 40))); + + StatementPrinter statementPrinter = new StatementPrinter(); + var result = statementPrinter.print(invoice, plays); + + assertThat(result).isEqualTo("Statement for BigCo\n" + + " Hamlet: $650.00 (55 seats)\n" + + " As You Like It: $580.00 (35 seats)\n" + + " Othello: $500.00 (40 seats)\n" + + "Amount owed is $1,730.00\n" + + "You earned 47 credits\n"); + } +}