Проблема в многопоточном приложении Java - синхронизированный метод не работает должным образом - PullRequest
Новая
       43

Проблема в многопоточном приложении Java - синхронизированный метод не работает должным образом

1 голос
/ 25 сентября 2011

Я кодирую программу, аналогичную производителю-потребителю в Java (но с частью только для потребителя, без потока производителя). Похоже, что критическая область выполняется одновременно несколькими потоками, несмотря на то, что я вызываю код синхронизированного метода.

Вот код:

Класс 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.");

}

}

Класс 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;
}

}

Класс 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, главная] Педидо: 0 Inicio: 2011/09/24 21:14:18 Fim: 2011/09/24 21:14:19 posicao 0

Тема: Тема [Тема-6,5, главная] Педидо: 0 Inicio: 2011/09/24 21:14:18 Fim: 2011/09/24 21:14:19 posicao 0

Тема: Тема [Тема-2,5, основная] Педидо: 0 Inicio: 2011/09/24 21:14:18 Fim: 2011/09/24 21:14:19 posicao 0

Тема: Тема [Тема-9,5, главная] Педидо: 0 Inicio: 2011/09/24 21:14:18 Fim: 2011/09/24 21:14:19 posicao 0

Тема: Тема [Тема-3,5, основная] Педидо: 4 Inicio: 2011/09/24 21:14:18 Fim: 2011/09/24 21:14:19 posicao 4

Тема: Тема [Тема-5,5, главная] Педидо: 5 Inicio: 2011/09/24 21:14:18 Fim: 2011/09/24 21:14:19 posicao 5

Тема: Тема [Тема-0,5, основная] Педидо: 0 Inicio: 2011/09/24 21:14:18 Fim: 2011/09/24 21:14:19 posicao 5

Тема: Тема [Тема-8,5, main] Педидо: 0 Inicio: 2011/09/24 21:14:18 Fim: 2011/09/24 21:14:19 posicao 5

Тема: Тема [Тема-4,5, основная] Педидо: 5 Inicio: 2011/09/24 21:14:18 Fim: 2011/09/24 21:14:19 posicao 5

Тема: Тема [Тема-1,5, основная] Педидо: 0 Inicio: 2011/09/24 21:14:18 Fim: 2011/09/24 21:14:19 posicao 0

Processamento encerrado.

Поймите, что значение позиции повторяется между различными потоками. В выходных данных posicao должен иметь разные значения в каждом потоке, который вызывает синхронизированный метод.

1 Ответ

2 голосов
/ 25 сентября 2011

Каждый экземпляр потока синхронизируется сам по себе.Чтобы сделать все потоки взаимоисключающими, они должны синхронизироваться на общем объекте.

То есть 

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, передайте им общий объект для синхронизации.

...