Форум программистов, компьютерный форум, киберфорум
Java SE (J2SE)
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.67/3: Рейтинг темы: голосов - 3, средняя оценка - 4.67
3 / 3 / 14
Регистрация: 26.01.2016
Сообщений: 481

Посоветуйте как улучшить код

10.03.2017, 19:19. Показов 664. Ответов 14
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
 
 
@Autowired
    private ClothFabricRepository clothFabricRepository;
    @Autowired
    private CurtainRepository curtainRepository;
    @Autowired
    private OrderCurtainRepository orderCurtainRepository;
    @Autowired
    private TulleRepository tulleRepository;
    @Autowired
    private UpholsteryFabricRepository upholsteryFabricRepository;
 
    @Transactional
    public void changeInfoInDB(final String idFabric, final String photo,
                               final String name, final Double quantityFromUI) {
       final String clothFabric = "clothFabric";
        final String curtainFabric = "curtain";
        final String orderFabric = "orderCurtain";
        final String tulleFabric = "tulle";
        final String upholsteryFabric = "upholsteryFabric";
 
        String[] split = photo.split("/");
//        System.out.println("SPLIT[0]  "+split[0]+"  SPLIT[1]"+ split[1]);
 
       if(split[1].contains(clothFabric)) {
 
           List<ClothFabricModel> clothFabricModels = clothFabricRepository.findById(id);
            ClothFabricModel model = clothFabricModels.get(0);
           Double quantityInDB = model.getQuantity();
           if (quantityInDB == 0) {
               System.out.println("In DB ZERO");
           } else {
 
               Double result = quantityInDB - quantityFromUI;
               model.setQuantity(result);
               clothFabricRepository.save(model);
           }
       } else if (split[1].contains(curtainFabric)){
 
           List<CurtainModel> listModel = curtainRepository.findById(id);
           CurtainModel model = listModel.get(0);
           Double quantityInDB = model.getQuantity();
           if (quantityInDB == 0) {
               System.out.println("In DB ZERO");
           } else {
 
               Double result = quantityInDB - quantityFromUI;
               model.setQuantity(result);
               curtainRepository.save(model);
           }
 
       } else if (split[1].contains(orderFabric)){
           List<OrderCurtainModel> listModel = orderCurtainRepository.findById(id);
           OrderCurtainModel model = listModel.get(0);
           Double quantityInDB = model.getQuantity();
           if (quantityInDB == 0) {
               System.out.println("In DB ZERO");
           } else {
 
               Double result = quantityInDB - quantityFromUI;
               model.setQuantity(result);
               orderCurtainRepository.save(model);
           }
       }
У меня получается один и тот же код повторяется, но разные репозитории и модели. Посоветуйте как его сделать лучше?
0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
10.03.2017, 19:19
Ответы с готовыми решениями:

Подскажите как улучшить код (убрать лишний цикл)
Всем доброго времени суток! Есть программа которая делает гистограмму изображения. Все уже работает,но нужно чуть допилить пару моментов...

Ввести с клавиатуры три числа, и вывести их в порядке убывания. Как улучшить код?
Подскажите , как улучшить это ? Сделать более компактным чтоль. А то мне напоминает индусский код. BufferedReader reader = new...

Посоветуйте, как оптимизировать код!
Программа переводит минуты в часы. import java.io.IOException; import java.util.InputMismatchException; import java.util.Scanner; ...

14
636 / 528 / 165
Регистрация: 01.04.2010
Сообщений: 1,843
10.03.2017, 21:05
Репозитории параметризуй генериками, модели должны имплементировать один интерфейс (или иметь общего предка), чтобы getQuantity/setQuantity дёргать можно было.
0
Эксперт Java
3639 / 2971 / 918
Регистрация: 05.07.2013
Сообщений: 14,220
10.03.2017, 21:33
а может эту всю фигню описать одним классом, добавить поле type, забить типы в енум и не парить мозг?
0
3 / 3 / 14
Регистрация: 26.01.2016
Сообщений: 481
11.03.2017, 11:31  [ТС]
Предложение aleksandy понял сам думал так сделать, а вот Ваше xoraxax не понял. Не понял эту часть
HTML5
1
добавить поле type, забить типы в енум
можно пример?
0
Эксперт Java
3639 / 2971 / 918
Регистрация: 05.07.2013
Сообщений: 14,220
11.03.2017, 12:58
Curtain, cloth и все остальное опиши в одном классе, например, goods. В класс добавь поле goodsType. Я просто пока не совсем понимаю, почему это все разные сущности.
0
3 / 3 / 14
Регистрация: 26.01.2016
Сообщений: 481
11.03.2017, 16:21  [ТС]
Это разные табл в БД, я саму БД создавал с помощью hibernate, поэтому каждую таблицу описывал отдельно. Мне так показывали, если возможно по другому, то буду рад повысить свои знания. каждая табл выглядит примерно так:
Кликните здесь для просмотра всего текста
Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
Entity
@Table(name = "CLOTH", schema = "public")
public class ClothModel extends AllFabricModel{
 
    @Column(name = "PHOTO")
    private String photo;
    @Column(name = "PHOTO01")
    private String photo01;
    @Column(name = "PHOTO02")
    private String photo02;
    @Column(name = "PHOTO03")
    private String photo03;
    @Column(name = "PHOTO04")
    private String photo04;
    @Column(name = "PHOTO05")
    private String photo05;
    @Id
    @Column(name = "ID")
    @GeneratedValue(strategy=GenerationType.IDENTITY)
    private Long id;
    @Column
    private String name;
    @Column(name = "DESCRIPTION")
    private String description;
    @Column(name = "STRUCTURE") //organza, curtain ....
    private String structure;
    @Column(name = "PAINT") //abstraction, geometric  ....
    private String paint;
    @Column(name = "HEIGHT")
    private String height;
    @Column(name = "COLOR")
    private String color;
    @Column(name = "QUANTITY")
    private Double quantity;
    @Column(name = "PRICE")
    private BigDecimal price;

у меня даже extends не было, т.к. делал предка в DTO, а к базе старался минимизировать доступ.
0
Эксперт Java
3639 / 2971 / 918
Регистрация: 05.07.2013
Сообщений: 14,220
11.03.2017, 16:34
Так если у тебя все сущности выглядят и ведут себя одинаково, зачем их хранить в разных таблицах, зачем описывать разными классами, если по сути это одно и то же, ты можешь объяснить?
0
3 / 3 / 14
Регистрация: 26.01.2016
Сообщений: 481
11.03.2017, 23:38  [ТС]
Я не пойму, а как можно было создать все эти базы без описания?

Добавлено через 13 минут
описать один класс а потом наследовать, как же название таблиц?

Добавлено через 4 часа 5 минут
Мда, а можно пример как параметризировать похожие репозитории генериками, а то что-то или суббота или все таки не понимаю(
0
Эксперт Java
 Аватар для KEKCoGEN
2399 / 2224 / 565
Регистрация: 28.12.2010
Сообщений: 8,672
11.03.2017, 23:42
Huge, похоже все ваши проблемы от неверного проектирования БД. Покажите ваши таблицы БД.
0
3 / 3 / 14
Регистрация: 26.01.2016
Сообщений: 481
12.03.2017, 10:15  [ТС]
вот одна из них таких, подобных, 5шт
Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
Entity
@Table(name = "CLOTH", schema = "public")
public class ClothModel extends AllFabricModel{
 
    @Column(name = "PHOTO")
    private String photo;
    @Column(name = "PHOTO01")
    private String photo01;
    @Column(name = "PHOTO02")
    private String photo02;
    @Column(name = "PHOTO03")
    private String photo03;
    @Column(name = "PHOTO04")
    private String photo04;
    @Column(name = "PHOTO05")
    private String photo05;
    @Id
    @Column(name = "ID")
    @GeneratedValue(strategy=GenerationType.IDENTITY)
    private Long id;
    @Column
    private String name;
    @Column(name = "DESCRIPTION")
    private String description;
    @Column(name = "STRUCTURE") //organza, curtain ....
    private String structure;
    @Column(name = "PAINT") //abstraction, geometric  ....
    private String paint;
    @Column(name = "HEIGHT")
    private String height;
    @Column(name = "COLOR")
    private String color;
    @Column(name = "QUANTITY")
    private Double quantity;
    @Column(name = "PRICE")
    private BigDecimal price;
Далее есть табл для авторизации и к ней персонализация (Юзер, Модератор, Админ)
Сейчас начинаю писать табл Заказчик/заказ(OneToMany) тоже будут вопросы.
Но хочу этот код упростить, т.к. Идея пишет ошибку, повторяющийся код
0
Эксперт Java
 Аватар для KEKCoGEN
2399 / 2224 / 565
Регистрация: 28.12.2010
Сообщений: 8,672
12.03.2017, 10:58
Цитата Сообщение от Huge Посмотреть сообщение
вот одна из них таких, подобных, 5шт
если у вас подобных 5 штук, у вас неверная архитектура. Нет причины держать 5 одинаковых таблиц в БД. Удалите 4 и оставьте одну.

Так же вынесите фото в коллекцию.
0
3 / 3 / 14
Регистрация: 26.01.2016
Сообщений: 481
12.03.2017, 22:55  [ТС]
хорошо оставлю одну, а как их создавать через hibernate?
0
Эксперт Java
 Аватар для KEKCoGEN
2399 / 2224 / 565
Регистрация: 28.12.2010
Сообщений: 8,672
13.03.2017, 00:16
Huge, ненадо их создавать через hibernate. Руками создавайте. Это хорошая практика.
0
3 / 3 / 14
Регистрация: 26.01.2016
Сообщений: 481
13.03.2017, 19:09  [ТС]
Это практика хорошая, но когда из-за деплоя приходится менять БД, то практика создавать базу из под hibernate очень себя оправдала. Когда мне с MySQL пришлось переводить все на PostgreSQL, именно hibernate я обязан за норм перевод manyToMany, OneToOne все сделал за 1 create.
А до этого, пока умные люди не сказали, что Hibernate сам может создавать БД перелопатил кучу литературы и все равно толком не выходило описать + не знаю как у кого, а у меня pgAdmin4 глючит, через раз загружаясь и слетая при входе в большую БД.

Добавлено через 14 минут
В общем то разобрался, код сократи,л создав отдельный класс для повторения. Спасибо за Ваши ответы
0
Эксперт Java
 Аватар для KEKCoGEN
2399 / 2224 / 565
Регистрация: 28.12.2010
Сообщений: 8,672
13.03.2017, 20:52
Huge, с хибернейт можно создать бд пока система еще не в пррдакшне. Дальше уже нельзя.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
13.03.2017, 20:52
Помогаю со студенческими работами здесь

Посоветуйте, как улучшить код?
Здравствуйте! Вот такую форму написал. Посоветуйте как улучшить код? &lt;html&gt; &lt;head&gt; &lt;title&gt;&lt;/title&gt; ...

Посоветуйте, пожалуйста, как улучшить сборку. 70к
Бюджет: 70к Цель: игры, геймдев, блендер Апгрейд: в ближайшем будущем не планируется Разгон: вообще нет Город: Пенза Сайт:...

Посоветуйте как улучшить старую конфигурацию до 15000 рублей?
На данный момент имею очень старую машину: Проц = DualCore Intel Core 2 Duo E6750, 2666 MHz (8 x 333) Мама = Asus P5B (3 PCI, 3 PCI-E...

Как улучшить код
unit Unit1; interface uses Windows, Messages, SysUtils, Variants, Classes, Graphics, Controls, Forms, Dialogs, StdCtrls,...

Как улучшить код?
В Main() код для вызова функции У меня есть много классов которые должны работать через Thread например: Thread s = new...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
15
Ответ Создать тему
Новые блоги и статьи
Отправка уведомления на почту при изменении наименования справочника
Maks 24.03.2026
Программная отправка письма электронной почты на примере изменения наименования справочника "Склады" в конфигурации БП3. Перед реализацией необходимо выполнить настройку системной учетной записи. . .
модель ЗдравоСохранения 5. Меньше увольнений- больше дохода!
anaschu 24.03.2026
Теперь система здравосохранения уменьшает количество увольнений. 9TO2GP2bpX4 a42b81fb172ffc12ca589c7898261ccb/ https:/ / rutube. ru/ video/ a42b81fb172ffc12ca589c7898261ccb/ Слева синяя линия -. . .
Midnight Chicago Blues
kumehtar 24.03.2026
Такой Midnight Chicago Blues, знаешь?. . Когда вечерние улицы становятся ночными, а ты не можешь уснуть. Ты идёшь в любимый старый бар, и бармен наливает тебе виски. Ты смотришь на пролетающие. . .
Контроль уникальности заводского номера - вариант №2
Maks 24.03.2026
В отличие от предыдущего варианта добавлено прерывание циклов, также добавлены новые переменные для сохранения контекста ошибки перед прерыванием цикла: Процедура ПередЗаписью(Отказ, РежимЗаписи,. . .
SDL3 для Desktop (MinGW): Вывод текста со шрифтом TTF с помощью библиотеки SDL3_ttf на Си и C++
8Observer8 24.03.2026
Содержание блога Финальные проекты на Си и на C++: finish-text-sdl3-c. zip finish-text-sdl3-cpp. zip
Жизнь в неопределённости
kumehtar 23.03.2026
Жизнь — это постоянное существование в неопределённости. Например, даже если у тебя есть список дел, невозможно дойти до точки, где всё окончательно завершено и больше ничего не осталось. В принципе,. . .
Модель здравоСохранения: работники работают быстрее после её введения.
anaschu 23.03.2026
geJalZw1fLo Корпорация до введения программа здравоохранения имела много невыполненных работниками заданий, после введения программы количество заданий выросло. Но на выплатах по больничным это. . .
Контроль уникальности заводского номера - вариант №1
Maks 23.03.2026
Алгоритм контроля уникальности заводского (или серийного) номера на примере документа выдачи шин для спецтехники с табличной частью в конфигурации КА2. Данные берутся из регистра сведений, по. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru