From 849d893f576bbaf3fa22728f061d0ccd33b9bed7 Mon Sep 17 00:00:00 2001 From: obvly Date: Thu, 17 Sep 2026 15:44:10 +0200 Subject: [PATCH 1/5] Final homework submission --- .gitattributes | 2 + .gitignore | 33 ++ .mvn/wrapper/maven-wrapper.properties | 3 + checkstyle.xml | 8 + mvnw | 295 ++++++++++++++++++ mvnw.cmd | 189 +++++++++++ pom.xml | 124 ++++++++ .../SpringbootWebApplication.java | 13 + .../controller/BookController.java | 30 ++ .../example/springbootweb/dto/BookDto.java | 15 + .../dto/CreateBookRequestDto.java | 14 + .../exception/EntityNotFoundException.java | 7 + .../springbootweb/mapper/BookMapper.java | 16 + .../org/example/springbootweb/model/Book.java | 24 ++ .../repository/BookRepository.java | 7 + .../springbootweb/service/BookService.java | 11 + .../service/impl/BookServiceImpl.java | 41 +++ src/main/resources/application.properties | 7 + .../SpringbootWebApplicationTests.java | 13 + 19 files changed, 852 insertions(+) create mode 100644 .gitattributes create mode 100644 .gitignore create mode 100644 .mvn/wrapper/maven-wrapper.properties create mode 100644 checkstyle.xml create mode 100755 mvnw create mode 100644 mvnw.cmd create mode 100644 pom.xml create mode 100644 src/main/java/org/example/springbootweb/SpringbootWebApplication.java create mode 100644 src/main/java/org/example/springbootweb/controller/BookController.java create mode 100644 src/main/java/org/example/springbootweb/dto/BookDto.java create mode 100644 src/main/java/org/example/springbootweb/dto/CreateBookRequestDto.java create mode 100644 src/main/java/org/example/springbootweb/exception/EntityNotFoundException.java create mode 100644 src/main/java/org/example/springbootweb/mapper/BookMapper.java create mode 100644 src/main/java/org/example/springbootweb/model/Book.java create mode 100644 src/main/java/org/example/springbootweb/repository/BookRepository.java create mode 100644 src/main/java/org/example/springbootweb/service/BookService.java create mode 100644 src/main/java/org/example/springbootweb/service/impl/BookServiceImpl.java create mode 100644 src/main/resources/application.properties create mode 100644 src/test/java/org/example/springbootweb/SpringbootWebApplicationTests.java diff --git a/.gitattributes b/.gitattributes new file mode 100644 index 0000000..3b41682 --- /dev/null +++ b/.gitattributes @@ -0,0 +1,2 @@ +/mvnw text eol=lf +*.cmd text eol=crlf diff --git a/.gitignore b/.gitignore new file mode 100644 index 0000000..667aaef --- /dev/null +++ b/.gitignore @@ -0,0 +1,33 @@ +HELP.md +target/ +.mvn/wrapper/maven-wrapper.jar +!**/src/main/**/target/ +!**/src/test/**/target/ + +### STS ### +.apt_generated +.classpath +.factorypath +.project +.settings +.springBeans +.sts4-cache + +### IntelliJ IDEA ### +.idea +*.iws +*.iml +*.ipr + +### NetBeans ### +/nbproject/private/ +/nbbuild/ +/dist/ +/nbdist/ +/.nb-gradle/ +build/ +!**/src/main/**/build/ +!**/src/test/**/build/ + +### VS Code ### +.vscode/ diff --git a/.mvn/wrapper/maven-wrapper.properties b/.mvn/wrapper/maven-wrapper.properties new file mode 100644 index 0000000..216df05 --- /dev/null +++ b/.mvn/wrapper/maven-wrapper.properties @@ -0,0 +1,3 @@ +wrapperVersion=3.3.4 +distributionType=only-script +distributionUrl=https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.9.16/apache-maven-3.9.16-bin.zip diff --git a/checkstyle.xml b/checkstyle.xml new file mode 100644 index 0000000..b971879 --- /dev/null +++ b/checkstyle.xml @@ -0,0 +1,8 @@ + + + + + + diff --git a/mvnw b/mvnw new file mode 100755 index 0000000..bd8896b --- /dev/null +++ b/mvnw @@ -0,0 +1,295 @@ +#!/bin/sh +# ---------------------------------------------------------------------------- +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOTICE file +# distributed with this work for additional information +# regarding copyright ownership. The ASF licenses this file +# to you under the Apache License, Version 2.0 (the +# "License"); you may not use this file except in compliance +# with the License. You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, +# software distributed under the License is distributed on an +# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY +# KIND, either express or implied. See the License for the +# specific language governing permissions and limitations +# under the License. +# ---------------------------------------------------------------------------- + +# ---------------------------------------------------------------------------- +# Apache Maven Wrapper startup batch script, version 3.3.4 +# +# Optional ENV vars +# ----------------- +# JAVA_HOME - location of a JDK home dir, required when download maven via java source +# MVNW_REPOURL - repo url base for downloading maven distribution +# MVNW_USERNAME/MVNW_PASSWORD - user and password for downloading maven +# MVNW_VERBOSE - true: enable verbose log; debug: trace the mvnw script; others: silence the output +# ---------------------------------------------------------------------------- + +set -euf +[ "${MVNW_VERBOSE-}" != debug ] || set -x + +# OS specific support. +native_path() { printf %s\\n "$1"; } +case "$(uname)" in +CYGWIN* | MINGW*) + [ -z "${JAVA_HOME-}" ] || JAVA_HOME="$(cygpath --unix "$JAVA_HOME")" + native_path() { cygpath --path --windows "$1"; } + ;; +esac + +# set JAVACMD and JAVACCMD +set_java_home() { + # For Cygwin and MinGW, ensure paths are in Unix format before anything is touched + 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" + JAVACCMD="$JAVA_HOME/jre/sh/javac" + else + JAVACMD="$JAVA_HOME/bin/java" + JAVACCMD="$JAVA_HOME/bin/javac" + + if [ ! -x "$JAVACMD" ] || [ ! -x "$JAVACCMD" ]; then + echo "The JAVA_HOME environment variable is not defined correctly, so mvnw cannot run." >&2 + echo "JAVA_HOME is set to \"$JAVA_HOME\", but \"\$JAVA_HOME/bin/java\" or \"\$JAVA_HOME/bin/javac\" does not exist." >&2 + return 1 + fi + fi + else + JAVACMD="$( + 'set' +e + 'unset' -f command 2>/dev/null + 'command' -v java + )" || : + JAVACCMD="$( + 'set' +e + 'unset' -f command 2>/dev/null + 'command' -v javac + )" || : + + if [ ! -x "${JAVACMD-}" ] || [ ! -x "${JAVACCMD-}" ]; then + echo "The java/javac command does not exist in PATH nor is JAVA_HOME set, so mvnw cannot run." >&2 + return 1 + fi + fi +} + +# hash string like Java String::hashCode +hash_string() { + str="${1:-}" h=0 + while [ -n "$str" ]; do + char="${str%"${str#?}"}" + h=$(((h * 31 + $(LC_CTYPE=C printf %d "'$char")) % 4294967296)) + str="${str#?}" + done + printf %x\\n $h +} + +verbose() { :; } +[ "${MVNW_VERBOSE-}" != true ] || verbose() { printf %s\\n "${1-}"; } + +die() { + printf %s\\n "$1" >&2 + exit 1 +} + +trim() { + # MWRAPPER-139: + # Trims trailing and leading whitespace, carriage returns, tabs, and linefeeds. + # Needed for removing poorly interpreted newline sequences when running in more + # exotic environments such as mingw bash on Windows. + printf "%s" "${1}" | tr -d '[:space:]' +} + +scriptDir="$(dirname "$0")" +scriptName="$(basename "$0")" + +# parse distributionUrl and optional distributionSha256Sum, requires .mvn/wrapper/maven-wrapper.properties +while IFS="=" read -r key value; do + case "${key-}" in + distributionUrl) distributionUrl=$(trim "${value-}") ;; + distributionSha256Sum) distributionSha256Sum=$(trim "${value-}") ;; + esac +done <"$scriptDir/.mvn/wrapper/maven-wrapper.properties" +[ -n "${distributionUrl-}" ] || die "cannot read distributionUrl property in $scriptDir/.mvn/wrapper/maven-wrapper.properties" + +case "${distributionUrl##*/}" in +maven-mvnd-*bin.*) + MVN_CMD=mvnd.sh _MVNW_REPO_PATTERN=/maven/mvnd/ + case "${PROCESSOR_ARCHITECTURE-}${PROCESSOR_ARCHITEW6432-}:$(uname -a)" in + *AMD64:CYGWIN* | *AMD64:MINGW*) distributionPlatform=windows-amd64 ;; + :Darwin*x86_64) distributionPlatform=darwin-amd64 ;; + :Darwin*arm64) distributionPlatform=darwin-aarch64 ;; + :Linux*x86_64*) distributionPlatform=linux-amd64 ;; + *) + echo "Cannot detect native platform for mvnd on $(uname)-$(uname -m), use pure java version" >&2 + distributionPlatform=linux-amd64 + ;; + esac + distributionUrl="${distributionUrl%-bin.*}-$distributionPlatform.zip" + ;; +maven-mvnd-*) MVN_CMD=mvnd.sh _MVNW_REPO_PATTERN=/maven/mvnd/ ;; +*) MVN_CMD="mvn${scriptName#mvnw}" _MVNW_REPO_PATTERN=/org/apache/maven/ ;; +esac + +# apply MVNW_REPOURL and calculate MAVEN_HOME +# maven home pattern: ~/.m2/wrapper/dists/{apache-maven-,maven-mvnd--}/ +[ -z "${MVNW_REPOURL-}" ] || distributionUrl="$MVNW_REPOURL$_MVNW_REPO_PATTERN${distributionUrl#*"$_MVNW_REPO_PATTERN"}" +distributionUrlName="${distributionUrl##*/}" +distributionUrlNameMain="${distributionUrlName%.*}" +distributionUrlNameMain="${distributionUrlNameMain%-bin}" +MAVEN_USER_HOME="${MAVEN_USER_HOME:-${HOME}/.m2}" +MAVEN_HOME="${MAVEN_USER_HOME}/wrapper/dists/${distributionUrlNameMain-}/$(hash_string "$distributionUrl")" + +exec_maven() { + unset MVNW_VERBOSE MVNW_USERNAME MVNW_PASSWORD MVNW_REPOURL || : + exec "$MAVEN_HOME/bin/$MVN_CMD" "$@" || die "cannot exec $MAVEN_HOME/bin/$MVN_CMD" +} + +if [ -d "$MAVEN_HOME" ]; then + verbose "found existing MAVEN_HOME at $MAVEN_HOME" + exec_maven "$@" +fi + +case "${distributionUrl-}" in +*?-bin.zip | *?maven-mvnd-?*-?*.zip) ;; +*) die "distributionUrl is not valid, must match *-bin.zip or maven-mvnd-*.zip, but found '${distributionUrl-}'" ;; +esac + +# prepare tmp dir +if TMP_DOWNLOAD_DIR="$(mktemp -d)" && [ -d "$TMP_DOWNLOAD_DIR" ]; then + clean() { rm -rf -- "$TMP_DOWNLOAD_DIR"; } + trap clean HUP INT TERM EXIT +else + die "cannot create temp dir" +fi + +mkdir -p -- "${MAVEN_HOME%/*}" + +# Download and Install Apache Maven +verbose "Couldn't find MAVEN_HOME, downloading and installing it ..." +verbose "Downloading from: $distributionUrl" +verbose "Downloading to: $TMP_DOWNLOAD_DIR/$distributionUrlName" + +# select .zip or .tar.gz +if ! command -v unzip >/dev/null; then + distributionUrl="${distributionUrl%.zip}.tar.gz" + distributionUrlName="${distributionUrl##*/}" +fi + +# verbose opt +__MVNW_QUIET_WGET=--quiet __MVNW_QUIET_CURL=--silent __MVNW_QUIET_UNZIP=-q __MVNW_QUIET_TAR='' +[ "${MVNW_VERBOSE-}" != true ] || __MVNW_QUIET_WGET='' __MVNW_QUIET_CURL='' __MVNW_QUIET_UNZIP='' __MVNW_QUIET_TAR=v + +# normalize http auth +case "${MVNW_PASSWORD:+has-password}" in +'') MVNW_USERNAME='' MVNW_PASSWORD='' ;; +has-password) [ -n "${MVNW_USERNAME-}" ] || MVNW_USERNAME='' MVNW_PASSWORD='' ;; +esac + +if [ -z "${MVNW_USERNAME-}" ] && command -v wget >/dev/null; then + verbose "Found wget ... using wget" + wget ${__MVNW_QUIET_WGET:+"$__MVNW_QUIET_WGET"} "$distributionUrl" -O "$TMP_DOWNLOAD_DIR/$distributionUrlName" || die "wget: Failed to fetch $distributionUrl" +elif [ -z "${MVNW_USERNAME-}" ] && command -v curl >/dev/null; then + verbose "Found curl ... using curl" + curl ${__MVNW_QUIET_CURL:+"$__MVNW_QUIET_CURL"} -f -L -o "$TMP_DOWNLOAD_DIR/$distributionUrlName" "$distributionUrl" || die "curl: Failed to fetch $distributionUrl" +elif set_java_home; then + verbose "Falling back to use Java to download" + javaSource="$TMP_DOWNLOAD_DIR/Downloader.java" + targetZip="$TMP_DOWNLOAD_DIR/$distributionUrlName" + cat >"$javaSource" <<-END + public class Downloader extends java.net.Authenticator + { + protected java.net.PasswordAuthentication getPasswordAuthentication() + { + return new java.net.PasswordAuthentication( System.getenv( "MVNW_USERNAME" ), System.getenv( "MVNW_PASSWORD" ).toCharArray() ); + } + public static void main( String[] args ) throws Exception + { + setDefault( new Downloader() ); + java.nio.file.Files.copy( java.net.URI.create( args[0] ).toURL().openStream(), java.nio.file.Paths.get( args[1] ).toAbsolutePath().normalize() ); + } + } + END + # For Cygwin/MinGW, switch paths to Windows format before running javac and java + verbose " - Compiling Downloader.java ..." + "$(native_path "$JAVACCMD")" "$(native_path "$javaSource")" || die "Failed to compile Downloader.java" + verbose " - Running Downloader.java ..." + "$(native_path "$JAVACMD")" -cp "$(native_path "$TMP_DOWNLOAD_DIR")" Downloader "$distributionUrl" "$(native_path "$targetZip")" +fi + +# If specified, validate the SHA-256 sum of the Maven distribution zip file +if [ -n "${distributionSha256Sum-}" ]; then + distributionSha256Result=false + if [ "$MVN_CMD" = mvnd.sh ]; then + echo "Checksum validation is not supported for maven-mvnd." >&2 + echo "Please disable validation by removing 'distributionSha256Sum' from your maven-wrapper.properties." >&2 + exit 1 + elif command -v sha256sum >/dev/null; then + if echo "$distributionSha256Sum $TMP_DOWNLOAD_DIR/$distributionUrlName" | sha256sum -c - >/dev/null 2>&1; then + distributionSha256Result=true + fi + elif command -v shasum >/dev/null; then + if echo "$distributionSha256Sum $TMP_DOWNLOAD_DIR/$distributionUrlName" | shasum -a 256 -c >/dev/null 2>&1; then + distributionSha256Result=true + fi + else + echo "Checksum validation was requested but neither 'sha256sum' or 'shasum' are available." >&2 + echo "Please install either command, or disable validation by removing 'distributionSha256Sum' from your maven-wrapper.properties." >&2 + exit 1 + fi + if [ $distributionSha256Result = false ]; then + echo "Error: Failed to validate Maven distribution SHA-256, your Maven distribution might be compromised." >&2 + echo "If you updated your Maven version, you need to update the specified distributionSha256Sum property." >&2 + exit 1 + fi +fi + +# unzip and move +if command -v unzip >/dev/null; then + unzip ${__MVNW_QUIET_UNZIP:+"$__MVNW_QUIET_UNZIP"} "$TMP_DOWNLOAD_DIR/$distributionUrlName" -d "$TMP_DOWNLOAD_DIR" || die "failed to unzip" +else + tar xzf${__MVNW_QUIET_TAR:+"$__MVNW_QUIET_TAR"} "$TMP_DOWNLOAD_DIR/$distributionUrlName" -C "$TMP_DOWNLOAD_DIR" || die "failed to untar" +fi + +# Find the actual extracted directory name (handles snapshots where filename != directory name) +actualDistributionDir="" + +# First try the expected directory name (for regular distributions) +if [ -d "$TMP_DOWNLOAD_DIR/$distributionUrlNameMain" ]; then + if [ -f "$TMP_DOWNLOAD_DIR/$distributionUrlNameMain/bin/$MVN_CMD" ]; then + actualDistributionDir="$distributionUrlNameMain" + fi +fi + +# If not found, search for any directory with the Maven executable (for snapshots) +if [ -z "$actualDistributionDir" ]; then + # enable globbing to iterate over items + set +f + for dir in "$TMP_DOWNLOAD_DIR"/*; do + if [ -d "$dir" ]; then + if [ -f "$dir/bin/$MVN_CMD" ]; then + actualDistributionDir="$(basename "$dir")" + break + fi + fi + done + set -f +fi + +if [ -z "$actualDistributionDir" ]; then + verbose "Contents of $TMP_DOWNLOAD_DIR:" + verbose "$(ls -la "$TMP_DOWNLOAD_DIR")" + die "Could not find Maven distribution directory in extracted archive" +fi + +verbose "Found extracted Maven distribution directory: $actualDistributionDir" +printf %s\\n "$distributionUrl" >"$TMP_DOWNLOAD_DIR/$actualDistributionDir/mvnw.url" +mv -- "$TMP_DOWNLOAD_DIR/$actualDistributionDir" "$MAVEN_HOME" || [ -d "$MAVEN_HOME" ] || die "fail to move MAVEN_HOME" + +clean || : +exec_maven "$@" diff --git a/mvnw.cmd b/mvnw.cmd new file mode 100644 index 0000000..92450f9 --- /dev/null +++ b/mvnw.cmd @@ -0,0 +1,189 @@ +<# : batch portion +@REM ---------------------------------------------------------------------------- +@REM Licensed to the Apache Software Foundation (ASF) under one +@REM or more contributor license agreements. See the NOTICE file +@REM distributed with this work for additional information +@REM regarding copyright ownership. The ASF licenses this file +@REM to you under the Apache License, Version 2.0 (the +@REM "License"); you may not use this file except in compliance +@REM with the License. You may obtain a copy of the License at +@REM +@REM http://www.apache.org/licenses/LICENSE-2.0 +@REM +@REM Unless required by applicable law or agreed to in writing, +@REM software distributed under the License is distributed on an +@REM "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY +@REM KIND, either express or implied. See the License for the +@REM specific language governing permissions and limitations +@REM under the License. +@REM ---------------------------------------------------------------------------- + +@REM ---------------------------------------------------------------------------- +@REM Apache Maven Wrapper startup batch script, version 3.3.4 +@REM +@REM Optional ENV vars +@REM MVNW_REPOURL - repo url base for downloading maven distribution +@REM MVNW_USERNAME/MVNW_PASSWORD - user and password for downloading maven +@REM MVNW_VERBOSE - true: enable verbose log; others: silence the output +@REM ---------------------------------------------------------------------------- + +@IF "%__MVNW_ARG0_NAME__%"=="" (SET __MVNW_ARG0_NAME__=%~nx0) +@SET __MVNW_CMD__= +@SET __MVNW_ERROR__= +@SET __MVNW_PSMODULEP_SAVE=%PSModulePath% +@SET PSModulePath= +@FOR /F "usebackq tokens=1* delims==" %%A IN (`powershell -noprofile "& {$scriptDir='%~dp0'; $script='%__MVNW_ARG0_NAME__%'; icm -ScriptBlock ([Scriptblock]::Create((Get-Content -Raw '%~f0'))) -NoNewScope}"`) DO @( + IF "%%A"=="MVN_CMD" (set __MVNW_CMD__=%%B) ELSE IF "%%B"=="" (echo %%A) ELSE (echo %%A=%%B) +) +@SET PSModulePath=%__MVNW_PSMODULEP_SAVE% +@SET __MVNW_PSMODULEP_SAVE= +@SET __MVNW_ARG0_NAME__= +@SET MVNW_USERNAME= +@SET MVNW_PASSWORD= +@IF NOT "%__MVNW_CMD__%"=="" ("%__MVNW_CMD__%" %*) +@echo Cannot start maven from wrapper >&2 && exit /b 1 +@GOTO :EOF +: end batch / begin powershell #> + +$ErrorActionPreference = "Stop" +if ($env:MVNW_VERBOSE -eq "true") { + $VerbosePreference = "Continue" +} + +# calculate distributionUrl, requires .mvn/wrapper/maven-wrapper.properties +$distributionUrl = (Get-Content -Raw "$scriptDir/.mvn/wrapper/maven-wrapper.properties" | ConvertFrom-StringData).distributionUrl +if (!$distributionUrl) { + Write-Error "cannot read distributionUrl property in $scriptDir/.mvn/wrapper/maven-wrapper.properties" +} + +switch -wildcard -casesensitive ( $($distributionUrl -replace '^.*/','') ) { + "maven-mvnd-*" { + $USE_MVND = $true + $distributionUrl = $distributionUrl -replace '-bin\.[^.]*$',"-windows-amd64.zip" + $MVN_CMD = "mvnd.cmd" + break + } + default { + $USE_MVND = $false + $MVN_CMD = $script -replace '^mvnw','mvn' + break + } +} + +# apply MVNW_REPOURL and calculate MAVEN_HOME +# maven home pattern: ~/.m2/wrapper/dists/{apache-maven-,maven-mvnd--}/ +if ($env:MVNW_REPOURL) { + $MVNW_REPO_PATTERN = if ($USE_MVND -eq $False) { "/org/apache/maven/" } else { "/maven/mvnd/" } + $distributionUrl = "$env:MVNW_REPOURL$MVNW_REPO_PATTERN$($distributionUrl -replace "^.*$MVNW_REPO_PATTERN",'')" +} +$distributionUrlName = $distributionUrl -replace '^.*/','' +$distributionUrlNameMain = $distributionUrlName -replace '\.[^.]*$','' -replace '-bin$','' + +$MAVEN_M2_PATH = "$HOME/.m2" +if ($env:MAVEN_USER_HOME) { + $MAVEN_M2_PATH = "$env:MAVEN_USER_HOME" +} + +if (-not (Test-Path -Path $MAVEN_M2_PATH)) { + New-Item -Path $MAVEN_M2_PATH -ItemType Directory | Out-Null +} + +$MAVEN_WRAPPER_DISTS = $null +if ((Get-Item $MAVEN_M2_PATH).Target[0] -eq $null) { + $MAVEN_WRAPPER_DISTS = "$MAVEN_M2_PATH/wrapper/dists" +} else { + $MAVEN_WRAPPER_DISTS = (Get-Item $MAVEN_M2_PATH).Target[0] + "/wrapper/dists" +} + +$MAVEN_HOME_PARENT = "$MAVEN_WRAPPER_DISTS/$distributionUrlNameMain" +$MAVEN_HOME_NAME = ([System.Security.Cryptography.SHA256]::Create().ComputeHash([byte[]][char[]]$distributionUrl) | ForEach-Object {$_.ToString("x2")}) -join '' +$MAVEN_HOME = "$MAVEN_HOME_PARENT/$MAVEN_HOME_NAME" + +if (Test-Path -Path "$MAVEN_HOME" -PathType Container) { + Write-Verbose "found existing MAVEN_HOME at $MAVEN_HOME" + Write-Output "MVN_CMD=$MAVEN_HOME/bin/$MVN_CMD" + exit $? +} + +if (! $distributionUrlNameMain -or ($distributionUrlName -eq $distributionUrlNameMain)) { + Write-Error "distributionUrl is not valid, must end with *-bin.zip, but found $distributionUrl" +} + +# prepare tmp dir +$TMP_DOWNLOAD_DIR_HOLDER = New-TemporaryFile +$TMP_DOWNLOAD_DIR = New-Item -Itemtype Directory -Path "$TMP_DOWNLOAD_DIR_HOLDER.dir" +$TMP_DOWNLOAD_DIR_HOLDER.Delete() | Out-Null +trap { + if ($TMP_DOWNLOAD_DIR.Exists) { + try { Remove-Item $TMP_DOWNLOAD_DIR -Recurse -Force | Out-Null } + catch { Write-Warning "Cannot remove $TMP_DOWNLOAD_DIR" } + } +} + +New-Item -Itemtype Directory -Path "$MAVEN_HOME_PARENT" -Force | Out-Null + +# Download and Install Apache Maven +Write-Verbose "Couldn't find MAVEN_HOME, downloading and installing it ..." +Write-Verbose "Downloading from: $distributionUrl" +Write-Verbose "Downloading to: $TMP_DOWNLOAD_DIR/$distributionUrlName" + +$webclient = New-Object System.Net.WebClient +if ($env:MVNW_USERNAME -and $env:MVNW_PASSWORD) { + $webclient.Credentials = New-Object System.Net.NetworkCredential($env:MVNW_USERNAME, $env:MVNW_PASSWORD) +} +[Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12 +$webclient.DownloadFile($distributionUrl, "$TMP_DOWNLOAD_DIR/$distributionUrlName") | Out-Null + +# If specified, validate the SHA-256 sum of the Maven distribution zip file +$distributionSha256Sum = (Get-Content -Raw "$scriptDir/.mvn/wrapper/maven-wrapper.properties" | ConvertFrom-StringData).distributionSha256Sum +if ($distributionSha256Sum) { + if ($USE_MVND) { + Write-Error "Checksum validation is not supported for maven-mvnd. `nPlease disable validation by removing 'distributionSha256Sum' from your maven-wrapper.properties." + } + Import-Module $PSHOME\Modules\Microsoft.PowerShell.Utility -Function Get-FileHash + if ((Get-FileHash "$TMP_DOWNLOAD_DIR/$distributionUrlName" -Algorithm SHA256).Hash.ToLower() -ne $distributionSha256Sum) { + Write-Error "Error: Failed to validate Maven distribution SHA-256, your Maven distribution might be compromised. If you updated your Maven version, you need to update the specified distributionSha256Sum property." + } +} + +# unzip and move +Expand-Archive "$TMP_DOWNLOAD_DIR/$distributionUrlName" -DestinationPath "$TMP_DOWNLOAD_DIR" | Out-Null + +# Find the actual extracted directory name (handles snapshots where filename != directory name) +$actualDistributionDir = "" + +# First try the expected directory name (for regular distributions) +$expectedPath = Join-Path "$TMP_DOWNLOAD_DIR" "$distributionUrlNameMain" +$expectedMvnPath = Join-Path "$expectedPath" "bin/$MVN_CMD" +if ((Test-Path -Path $expectedPath -PathType Container) -and (Test-Path -Path $expectedMvnPath -PathType Leaf)) { + $actualDistributionDir = $distributionUrlNameMain +} + +# If not found, search for any directory with the Maven executable (for snapshots) +if (!$actualDistributionDir) { + Get-ChildItem -Path "$TMP_DOWNLOAD_DIR" -Directory | ForEach-Object { + $testPath = Join-Path $_.FullName "bin/$MVN_CMD" + if (Test-Path -Path $testPath -PathType Leaf) { + $actualDistributionDir = $_.Name + } + } +} + +if (!$actualDistributionDir) { + Write-Error "Could not find Maven distribution directory in extracted archive" +} + +Write-Verbose "Found extracted Maven distribution directory: $actualDistributionDir" +Rename-Item -Path "$TMP_DOWNLOAD_DIR/$actualDistributionDir" -NewName $MAVEN_HOME_NAME | Out-Null +try { + Move-Item -Path "$TMP_DOWNLOAD_DIR/$MAVEN_HOME_NAME" -Destination $MAVEN_HOME_PARENT | Out-Null +} catch { + if (! (Test-Path -Path "$MAVEN_HOME" -PathType Container)) { + Write-Error "fail to move MAVEN_HOME" + } +} finally { + try { Remove-Item $TMP_DOWNLOAD_DIR -Recurse -Force | Out-Null } + catch { Write-Warning "Cannot remove $TMP_DOWNLOAD_DIR" } +} + +Write-Output "MVN_CMD=$MAVEN_HOME/bin/$MVN_CMD" diff --git a/pom.xml b/pom.xml new file mode 100644 index 0000000..14b5412 --- /dev/null +++ b/pom.xml @@ -0,0 +1,124 @@ + + + 4.0.0 + + org.springframework.boot + spring-boot-starter-parent + 3.3.4 + + + org.example + springboot-web + 0.0.1-SNAPSHOT + springboot-web + springboot-web + + + 17 + 1.5.5.Final + 1.18.34 + 0.2.0 + 3.3.0 + + + + + org.springframework.boot + spring-boot-starter-data-jpa + + + org.springframework.boot + spring-boot-starter-web + + + com.h2database + h2 + runtime + + + com.mysql + mysql-connector-j + runtime + + + org.projectlombok + lombok + ${lombok.version} + true + + + org.mapstruct + mapstruct + ${mapstruct.version} + + + org.springframework.boot + spring-boot-starter-test + test + + + + + + + org.springframework.boot + spring-boot-maven-plugin + + + + org.projectlombok + lombok + + + + + + org.apache.maven.plugins + maven-compiler-plugin + 3.13.0 + + ${java.version} + ${java.version} + + + org.projectlombok + lombok + ${lombok.version} + + + org.projectlombok + lombok-mapstruct-binding + ${lombok.mapstruct.binding.version} + + + org.mapstruct + mapstruct-processor + ${mapstruct.version} + + + + + + org.apache.maven.plugins + maven-checkstyle-plugin + ${checkstyle.plugin.version} + + + compile + + check + + + + + checkstyle.xml + true + true + false + src/main + + + + + diff --git a/src/main/java/org/example/springbootweb/SpringbootWebApplication.java b/src/main/java/org/example/springbootweb/SpringbootWebApplication.java new file mode 100644 index 0000000..d28729f --- /dev/null +++ b/src/main/java/org/example/springbootweb/SpringbootWebApplication.java @@ -0,0 +1,13 @@ +package org.example.springbootweb; + +import org.springframework.boot.SpringApplication; +import org.springframework.boot.autoconfigure.SpringBootApplication; + +@SpringBootApplication +public class SpringbootWebApplication { + + public static void main(String[] args) { + SpringApplication.run(SpringbootWebApplication.class, args); + } + +} diff --git a/src/main/java/org/example/springbootweb/controller/BookController.java b/src/main/java/org/example/springbootweb/controller/BookController.java new file mode 100644 index 0000000..74a34b2 --- /dev/null +++ b/src/main/java/org/example/springbootweb/controller/BookController.java @@ -0,0 +1,30 @@ +package org.example.springbootweb.controller; + +import lombok.RequiredArgsConstructor; +import org.example.springbootweb.dto.BookDto; +import org.example.springbootweb.dto.CreateBookRequestDto; +import org.example.springbootweb.service.BookService; +import org.springframework.web.bind.annotation.*; +import java.util.List; + +@RestController +@RequestMapping("/api/books") +@RequiredArgsConstructor +public class BookController { + private final BookService bookService; + + @GetMapping + public List getAll() { + return bookService.getAll(); + } + + @GetMapping("/{id}") + public BookDto getBookById(@PathVariable Long id) { + return bookService.getBookById(id); + } + @PostMapping + public BookDto createBook(@RequestBody CreateBookRequestDto bookDto) { + return bookService.createBook(bookDto); + } +} + diff --git a/src/main/java/org/example/springbootweb/dto/BookDto.java b/src/main/java/org/example/springbootweb/dto/BookDto.java new file mode 100644 index 0000000..b24d7d0 --- /dev/null +++ b/src/main/java/org/example/springbootweb/dto/BookDto.java @@ -0,0 +1,15 @@ +package org.example.springbootweb.dto; + +import lombok.Data; +import java.math.BigDecimal; + +@Data +public class BookDto { + private Long id; + private String title; + private String author; + private String isbn; + private BigDecimal price; + private String description; + private String coverImage; +} diff --git a/src/main/java/org/example/springbootweb/dto/CreateBookRequestDto.java b/src/main/java/org/example/springbootweb/dto/CreateBookRequestDto.java new file mode 100644 index 0000000..0d69a4b --- /dev/null +++ b/src/main/java/org/example/springbootweb/dto/CreateBookRequestDto.java @@ -0,0 +1,14 @@ +package org.example.springbootweb.dto; + +import lombok.Data; +import java.math.BigDecimal; + +@Data +public class CreateBookRequestDto { + private String title; + private String author; + private String isbn; + private BigDecimal price; + private String description; + private String coverImage; +} diff --git a/src/main/java/org/example/springbootweb/exception/EntityNotFoundException.java b/src/main/java/org/example/springbootweb/exception/EntityNotFoundException.java new file mode 100644 index 0000000..b4a7eb1 --- /dev/null +++ b/src/main/java/org/example/springbootweb/exception/EntityNotFoundException.java @@ -0,0 +1,7 @@ +package org.example.springbootweb.exception; + +public class EntityNotFoundException extends RuntimeException { + public EntityNotFoundException(String message) { + super(message); + } +} diff --git a/src/main/java/org/example/springbootweb/mapper/BookMapper.java b/src/main/java/org/example/springbootweb/mapper/BookMapper.java new file mode 100644 index 0000000..e87ca58 --- /dev/null +++ b/src/main/java/org/example/springbootweb/mapper/BookMapper.java @@ -0,0 +1,16 @@ +package org.example.springbootweb.mapper; + +import org.example.springbootweb.dto.BookDto; +import org.example.springbootweb.dto.CreateBookRequestDto; +import org.mapstruct.Mapper; +import org.example.springbootweb.model.Book; +import org.mapstruct.Mapping; + +@Mapper(componentModel = "spring") +public interface BookMapper { + + BookDto toDto(Book book); + + @Mapping(target = "id", ignore = true) + Book toModel(CreateBookRequestDto requestDto); +} diff --git a/src/main/java/org/example/springbootweb/model/Book.java b/src/main/java/org/example/springbootweb/model/Book.java new file mode 100644 index 0000000..71e4af6 --- /dev/null +++ b/src/main/java/org/example/springbootweb/model/Book.java @@ -0,0 +1,24 @@ +package org.example.springbootweb.model; + +import jakarta.persistence.Entity; +import jakarta.persistence.GeneratedValue; +import jakarta.persistence.GenerationType; +import jakarta.persistence.Id; +import jakarta.persistence.Table; +import lombok.Data; +import java.math.BigDecimal; + +@Entity +@Table(name = "books") +@Data +public class Book { + @Id + @GeneratedValue(strategy = GenerationType.IDENTITY) + private Long id; + private String title; + private String author; + private String isbn; + private BigDecimal price; + private String description; + private String coverImage; +} diff --git a/src/main/java/org/example/springbootweb/repository/BookRepository.java b/src/main/java/org/example/springbootweb/repository/BookRepository.java new file mode 100644 index 0000000..728f56e --- /dev/null +++ b/src/main/java/org/example/springbootweb/repository/BookRepository.java @@ -0,0 +1,7 @@ +package org.example.springbootweb.repository; + +import org.example.springbootweb.model.Book; +import org.springframework.data.jpa.repository.JpaRepository; + +public interface BookRepository extends JpaRepository { +} diff --git a/src/main/java/org/example/springbootweb/service/BookService.java b/src/main/java/org/example/springbootweb/service/BookService.java new file mode 100644 index 0000000..86f300f --- /dev/null +++ b/src/main/java/org/example/springbootweb/service/BookService.java @@ -0,0 +1,11 @@ +package org.example.springbootweb.service; + +import org.example.springbootweb.dto.BookDto; +import org.example.springbootweb.dto.CreateBookRequestDto; +import java.util.List; + +public interface BookService { + BookDto createBook(CreateBookRequestDto bookDto); + List getAll(); + BookDto getBookById(Long id); +} diff --git a/src/main/java/org/example/springbootweb/service/impl/BookServiceImpl.java b/src/main/java/org/example/springbootweb/service/impl/BookServiceImpl.java new file mode 100644 index 0000000..1818535 --- /dev/null +++ b/src/main/java/org/example/springbootweb/service/impl/BookServiceImpl.java @@ -0,0 +1,41 @@ +package org.example.springbootweb.service.impl; + +import lombok.RequiredArgsConstructor; +import org.example.springbootweb.dto.BookDto; +import org.example.springbootweb.dto.CreateBookRequestDto; +import org.example.springbootweb.exception.EntityNotFoundException; +import org.example.springbootweb.mapper.BookMapper; +import org.example.springbootweb.model.Book; +import org.example.springbootweb.repository.BookRepository; +import org.example.springbootweb.service.BookService; +import org.springframework.stereotype.Service; +import java.util.List; + +@Service +@RequiredArgsConstructor +public class BookServiceImpl implements BookService { + private final BookRepository bookRepository; + private final BookMapper bookMapper; + + @Override + public BookDto createBook(CreateBookRequestDto requestDto) { + Book book = bookMapper.toModel(requestDto); + return bookMapper.toDto(bookRepository.save(book)); + } + + @Override + public List getAll() { + return bookRepository.findAll() + .stream() + .map(bookMapper::toDto) + .toList(); + } + + @Override + public BookDto getBookById(Long id) { + Book book = bookRepository.findById(id).orElseThrow( + () -> new EntityNotFoundException("Can't find book by id " + id) + ); + return bookMapper.toDto(book); + } +} diff --git a/src/main/resources/application.properties b/src/main/resources/application.properties new file mode 100644 index 0000000..ae9d5c0 --- /dev/null +++ b/src/main/resources/application.properties @@ -0,0 +1,7 @@ +spring.datasource.url=jdbc:h2:mem:testdb +spring.datasource.driverClassName=org.h2.Driver +spring.datasource.username=sa +spring.datasource.password=password +spring.jpa.database-platform=org.hibernate.dialect.H2Dialect +spring.jpa.show-sql=true +spring.jpa.hibernate.ddl-auto=update diff --git a/src/test/java/org/example/springbootweb/SpringbootWebApplicationTests.java b/src/test/java/org/example/springbootweb/SpringbootWebApplicationTests.java new file mode 100644 index 0000000..8eafb94 --- /dev/null +++ b/src/test/java/org/example/springbootweb/SpringbootWebApplicationTests.java @@ -0,0 +1,13 @@ +package org.example.springbootweb; + +import org.junit.jupiter.api.Test; +import org.springframework.boot.test.context.SpringBootTest; + +@SpringBootTest +class SpringbootWebApplicationTests { + + @Test + void contextLoads() { + } + +} From 867575d31b5373da7f8ad6e833544cd62f8e2743 Mon Sep 17 00:00:00 2001 From: obvly Date: Sat, 19 Sep 2026 01:07:33 +0200 Subject: [PATCH 2/5] fix: address review comments, switch to SessionFactory and MySQL --- .../controller/BookController.java | 3 + .../org/example/springbootweb/model/Book.java | 22 +++++--- .../repository/BookRepository.java | 8 ++- .../repository/BookRepositoryImpl.java | 56 +++++++++++++++++++ .../springbootweb/service/BookService.java | 2 + src/main/resources/application.properties | 13 +++-- 6 files changed, 89 insertions(+), 15 deletions(-) create mode 100644 src/main/java/org/example/springbootweb/repository/BookRepositoryImpl.java diff --git a/src/main/java/org/example/springbootweb/controller/BookController.java b/src/main/java/org/example/springbootweb/controller/BookController.java index 74a34b2..49b7f7e 100644 --- a/src/main/java/org/example/springbootweb/controller/BookController.java +++ b/src/main/java/org/example/springbootweb/controller/BookController.java @@ -4,6 +4,7 @@ import org.example.springbootweb.dto.BookDto; import org.example.springbootweb.dto.CreateBookRequestDto; import org.example.springbootweb.service.BookService; +import org.springframework.http.HttpStatus; import org.springframework.web.bind.annotation.*; import java.util.List; @@ -22,7 +23,9 @@ public List getAll() { public BookDto getBookById(@PathVariable Long id) { return bookService.getBookById(id); } + @PostMapping + @ResponseStatus(HttpStatus.CREATED) // Add status 201 public BookDto createBook(@RequestBody CreateBookRequestDto bookDto) { return bookService.createBook(bookDto); } diff --git a/src/main/java/org/example/springbootweb/model/Book.java b/src/main/java/org/example/springbootweb/model/Book.java index 71e4af6..ab6f130 100644 --- a/src/main/java/org/example/springbootweb/model/Book.java +++ b/src/main/java/org/example/springbootweb/model/Book.java @@ -1,24 +1,32 @@ package org.example.springbootweb.model; -import jakarta.persistence.Entity; -import jakarta.persistence.GeneratedValue; -import jakarta.persistence.GenerationType; -import jakarta.persistence.Id; -import jakarta.persistence.Table; -import lombok.Data; +import jakarta.persistence.*; +import lombok.Getter; +import lombok.Setter; + import java.math.BigDecimal; @Entity @Table(name = "books") -@Data +@Getter +@Setter public class Book { @Id @GeneratedValue(strategy = GenerationType.IDENTITY) private Long id; + + @Column(nullable = false) private String title; + + @Column(nullable = false) private String author; + + @Column(unique = true, nullable = false) private String isbn; + + @Column(nullable = false) private BigDecimal price; + private String description; private String coverImage; } diff --git a/src/main/java/org/example/springbootweb/repository/BookRepository.java b/src/main/java/org/example/springbootweb/repository/BookRepository.java index 728f56e..245ea5a 100644 --- a/src/main/java/org/example/springbootweb/repository/BookRepository.java +++ b/src/main/java/org/example/springbootweb/repository/BookRepository.java @@ -1,7 +1,11 @@ package org.example.springbootweb.repository; import org.example.springbootweb.model.Book; -import org.springframework.data.jpa.repository.JpaRepository; +import java.util.List; +import java.util.Optional; -public interface BookRepository extends JpaRepository { +public interface BookRepository { + Book save(Book book); + List findAll(); + Optional findById(Long id); } diff --git a/src/main/java/org/example/springbootweb/repository/BookRepositoryImpl.java b/src/main/java/org/example/springbootweb/repository/BookRepositoryImpl.java new file mode 100644 index 0000000..7316bbf --- /dev/null +++ b/src/main/java/org/example/springbootweb/repository/BookRepositoryImpl.java @@ -0,0 +1,56 @@ +package org.example.springbootweb.repository; + +import java.util.List; +import java.util.Optional; +import lombok.RequiredArgsConstructor; +import org.example.springbootweb.model.Book; +import org.hibernate.Session; +import org.hibernate.SessionFactory; +import org.hibernate.Transaction; +import org.springframework.stereotype.Repository; + +@Repository +@RequiredArgsConstructor +public class BookRepositoryImpl implements BookRepository { + private final SessionFactory sessionFactory; + + @Override + public Book save(Book book) { + Session session = null; + Transaction transaction = null; + try { + session = sessionFactory.openSession(); + transaction = session.beginTransaction(); + session.persist(book); + transaction.commit(); + return book; + } catch (Exception e) { + if (transaction != null) { + transaction.rollback(); + } + throw new RuntimeException("Can't insert book: " + book, e); + } finally { + if (session != null) { + session.close(); + } + } + } + + @Override + public List findAll() { + try (Session session = sessionFactory.openSession()) { + return session.createQuery("from Book", Book.class).list(); + } catch (Exception e) { + throw new RuntimeException("Can't get all books", e); + } + } + + @Override + public Optional findById(Long id) { + try (Session session = sessionFactory.openSession()) { + return Optional.ofNullable(session.get(Book.class, id)); + } catch (Exception e) { + throw new RuntimeException("Can't find book by id: " + id, e); + } + } +} diff --git a/src/main/java/org/example/springbootweb/service/BookService.java b/src/main/java/org/example/springbootweb/service/BookService.java index 86f300f..8873700 100644 --- a/src/main/java/org/example/springbootweb/service/BookService.java +++ b/src/main/java/org/example/springbootweb/service/BookService.java @@ -6,6 +6,8 @@ public interface BookService { BookDto createBook(CreateBookRequestDto bookDto); + List getAll(); + BookDto getBookById(Long id); } diff --git a/src/main/resources/application.properties b/src/main/resources/application.properties index ae9d5c0..6c9c52a 100644 --- a/src/main/resources/application.properties +++ b/src/main/resources/application.properties @@ -1,7 +1,8 @@ -spring.datasource.url=jdbc:h2:mem:testdb -spring.datasource.driverClassName=org.h2.Driver -spring.datasource.username=sa -spring.datasource.password=password -spring.jpa.database-platform=org.hibernate.dialect.H2Dialect +spring.datasource.url=jdbc:mysql://localhost:3306/online_bookstore?allowPublicKeyRetrieval=true&useSSL=false&serverTimezone=UTC +spring.datasource.username=root +spring.datasource.password=1234 +spring.datasource.driver-class-name=com.mysql.cj.jdbc.Driver + +spring.jpa.hibernate.ddl-auto=create-drop spring.jpa.show-sql=true -spring.jpa.hibernate.ddl-auto=update +spring.jpa.properties.hibernate.dialect=org.hibernate.dialect.MySQLDialect From 52c1d68b6d40792bbf14601458f38198e598f8ca Mon Sep 17 00:00:00 2001 From: obvly Date: Mon, 21 Sep 2026 13:55:40 +0200 Subject: [PATCH 3/5] fix: resolve checkstyle star imports and finalize repository --- checkstyle.xml | 14 ++++++++++++++ pom.xml | 3 ++- .../springbootweb/controller/BookController.java | 10 ++++++++-- .../java/org/example/springbootweb/model/Book.java | 8 ++++++-- .../springbootweb/repository/BookRepository.java | 2 ++ 5 files changed, 32 insertions(+), 5 deletions(-) diff --git a/checkstyle.xml b/checkstyle.xml index b971879..9ad5c7f 100644 --- a/checkstyle.xml +++ b/checkstyle.xml @@ -4,5 +4,19 @@ "https://checkstyle.org/dtds/configuration_1_3.dtd"> + + + + + + + + + + + + + + diff --git a/pom.xml b/pom.xml index 14b5412..705cffb 100644 --- a/pom.xml +++ b/pom.xml @@ -12,7 +12,6 @@ springboot-web 0.0.1-SNAPSHOT springboot-web - springboot-web 17 @@ -73,6 +72,7 @@ + org.apache.maven.plugins maven-compiler-plugin @@ -99,6 +99,7 @@ + org.apache.maven.plugins maven-checkstyle-plugin diff --git a/src/main/java/org/example/springbootweb/controller/BookController.java b/src/main/java/org/example/springbootweb/controller/BookController.java index 49b7f7e..cb7d678 100644 --- a/src/main/java/org/example/springbootweb/controller/BookController.java +++ b/src/main/java/org/example/springbootweb/controller/BookController.java @@ -5,7 +5,13 @@ import org.example.springbootweb.dto.CreateBookRequestDto; import org.example.springbootweb.service.BookService; import org.springframework.http.HttpStatus; -import org.springframework.web.bind.annotation.*; +import org.springframework.web.bind.annotation.GetMapping; +import org.springframework.web.bind.annotation.PathVariable; +import org.springframework.web.bind.annotation.PostMapping; +import org.springframework.web.bind.annotation.RequestBody; +import org.springframework.web.bind.annotation.RequestMapping; +import org.springframework.web.bind.annotation.ResponseStatus; +import org.springframework.web.bind.annotation.RestController; import java.util.List; @RestController @@ -25,7 +31,7 @@ public BookDto getBookById(@PathVariable Long id) { } @PostMapping - @ResponseStatus(HttpStatus.CREATED) // Add status 201 + @ResponseStatus(HttpStatus.CREATED) public BookDto createBook(@RequestBody CreateBookRequestDto bookDto) { return bookService.createBook(bookDto); } diff --git a/src/main/java/org/example/springbootweb/model/Book.java b/src/main/java/org/example/springbootweb/model/Book.java index ab6f130..65b0846 100644 --- a/src/main/java/org/example/springbootweb/model/Book.java +++ b/src/main/java/org/example/springbootweb/model/Book.java @@ -1,9 +1,13 @@ package org.example.springbootweb.model; -import jakarta.persistence.*; +import jakarta.persistence.Column; +import jakarta.persistence.Entity; +import jakarta.persistence.GeneratedValue; +import jakarta.persistence.GenerationType; +import jakarta.persistence.Id; +import jakarta.persistence.Table; import lombok.Getter; import lombok.Setter; - import java.math.BigDecimal; @Entity diff --git a/src/main/java/org/example/springbootweb/repository/BookRepository.java b/src/main/java/org/example/springbootweb/repository/BookRepository.java index 245ea5a..0076d21 100644 --- a/src/main/java/org/example/springbootweb/repository/BookRepository.java +++ b/src/main/java/org/example/springbootweb/repository/BookRepository.java @@ -6,6 +6,8 @@ public interface BookRepository { Book save(Book book); + List findAll(); + Optional findById(Long id); } From 553f42447ce468b4eac843410b623c951b295cc4 Mon Sep 17 00:00:00 2001 From: obvly Date: Mon, 21 Sep 2026 23:34:07 +0200 Subject: [PATCH 4/5] fix: resolve checkstyle violations, add CI/CD workflow and format repository --- .github/workflows/ci.yml | 20 ++ checkstyle.xml | 240 +++++++++++++++++- .../controller/BookController.java | 2 +- .../example/springbootweb/dto/BookDto.java | 2 +- .../dto/CreateBookRequestDto.java | 2 +- .../springbootweb/mapper/BookMapper.java | 2 +- .../org/example/springbootweb/model/Book.java | 2 +- .../repository/BookRepository.java | 2 +- .../springbootweb/service/BookService.java | 2 +- .../service/impl/BookServiceImpl.java | 2 +- src/main/resources/application.properties | 3 +- 11 files changed, 264 insertions(+), 15 deletions(-) create mode 100644 .github/workflows/ci.yml diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml new file mode 100644 index 0000000..bc4db7d --- /dev/null +++ b/.github/workflows/ci.yml @@ -0,0 +1,20 @@ +name: Java CI + +on: + - push + - pull_request + +jobs: + build: + runs-on: ubuntu-latest + + steps: + - uses: actions/checkout@v4 + - name: Set up JDK 17 + uses: actions/setup-java@v4 + with: + java-version: '17' + distribution: 'temurin' + cache: maven + - name: Build with Maven + run: mvn --batch-mode --update-snapshots verify diff --git a/checkstyle.xml b/checkstyle.xml index 9ad5c7f..ea0a529 100644 --- a/checkstyle.xml +++ b/checkstyle.xml @@ -2,21 +2,249 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - + + - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/src/main/java/org/example/springbootweb/controller/BookController.java b/src/main/java/org/example/springbootweb/controller/BookController.java index cb7d678..88fa56c 100644 --- a/src/main/java/org/example/springbootweb/controller/BookController.java +++ b/src/main/java/org/example/springbootweb/controller/BookController.java @@ -1,5 +1,6 @@ package org.example.springbootweb.controller; +import java.util.List; import lombok.RequiredArgsConstructor; import org.example.springbootweb.dto.BookDto; import org.example.springbootweb.dto.CreateBookRequestDto; @@ -12,7 +13,6 @@ import org.springframework.web.bind.annotation.RequestMapping; import org.springframework.web.bind.annotation.ResponseStatus; import org.springframework.web.bind.annotation.RestController; -import java.util.List; @RestController @RequestMapping("/api/books") diff --git a/src/main/java/org/example/springbootweb/dto/BookDto.java b/src/main/java/org/example/springbootweb/dto/BookDto.java index b24d7d0..df11351 100644 --- a/src/main/java/org/example/springbootweb/dto/BookDto.java +++ b/src/main/java/org/example/springbootweb/dto/BookDto.java @@ -1,7 +1,7 @@ package org.example.springbootweb.dto; -import lombok.Data; import java.math.BigDecimal; +import lombok.Data; @Data public class BookDto { diff --git a/src/main/java/org/example/springbootweb/dto/CreateBookRequestDto.java b/src/main/java/org/example/springbootweb/dto/CreateBookRequestDto.java index 0d69a4b..ea403e8 100644 --- a/src/main/java/org/example/springbootweb/dto/CreateBookRequestDto.java +++ b/src/main/java/org/example/springbootweb/dto/CreateBookRequestDto.java @@ -1,7 +1,7 @@ package org.example.springbootweb.dto; -import lombok.Data; import java.math.BigDecimal; +import lombok.Data; @Data public class CreateBookRequestDto { diff --git a/src/main/java/org/example/springbootweb/mapper/BookMapper.java b/src/main/java/org/example/springbootweb/mapper/BookMapper.java index e87ca58..33c6b2f 100644 --- a/src/main/java/org/example/springbootweb/mapper/BookMapper.java +++ b/src/main/java/org/example/springbootweb/mapper/BookMapper.java @@ -2,8 +2,8 @@ import org.example.springbootweb.dto.BookDto; import org.example.springbootweb.dto.CreateBookRequestDto; -import org.mapstruct.Mapper; import org.example.springbootweb.model.Book; +import org.mapstruct.Mapper; import org.mapstruct.Mapping; @Mapper(componentModel = "spring") diff --git a/src/main/java/org/example/springbootweb/model/Book.java b/src/main/java/org/example/springbootweb/model/Book.java index 65b0846..c0fe5d2 100644 --- a/src/main/java/org/example/springbootweb/model/Book.java +++ b/src/main/java/org/example/springbootweb/model/Book.java @@ -6,9 +6,9 @@ import jakarta.persistence.GenerationType; import jakarta.persistence.Id; import jakarta.persistence.Table; +import java.math.BigDecimal; import lombok.Getter; import lombok.Setter; -import java.math.BigDecimal; @Entity @Table(name = "books") diff --git a/src/main/java/org/example/springbootweb/repository/BookRepository.java b/src/main/java/org/example/springbootweb/repository/BookRepository.java index 0076d21..1d43864 100644 --- a/src/main/java/org/example/springbootweb/repository/BookRepository.java +++ b/src/main/java/org/example/springbootweb/repository/BookRepository.java @@ -1,8 +1,8 @@ package org.example.springbootweb.repository; -import org.example.springbootweb.model.Book; import java.util.List; import java.util.Optional; +import org.example.springbootweb.model.Book; public interface BookRepository { Book save(Book book); diff --git a/src/main/java/org/example/springbootweb/service/BookService.java b/src/main/java/org/example/springbootweb/service/BookService.java index 8873700..47b4930 100644 --- a/src/main/java/org/example/springbootweb/service/BookService.java +++ b/src/main/java/org/example/springbootweb/service/BookService.java @@ -1,8 +1,8 @@ package org.example.springbootweb.service; +import java.util.List; import org.example.springbootweb.dto.BookDto; import org.example.springbootweb.dto.CreateBookRequestDto; -import java.util.List; public interface BookService { BookDto createBook(CreateBookRequestDto bookDto); diff --git a/src/main/java/org/example/springbootweb/service/impl/BookServiceImpl.java b/src/main/java/org/example/springbootweb/service/impl/BookServiceImpl.java index 1818535..ca8b3c6 100644 --- a/src/main/java/org/example/springbootweb/service/impl/BookServiceImpl.java +++ b/src/main/java/org/example/springbootweb/service/impl/BookServiceImpl.java @@ -1,5 +1,6 @@ package org.example.springbootweb.service.impl; +import java.util.List; import lombok.RequiredArgsConstructor; import org.example.springbootweb.dto.BookDto; import org.example.springbootweb.dto.CreateBookRequestDto; @@ -9,7 +10,6 @@ import org.example.springbootweb.repository.BookRepository; import org.example.springbootweb.service.BookService; import org.springframework.stereotype.Service; -import java.util.List; @Service @RequiredArgsConstructor diff --git a/src/main/resources/application.properties b/src/main/resources/application.properties index 6c9c52a..6021d62 100644 --- a/src/main/resources/application.properties +++ b/src/main/resources/application.properties @@ -1,4 +1,5 @@ -spring.datasource.url=jdbc:mysql://localhost:3306/online_bookstore?allowPublicKeyRetrieval=true&useSSL=false&serverTimezone=UTC +spring.datasource.url=jdbc:mysql://localhost:3306/online_bookstore?\ +allowPublicKeyRetrieval=true&useSSL=false&serverTimezone=UTC spring.datasource.username=root spring.datasource.password=1234 spring.datasource.driver-class-name=com.mysql.cj.jdbc.Driver From 4e0cf8937697a4cf1f9647267b9b312f49f35d00 Mon Sep 17 00:00:00 2001 From: obvly Date: Mon, 21 Sep 2026 23:41:53 +0200 Subject: [PATCH 5/5] fix: resolve CI/CD and Checkstyle line length issues --- src/test/resources/application.properties | 7 +++++++ 1 file changed, 7 insertions(+) create mode 100644 src/test/resources/application.properties diff --git a/src/test/resources/application.properties b/src/test/resources/application.properties new file mode 100644 index 0000000..60741b2 --- /dev/null +++ b/src/test/resources/application.properties @@ -0,0 +1,7 @@ +spring.datasource.url=jdbc:h2:mem:testdb;DB_CLOSE_DELAY=-1 +spring.datasource.driverClassName=org.h2.Driver +spring.datasource.username=sa +spring.datasource.password= + +spring.jpa.database-platform=org.hibernate.dialect.H2Dialect +spring.jpa.hibernate.ddl-auto=create-drop