Vamos ver a sua primeira classe:
class ligacao {
// ...
public licacao(boolean b){
// ...
Aqui temos um erro de compilação, pois o nome do construtor deveria ser ligacao
, e não licacao
. Além disso, a convenção de nomes dita que nomes de classes iniciam com letras maiúsculas, e desobedecer essa conveção não é uma boa ideia.
Um pouco mais abaixo:
filtrar.removeAll(obls);
Não há nenhuma variável obls
. Acho que você quis dizer obs
.
Também neste método:
if(b == true){
Por favor, NUNCA compare uma expressão ou variável booleana com == true
ou == false
. Isso é uma má prática de programação. Ao invés disso, use simplesmente assim:
if (b) {
E se quisesse comparar com false
, bastaria usar isso:
if (!b) {
O método também tem como retorno FilteredList
bruto sem o tipo genérico ao invés de FilteredList<Person>
. O compilador deve estar dando algumas warnings dos tipos rawtypes e unchecked por causa disso.
Olhando aonde você usa a classe ligacao
de fora dela, temos estes dois lugares:
new ligacao(b);
ligacao.get(obs(), obs2());
No primeiro caso, a instância de ligacao
recém-criada não será utilizada para nada e será perdida para ser devorada pelo coletor de lixo. O único efeito disso será alterar o valor da variável estática b
(que não vejo sentido ser protected
ao invés de private
). No segundo caso, o método get
é estático. Como conclusão, instâncias da classe ligacao
são inúteis, e portanto não tem sentido de serem instanciadas. O construtor que você usou nada mais é do que um setter disfarçado, e usar construtores para isso é algo horroroso. A sugestão neste caso seria eliminar o construtor e usar um setter estático:
class Ligacao {
private static boolean b;
public static void setFlag(boolean novoValor) {
b = novoValor;
}
// Na segunda classe:
Ligacao.setFlag(b);
Vamos aproveitar e dar uma olhada no código que veio da segunda classe, e tentar rastrear qual é o valor do b
dela:
carne.setOnAction((Carne) -> {
try {
boolean b = false;
System.out.println(false); // Escrevia b, mas b é sabidamente false.
if(carne.getText().endsWith("Carnê")){
carne.setText("Tabela");
System.out.println(false); // Escrevia b, mas b é sabidamente false.
b = true;
System.out.println(true); // Escrevia b, mas b é sabidamente true.
}else{
carne.setText("Carnê");
System.out.println(false); // Escrevia b, mas b é sabidamente false.
b = false; // Não faz nada, pois b já é false.
System.out.println(false); // Escrevia b, mas b é sabidamente false.
}
System.out.println(b);
Ligacao.setFlag(b);
System.out.println(b + "\n");
} catch (Exception e) {
e.printStackTrace();
}
});
Vamos simplificar então, tirando os System.out.println
desnecessários e as instruções desnecessárias:
carne.setOnAction((Carne) -> {
try {
boolean b = false;
if (carne.getText().endsWith("Carnê")) {
carne.setText("Tabela");
b = true;
} else {
carne.setText("Carnê");
}
Ligacao.setFlag(b);
System.out.println(b + "\n");
} catch (Exception e) {
e.printStackTrace();
}
});
Podemos simplificar um pouco mais se notarmos que b
só será verdadeiro se a condição do if
for verdadeira e com isso podemos usá-la diretamente como o valor de b
:
carne.setOnAction((Carne) -> {
try {
boolean b = carne.getText().endsWith("Carnê");
carne.setText(b ? "Tabela" : "Carnê");
Ligacao.setFlag(b);
System.out.println(b + "\n");
} catch (Exception e) {
e.printStackTrace();
}
});
Aqui, ainda cabe questionar o nome do parâmetro do lambda: Carne
. Isso não é um nome apropriado, pois isso é uma variável e portanto deveria começar com letra minúscula. Além disso, ter duas variáveis cujo nome difere apenas pelas maiúsculas e minúsculas, a saber carne
e Carne
é uma má prática de programação pois deixa o código bem confuso. Sugiro renomear o Carne
para alguma outra coisa.
Agora, voltando a sua primeira classe, percebe-se que você esqueceu o private
na declaração do campo estático filtrar
, mas vamos dar uma olhada no que o método get
faz (já aplicadas as correções que sugeri antes):
private static FilteredList<Person> filtrar;
public static FilteredList<Person> get(ObservableList<Person> obs, ObservableList<Person> obs2) {
filtrar = new FilteredList<>(obs, p -> true);
if (b) {
filtrar.removeAll(obs);
filtrar = new FilteredList<>(obs2, p -> true);
}else{
filtrar.removeAll(obs2);
filtrar = new FilteredList<>(obs, p -> true);
}
return filtrar;
}
Se b
for verdadeiro eis o que ocorre:
- Cria uma lista com todos os elementos de
obs
e o atribui à variável estática filtra
, portanto perdendo a referência anterior da lista não importando o que ela era.
- Remove todos os elementos de
obs
que estão na lista (e portanto todos os elementos), o que resulta em uma lista vazia.
- Cria uma outra lista com todos os elementos de
obs2
. A lista anterior é perdida para o coletor de lixo, e portanto o trabalho realizado nela nos passos 1 e 2 foi desperdiçado.
- Retorna a lista com os elementos que estavam em
obs2
.
Se b
for falso:
- Cria uma lista com todos os elementos de
obs
e o atribui à variável estática filtra
, portanto perdendo a referência anterior da lista não importando o que ela era.
- Remove todos os elementos de
obs2
que estão na lista, o que resulta em uma lista com os elementos de obs
que não estão em obs2
. Se não houver nenhum elemento que esteja em ambas as listas, isso não vai fazer nada.
- Cria uma outra lista com todos os elementos de
obs
. A lista anterior é perdida para o coletor de lixo, e portanto o trabalho realizado nela nos passos 1 e 2 foi desperdiçado.
- Retorna a lista com os elementos que estavam em
obs
.
Observe que em ambos os casos, a lista que fica guardada ao final é irrelevante, pois da forma como o método get
foi codificado, qualquer tentativa de acessar o conteúdo desta lista necessariamente a fará ser perdida.
Tudo isso ficaria muito mais simples assim:
public static FilteredList<Person> get(ObservableList<Person> obs, ObservableList<Person> obs2) {
return new FilteredList<>(b ? obs : obs2, p -> true);
}
E é assim que o seu código fica no final das contas:
class Ligacao {
private static boolean b;
public static void setFlag(boolean novoValor) {
b = novoValor;
}
public static FilteredList<Person> get(ObservableList<Person> obs, ObservableList<Person> obs2) {
return new FilteredList<>(b ? obs : obs2, p -> true);
}
}
carne.setOnAction(x -> {
try {
boolean b = carne.getText().endsWith("Carnê");
carne.setText(b ? "Tabela" : "Carnê");
Ligacao.setFlag(b);
System.out.println(b + "\n");
} catch (Exception e) {
e.printStackTrace();
}
});
FilteredList<Person> filtrar = Ligacao.get(obs(), obs2());
Por fim, algumas pessoas podem achar que acessar o campo estático usando this
causa um erro de compilação. Isso não é verdade, pois é permitido acessar um campo estático usando uma variável ao invés do nome da classe e no caso, a variável é o this
). Entretanto, isso é uma péssima prática de programação e não deveria ser feito nunca, mas a linguagem permite.
E também acessar uma variável estática de dentro do construtor é perfeitamente válido na linguagem Java. Na maioria das vezes (nem sempre, existem algumas exceções), fazer isso não é boa prática de programação e tende a deixar o código ruim e propenso a ocorrência de bugs, mas a linguagem Java permite.
b
comopublic static boolean b;
b
não estava declarado como estático e era acedido no método estáticoget
.