多线程Java应用程序中的问题 - 同步方法无法按预期工作

时间:2011-09-25 00:47:43

标签: java multithreading concurrency synchronized synchronize

我正在编写一个类似于java中的producer-consumer的程序(但是只有消费者部分,没有生产者线程)。看起来关键区域正由多个线程同时执行,尽管事实上我正在调用同步方法的代码。

以下是代码:

Class Main.java:

package principal;

public class Main {

/**
 * @param args
 */
public static void main(String[] args) {
    final int tamanho_buffer=10;
    final int quantidade_threads=10;

    Pedido buffer[] = new Pedido[tamanho_buffer];
    Consumidor consumidor[] = new Consumidor[quantidade_threads];

    for (int i=0;i<tamanho_buffer;i++) {
        buffer[i]=new Pedido();
    }

    for (int i=0;i<quantidade_threads;i++) {
        consumidor[i]=new Consumidor();
    }

    for (int i=0;i<tamanho_buffer;i++) {
        int identificador[]=new int[Pedido.getTamanho_identificador()];
        identificador[0]=i;
        buffer[i].setIdentificador(identificador);
        buffer[i].setTexto("pacote de dados");
    }

    Consumidor.setBuffer(buffer);
    Consumidor.setTamanho_buffer(tamanho_buffer);

    for (int i=0;i<quantidade_threads;i++) {
        consumidor[i].start();
    }

    for (int i=0;i<quantidade_threads;i++) {
        try {
            consumidor[i].join();
        }catch(InterruptedException e ){
            System.out.println("InterruptedException lancada");
        }
    }

    System.out.println("Processamento encerrado.");

}

}

Class Pedido.java:

package principal;

    public class Pedido {
private int identificador[];
private String texto;
static int tamanho_identificador=10;
int ti=tamanho_identificador;

public Pedido() {
    this.identificador= new int[this.ti];
}

public static int getTamanho_identificador() {
    return tamanho_identificador;
}

public int[] getIdentificador() {
    return identificador;
}
public void setIdentificador(int[] identificador) {
    this.identificador = identificador;
}
public String getTexto() {
    return texto;
}
public void setTexto(String texto) {
    this.texto = texto;
}

}

Class Consumidor.java

package principal;

import java.util.Date;
import java.text.DateFormat;
import java.text.SimpleDateFormat;

public class Consumidor extends Thread {

private static Pedido buffer[];
private static int tamanho_buffer;
private static int posicao=0;

public static void setTamanho_buffer(int tamanhoBuffer) {
    tamanho_buffer = tamanhoBuffer;
}

public static void setBuffer(Pedido[] buffer) {
    Consumidor.buffer = buffer;
}

public void run() {
    DateFormat dateFormat = new SimpleDateFormat("yyyy/MM/dd HH:mm:ss");
    while (posicao < Consumidor.tamanho_buffer ) {
   //           int identificador;
   //           identificador=buffer[posicao].getIdentificador()[0];;

        Date datainicio = new Date();
        String inicio=dateFormat.format(datainicio);

        try {
            Consumidor.sleep(1000);
        } catch(InterruptedException e) {
            System.out.println("InterruptedException lancada");
        }

        Date datafim = new Date();
        String fim=dateFormat.format(datafim);

        consomebuffer(inicio,fim);

    }
}

synchronized void consomebuffer(String inicio, String fim) {
    if (posicao < Consumidor.tamanho_buffer ) {
        int identificador;
        identificador=buffer[posicao].getIdentificador()[0];
        System.out.println("Thread: "+Thread.currentThread()+" Pedido: "+identificador+" Inicio: "+inicio+" Fim: "+fim+" posicao "+posicao);
        posicao++;
    }
}
}

方法consomebuffer是同步的,但它看起来像其他线程同时访问变量posicao(位置的葡萄牙语名称),这不应该发生,因为它是一个同步方法。程序输出是这样的:

线程:线程[线程-7,5,主要] Pedido:0 Inicio:2011/09/24 21:14:18 Fim:2011/09/24 21:14:19 posicao 0

线程:主题[主题6,5,主要] Pedido:0 Inicio:2011/09/24 21:14:18 Fim:2011/09/24 21:14:19 posicao 0

线程:主题[主题-2.5,主要] Pedido:0 Inicio:2011/09/24 21:14:18 Fim:2011/09/24 21:14:19 posicao 0

主题:线程[主题-9,5,主要] Pedido:0 Inicio:2011/09/24 21:14:18 Fim:2011/09/24 21:14:19 posicao 0

线程:线程[主题-3.5,主要] Pedido:4 Inicio:2011/09/24 21:14:18 Fim:2011/09/24 21:14:19 posicao 4

线程:线程[主线-5,5,主要] Pedido:5 Inicio:2011/09/24 21:14:18 Fim:2011/09/24 21:14:19 posicao 5

线程:线程[线程-0,5,主要] Pedido:0 Inicio:2011/09/24 21:14:18 Fim:2011/09/24 21:14:19 posicao 5

线程:线程[线程-8,5,主要] Pedido:0 Inicio:2011/09/24 21:14:18 Fim:2011/09/24 21:14:19 posicao 5

线程:线程[主线-4,5,主要] Pedido:5 Inicio:2011/09/24 21:14:18 Fim:2011/09/24 21:14:19 posicao 5

线程:主题[主题-1,4,主要] Pedido:0 Inicio:2011/09/24 21:14:18 Fim:2011/09/24 21:14:19 posicao 0

Processamento encerrado。

意识到位置值在不同线程之间重复出现。在输出中,posicao在每个调用synchronized方法的线程中都应该有不同的值。

1 个答案:

答案 0 :(得分:2)

每个Thread实例都在自身同步。为了使所有线程互斥,它们必须在公共对象上同步。

即,

public synchronized method(int parameter)
{
    //do some stuff
}

的简写
public method(int parameter)
{
    synchronized (this)
    {
        //do some stuff
    }
}

如果你想让一堆线程相互同步,你需要为它们提供一个可以同步的公共对象。

例如,您可以添加Consumidor.java的构造函数

public Consumidator(Object monitor)
{
     myMonitor = monitor
}

然后在奔跑中

void consomebuffer(String inicio, String fim) {
  synchronized (myMonitor)
  {
    if (posicao < Consumidor.tamanho_buffer ) {
        int identificador;
        identificador=buffer[posicao].getIdentificador()[0];
        System.out.println("Thread: "+Thread.currentThread()+" Pedido: "+identificador+" Inicio: "+inicio+" Fim: "+fim+" posicao "+posicao);
        posicao++;
    }
  }
}

然后在创建Consumidadors数组时,向它们传递一个共享对象以进行同步。

相关问题