是否有可能重构此Java代码？

Question

我有一个简单的代码如下:

import java.util.ArrayList;

public class BoidList extends ArrayList
{
  synchronized public Boid GetBoid( int idx_ ) throws ArrayIndexOutOfBoundsException
  {
    if( idx_ <  super.size() &&
        idx_ >= 0 )
    {
      return (Boid) super.get(idx_);
    }
    else
    {
      throw new ArrayIndexOutOfBoundsException();
    }
  }
  synchronized public void RemoveBoid( int idx_ ) throws ArrayIndexOutOfBoundsException
  {
    if( idx_ <  super.size() &&
        idx_ >= 0 )
    {
      super.remove(idx_);
    }
    else
    {
      throw new ArrayIndexOutOfBoundsException();
    }    
  }
}

两种方法之间有很多相似之处,但它们做了两件不同的事情.有可能重构这个吗？

Answer 1

这个的真正目的是BoidList什么？考虑以下:

List<Boid> boids = Collections.synchronizedList(new ArrayList<Boid>());

这行代码或多或少等同于您尝试创建的子类:

• 类型安全在编译时强制执行
• ArrayListIndexOutOfBoundsException如果索引无效,则抛出一个
• Collections.synchronizedList() 确保对列表的同步访问

根据您提供的源代码,我认为没有任何理由创建您自己的子类.

Answer 2

public class BoidList extends ArrayList<Boid> {

    private void checkIndex(int idx) {
        if (idx >= super.size() || idx < 0) {
            throw new ArrayIndexOutOfBoundsException(String.valueOf(idx));
        }
    }

    synchronized public Boid getBoid(int idx) {
        checkIndex(idx);
        return super.get(idx);
    }

    synchronized public void removeBoid(int idx) {
        checkIndex(idx);
        super.remove(idx);
    }
}

这会以多种方式修复您的代码:

• checkIndex()方法是你想要的
• 永远不会声明抛出RuntimeException,因为它是一个编程错误
• 遵循java命名约定(GetBoid - > getBoid,RemoveBoid - > removeBoid)
• 删除无用的转换只是扩展ArrayList

编辑

您可能还想检查CopyOnWriteArrayList <Boid>