Skip to content

Commit 284e1be

Browse files
committed
AbstractConvertService: apply code template
1 parent 1e100de commit 284e1be

File tree

1 file changed

+23
-25
lines changed

1 file changed

+23
-25
lines changed

src/main/java/org/scijava/convert/AbstractConvertService.java

Lines changed: 23 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -32,31 +32,27 @@
3232
package org.scijava.convert;
3333

3434
import java.lang.reflect.Type;
35-
import java.util.ArrayList;
3635
import java.util.Collection;
3736
import java.util.HashSet;
38-
import java.util.LinkedHashSet;
3937
import java.util.Set;
4038

4139
import org.scijava.plugin.AbstractHandlerService;
4240
import org.scijava.util.ConversionUtils;
4341

4442
/**
45-
* Abstract superclass for {@link ConvertService} implementations. Sets
46-
* this service as the active delegate service in {@link ConversionUtils}.
43+
* Abstract superclass for {@link ConvertService} implementations. Sets this
44+
* service as the active delegate service in {@link ConversionUtils}.
4745
*
4846
* @author Mark Hiner
4947
*/
50-
public abstract class AbstractConvertService extends
51-
AbstractHandlerService<ConversionRequest, Converter<?, ?>> implements
52-
ConvertService
53-
{
48+
public abstract class AbstractConvertService extends AbstractHandlerService<ConversionRequest, Converter<?, ?>>
49+
implements ConvertService {
5450

5551
// -- ConversionService methods --
5652
@SuppressWarnings({ "unchecked", "rawtypes" })
5753
@Override
5854
public Class<Converter<?, ?>> getPluginType() {
59-
return (Class)Converter.class;
55+
return (Class) Converter.class;
6056
}
6157

6258
@Override
@@ -105,8 +101,8 @@ public boolean supports(final Class<?> src, final Type dest) {
105101
}
106102

107103
@Override
108-
public Collection<Object> getCompatibleInputs(Class<?> dest) {
109-
Set<Object> objects = new HashSet<Object>();
104+
public Collection<Object> getCompatibleInputs(final Class<?> dest) {
105+
final Set<Object> objects = new HashSet<Object>();
110106

111107
for (final Converter<?, ?> c : getInstances()) {
112108
if (dest.isAssignableFrom(c.getOutputType())) {
@@ -118,40 +114,40 @@ public Collection<Object> getCompatibleInputs(Class<?> dest) {
118114
}
119115

120116
@Override
121-
public Object convert(Object src, Type dest) {
117+
public Object convert(final Object src, final Type dest) {
122118
return convert(new ConversionRequest(src, dest));
123119
}
124120

125121
@Override
126-
public <T> T convert(Object src, Class<T> dest) {
122+
public <T> T convert(final Object src, final Class<T> dest) {
127123
// NB: repeated code with convert(ConversionRequest), because the
128124
// handler's convert method respects the T provided
129-
Converter<?, ?> handler = getHandler(src, dest);
125+
final Converter<?, ?> handler = getHandler(src, dest);
130126
return handler == null ? null : handler.convert(src, dest);
131127
}
132128

133129
@Override
134-
public Object convert(ConversionRequest request) {
135-
Converter<?, ?> handler = getHandler(request);
130+
public Object convert(final ConversionRequest request) {
131+
final Converter<?, ?> handler = getHandler(request);
136132
return handler == null ? null : handler.convert(request);
137133
}
138134

139135
@Override
140-
public Collection<Class<?>> getCompatibleInputClasses(Class<?> dest) {
141-
Set<Class<?>> compatibleClasses = new HashSet<Class<?>>();
136+
public Collection<Class<?>> getCompatibleInputClasses(final Class<?> dest) {
137+
final Set<Class<?>> compatibleClasses = new HashSet<Class<?>>();
142138

143-
for (Converter<?, ?> converter : getInstances()) {
139+
for (final Converter<?, ?> converter : getInstances()) {
144140
addIfMatches(dest, converter.getOutputType(), converter.getInputType(), compatibleClasses);
145141
}
146142

147143
return compatibleClasses;
148144
}
149145

150146
@Override
151-
public Collection<Class<?>> getCompatibleOutputClasses(Class<?> source) {
152-
Set<Class<?>> compatibleClasses = new HashSet<Class<?>>();
147+
public Collection<Class<?>> getCompatibleOutputClasses(final Class<?> source) {
148+
final Set<Class<?>> compatibleClasses = new HashSet<Class<?>>();
153149

154-
for (Converter<?, ?> converter : getInstances()) {
150+
for (final Converter<?, ?> converter : getInstances()) {
155151
addIfMatches(source, converter.getInputType(), converter.getOutputType(), compatibleClasses);
156152
}
157153

@@ -168,9 +164,11 @@ public void initialize() {
168164
// -- Helper methods --
169165

170166
/**
171-
* Test two classes; if they match, a third class is added to the provided set of classes.
167+
* Test two classes; if they match, a third class is added to the provided
168+
* set of classes.
172169
*/
173-
private void addIfMatches(Class<?> c1, Class<?> c2, Class<?> toAdd, Set<Class<?>> classes) {
174-
if (c1 == c2) classes.add(toAdd);
170+
private void addIfMatches(final Class<?> c1, final Class<?> c2, final Class<?> toAdd, final Set<Class<?>> classes) {
171+
if (c1 == c2)
172+
classes.add(toAdd);
175173
}
176174
}

0 commit comments

Comments
 (0)