我正在使用以下方法在任何可能的情况下打印所有捆绑包内容。问题在于PMD正在报告NcssCount
和CyclomaticComplexity
。我想改进代码,以消除警告,而不仅仅是抑制警告。
private static String bundleToString(Bundle bundle) {
StringBuilder out = new StringBuilder("Bundle[");
if (bundle == null) {
out.append("null");
} else {
boolean first = true;
for (String key : bundle.keySet()) {
if (!first) {
out.append(", ");
}
out.append(key).append('=');
Object value = bundle.get(key);
if (value instanceof int[]) {
out.append(Arrays.toString((int[]) value));
} else if (value instanceof byte[]) {
out.append(Arrays.toString((byte[]) value));
} else if (value instanceof boolean[]) {
out.append(Arrays.toString((boolean[]) value));
} else if (value instanceof short[]) {
out.append(Arrays.toString((short[]) value));
} else if (value instanceof long[]) {
out.append(Arrays.toString((long[]) value));
} else if (value instanceof float[]) {
out.append(Arrays.toString((float[]) value));
} else if (value instanceof double[]) {
out.append(Arrays.toString((double[]) value));
} else if (value instanceof String[]) {
out.append(Arrays.toString((String[]) value));
} else if (value instanceof CharSequence[]) {
out.append(Arrays.toString((CharSequence[]) value));
} else if (value instanceof Parcelable[]) {
out.append(Arrays.toString((Parcelable[]) value));
} else if (value instanceof Bundle) {
out.append(bundleToString((Bundle) value));
} else {
out.append(value);
}
first = false;
}
}
out.append(']');
return out.toString();
}
我已尝试使用以下逻辑将代码分为两部分,但NcssCount
和CyclomaticComplexity
刚刚移至执行所有数组转换的新方法。
if (value != null && value.getClass().isArray()) {
// the new method to call all the array casts
newMethodThatSuffersFromTheSameProblem(out, value);
} else if (value instanceof Bundle) {
out.append(bundleToString((Bundle) value));
} else {
out.append(value);
}
我可以以某种方式对每种数组类型进行动态转换吗?
if (value != null && value.getClass().isArray()) {
out.append(Arrays.toString((<cast-to-class-array-[]>) value));
}
我也尝试过out.append(Arrays.toString(value.getClass().cast(value)));
,但遇到编译错误Cannot resolve method 'toString(capture<? extends java.lang.Object>)'
答案 0 :(得分:4)
} else if (value instanceof String[]) {
out.append(Arrays.toString((String[]) value));
} else if (value instanceof CharSequence[]) {
out.append(Arrays.toString((CharSequence[]) value));
} else if (value instanceof Parcelable[]) {
out.append(Arrays.toString((Parcelable[]) value));
这3个if
可以替换为一个
} else if (value != null && value.getClass().isArray())
out.append(Arrays.toString((Object[]) value));
对于其余的内容,您可以使用静态的Map<Class, Function<Object, String>> convertors
并将其预先填充为
convertors.put(int[].class, value -> Arrays.toString((int [])value));
...
然后使用
Function<Object, String> convertor = convertors.get(value.getClass());
if (convertor != null) {
out.append(convertor.apply(value));
} else ...
针对无法使用Lambda的旧版机器人
interface ToString { String convert(Object object); }
...
static Map<Class, ToString> convertors = new HashMap();
convertors.put(int[].class, new ToString() { public String convert(Object value) { return Arrays.toString((int [])value); }});
...
然后使用
ToString convertor = convertors.get(value.getClass());
if (convertor != null) {
out.append(convertor.convert(value));
} else ...
答案 1 :(得分:0)
我现在无法测试,但是出了什么问题
if (value == null) {
// do whatever you want;
} else if (value instanceof Bundle) {
out.append(bundleToString((Bundle) value));
} else if (value.getClass().isArray()) {
out.append(Arrays.toString((Object[]) value));
} else {
out.append(value);
}
BTW 1:NCSS和相关参数是提示。如果您有20种不同的情况,则可以。虽然您总是应该考虑太大的数字,但有时@SuppressWarning是正确的解决方案。
BTW 2:创建静态bundleToString(Bundle)而不是覆盖Bundle.toString()的原因是什么?