关于for和foreach,兼顾效率与安全

对于数组的访问,是应该使用for的方式的,因为这样性能更高。以下代码是恰当的。 Object[] objArray = ...;
int objArrayLength = objArray.Length;
for (int i = 0; i < objArrayLength; ++i)
{
    // do something ...
}

String str = ...;
int strLength = str.Length;
for (int i = 0; i < strLength; ++i)
{
   // do something ...
}
对ArrayList这样的可使用下标进行随机访问的数据结构,使用下标访问,要比foreach的方式进行顺序访问,速度要快一些。foreach这样 写法,使用的过程产生一个额外的对象Enumerator,而且每次访问需要更多的操作,降低性能。下面的两种写法编译出的代码是一样的:
第一种写法:
IList list = new ArrayList();
IEnumerator iter = list.GetEnumerator();
try
{
    while (iter.MoveNext())
    {
        Object obj = iter.Current;
        //do something ...
    }
}
finally
{
    IDisposable disposableObj = iter as IDisposable;
    if (disposableObj != null)
    {
        disposableObj.Dispose();
    }
}
第二种写法: IList list = new ArrayList();
foreach (Object obj in list)
{
    //do something ...
}
对比这两种写法,第一种写法非常罗嗦,所以C#引入了foreach的语法。通过观察第一种写法,foreach是通过GetEnumerator获得一 个IEnumerator对象,通过IEnumerator对象执行MoveNext()方法和获取Current属性进行遍历的。

我们再通过Reflector工具,查看mscorlib.dll中System.Collection.ArrayList的实现: //为了简单起见,我只列出ArrayList的Add、Clear、GetEnumerator的代码
public class ArrayList
{
    //这是一个版本标识,ArrayList对象,每做一个修改操作,_version都会加1
    private int _version;

    public virtual int Add(object value)
    {
        int num1;
        if (this._size == this._items.Length)
        {
            this.EnsureCapacity((this._size + 1));
        }
        this._items[this._size] = value;
        ++this._version; //注意此处
        this._size = ((num1 = this._size) + 1);
        return num1;
    }

    public virtual void Clear()
    {
        Array.Clear(this._items, 0, this._size);
        this._size = 0;
        ++this._version; //注意此处
    }

    //每次调用GetEnumerator方法,都会构造一个FastArrayListEnumerator
    //或者ArrayListEnumeratorSimple对象。
    public virtual IEnumerator GetEnumerator()
    {
        if (base.GetType() == typeof(ArrayList))
        {
            return new ArrayList.FastArrayListEnumerator(this);
        }
        return new ArrayList.ArrayListEnumeratorSimple(this);
    }
}

通过上述代码可以看到,ArrayList是通过_version成员变量作版本标识的,每次执行Add、Clear等修改ArrayList内容的操 作,都会将版本号加1,而每次调用GetEnumerator方法,都会构造一个FastArrayListEnumerator或者 ArrayListEnumeratorSimple对象。我们再看FastArrayListEnumerator的实现: class FastArrayListEnumerator
{
    private int version;

    internal FastArrayListEnumerator(ArrayList list)
    {
        this.list = list;
        this.index = -1;

        //获取构建FastArrayListEnumerator对象时ArrayList的版本号
        this.version = list._version;

        this.lastIndex = (list._size - 1);
    }

    public bool MoveNext()
    {
        int num1;

        //比较ArrayList当前的版本号,
        //是否和构建FastArrayListEnumerator对象时的版本号一致
        //如果不一致,则抛出异常。
        if (this.version != this.list._version)
        {
            throw new InvalidOperationException(
                Environment.GetResourceString("InvalidOperation_EnumFailedVersion"
                );
        }

        //... ...
    }
}
FastArrayListEnumerator对象构建时,当时时ArrayList的版本号。当执行MoveNext()操作时,检查 ArrayList当前的版本号是否和FastArrayListEnumerator对象构建时的版本号一致,如果不一致就会抛出异常。

由于Enumerator中,做了版本检查处理的工作,所以使用foreach是线程安全,而使用for则不时。为什么呢?如果在使用foreach遍历 对象的过程中,其他线程修改了List的内容,例如添加或者删除,就会出现不可知的错误,而使用foreach则能够正确抛出错误信息。

综上所述,结论如下:
使用for,更高效率。
使用foreach,更安全。

那么如何选择呢?我的建议是,在一些全局的,多线程可以访问的数据结构对象,使用foreach。而对本地变量,则使用for,效率和安全兼顾!例如: public void F1(IList globalList)
{
    IList waitForDeleteList = new ArrayList();

    //全局变量,使用foreach,保证线程
    foreach (Object item in globalList)
    {

        if (condition)
        {
            waitForDeleteList.Add(item);
        }
    }

    //本地变量使用for,保证效率
    int waitForDeleteListCount = waitForDeleteList.Count;
    for (int i = 0; i < waitForDeleteListCount; ++i)
    {
        globalList.Remove(waitForDeleteList);
    }
}
以上建议,对于在Java环境下也使用,我阅读过JDK 1.4的java.util.ArrayList的实现,.NET Framework的实现和JDK的实现,几乎是一样的,是否抄袭,见仁见智。上述的C#代码在Java环境中应为: public void f1(List globalList) {
    List waitForDeleteList = new ArrayList();
    //全局变量,使用Iterator遍历,保证线程
    Iterator iter = globalList.iterator();
    while (iter.hasNext()) {
        Object item = iter.next();
        if (condition) {
            waitForDeleteList.add(item);
        }
    }
   
    //本地变量使用for,保证效率
    int waitForDeleteListCount = waitForDeleteList.size();
    for (int i = 0; i < waitForDeleteListCount; ++i) {
        globalList.remove(waitForDeleteList.get(i));
    }
}

