Skip to content

Commit 33462ec

Browse files
committed
SeriesService.add: extracted 4 methods for setting catalog numbers.
No functional changes.
1 parent e637a9e commit 33462ec

File tree

1 file changed

+40
-26
lines changed

1 file changed

+40
-26
lines changed

src/main/java/ru/mystamps/web/service/SeriesService.java

Lines changed: 40 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -101,33 +101,11 @@ public Series add(AddSeriesDto dto) {
101101
series.setQuantity(dto.getQuantity());
102102
series.setPerforated(dto.getPerforated());
103103

104-
Set<MichelCatalog> michelNumbers =
105-
CatalogUtils.fromString(dto.getMichelNumbers(), MichelCatalog.class);
106-
if (!michelNumbers.isEmpty()) {
107-
michelCatalogDao.save(michelNumbers);
108-
series.setMichel(michelNumbers);
109-
}
104+
setMichelNumbersIfProvided(dto, series);
105+
setScottNumbersIfProvided(dto, series);
106+
setYvertNumbersIfProvided(dto, series);
107+
setGibbonsNumbersIfProvided(dto, series);
110108

111-
Set<ScottCatalog> scottNumbers =
112-
CatalogUtils.fromString(dto.getScottNumbers(), ScottCatalog.class);
113-
if (!scottNumbers.isEmpty()) {
114-
scottCatalogDao.save(scottNumbers);
115-
series.setScott(scottNumbers);
116-
}
117-
118-
Set<YvertCatalog> yvertNumbers =
119-
CatalogUtils.fromString(dto.getYvertNumbers(), YvertCatalog.class);
120-
if (!yvertNumbers.isEmpty()) {
121-
yvertCatalogDao.save(yvertNumbers);
122-
series.setYvert(yvertNumbers);
123-
}
124-
125-
Set<GibbonsCatalog> gibbonsNumbers =
126-
CatalogUtils.fromString(dto.getGibbonsNumbers(), GibbonsCatalog.class);
127-
if (!gibbonsNumbers.isEmpty()) {
128-
gibbonsCatalogDao.save(gibbonsNumbers);
129-
series.setGibbons(gibbonsNumbers);
130-
}
131109

132110
String imageUrl = imageService.save(dto.getImage());
133111
Validate.validState(imageUrl != null, "Image url must be non null");
@@ -162,4 +140,40 @@ public Series findById(Integer id) {
162140
return seriesDao.findOne(id);
163141
}
164142

143+
private void setMichelNumbersIfProvided(AddSeriesDto dto, Series series) {
144+
Set<MichelCatalog> michelNumbers =
145+
CatalogUtils.fromString(dto.getMichelNumbers(), MichelCatalog.class);
146+
if (!michelNumbers.isEmpty()) {
147+
michelCatalogDao.save(michelNumbers);
148+
series.setMichel(michelNumbers);
149+
}
150+
}
151+
152+
private void setScottNumbersIfProvided(AddSeriesDto dto, Series series) {
153+
Set<ScottCatalog> scottNumbers =
154+
CatalogUtils.fromString(dto.getScottNumbers(), ScottCatalog.class);
155+
if (!scottNumbers.isEmpty()) {
156+
scottCatalogDao.save(scottNumbers);
157+
series.setScott(scottNumbers);
158+
}
159+
}
160+
161+
private void setYvertNumbersIfProvided(AddSeriesDto dto, Series series) {
162+
Set<YvertCatalog> yvertNumbers =
163+
CatalogUtils.fromString(dto.getYvertNumbers(), YvertCatalog.class);
164+
if (!yvertNumbers.isEmpty()) {
165+
yvertCatalogDao.save(yvertNumbers);
166+
series.setYvert(yvertNumbers);
167+
}
168+
}
169+
170+
private void setGibbonsNumbersIfProvided(AddSeriesDto dto, Series series) {
171+
Set<GibbonsCatalog> gibbonsNumbers =
172+
CatalogUtils.fromString(dto.getGibbonsNumbers(), GibbonsCatalog.class);
173+
if (!gibbonsNumbers.isEmpty()) {
174+
gibbonsCatalogDao.save(gibbonsNumbers);
175+
series.setGibbons(gibbonsNumbers);
176+
}
177+
}
178+
165179
}

0 commit comments

Comments
 (0)