From 46a080a813190a05b0dea0464116509b5f72249b Mon Sep 17 00:00:00 2001 From: xilser Date: Fri, 27 Jun 2025 20:40:54 +0300 Subject: [PATCH 1/3] Update Main.java --- src/main/java/ru/cu/Main.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/main/java/ru/cu/Main.java b/src/main/java/ru/cu/Main.java index 4363c0a..7f36a11 100644 --- a/src/main/java/ru/cu/Main.java +++ b/src/main/java/ru/cu/Main.java @@ -14,4 +14,5 @@ public static void main(String[] args) { System.out.println("i = " + i); } } -} \ No newline at end of file +} + From ffa30ae018028e436f5edbbd9b2029d2d3eae465 Mon Sep 17 00:00:00 2001 From: xilser Date: Fri, 27 Jun 2025 20:42:20 +0300 Subject: [PATCH 2/3] Update OrderProcessor.java --- src/main/java/ru/cu/OrderProcessor.java | 1 + 1 file changed, 1 insertion(+) diff --git a/src/main/java/ru/cu/OrderProcessor.java b/src/main/java/ru/cu/OrderProcessor.java index a49a911..7381b10 100644 --- a/src/main/java/ru/cu/OrderProcessor.java +++ b/src/main/java/ru/cu/OrderProcessor.java @@ -4,6 +4,7 @@ * Smell Safari Exercise */ + import java.io.Closeable; import java.io.IOException; import java.util.ArrayList; From 8aa19ed8b4d087521be50b926b808ba8f532361f Mon Sep 17 00:00:00 2001 From: xilser Date: Fri, 27 Jun 2025 21:00:55 +0300 Subject: [PATCH 3/3] =?UTF-8?q?=D1=87=D0=B5=20=D1=83=D1=81=D0=BF=D0=B5?= =?UTF-8?q?=D0=BB=D0=B8=20=D1=82=D0=BE=20=D1=83=D1=81=D0=BF=D0=B5=D0=BB?= =?UTF-8?q?=D0=B8?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/main/java/ru/cu/Main.java | 7 ++-- src/main/java/ru/cu/OrderProcessor.java | 47 +++++++++++++------------ 2 files changed, 29 insertions(+), 25 deletions(-) diff --git a/src/main/java/ru/cu/Main.java b/src/main/java/ru/cu/Main.java index 7f36a11..85206c2 100644 --- a/src/main/java/ru/cu/Main.java +++ b/src/main/java/ru/cu/Main.java @@ -1,16 +1,19 @@ package ru.cu; //TIP To Run code, press or -// click the icon in the gutter. +// click the icon in the gutter. +// слишком много комментариев public class Main { public static void main(String[] args) { //TIP Press with your caret at the highlighted text // to see how IntelliJ IDEA suggests fixing it. + // слишком много комментариев System.out.printf("Hello and welcome!"); for (int i = 1; i <= 5; i++) { //TIP Press to start debugging your code. We have set one breakpoint - // for you, but you can always add more by pressing . + // for you, but you can always add more by pressing . + // слишком много комментариев System.out.println("i = " + i); } } diff --git a/src/main/java/ru/cu/OrderProcessor.java b/src/main/java/ru/cu/OrderProcessor.java index 7381b10..afee293 100644 --- a/src/main/java/ru/cu/OrderProcessor.java +++ b/src/main/java/ru/cu/OrderProcessor.java @@ -4,10 +4,10 @@ * Smell Safari Exercise */ - + import java.io.Closeable; import java.io.IOException; -import java.util.ArrayList; +import java.util.ArrayList; // не используется import java.util.HashMap; import java.util.List; import java.util.Map; @@ -27,13 +27,13 @@ public class OrderProcessor extends BaseOrderProcessor implements Closeable { private double taxRate = 0.2; - private String lastCustomerName; - private double lastGrandTotal; + private String lastCustomerName; // не используется + private double lastGrandTotal; // не используется private int operationsCounter; - private final Map loyaltyCache = new HashMap<>(); + private final Map loyaltyCache = new HashMap<>(); // не используется - private String tempReport; + private String tempReport; // временная переменная должна быть внутри нужного метода public void processOrders(List orders, String customerName, @@ -49,12 +49,13 @@ public void processOrders(List orders, int shippingSpeed, boolean sendEmailFlag, boolean sendSmsFlag, - boolean sendPushFlag) { + boolean sendPushFlag) { // слишком много параметров, надо разбивать на классы + // да и метод чет слишком длинный, его тоже можно разбить на несколько beforeProcess(); operationsCounter++; - logger.info("--- Start processing batch #" + operationsCounter + " ---"); + logger.info("--- Start processing batch #" + operationsCounter + " ---"); double total = 0; for (Order o : orders) { @@ -74,17 +75,17 @@ public void processOrders(List orders, } } - double shippingCost = calculateShipping(shippingMethod, shippingSpeed); - double tax = total * taxRate; - double grandTotal = total + shippingCost + tax; + double shippingCost = calculateShipping(shippingMethod, shippingSpeed); // primitive obsession + double tax = total * taxRate; // primitive obsession + double grandTotal = total + shippingCost + tax; // primitive obsession - if (sendEmailFlag) { + if (sendEmailFlag) { // флаги это плохо sendEmail(customerName, addressLine1, addressLine2, postalCode, city, country, grandTotal); } - if (sendSmsFlag) { + if (sendSmsFlag) { // флаги это плохо sendSMS(customerName, country, grandTotal); } - if (sendPushFlag) { + if (sendPushFlag) { // флаги это плохо sendPush(customerName, grandTotal); } @@ -97,7 +98,7 @@ public void processOrders(List orders, logger.info("Non-Finland loyalty points added: " + loyalty); } - tempReport = generateReport(customerName, orders, grandTotal); + tempReport = generateReport(customerName, orders, grandTotal); // используется deprecated метод logger.fine(tempReport); lastCustomerName = customerName; @@ -132,21 +133,21 @@ public String generateReport(String customerName, List orders, double gra sb.append("TOTAL: ").append(grandTotal).append("\n"); logger.info("Report generated for " + customerName); return sb.toString(); - } + } // dead code public String getCustomerPhone(Order o) { return o.getCustomer().getPhone(); } public String getCustomerEmail(Order o) { return o.getCustomer().getEmail(); - } + } // ??? - private void sendEmail(String name, String a1, String a2, String pc, String city, String country, double total) { + private void sendEmail(String name, String a1, String a2, String pc, String city, String country, double total) { // плохие названия try { if (total < 0) throw new IOException("Negative total!"); logger.info("[EMAIL] " + name + " spent " + total); } catch (Exception ignored) { - } + } // пустой catch блок } private void sendSMS(String name, String country, double total) { @@ -169,13 +170,13 @@ public double calculateTotalAmounts(List orders) { return sum; } - private double oldCalculate(List orders) { - return 42; + private double oldCalculate(List orders) { // never used + return 42; // magic number } @Override - public void close() { - } + public void close() { + } // пустой deprecated метод public static class Order { private final String id;