【发布时间】：2017-03-20 13:26:11
【问题描述】：
我希望你们一切都好。所以来到这个问题。我有这部分代码：
private static ArrayList<String> primelist = new ArrayList<>();
static void addToList(String list_elm) {
primelist.add(list_elm);
}
基本上这个列表被我在完整代码（如下）中创建的多个线程同时访问，这些线程负责进行一些计算并返回结果，然后将其添加到这个primelist通过调用方法addToList(String list_elm)。
但是，在所有线程都被终止（即：完成它们的工作）之后，primelist 中存在一些空值。所以经过一些研究，事实证明 ArrayList 不是线程安全的类，因此它的方法也不是。我想问以下（也许很深）问题：
线程在执行一行代码时是否会被放入（等待//定时等待），也就是说它正在调用方法addToList(String list_elm)并且它到达primelist.add(list_elm);的行但同时添加它刚刚停止的元素！
如果不能，请澄清我对（尤其是）ArrayList 案例的困惑。 ((基本上是怎么回事？^^))
完整代码：
import java.util.ArrayList;
import java.util.Iterator;
public class CreatingAThreadThree
{
private static ArrayList<String> primelist = new ArrayList<>();
static void addToList(String list_elm)
{
primelist.add(list_elm);
}
static ArrayList<String> getListReference(){
return primelist;
}
public static void main(String[] args)
{
for(long x = 6223372036854775899L; x<=(6223372036854775999L); x+=2)
{
new Thread (new MyCalcRunnable(x)).start();
}
for(long x = 9223372036854774703L; x<=9223372036854774789L; x+=2 )
{
new MyCalcThread(x, "myChildThread"+x);
}
Thread mainThread = Thread.currentThread();
int spinner =0;
char animation = ' ';
System.out.println("Total number of active threads: " + Thread.activeCount());
System.out.print("Calculating primes: ");
while(Thread.activeCount() >1)
{
spinner ++;
switch(spinner)
{
case 1:
animation = '|';
break;
case 2:
animation = '/';
break;
case 3:
animation = '-';
break;
case 4:
animation = '\\';
spinner = 0;
break;
}
System.out.print("\b" + animation);
try
{
Thread.sleep(200);
}catch(InterruptedException ex)
{
}
}
System.out.println("Total number of active threads: " + Thread.activeCount());
System.out.println("Results List:");
Iterator<?> iterator = (getListReference().iterator());
while(iterator.hasNext())
{
System.out.println(iterator.next());
}
}
}
class MyCalcThread extends Thread
{
private long numberToFactor = 0;
MyCalcThread(long numberToFactor, String name)
{
super(name);
this.numberToFactor = numberToFactor;
start();
}
@Override
public void run()
{
CreatingAThreadThree.addToList(new PrimeStuff().isItPrime(this.numberToFactor));
}
}
class MyCalcRunnable implements Runnable
{
private long numberToFactor = 0;
MyCalcRunnable(long numberToFactor)
{
this.numberToFactor = numberToFactor;
}
@Override
public void run()
{
CreatingAThreadThree.addToList(new PrimeStuff().isItPrime(this.numberToFactor));
}
}
class PrimeStuff
{
String isItPrime(long numberToFactor)
{
if(numberToFactor % 2 == 0)
return (numberToFactor +"is Not prime....divisible by 2");
long squareRoot = (long)(Math.sqrt(numberToFactor));
for(long i=3; i<squareRoot; i++)
{
if(numberToFactor % i == 0)
{
return (numberToFactor +"is Not prime....first divisible by " + i);
}
}
return (numberToFactor + " is Prime!!");
}
}
【问题讨论】：
-
这里有一点需要注意：您想了解 Java 编码风格约定。例如，您不要在普通变量名中使用“_”；这些仅适用于 SOME_CONSTANT。
-
据我所知，当所有字母都是大写时，“_”按约定用于常量，而我的代码中并非如此。
-
我的意思是：_ 仅用于常量。例如，它不像您的 list_elem 那样使用。
-
@Xinchao 在 question 中重构代码是一个真的 坏主意，因为它可能会混淆 OP 和答案以及未来的读者。如果您想要一个规范的MCVE，我建议您提出一个新问题作为一个较小的邪恶（其中一个问题将是重复的。）
标签： arraylist thread-safety copyonwritearraylist