Java:なぜこのメソッドには副作用があるのですか?
-
05-07-2019 - |
質問
特定の変数が final
とマークされていても、副作用を引き起こすメソッドがあります。どうしてこれなの?おそらく、 final
の機能について混乱しています。
@Test
public void testSubGraph() {
WeightedGraph<String, DefaultWeightedEdge> g = generateSimpleCaseGraph();
Graph<String, DefaultWeightedEdge> sub = ChooseRoot.subgraphInDirection(g, "alpha", "l");
assertEquals(g, generateSimpleCaseGraph()); //fails
}
public static <V, E extends DefaultEdge> Graph<V, E> subgraphInDirection(final Graph<V, E> g, final V start, final V sink) {
Graph<V, E> sub = removeEdges(g, start, sink);
return removeUnconnectedNodes(sub, start);
}
private static <Vertex, Edge extends DefaultEdge> Graph<Vertex, Edge> removeEdges(final Graph<Vertex, Edge> g, Vertex start, Vertex sink) {
final Set<Edge> outEdges = new HashSet<Edge>(g.edgesOf(start));
boolean removedEdge;
for (Edge e : outEdges) {
if (! (g.getEdgeTarget(e).equals(sink) || g.getEdgeSource(e).equals(sink))) {
removedEdge = g.removeEdge(e);
assert removedEdge;
}
}
return g;
}
private static <Vertex, Edge> Graph<Vertex, Edge> removeUnconnectedNodes(Graph<Vertex, Edge> g, Vertex start) {
ConnectivityInspector<Vertex, Edge> conn = new ConnectivityInspector<Vertex, Edge>((UndirectedGraph<Vertex, Edge>) g);
boolean removedVertex;
final Set<Vertex> nodes = new HashSet<Vertex>(g.vertexSet());
for (Vertex v : nodes) {
if (! conn.pathExists(start, v)) {
removedVertex = g.removeVertex(v);
assert removedVertex;
}
}
return g;
}
解決
final
修飾子は、参照を再割り当てできないことを意味するだけです。オブジェクトの状態が変更されるのを防ぎません。
編集:トムのみ:
public void doSomething1(Object arg)
{
arg = new Object(); // OK.
}
public void doSomething2(final Object arg)
{
arg = new Object(); // Compile error.
}
どちらの場合でも、状態を変更するメソッドを含め、 arg
が指すオブジェクトのメソッドを呼び出すことができます。
他のヒント
ダンはファイナルで正解です。あとは、JavaにはないC ++のconstに似ています。これをシミュレートできます:
public class Foo
{
protected int x;
public Foo(final int val)
{
x = val;
}
public int getX()
{
return (x);
}
}
public class MutableFoo
extends Foo
{
public MutableFoo(final int val)
{
super(val);
}
public void setX(final int val)
{
x = val;
}
}
それから:
void bar(final Foo foo)
{
foo.setX(5); // will not compile
}
void bar(final MutableFoo foo)
{
foo.setX(5); // will compile
}
きれいではありませんが、動作します。コツは、親クラス(Foo)のメソッドがインスタンス変数を変更しないようにすることです-状態を変更できるメソッドを持つことができるのはMutableFooだけです。
もちろん、できる限り最善の方法は、不変クラスを記述し(すべての変数を最終的にする)、副作用のあるインスタンス/クラス変数のメソッドを呼び出さないことです。 p>
所属していません StackOverflow