注意,以上代码并不是做该项工作的最优算法,如果需要更高的效率,修改如下:
C#版本
public void F1(IList globalList)
{
    bool condition = true;
    IList waitForDeleteList = new ArrayList();

    //全局变量,使用foreach,保证线程
    int itemIndex = 0;
    foreach (Object item in globalList)
    {
        if (condition)
        {
            waitForDeleteList.Add(index);
        }
        ++itemIndex;
    }

    //本地变量使用for,保证效率
    int waitForDeleteListCount = waitForDeleteList.Count;
    for (int i = waitForDeleteListCount - 1; i >= 0; --i)
    {
        index = (int) waitForDeleteList;
        globalList.RemoveAt(itemIndex);
    }
}
Java版本:
public void f1(List globalList) {
    List waitForDeleteList = new ArrayList();
    //全局变量,使用Iterator遍历,保证线程
    Iterator iter = globalList.iterator();
    int index = 0;
    while (iter.hasNext()) {
        Object item = iter.next();
        if (condition) {
            waitForDeleteList.add(new Integer(index));
        }
        ++index;
    }
   
    //本地变量使用for,保证效率
    int waitForDeleteListCount = waitForDeleteList.size();
    for (int i = waitForDeleteListCount - 1; i >= 0; --i) {
       index = ((Integer) waitForDeleteList.get(i)).intValue();
        globalList.remove(index);
    }
}

呵呵,你说的意思我明白,不过有一些东西我还是觉得不能够完全认同你的。

1、线程安全。
这里其实不涉及线程安全,所谓线程安全在foreach里面可以部分的保证,但是不能够完全保证。SyncRoot才是保证线程安全的东 西,_version所保护的并不是线程安全,而是类似iterator模式里面的“安全遍历”问题。.NET Framework在这里有点“偷懒”并不能够保证“安全遍历”,如果在遍历期间修改了元素的内容,则会引发异常。既然是引发异常,那就不是“XX安全” 或者“安全XX”,因为他认为这是一种异常情况,而不是“安全”情况。

你可以分析一下SyncRoot属性的行为,应该可以看到有一个什么特殊的类型,用于“线程安全”操作的。(这个是我估计的,因为在绝大多数的集合类型里 面都会采用一个内部嵌套的类来提供线程安全问题。不过在.NET 2.0里面的Generic集合类型里面不提供这样的操作了,也就是不提供SyncRoot了,因为实际上还是有可能线程不安全的。)在这里面你会看到实 际上是通过lock(或者类似的其它东西,目前我没有见到除了lock之外的其他东西,但不排除)来达到线程安全的,实际上并不会引发任何的 exception。

试想一下,如果说一个操作能够引发异常,那么这个操作无论如何都不能够说是“安全”的操作。如果你的程序在foreach外面没有try...catch,甚至会让你的程序崩溃掉,试问,这样是安全的吗?不是吧?我宁愿认为for是“安全的”,尽管他不完全正确。

实际上如果你在for里面进行add/insert/remove操作,是完全可以判断应该怎么修改index的值,以达到安全遍历的效果。(例如对于 add,index完全不需要修改;对于insert,如果insert_index < index 那么 index++ 否则不处理;remove和insert一样,除了把++换成--)

2、另外,我在你那篇文章里面回复的内容,实际上不会有安全问题,因为我把它copy出来了再进行操作。或者说,遍历的是keys和values数组,修改的是ht哈希表,对哈希表的修改不会影响keys和values,所以完全不会构成冲突。

不知道我说的是否有理呢?  

你的回复中,对于安全的描述,你说的也有一定道理,既然引发了异常就不是安全了。
其实我想表达的安全,就是你所说的安全遍历。一般编写多线程程序时,使用for都要很小心,因为for无法保证安全遍历,一旦出错,谁也不知道为什么,这种错误,可能极难被人发现。
记得以前刚学多线程编程的时候,在一本Java关于多线程编程的书上,就介绍,多线程慎用for,常识呀。

你对Dictionay数据结构的遍历方式,我对此还是持保留态度。我认为,这种方式,数据量小,不一定快,大数据量肯定慢,如果中途有break退出,就更不值得了,因此不推荐使用。

posted @ 2011-09-30 16:11  hanmos  阅读(1856)  评论(0编辑  收藏  举报