From e9aceedf03b7acff469fcb1a42d08b560ffca9da Mon Sep 17 00:00:00 2001 From: Ali Safarli Date: Fri, 6 Mar 2026 11:52:18 +0400 Subject: [PATCH 1/3] Lab was completed --- .gitattributes | 2 + .gitignore | 33 ++ .mvn/wrapper/maven-wrapper.properties | 3 + mvnw | 295 ++++++++++++++++++ mvnw.cmd | 189 +++++++++++ pom.xml | 69 ++++ .../labrestapi/LabRestApiApplication.java | 13 + .../controller/CustomerController.java | 50 +++ .../controller/ProductController.java | 73 +++++ .../exception/GlobalExceptionHandler.java | 59 ++++ .../exception/InvalidApiKeyException.java | 7 + .../exception/InvalidPriceRangeException.java | 7 + .../exception/ProductNotFoundException.java | 7 + .../ironhack/labrestapi/model/Customer.java | 62 ++++ .../ironhack/labrestapi/model/Product.java | 63 ++++ .../labrestapi/service/CustomerService.java | 58 ++++ .../labrestapi/service/ProductService.java | 118 +++++++ src/main/resources/application.properties | 1 + .../LabRestApiApplicationTests.java | 13 + 19 files changed, 1122 insertions(+) create mode 100644 .gitattributes create mode 100644 .gitignore create mode 100644 .mvn/wrapper/maven-wrapper.properties create mode 100644 mvnw create mode 100644 mvnw.cmd create mode 100644 pom.xml create mode 100644 src/main/java/com/ironhack/labrestapi/LabRestApiApplication.java create mode 100644 src/main/java/com/ironhack/labrestapi/controller/CustomerController.java create mode 100644 src/main/java/com/ironhack/labrestapi/controller/ProductController.java create mode 100644 src/main/java/com/ironhack/labrestapi/exception/GlobalExceptionHandler.java create mode 100644 src/main/java/com/ironhack/labrestapi/exception/InvalidApiKeyException.java create mode 100644 src/main/java/com/ironhack/labrestapi/exception/InvalidPriceRangeException.java create mode 100644 src/main/java/com/ironhack/labrestapi/exception/ProductNotFoundException.java create mode 100644 src/main/java/com/ironhack/labrestapi/model/Customer.java create mode 100644 src/main/java/com/ironhack/labrestapi/model/Product.java create mode 100644 src/main/java/com/ironhack/labrestapi/service/CustomerService.java create mode 100644 src/main/java/com/ironhack/labrestapi/service/ProductService.java create mode 100644 src/main/resources/application.properties create mode 100644 src/test/java/com/ironhack/labrestapi/LabRestApiApplicationTests.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..8dea6c2 --- /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.12/apache-maven-3.9.12-bin.zip diff --git a/mvnw b/mvnw new file mode 100644 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..c3fdff7 --- /dev/null +++ b/pom.xml @@ -0,0 +1,69 @@ + + + 4.0.0 + + org.springframework.boot + spring-boot-starter-parent + 4.0.3 + + + com.ironhack + lab-rest-api + 0.0.1-SNAPSHOT + lab-rest-api + lab-rest-api + + + + + + + + + + + + + + + 17 + + + + org.springframework.boot + spring-boot-starter-validation + + + org.springframework.boot + spring-boot-starter-webmvc + + + + org.springframework.boot + spring-boot-devtools + runtime + true + + + org.springframework.boot + spring-boot-starter-validation-test + test + + + org.springframework.boot + spring-boot-starter-webmvc-test + test + + + + + + + org.springframework.boot + spring-boot-maven-plugin + + + + + diff --git a/src/main/java/com/ironhack/labrestapi/LabRestApiApplication.java b/src/main/java/com/ironhack/labrestapi/LabRestApiApplication.java new file mode 100644 index 0000000..ecc9938 --- /dev/null +++ b/src/main/java/com/ironhack/labrestapi/LabRestApiApplication.java @@ -0,0 +1,13 @@ +package com.ironhack.labrestapi; + +import org.springframework.boot.SpringApplication; +import org.springframework.boot.autoconfigure.SpringBootApplication; + +@SpringBootApplication +public class LabRestApiApplication { + + public static void main(String[] args) { + SpringApplication.run(LabRestApiApplication.class, args); + } + +} diff --git a/src/main/java/com/ironhack/labrestapi/controller/CustomerController.java b/src/main/java/com/ironhack/labrestapi/controller/CustomerController.java new file mode 100644 index 0000000..6636c08 --- /dev/null +++ b/src/main/java/com/ironhack/labrestapi/controller/CustomerController.java @@ -0,0 +1,50 @@ +package com.ironhack.labrestapi.controller; + +import com.ironhack.labrestapi.model.Customer; +import com.ironhack.labrestapi.service.CustomerService; +import jakarta.validation.Valid; +import org.springframework.http.HttpStatus; +import org.springframework.http.ResponseEntity; +import org.springframework.web.bind.annotation.*; + +import java.util.ArrayList; +import java.util.List; + +@RestController +@RequestMapping("/customers") +public class CustomerController { + + private final CustomerService customerService; + + public CustomerController(CustomerService customerService) { + this.customerService = customerService; + } + + + @PostMapping + public ResponseEntity addCustomer(@Valid @RequestBody Customer customer) { + Customer createdCustomer = customerService.create(customer.getName(), customer.getEmail(), customer.getAge(), customer.getAddress()); + return ResponseEntity.status(HttpStatus.CREATED).body(createdCustomer); + } + + + + @GetMapping + public ResponseEntity> getCustomers( + @RequestParam (required = false) String email + ) { + + if (email != null) { + List customersByEmail = new ArrayList<>(); + customersByEmail.add(customerService.findByEmail(email)); + return ResponseEntity.ok(customersByEmail); + } + + return ResponseEntity.ok(customerService.getAllCustomers()); + + + } + + + +} diff --git a/src/main/java/com/ironhack/labrestapi/controller/ProductController.java b/src/main/java/com/ironhack/labrestapi/controller/ProductController.java new file mode 100644 index 0000000..649fbf1 --- /dev/null +++ b/src/main/java/com/ironhack/labrestapi/controller/ProductController.java @@ -0,0 +1,73 @@ +package com.ironhack.labrestapi.controller; + +import com.ironhack.labrestapi.exception.InvalidApiKeyException; +import com.ironhack.labrestapi.model.Product; +import com.ironhack.labrestapi.service.ProductService; +import org.springframework.http.HttpStatus; +import org.springframework.http.ResponseEntity; +import org.springframework.web.bind.annotation.*; + +import java.util.ArrayList; +import java.util.List; + + +@RestController +@RequestMapping("/products") +public class ProductController { + + private final ProductService productService; + private static final String API_KEY= "12345"; + + public ProductController(ProductService productService) { + this.productService = productService; + } + + + private void validateApiKey(String apiKey) { + if (!API_KEY.equals(apiKey)) { + throw new InvalidApiKeyException("Invalid API Key"); + } + } + + @GetMapping + public ResponseEntity> getProducts( + @RequestHeader(value = "API-Key") String apiKey, + @RequestParam(required = false) String name, + @RequestParam(required = false) String category, + @RequestParam(required = false) Double startPrice, + @RequestParam(required = false) Double endPrice){ + + + + validateApiKey(apiKey); + + + if(name!=null){ + List productsByName = new ArrayList<>(); + productsByName.add(productService.findByName(name)); + return ResponseEntity.ok(productsByName); + } + + if(category!=null){ + return ResponseEntity.ok(productService.findByCategory(category)); + } + + if(startPrice!=null && endPrice!=null){ + return ResponseEntity.ok(productService.findByPriceRange(startPrice, endPrice)); + } + + return ResponseEntity.ok(productService.findAll()); + + + } + + @PostMapping + public ResponseEntity createProduct(@RequestHeader(value = "API-Key") String apiKey, @RequestBody Product product) { + validateApiKey(apiKey); + Product createdProduct = productService.create(product.getName(), product.getPrice(), product.getCategory(), product.getQuantity()); + return ResponseEntity.status(HttpStatus.CREATED).body(createdProduct); + } + + +} + diff --git a/src/main/java/com/ironhack/labrestapi/exception/GlobalExceptionHandler.java b/src/main/java/com/ironhack/labrestapi/exception/GlobalExceptionHandler.java new file mode 100644 index 0000000..172574b --- /dev/null +++ b/src/main/java/com/ironhack/labrestapi/exception/GlobalExceptionHandler.java @@ -0,0 +1,59 @@ +package com.ironhack.labrestapi.exception; + +import org.springframework.http.HttpStatus; +import org.springframework.http.ResponseEntity; +import org.springframework.validation.FieldError; +import org.springframework.web.bind.MethodArgumentNotValidException; +import org.springframework.web.bind.annotation.ExceptionHandler; +import org.springframework.web.bind.annotation.ResponseStatus; +import org.springframework.web.bind.annotation.RestControllerAdvice; + +import java.time.LocalDateTime; +import java.util.HashMap; +import java.util.Map; + +@RestControllerAdvice +public class GlobalExceptionHandler { + + @ExceptionHandler(MethodArgumentNotValidException.class) + @ResponseStatus(HttpStatus.BAD_REQUEST) + public Map handleValidation(MethodArgumentNotValidException ex) { + + Map errors = new HashMap<>(); + + ex.getBindingResult().getFieldErrors() + .forEach(e -> errors.put(e.getField(), e.getDefaultMessage())); + + return errors; + } + + @ExceptionHandler(InvalidApiKeyException.class) + @ResponseStatus(HttpStatus.UNAUTHORIZED) + public Map handleMissingApiKey(InvalidApiKeyException ex) { + Map error = new HashMap<>(); + error.put("error", ex.getMessage()); + return error; + + } + + @ExceptionHandler(ProductNotFoundException.class) + @ResponseStatus(HttpStatus.NOT_FOUND) + public Map handleProductNotFound(ProductNotFoundException ex) { + Map error = new HashMap<>(); + error.put("error", ex.getMessage()); + return error; + + + } + + @ExceptionHandler(InvalidPriceRangeException.class) + @ResponseStatus(HttpStatus.BAD_REQUEST) + public Map handleInvalidPriceRange(InvalidPriceRangeException ex) { + Map error = new HashMap<>(); + error.put("error", ex.getMessage()); + return error; + + + + } +} diff --git a/src/main/java/com/ironhack/labrestapi/exception/InvalidApiKeyException.java b/src/main/java/com/ironhack/labrestapi/exception/InvalidApiKeyException.java new file mode 100644 index 0000000..1ea13f1 --- /dev/null +++ b/src/main/java/com/ironhack/labrestapi/exception/InvalidApiKeyException.java @@ -0,0 +1,7 @@ +package com.ironhack.labrestapi.exception; + +public class InvalidApiKeyException extends RuntimeException { + public InvalidApiKeyException(String message) { + super(message); + } +} \ No newline at end of file diff --git a/src/main/java/com/ironhack/labrestapi/exception/InvalidPriceRangeException.java b/src/main/java/com/ironhack/labrestapi/exception/InvalidPriceRangeException.java new file mode 100644 index 0000000..903b09a --- /dev/null +++ b/src/main/java/com/ironhack/labrestapi/exception/InvalidPriceRangeException.java @@ -0,0 +1,7 @@ +package com.ironhack.labrestapi.exception; + +public class InvalidPriceRangeException extends RuntimeException { + public InvalidPriceRangeException(String message) { + super(message); + } +} \ No newline at end of file diff --git a/src/main/java/com/ironhack/labrestapi/exception/ProductNotFoundException.java b/src/main/java/com/ironhack/labrestapi/exception/ProductNotFoundException.java new file mode 100644 index 0000000..40a5bc6 --- /dev/null +++ b/src/main/java/com/ironhack/labrestapi/exception/ProductNotFoundException.java @@ -0,0 +1,7 @@ +package com.ironhack.labrestapi.exception; + +public class ProductNotFoundException extends RuntimeException { + public ProductNotFoundException(String message) { + super(message); + } +} \ No newline at end of file diff --git a/src/main/java/com/ironhack/labrestapi/model/Customer.java b/src/main/java/com/ironhack/labrestapi/model/Customer.java new file mode 100644 index 0000000..8d5f474 --- /dev/null +++ b/src/main/java/com/ironhack/labrestapi/model/Customer.java @@ -0,0 +1,62 @@ +package com.ironhack.labrestapi.model; + +import jakarta.validation.constraints.Email; +import jakarta.validation.constraints.Min; +import jakarta.validation.constraints.NotBlank; + +public class Customer { + + @NotBlank(message = "Customer name cannot be blank") + private String name; + + @Email(message = "Email should be valid") + @NotBlank(message = "Email cannot be blank") + private String email; + + @Min(value = 18, message = "Age must be at least 18") + private int age; + + @NotBlank(message = "Address cannot be blank") + private String address; + + public Customer() {} + + public Customer(String name, String email, int age, String address) { + this.name = name; + this.email = email; + this.age = age; + this.address = address; + } + + public String getName() { + return name; + } + + public void setName(String name) { + this.name = name; + } + + public String getEmail() { + return email; + } + + public void setEmail(String email) { + this.email = email; + } + + public int getAge() { + return age; + } + + public void setAge(int age) { + this.age = age; + } + + public String getAddress() { + return address; + } + + public void setAddress(String address) { + this.address = address; + } +} diff --git a/src/main/java/com/ironhack/labrestapi/model/Product.java b/src/main/java/com/ironhack/labrestapi/model/Product.java new file mode 100644 index 0000000..ff513f1 --- /dev/null +++ b/src/main/java/com/ironhack/labrestapi/model/Product.java @@ -0,0 +1,63 @@ +package com.ironhack.labrestapi.model; + +import jakarta.validation.constraints.NegativeOrZero; +import jakarta.validation.constraints.NotBlank; +import jakarta.validation.constraints.Positive; +import jakarta.validation.constraints.Size; + +public class Product { + + @NotBlank(message = "Product name cannot be blank") + @Size(min = 3, max = 100, message = "Product name must be between 3 and 100 characters") + private String name; + + @Positive(message = "Price must be greater than zero") + private double price; + + @NotBlank(message = "Category cannot be blank") + private String category; + + @Positive(message = "Quantity must be greater than zero") + private int quantity; + + public Product() {} + + public Product(String name, double price, String category, int quantity) { + this.name = name; + this.price = price; + this.category = category; + this.quantity = quantity; + } + + public String getName() { + return name; + } + + public void setName(String name) { + this.name = name; + } + + public double getPrice() { + return price; + } + + public void setPrice(double price) { + this.price = price; + } + + public String getCategory() { + return category; + } + + public void setCategory(String category) { + this.category = category; + } + + public int getQuantity() { + return quantity; + } + + public void setQuantity(int quantity) { + this.quantity = quantity; + } +} diff --git a/src/main/java/com/ironhack/labrestapi/service/CustomerService.java b/src/main/java/com/ironhack/labrestapi/service/CustomerService.java new file mode 100644 index 0000000..f7e1cd5 --- /dev/null +++ b/src/main/java/com/ironhack/labrestapi/service/CustomerService.java @@ -0,0 +1,58 @@ +package com.ironhack.labrestapi.service; + +import com.ironhack.labrestapi.model.Customer; +import org.springframework.stereotype.Service; + +import java.util.ArrayList; +import java.util.HashMap; +import java.util.List; +import java.util.Map; + +@Service +public class CustomerService { + + private final Map customers= new HashMap<>(); + private Long nextId = 1L; + + public CustomerService() { + //seed customer1 + Customer c1 = new Customer(); + c1.setName("John Doe"); + c1.setEmail("alisada12@gmail.com"); + c1.setAge(30); + c1.setAddress("123 Main St"); + customers.put(nextId++, c1); + + //seed customer2 + Customer c2 = new Customer(); + c2.setName("Jane Smith"); + c2.setEmail("Jane1232@gmail.com"); + c2.setAge(25); + c2.setAddress("456 Elm St"); + customers.put(nextId++, c2); + } + + public List getAllCustomers() { + return new ArrayList<>(customers.values()); + } + + public Customer create(String name, String email, int age, String address) { + Customer customer = new Customer(); + customer.setName(name); + customer.setEmail(email); + customer.setAge(age); + customer.setAddress(address); + customers.put(nextId++, customer); + return customer; + } + + public Customer findByEmail(String email) { + for (Customer customer : customers.values()) { + if (customer.getEmail().equalsIgnoreCase(email)) { + return customer; + } + } + + throw new RuntimeException("Customer not found"); + } +} diff --git a/src/main/java/com/ironhack/labrestapi/service/ProductService.java b/src/main/java/com/ironhack/labrestapi/service/ProductService.java new file mode 100644 index 0000000..911dcde --- /dev/null +++ b/src/main/java/com/ironhack/labrestapi/service/ProductService.java @@ -0,0 +1,118 @@ +package com.ironhack.labrestapi.service; + +import com.ironhack.labrestapi.exception.InvalidPriceRangeException; +import com.ironhack.labrestapi.exception.ProductNotFoundException; +import com.ironhack.labrestapi.model.Product; +import org.springframework.stereotype.Service; + +import java.util.ArrayList; +import java.util.HashMap; +import java.util.List; +import java.util.Map; + +@Service +public class ProductService { + + private final Map products = new HashMap<>(); + + + + + public ProductService() { + //seed product1 + Product p1 = new Product(); + p1.setName("Product 1"); + p1.setPrice(12); + p1.setCategory("Food"); + p1.setQuantity(5); + products.put(p1.getName(), p1); + + + //seed product2 + Product p2 = new Product(); + p2.setName("Product 2"); + p2.setPrice(20); + p2.setCategory("Electronics"); + p2.setQuantity(3); + products.put(p2.getName(), p2); + + //seed product3 + Product p3 = new Product(); + p3.setName("Product 3"); + p3.setPrice(15); + p3.setCategory("Clothing"); + p3.setQuantity(10); + products.put(p3.getName(), p3); + + } + + public List findAll() { + return new ArrayList<>(products.values()); + } + + + public Product create(String name, double price, String category, int quantity) { + Product product = new Product(); + product.setName(name); + product.setPrice(price); + product.setCategory(category); + product.setQuantity(quantity); + products.put(product.getName(), product); + return product; + } + + + public List findByPriceRange(double startPrice, double endPrice) { + if (startPrice<0 || endPrice < 0) { + throw new InvalidPriceRangeException("Price cannot be negative"); + } + + if (startPrice > endPrice) { + throw new InvalidPriceRangeException("Start price cannot be greater than end price"); + } + + + List productsByRange = new ArrayList<>(); + + for (Product product : products.values()) { + if (product.getPrice() >= startPrice && product.getPrice() <= endPrice) { + productsByRange.add(product); + } + + } + return productsByRange; + } + + + public List findByCategory(String category) { + List productsByCategory = new ArrayList<>(); + + for (Product product : products.values()) { + if (product.getCategory().equalsIgnoreCase(category)) { + productsByCategory.add(product); + } + } + return productsByCategory; + + } + + public Product findByName(String Name) { + for (Product product : products.values()) { + if (product.getName().equalsIgnoreCase(Name)) { + + return product; + } + + } + throw new ProductNotFoundException("Product not found: " + Name); +} + + + + + + + + + +} diff --git a/src/main/resources/application.properties b/src/main/resources/application.properties new file mode 100644 index 0000000..6c809f6 --- /dev/null +++ b/src/main/resources/application.properties @@ -0,0 +1 @@ +spring.application.name=lab-rest-api diff --git a/src/test/java/com/ironhack/labrestapi/LabRestApiApplicationTests.java b/src/test/java/com/ironhack/labrestapi/LabRestApiApplicationTests.java new file mode 100644 index 0000000..0db6c27 --- /dev/null +++ b/src/test/java/com/ironhack/labrestapi/LabRestApiApplicationTests.java @@ -0,0 +1,13 @@ +package com.ironhack.labrestapi; + +import org.junit.jupiter.api.Test; +import org.springframework.boot.test.context.SpringBootTest; + +@SpringBootTest +class LabRestApiApplicationTests { + + @Test + void contextLoads() { + } + +} From f7cb7ddec833efeec12511c51274d1f1c8df7e99 Mon Sep 17 00:00:00 2001 From: Ali Safarli Date: Fri, 6 Mar 2026 12:01:40 +0400 Subject: [PATCH 2/3] Some problems solved --- .../labrestapi/controller/ProductController.java | 10 +++++----- .../ironhack/labrestapi/service/ProductService.java | 8 ++++---- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/src/main/java/com/ironhack/labrestapi/controller/ProductController.java b/src/main/java/com/ironhack/labrestapi/controller/ProductController.java index 649fbf1..cd99cb8 100644 --- a/src/main/java/com/ironhack/labrestapi/controller/ProductController.java +++ b/src/main/java/com/ironhack/labrestapi/controller/ProductController.java @@ -16,7 +16,7 @@ public class ProductController { private final ProductService productService; - private static final String API_KEY= "12345"; + private static final String API_KEY= "123456"; public ProductController(ProductService productService) { this.productService = productService; @@ -34,8 +34,8 @@ public ResponseEntity> getProducts( @RequestHeader(value = "API-Key") String apiKey, @RequestParam(required = false) String name, @RequestParam(required = false) String category, - @RequestParam(required = false) Double startPrice, - @RequestParam(required = false) Double endPrice){ + @RequestParam(required = false) Double min, + @RequestParam(required = false) Double max){ @@ -52,8 +52,8 @@ public ResponseEntity> getProducts( return ResponseEntity.ok(productService.findByCategory(category)); } - if(startPrice!=null && endPrice!=null){ - return ResponseEntity.ok(productService.findByPriceRange(startPrice, endPrice)); + if(min!=null && max!=null){ + return ResponseEntity.ok(productService.findByPriceRange(min, max)); } return ResponseEntity.ok(productService.findAll()); diff --git a/src/main/java/com/ironhack/labrestapi/service/ProductService.java b/src/main/java/com/ironhack/labrestapi/service/ProductService.java index 911dcde..81126d2 100644 --- a/src/main/java/com/ironhack/labrestapi/service/ProductService.java +++ b/src/main/java/com/ironhack/labrestapi/service/ProductService.java @@ -62,12 +62,12 @@ public Product create(String name, double price, String category, int quantity) } - public List findByPriceRange(double startPrice, double endPrice) { - if (startPrice<0 || endPrice < 0) { + public List findByPriceRange(double min, double max) { + if (min <0 || max < 0) { throw new InvalidPriceRangeException("Price cannot be negative"); } - if (startPrice > endPrice) { + if (min > max) { throw new InvalidPriceRangeException("Start price cannot be greater than end price"); } @@ -75,7 +75,7 @@ public List findByPriceRange(double startPrice, double endPrice) { List productsByRange = new ArrayList<>(); for (Product product : products.values()) { - if (product.getPrice() >= startPrice && product.getPrice() <= endPrice) { + if (product.getPrice() >= min && product.getPrice() <= max) { productsByRange.add(product); } From 8bbd863fa18c72db1a67013f68015eff74e60640 Mon Sep 17 00:00:00 2001 From: Ali Safarli Date: Wed, 11 Mar 2026 02:13:28 +0400 Subject: [PATCH 3/3] Add some important changes --- .../controller/CustomerController.java | 27 +++++--- .../controller/ProductController.java | 42 ++++++------- .../exception/GlobalExceptionHandler.java | 7 +-- .../ironhack/labrestapi/model/Customer.java | 17 +++++- .../ironhack/labrestapi/model/Product.java | 19 ++++-- .../labrestapi/service/CustomerService.java | 45 ++++++++++---- .../labrestapi/service/ProductService.java | 61 ++++++++++++------- 7 files changed, 143 insertions(+), 75 deletions(-) diff --git a/src/main/java/com/ironhack/labrestapi/controller/CustomerController.java b/src/main/java/com/ironhack/labrestapi/controller/CustomerController.java index 6636c08..0cf82d8 100644 --- a/src/main/java/com/ironhack/labrestapi/controller/CustomerController.java +++ b/src/main/java/com/ironhack/labrestapi/controller/CustomerController.java @@ -13,7 +13,6 @@ @RestController @RequestMapping("/customers") public class CustomerController { - private final CustomerService customerService; public CustomerController(CustomerService customerService) { @@ -23,16 +22,29 @@ public CustomerController(CustomerService customerService) { @PostMapping public ResponseEntity addCustomer(@Valid @RequestBody Customer customer) { - Customer createdCustomer = customerService.create(customer.getName(), customer.getEmail(), customer.getAge(), customer.getAddress()); - return ResponseEntity.status(HttpStatus.CREATED).body(createdCustomer); + return ResponseEntity.status(HttpStatus.CREATED).body(customerService.create(customer)); } + @PutMapping("/{id}") + public ResponseEntity updateCustomer(@PathVariable Long id, @Valid @RequestBody Customer customer) { + return ResponseEntity.ok(customerService.update(id, customer)); + } + + @DeleteMapping("/{id}") + public ResponseEntity deleteCustomer(@PathVariable Long id) { + customerService.delete(id); + return ResponseEntity.noContent().build(); + } + + @GetMapping("/{id}") + public ResponseEntity getCustomerById( + @PathVariable Long id) { + return ResponseEntity.ok(customerService.findById(id)); + } @GetMapping - public ResponseEntity> getCustomers( - @RequestParam (required = false) String email - ) { + public ResponseEntity> getCustomers(@RequestParam(required = false) String email) { if (email != null) { List customersByEmail = new ArrayList<>(); @@ -40,11 +52,10 @@ public ResponseEntity> getCustomers( return ResponseEntity.ok(customersByEmail); } - return ResponseEntity.ok(customerService.getAllCustomers()); + return ResponseEntity.ok(customerService.getAll()); } - } diff --git a/src/main/java/com/ironhack/labrestapi/controller/ProductController.java b/src/main/java/com/ironhack/labrestapi/controller/ProductController.java index cd99cb8..eb64b21 100644 --- a/src/main/java/com/ironhack/labrestapi/controller/ProductController.java +++ b/src/main/java/com/ironhack/labrestapi/controller/ProductController.java @@ -3,6 +3,7 @@ import com.ironhack.labrestapi.exception.InvalidApiKeyException; import com.ironhack.labrestapi.model.Product; import com.ironhack.labrestapi.service.ProductService; +import jakarta.validation.Valid; import org.springframework.http.HttpStatus; import org.springframework.http.ResponseEntity; import org.springframework.web.bind.annotation.*; @@ -14,15 +15,13 @@ @RestController @RequestMapping("/products") public class ProductController { - + private static final String API_KEY = "123456"; private final ProductService productService; - private static final String API_KEY= "123456"; public ProductController(ProductService productService) { this.productService = productService; } - private void validateApiKey(String apiKey) { if (!API_KEY.equals(apiKey)) { throw new InvalidApiKeyException("Invalid API Key"); @@ -30,42 +29,41 @@ private void validateApiKey(String apiKey) { } @GetMapping - public ResponseEntity> getProducts( - @RequestHeader(value = "API-Key") String apiKey, - @RequestParam(required = false) String name, - @RequestParam(required = false) String category, - @RequestParam(required = false) Double min, - @RequestParam(required = false) Double max){ - - + public ResponseEntity> getProducts(@RequestHeader(value = "API-Key") String apiKey, @RequestParam(required = false) String name, @RequestParam(required = false) String category, @RequestParam(required = false) Double min, @RequestParam(required = false) Double max) { validateApiKey(apiKey); - - if(name!=null){ + if (name != null) { List productsByName = new ArrayList<>(); productsByName.add(productService.findByName(name)); return ResponseEntity.ok(productsByName); } - - if(category!=null){ + if (category != null) { return ResponseEntity.ok(productService.findByCategory(category)); } - - if(min!=null && max!=null){ + if (min != null && max != null) { return ResponseEntity.ok(productService.findByPriceRange(min, max)); } - return ResponseEntity.ok(productService.findAll()); + } + @PostMapping + public ResponseEntity createProduct(@RequestHeader(value = "API-Key") String apiKey, @Valid @RequestBody Product product) { + validateApiKey(apiKey); + return ResponseEntity.status(HttpStatus.CREATED).body(productService.create(product)); + } + @PutMapping("/{id}") + public ResponseEntity updateProduct(@RequestHeader(value = "API-Key") String apiKey, @PathVariable Long id, @Valid @RequestBody Product product) { + validateApiKey(apiKey); + return ResponseEntity.ok(productService.update(id, product)); } - @PostMapping - public ResponseEntity createProduct(@RequestHeader(value = "API-Key") String apiKey, @RequestBody Product product) { + @DeleteMapping("/{id}") + public ResponseEntity deleteProduct(@RequestHeader(value = "API-Key") String apiKey, @PathVariable Long id) { validateApiKey(apiKey); - Product createdProduct = productService.create(product.getName(), product.getPrice(), product.getCategory(), product.getQuantity()); - return ResponseEntity.status(HttpStatus.CREATED).body(createdProduct); + productService.delete(id); + return ResponseEntity.noContent().build(); } diff --git a/src/main/java/com/ironhack/labrestapi/exception/GlobalExceptionHandler.java b/src/main/java/com/ironhack/labrestapi/exception/GlobalExceptionHandler.java index 172574b..c231ecf 100644 --- a/src/main/java/com/ironhack/labrestapi/exception/GlobalExceptionHandler.java +++ b/src/main/java/com/ironhack/labrestapi/exception/GlobalExceptionHandler.java @@ -1,14 +1,11 @@ package com.ironhack.labrestapi.exception; import org.springframework.http.HttpStatus; -import org.springframework.http.ResponseEntity; -import org.springframework.validation.FieldError; import org.springframework.web.bind.MethodArgumentNotValidException; import org.springframework.web.bind.annotation.ExceptionHandler; import org.springframework.web.bind.annotation.ResponseStatus; import org.springframework.web.bind.annotation.RestControllerAdvice; -import java.time.LocalDateTime; import java.util.HashMap; import java.util.Map; @@ -21,8 +18,7 @@ public Map handleValidation(MethodArgumentNotValidException ex) Map errors = new HashMap<>(); - ex.getBindingResult().getFieldErrors() - .forEach(e -> errors.put(e.getField(), e.getDefaultMessage())); + ex.getBindingResult().getFieldErrors().forEach(e -> errors.put(e.getField(), e.getDefaultMessage())); return errors; } @@ -54,6 +50,5 @@ public Map handleInvalidPriceRange(InvalidPriceRangeException ex return error; - } } diff --git a/src/main/java/com/ironhack/labrestapi/model/Customer.java b/src/main/java/com/ironhack/labrestapi/model/Customer.java index 8d5f474..38d934d 100644 --- a/src/main/java/com/ironhack/labrestapi/model/Customer.java +++ b/src/main/java/com/ironhack/labrestapi/model/Customer.java @@ -6,6 +6,8 @@ public class Customer { + private Long id; + @NotBlank(message = "Customer name cannot be blank") private String name; @@ -19,15 +21,26 @@ public class Customer { @NotBlank(message = "Address cannot be blank") private String address; - public Customer() {} + public Customer() { + } - public Customer(String name, String email, int age, String address) { + public Customer(Long id, String name, String email, int age, String address) { + this.id = id; this.name = name; this.email = email; this.age = age; this.address = address; } + public Long getId() { + return id; + } + + public void setId(Long id) { + this.id = id; + } + + public String getName() { return name; } diff --git a/src/main/java/com/ironhack/labrestapi/model/Product.java b/src/main/java/com/ironhack/labrestapi/model/Product.java index ff513f1..7625464 100644 --- a/src/main/java/com/ironhack/labrestapi/model/Product.java +++ b/src/main/java/com/ironhack/labrestapi/model/Product.java @@ -1,11 +1,12 @@ package com.ironhack.labrestapi.model; -import jakarta.validation.constraints.NegativeOrZero; import jakarta.validation.constraints.NotBlank; import jakarta.validation.constraints.Positive; import jakarta.validation.constraints.Size; public class Product { + private Long id; + @NotBlank(message = "Product name cannot be blank") @Size(min = 3, max = 100, message = "Product name must be between 3 and 100 characters") @@ -15,20 +16,30 @@ public class Product { private double price; @NotBlank(message = "Category cannot be blank") - private String category; + private String category; @Positive(message = "Quantity must be greater than zero") private int quantity; - public Product() {} + public Product() { + } - public Product(String name, double price, String category, int quantity) { + public Product(Long id, String name, double price, String category, int quantity) { + this.id = id; this.name = name; this.price = price; this.category = category; this.quantity = quantity; } + public Long getId() { + return id; + } + + public void setId(Long id) { + this.id = id; + } + public String getName() { return name; } diff --git a/src/main/java/com/ironhack/labrestapi/service/CustomerService.java b/src/main/java/com/ironhack/labrestapi/service/CustomerService.java index f7e1cd5..fb977f9 100644 --- a/src/main/java/com/ironhack/labrestapi/service/CustomerService.java +++ b/src/main/java/com/ironhack/labrestapi/service/CustomerService.java @@ -11,38 +11,36 @@ @Service public class CustomerService { - private final Map customers= new HashMap<>(); + private final Map customers = new HashMap<>(); private Long nextId = 1L; public CustomerService() { //seed customer1 Customer c1 = new Customer(); + c1.setId(nextId++); c1.setName("John Doe"); c1.setEmail("alisada12@gmail.com"); c1.setAge(30); c1.setAddress("123 Main St"); - customers.put(nextId++, c1); + customers.put(c1.getId(), c1); //seed customer2 Customer c2 = new Customer(); + c2.setId(nextId++); c2.setName("Jane Smith"); c2.setEmail("Jane1232@gmail.com"); c2.setAge(25); c2.setAddress("456 Elm St"); - customers.put(nextId++, c2); + customers.put(c2.getId(), c2); } - public List getAllCustomers() { + public List getAll() { return new ArrayList<>(customers.values()); } - public Customer create(String name, String email, int age, String address) { - Customer customer = new Customer(); - customer.setName(name); - customer.setEmail(email); - customer.setAge(age); - customer.setAddress(address); - customers.put(nextId++, customer); + public Customer create(Customer customer) { + customer.setId(nextId++); + customers.put(customer.getId(), customer); return customer; } @@ -55,4 +53,29 @@ public Customer findByEmail(String email) { throw new RuntimeException("Customer not found"); } + + public Customer findById(Long id) { + Customer customer = customers.get(id); + if (customer == null) { + throw new RuntimeException("Customer not found with id: " + id); + } + return customer; + } + + public Customer update(Long id, Customer updatedCustomer) { + Customer existingCustomer = findById(id); + if (existingCustomer == null) { + throw new RuntimeException("Customer not found"); + } + existingCustomer.setName(updatedCustomer.getName()); + existingCustomer.setEmail(updatedCustomer.getEmail()); + existingCustomer.setAge(updatedCustomer.getAge()); + existingCustomer.setAddress(updatedCustomer.getAddress()); + return existingCustomer; + } + + public void delete(Long id) { + findById(id); + customers.remove(id); + } } diff --git a/src/main/java/com/ironhack/labrestapi/service/ProductService.java b/src/main/java/com/ironhack/labrestapi/service/ProductService.java index 81126d2..9f15d30 100644 --- a/src/main/java/com/ironhack/labrestapi/service/ProductService.java +++ b/src/main/java/com/ironhack/labrestapi/service/ProductService.java @@ -13,36 +13,38 @@ @Service public class ProductService { - private final Map products = new HashMap<>(); - - + private final Map products = new HashMap<>(); + private Long nextId = 1L; public ProductService() { //seed product1 Product p1 = new Product(); + p1.setId(nextId++); p1.setName("Product 1"); p1.setPrice(12); p1.setCategory("Food"); p1.setQuantity(5); - products.put(p1.getName(), p1); + products.put(p1.getId(), p1); //seed product2 Product p2 = new Product(); + p2.setId(nextId++); p2.setName("Product 2"); p2.setPrice(20); p2.setCategory("Electronics"); p2.setQuantity(3); - products.put(p2.getName(), p2); + products.put(p2.getId(), p2); //seed product3 Product p3 = new Product(); + p3.setId(nextId++); p3.setName("Product 3"); p3.setPrice(15); p3.setCategory("Clothing"); p3.setQuantity(10); - products.put(p3.getName(), p3); + products.put(p3.getId(), p3); } @@ -51,19 +53,15 @@ public List findAll() { } - public Product create(String name, double price, String category, int quantity) { - Product product = new Product(); - product.setName(name); - product.setPrice(price); - product.setCategory(category); - product.setQuantity(quantity); - products.put(product.getName(), product); + public Product create(Product product) { + product.setId(nextId++); + products.put(product.getId(), product); return product; } public List findByPriceRange(double min, double max) { - if (min <0 || max < 0) { + if (min < 0 || max < 0) { throw new InvalidPriceRangeException("Price cannot be negative"); } @@ -96,23 +94,42 @@ public List findByCategory(String category) { } - public Product findByName(String Name) { + public Product findByName(String name) { for (Product product : products.values()) { - if (product.getName().equalsIgnoreCase(Name)) { - - return product; - } + if (product.getName().equalsIgnoreCase(name)) { - } - throw new ProductNotFoundException("Product not found: " + Name); -} + return product; + } + } + throw new ProductNotFoundException("Product not found: " + name); + } + public Product update(Long id, Product product) { + Product existingProduct = products.get(id); + if (existingProduct == null) { + throw new ProductNotFoundException("Product not found with id: " + id); + } else { + existingProduct.setName(product.getName()); + existingProduct.setPrice(product.getPrice()); + existingProduct.setCategory(product.getCategory()); + existingProduct.setQuantity(product.getQuantity()); + return existingProduct; + } + } + public void delete(Long id) { + Product existingProduct = products.get(id); + if (existingProduct == null) { + throw new ProductNotFoundException("Product not found with id: " + id); + } else { + products.remove(id); + } + } }