Android/Java 方法的奇怪问题。控制似乎跳过 if/else 块

Question

在下面的 Android 示例中，我有一组 JSON 对象（称为模板），并且正在对它们进行某种继承处理。如果一个 JSON 对象有一个键“extends”，则相应的值将引用另一个 JSON 对象。我只是将引用的 JSON 对象中的映射复制到当前对象中。每个属性（在代码中称为组件）都会有子属性，因此每个子属性都是单独复制的。 所有 JSON 对象都存储在 HashMap 中。 编辑：发布演示该问题的示例代码。

package com.trial;

import java.util.HashMap;
import java.util.Iterator;

import org.json.JSONException;
import org.json.JSONObject;

import android.app.Activity;
import android.os.Bundle;

public class Trial extends Activity {

    HashMap<String, JSONObject> templates = new HashMap<String, JSONObject>();

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);
        try {
            templates.put("entity", new JSONObject("{\"Health\":{\"health\":50,\"maxHealth\":100}}"));
            templates.put("unit", new JSONObject("{\"Health\":{\"health\":70},\"extends\":\"entity\"}"));
            Iterator<String> templatesIterator = templates.keySet().iterator();
            while (templatesIterator.hasNext()) {
                String templateName = templatesIterator.next();
                JSONObject template = templates.get(templateName); 
                if (template.has("extends"))
                    templates.put(templateName, processInheritance(template));
            }
        } catch (JSONException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
    }

    private JSONObject processInheritance(JSONObject template) throws JSONException {
        if (template.has("extends")) {
            //If current template extends from another template
            String parentTemplateName = template.getString("extends");
            template.remove("extends");
            JSONObject parentTemplate = processInheritance(templates.get(parentTemplateName));
            //Create a clone of the parent to which child properties/components can be added
            JSONObject processedTemplate = new JSONObject(parentTemplate.toString());
            Iterator<String> it = template.keys();
            while (it.hasNext()) {
                String componentName = it.next();
                if (processedTemplate.has(componentName)) {
                    //Component is already present. Loop through Component properties and make
                    //specific replacements
                    JSONObject componentData = template.getJSONObject(componentName);
                    Iterator<String> it2 = componentData.keys();
                    JSONObject processedComponentData = processedTemplate.getJSONObject(componentName); 
                    while (it2.hasNext()) {
                        String propertyName = it2.next();
                        processedComponentData.put(propertyName, componentData.get(propertyName));
                    }
                    processedTemplate.put(componentName, processedComponentData);
                } else {
                    //Component is not already present. Simply copy
                    processedTemplate.put(componentName, template.get(componentName));
                }
            }
            return processedTemplate;
        } else {
            return template;
        }
    }
}

在调试时，我注意到代码在 while 循环之后执行得很好。 while 循环结束后不久，它会跳过 if/else 并返回“template”而不是“processedTemplate”。甚至结果对象也表明正在返回“模板”。我尝试清除二进制文件，重新创建模拟器，然后在实际设备上执行它。这种奇怪的行为仍然存在。有人能告诉我为什么会发生这种情况吗？

编辑：返回正确的对象。我之前评论说“模板”被返回是因为一个错误，我没有将返回的流程对象保存回 HashMap。 我只是在做

if (template.has("extends"))
    processInheritance(template));

但是控制似乎仍然跳跃。我想是 Eclipse 的问题。

Answer 1

我尝试使用 Eclipse 调试您的代码，如果我正确理解它的目标，它似乎可以工作。调试器的行为似乎如您所描述的那样：当它应该返回processedTemplate时，它会跳到带有template的行。然而，正如预期的那样，processedTemplate 被返回。调试器似乎总是从最后一个 return 语句跳出函数。

我想说问题不在于这段代码。也许您应该发布一些出现问题的 JSON 示例文件。

Answer 2

我的猜测是因为它是一种递归方法，并且您有效地删除了一个键，然后迭代了剩余键的列表，因此在递归过程中这会以某种方式搞砸。尝试不要删除“extends”键，而是在迭代键集时忽略它。

Answer 3

虚拟机永远不会出现这种错误，因此您的递归方法很可能是一个错误。
我无法立即确定它，但有几个 System.out.println(template);可能有帮助。

Answer 4

更改如下代码似乎已经解决了问题，但原始代码片段的问题仍然存在。

private JSONObject processInheritance(JSONObject template) throws AssetLoadingException {
        try {
            if (!template.has("extends")) {
                return template;
            } else {
                //Everything else
            }
        } catch (JSONException ex) {
            throw new AssetLoadingException(ex);
        }
    }

Answer 5

发生这种情况是因为您将处理所有对象的继承，直到获得根对象。 （没有“extends”值的那个。）

因此，您找到的第一个返回是 else 分支中的返回。

重复我的解释：

这是你的精简代码：

private JSONObject processInheritance(JSONObject template) throws AssetLoadingException {
        try {
            if (template.has("extends")) {

                JSONObject parentTemplate = processInheritance(templates.get(parentTemplateName));

                }
                return processedTemplate;
            } else {
                return template;
            }
        } catch (JSONException ex) {
            throw new AssetLoadingException(ex);
        }
    }

如果你有序列 ROOT <- C <- B <- A （其中 B 扩展 A 等等）并且开始评估 A，将会发生以下情况：

1. 进程 A
2. A 正在扩展 B（有“extends”），因此进程 B
3. B 正在扩展 C，因此进程 C
4. C 正在扩展 ROOT，因此进程 ROOT
5. ROOT不扩展任何内容，因此输入 else 并返回模板。

（那么处理完C、B、A后会有一个顺序返回）