Skip to main content
4 de 6
adicionou 1211 caracteres ao conteúdo
Piovezan
  • 18mil
  • 7
  • 53
  • 91

Na minha experiência não é comum você ter duas listas independentes necessitando de acesso sincronizado em um objeto. Pode ser indício de uma violação do Princípio de Responsabilidade Única. Mas isso é uma questão de design e não vem ao caso.

Tenha em mente que o acesso sincronizado a um trecho de código depende de um "token", isto é, um objeto do qual só existe uma instância, de forma que quem tiver ele num determinado momento pode executar a ação desejada, caso contrário terá que esperar o dono do token liberá-lo para que ele possa adquirir esse token e efetuar a ação. Com a diferença que o termo usado não é "token" e sim "lock" (ou semáforo, monitor, etc). O que quero dizer é que a função dele é ser um token. É disso que se trata o acesso sincronizado: o thread que possui o lock pode entrar no trecho de código sincronizado e executá-lo (e fazer operações sobre as suas listas, por exemplo).

Qualquer objeto pode ser um "lock". Quando um método é synchronized, o lock é o próprio objeto que contém aquele método. No caso do seu exemplo, se você tiver um código assim...

Objt bijeto = new Objt();
bijeto.handleLists(list);

...o objeto bijeto será o lock para o método sincronizado handleLists().

A escolha do lock (ou dos locks) mais apropriado(s) para o seu caso vai depender da sua intenção. Quando um thread tem acesso a list_t1, você quer que ele também tenha acesso a list_t2? Nesse caso, pode usar um lock só para as duas listas. Esse lock pode ser o próprio objeto Objt, ou então um outro objeto que você define como atributo de Objt.

Ou quando um thread tem acesso a list_t1, você NÃO quer que ele tenha acesso a list_t2? Nesse caso precisará de dois locks diferentes. As próprias listas, se forem não-nulas e declaradas como final, podem ser usadas como locks para acesso a si mesmas. Ou então você pode criar dois atributos lock_t1 e lock_t2.

Importante notar que os dois trechos de código abaixo são equivalentes:

Trecho 1:

public class Objt {

    public synchronized handleList() {
        ...
    }
}

Trecho 2:

public class Objt {

    public handleList() {
        synchronized(this) { // "this" é referência para o próprio
            ...              // objeto de classe Objt
        }
    }
}

Edição:

Você esclareceu que quer ter um método que recebe como parâmetro uma lista e faz alterações nela. Um thread irá chamar esse método passando uma lista list_t1 como parâmetro e outro thread irá chamar esse método passando outra lista list_t2 como parâmetro.

Nesse caso SE esse for o único ponto em que as listas são modificadas, o paralelismo funcionará sem problemas e não haverá necessidade de declarar o método synchronized, como o @bigown falou na resposta dele, que responde perfeitamente a pergunta. Vai funcionar porque o método não está fazendo nenhuma alteração de estado, a não ser no estado dos parâmetros passados a ele.

Mas evite declarar as listas na classe que contém esse mesmo método, isso só cria confusão (pois se elas são atributos de uma classe não seria necessário passá-las como parâmetro para métodos dessa mesma classe).

Ou então pare de declarar tudo como public e procure aprender a usar os modificadores de acesso do Java para refletirem corretamente suas intenções no código.

Se quiser deixar o código mais elegante, dê mais detalhes sobre o que deseja fazer e inclua mais código na pergunta, inclusive dos threads que chamam o tal método, que mostramos a você como ele pode ficar.

Piovezan
  • 18mil
  • 7
  • 53
  • 91