Skip to content

Commit 4675fe6

Browse files
committed
Create NullConverter
Adds a new Converter type to handle null source or destination types. Without this converter, the null logic was managed in a mix between AbstractConverter and DefaultConverter - which means that all converters had some null logic in them (potentially wrong for that given converter) and could not run before DefaultConverter, without copying its null logic.
1 parent d9d39e9 commit 4675fe6

File tree

1 file changed

+81
-0
lines changed

1 file changed

+81
-0
lines changed
Lines changed: 81 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,81 @@
1+
/*
2+
* #%L
3+
* SciJava Common shared library for SciJava software.
4+
* %%
5+
* Copyright (C) 2009 - 2014 Board of Regents of the University of
6+
* Wisconsin-Madison, Broad Institute of MIT and Harvard, and Max Planck
7+
* Institute of Molecular Cell Biology and Genetics.
8+
* %%
9+
* Redistribution and use in source and binary forms, with or without
10+
* modification, are permitted provided that the following conditions are met:
11+
*
12+
* 1. Redistributions of source code must retain the above copyright notice,
13+
* this list of conditions and the following disclaimer.
14+
* 2. Redistributions in binary form must reproduce the above copyright notice,
15+
* this list of conditions and the following disclaimer in the documentation
16+
* and/or other materials provided with the distribution.
17+
*
18+
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
19+
* AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
20+
* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
21+
* ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDERS OR CONTRIBUTORS BE
22+
* LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
23+
* CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
24+
* SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
25+
* INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
26+
* CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
27+
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
28+
* POSSIBILITY OF SUCH DAMAGE.
29+
* #L%
30+
*/
31+
32+
package org.scijava.convert;
33+
34+
import org.scijava.Priority;
35+
import org.scijava.plugin.Plugin;
36+
import org.scijava.util.ConversionUtils;
37+
38+
/**
39+
* {@link Converter} implementation for handling {@code null} values. Performs
40+
* basic casting when given a {@code null} source and returns {@code} null
41+
* directly when given a {@code} null destination.
42+
* <p>
43+
* By running at {@link Priority#FIRST_PRIORITY}, other converters should
44+
* not need to worry about {@code null} source or destination parameters.
45+
* </p>
46+
*
47+
* @author Mark Hiner
48+
*/
49+
@Plugin(type = Converter.class, priority = Priority.FIRST_PRIORITY)
50+
public class NullConverter extends AbstractConverter<Object, Object> {
51+
52+
@Override
53+
public boolean canConvert(final Object src, final Class<?> dest) {
54+
return src == null || super.canConvert(src, dest);
55+
}
56+
57+
@Override
58+
public boolean canConvert(final Class<?> src, final Class<?> dest) {
59+
return src == null || dest == null;
60+
}
61+
62+
@Override
63+
public <T> T convert(final Object src, final Class<T> dest) {
64+
if (dest == null) return null;
65+
if (src == null) return ConversionUtils.getNullValue(dest);
66+
67+
throw new IllegalArgumentException("Attempting non-null conversion: " +
68+
src + " > " + dest + " using NullConverter.");
69+
}
70+
71+
@Override
72+
public Class<Object> getOutputType() {
73+
return Object.class;
74+
}
75+
76+
@Override
77+
public Class<Object> getInputType() {
78+
return Object.class;
79+
}
80+
81+
}

0 commit comments

Comments
 (0)