Skip to content

Commit 8f59872

Browse files
Minor refactoring
1 parent 1d91541 commit 8f59872

2 files changed

Lines changed: 5 additions & 18 deletions

File tree

spring-data-eclipse-store/src/main/java/software/xdev/spring/data/eclipse/store/repository/access/modifier/FieldAccessibleMaker.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
* the data from the field.<br/> }<br/>
3333
* </code>
3434
*/
35-
@SuppressWarnings({"java:S3011", "PMD:PreserveStackTrace"})
35+
@SuppressWarnings({"java:S3011", "PMD.PreserveStackTrace"})
3636
public class FieldAccessibleMaker<E> implements FieldAccessModifier<E>
3737
{
3838
private static final Logger LOG = LoggerFactory.getLogger(FieldAccessibleMaker.class);

spring-data-eclipse-store/src/main/java/software/xdev/spring/data/eclipse/store/repository/support/SimpleEclipseStoreRepository.java

Lines changed: 4 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -234,14 +234,13 @@ public void deleteById(@Nonnull final ID id)
234234
final EclipseStoreTransaction transaction = this.transactionManager.getTransaction();
235235
transaction.addAction(() ->
236236
this.storage.getReadWriteLock().write(
237-
() -> {
237+
() ->
238238
this
239239
.idManager
240240
.findById(id)
241241
.ifPresent(
242242
foundEntity -> this.storage.delete(this.domainClass, foundEntity)
243-
);
244-
}
243+
)
245244
)
246245
);
247246
}
@@ -264,27 +263,15 @@ public void delete(@Nonnull final T entity)
264263
public void deleteAllById(final Iterable<? extends ID> ids)
265264
{
266265
this.storage.getReadWriteLock().write(
267-
() ->
268-
{
269-
for(final ID id : ids)
270-
{
271-
this.deleteById(id);
272-
}
273-
}
266+
() -> ids.forEach(this::deleteById)
274267
);
275268
}
276269

277270
@Override
278271
public void deleteAll(final Iterable<? extends T> entities)
279272
{
280273
this.storage.getReadWriteLock().write(
281-
() ->
282-
{
283-
for(final T entity : entities)
284-
{
285-
this.delete(entity);
286-
}
287-
}
274+
() -> entities.forEach(this::delete)
288275
);
289276
}
290277

0 commit comments

Comments
 (0)