mirror of
https://github.com/Card-Forge/forge.git
synced 2025-11-17 11:18:01 +00:00
Simple classes do not need getters
This commit is contained in:
@@ -1002,16 +1002,13 @@ final class BridgeToInstance<T, U> extends Predicate<U> {
|
|||||||
*
|
*
|
||||||
* @param <T>
|
* @param <T>
|
||||||
*/
|
*/
|
||||||
|
/* PERFORMANCE CRITICAL - DO NOT GENERATE GETTERS HERE*/
|
||||||
class Node<T> extends Predicate<T> {
|
class Node<T> extends Predicate<T> {
|
||||||
|
/* PERFORMANCE CRITICAL - DO NOT GENERATE GETTERS HERE*/
|
||||||
private final PredicatesOp operator;
|
private final PredicatesOp operator;
|
||||||
/**
|
protected final Predicate<T> filter1;
|
||||||
*
|
protected final Predicate<T> filter2;
|
||||||
*/
|
/* PERFORMANCE CRITICAL - DO NOT GENERATE GETTERS HERE*/
|
||||||
private final Predicate<T> filter1;
|
|
||||||
/**
|
|
||||||
*
|
|
||||||
*/
|
|
||||||
private final Predicate<T> filter2;
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
*
|
*
|
||||||
@@ -1031,39 +1028,27 @@ class Node<T> extends Predicate<T> {
|
|||||||
public boolean isTrue(final T card) {
|
public boolean isTrue(final T card) {
|
||||||
switch (this.operator) {
|
switch (this.operator) {
|
||||||
case AND:
|
case AND:
|
||||||
return this.getFilter1().isTrue(card) && this.getFilter2().isTrue(card);
|
return this.filter1.isTrue(card) && this.filter2.isTrue(card);
|
||||||
case GT:
|
case GT:
|
||||||
return this.getFilter1().isTrue(card) && !this.getFilter2().isTrue(card);
|
return this.filter1.isTrue(card) && !this.filter2.isTrue(card);
|
||||||
case LT:
|
case LT:
|
||||||
return !this.getFilter1().isTrue(card) && this.getFilter2().isTrue(card);
|
return !this.filter1.isTrue(card) && this.filter2.isTrue(card);
|
||||||
case NAND:
|
case NAND:
|
||||||
return !(this.getFilter1().isTrue(card) && this.getFilter2().isTrue(card));
|
return !(this.filter1.isTrue(card) && this.filter2.isTrue(card));
|
||||||
case OR:
|
case OR:
|
||||||
return this.getFilter1().isTrue(card) || this.getFilter2().isTrue(card);
|
return this.filter1.isTrue(card) || this.filter2.isTrue(card);
|
||||||
case NOR:
|
case NOR:
|
||||||
return !(this.getFilter1().isTrue(card) || this.getFilter2().isTrue(card));
|
return !(this.filter1.isTrue(card) || this.filter2.isTrue(card));
|
||||||
case XOR:
|
case XOR:
|
||||||
return this.getFilter1().isTrue(card) ^ this.getFilter2().isTrue(card);
|
return this.filter1.isTrue(card) ^ this.filter2.isTrue(card);
|
||||||
case EQ:
|
case EQ:
|
||||||
return this.getFilter1().isTrue(card) == this.getFilter2().isTrue(card);
|
return this.filter1.isTrue(card) == this.filter2.isTrue(card);
|
||||||
default:
|
default:
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* @return the filter1
|
|
||||||
*/
|
|
||||||
public Predicate<T> getFilter1() {
|
|
||||||
return filter1;
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @return the filter2
|
|
||||||
*/
|
|
||||||
public Predicate<T> getFilter2() {
|
|
||||||
return filter2;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -1086,7 +1071,7 @@ final class NodeOr<T> extends Node<T> {
|
|||||||
|
|
||||||
@Override
|
@Override
|
||||||
public boolean isTrue(final T card) {
|
public boolean isTrue(final T card) {
|
||||||
return this.getFilter1().isTrue(card) || this.getFilter2().isTrue(card);
|
return this.filter1.isTrue(card) || this.filter2.isTrue(card);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -1110,7 +1095,7 @@ final class NodeAnd<T> extends Node<T> {
|
|||||||
|
|
||||||
@Override
|
@Override
|
||||||
public boolean isTrue(final T card) {
|
public boolean isTrue(final T card) {
|
||||||
return this.getFilter1().isTrue(card) && this.getFilter2().isTrue(card);
|
return this.filter1.isTrue(card) && this.filter2.isTrue(card);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user