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 @Maniero falou na resposta dele, que responde perfeitamente a pergunta. Vai funcionar porque o método não está fazendo nenhuma alteração em estado compartilhado, somente no estado dos parâmetros passados a ele, que no caso são independentes um do outro.
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.
synchronized
parece desnecessário. Porque precisaria criar outro método para fazer a mesma coisa? Crie cada thread chamando o mesmo método passando cada lista em uma thread.synchronized
está ali supondo que eu implementasse o caso de duas threads usando o mesmo método daquele objeto cada uma passando uma lista. Com a solução que você falou aí que vai dar problema, as threads mudarão a referência à variávellist
do método toda hora, e assim vou obter um resultado completamente indesejável, porque eu chamo o mesmo método do mesmo objeto toda vez.