背景

Question

背景

我正在开发一个需要在服务器上使用JSON服务的应用程序，然后将该JSON重新打包到可以使用视图中的Razor语法引用的视图模型中。出于各种原因，必须对此代码进行服务器端操作。

我们正在使用C＃，.NET 4，MVC3，Razor，JsonFx。

我们目前的代码工作得很好，但是在收到的JSON对象中迭代250个项目需要花费一分钟时间，这是不可接受的（更不用说令人费解了）。我已经将问题隔离到以下循环中; JSON快速闪电，这不是问题所在。这是工作但非常慢的代码：

        var reader = new JsonReader();
        var json = GetJson(SPListName);

        var admItems = new List<IDictionary<String, object>>();
        dynamic _items = reader.Read(json); //This part is REALLY fast.  No problem here.
        foreach (var itm in _items)
        {
            dynamic obj = new ExpandoObject();
            foreach (dynamic admObj in itm)//Here begins the slow part.
            {
                var item = obj as IDictionary<String, object>;
                var encodedValue = "";
                try
                {
                    if(admObj.Key == "Title")
                    {
                        encodedValue = admObj.Value.ToString();
                    }else
                    {
                        encodedValue = admObj.Value[0].ToString();
                    }
                }
                catch (Exception)
                {
                    encodedValue = admObj.Value.ToString();                   
                }

                item[admObj.Key] = encodedValue.EncodeNonAscii().FixHtmlEntities();
            }
            admItems.Add(obj);
        }
        return admItems;

您可能还会注意到一些自定义扩展方法。他们在这里（如果重要的话）：

public static string EncodeNonAscii(this Object str)
            {
                StringBuilder sb = new StringBuilder();
                foreach (char c in str.ToString())
                {
                    if (c > 127)
                    {
                        // This character is too big for ASCII
                        string encodedValue = "\\u" + ((int) c).ToString("x4");
                        sb.Append(encodedValue);
                    }
                    else
                    {
                        sb.Append(c);
                    }
                }
                return sb.ToString();
            }

            public static string FixHtmlEntities(this Object str)
            {
                var fixedString = str.ToString().Replace("\\u00c2\\u00ae", "&reg;");
                return fixedString;
            }

问题

我做错了什么/如何加快速度呢？我的大脑现在是汉堡包，所以我希望有人指出一个简单的疏忽。

更新/分辨率

Rophuine和Joshua Enfield都指出了速度问题的根源：捕获异常会减慢一切速度。

许多人建议我使用Json.Net或类似的东西。虽然我很欣赏这个建议，但它确实不是我问题的根源（即使它可能是这样出现的）;我过去曾广泛使用Json.Net，几个月前更喜欢JsonFx。在这种特殊情况下，我更关心视图模型对象的构造，因为我已经使用JsonFx对JSON进行了反序列化。如果你仍然认为我错过了你的观点，请告诉我。）。

我的脑死亡尝试/捕获方案的原因是我需要对JSON字符串的每个属性执行不同的操作，具体取决于它的类型（字符串，字符串[]，对象[]等）。我忘了System.Type可以为我处理。所以这是最终的代码：

var reader = new JsonReader();
                var json = GetJson(SPListName);

                var admItems = new List<IDictionary<String, object>>();
                dynamic _items = reader.Read(json);
                foreach (var itm in _items)
                {
                    dynamic obj = new ExpandoObject();
                    foreach (dynamic admObj in itm)
                    {
                        var item = obj as IDictionary<String, object>;
                        var encodedValue = "";

                        Type typeName = admObj.Value.GetType();

                        switch (typeName.ToString())
                        {
                            case("System.String[]"):
                                encodedValue = admObj.Value[0].ToString();
                                break;
                            default:
                                encodedValue = admObj.Value.ToString();
                                break;
                        }

                        item[admObj.Key] = encodedValue.EncodeNonAscii().FixHtmlEntities();
                    }
                    admItems.Add(obj);
                }
                return admItems;

希望这有助于某人！

Answer 1

            try
            {
                if(admObj.Key == "Title")
                {
                    encodedValue = admObj.Value.ToString();
                }else
                {
                    encodedValue = admObj.Value[0].ToString();
                }
            }
            catch (Exception)
            {
                encodedValue = admObj.Value.ToString();                   
            }

我面前没有C＃编译器，也没有访问您的数据，但这看起来对我很怀疑。例外情况非常慢 - 您是否经常访问catch块？如果你是的话，试着找出造成异常的原因并处理它们而不会引起异常 - 留下例外来处理你没想到的罕见情况。

编辑：最终解决方案在问题编辑中 - 我不会在这里重复。如果您遇到性能问题并且在循环中的某个地方进行了异常处理，那么这通常是首先尝试消除的问题。例外情况非常缓慢 - 远比你想象的要多得多。他们最好保留在非常特殊的情况下。

在C＃中迭代动态对象的最快方法是什么？

背景

问题

更新/分辨率

1 个答案: