半径を格納するメンバー 'r' を持つオブジェクトの Circle クラス タイプがあります。これらのオブジェクトを頻繁に使用するため、他のクラスが getR() メソッドを使用する必要があり、コードが読みにくくなるため、radius をプライベートにしたくありません。たとえば、次のようなコードを読む方がはるかに簡単です: double alpha = b1.r + b2.r -2*b1.r*b2.r; これを読むより: double alpha = b1.getR() + b2.getR() - 2*b1.getR()*b2.getR();
しかし、これは半径であり、その平方も頻繁に使用され、一度計算したいので、半径のセッターが必要なため、r とその平方の両方を同時に更新できます。したがって、setR() の使用を強制するために r を非公開にしたいのですが、読み取り用に参照できるように公開したいのです。Java でこれを行う方法を知っている人はいますか?
Java には、読み取り専用プロパティの概念がありません。
r を読み取り可能にすると、誰かがそれをランダムに変更できるようになります。その上、それは悪い習慣です。ただしないでください。大まかな例は次のようになります: これが奇妙な非ユークリッド空間で機能する必要がある場合はどうなるでしょうか? とにかく、混乱を解消して修正する必要があります。
現実的なオプション:
標準から逸脱して、ゲッターに r() という名前を付けることができます。ほぼ同じくらい短い。
または、そのような数学を circle クラスにプッシュして、そのメンバーを直接参照できるようにすることもできます。短い名前を使用すると、カプセル化のより良い部分が得られます。
class Circle() {
static double alpha(Circle c1, Circle c2) {
return c1.r + c2.r -2*c1.r*c2.r
}
...
}
a)。コードにカプセル化を提供するためにゲッターとセッターの使用をスキップしないでください。
b)。可読性が問題の場合は、 getR() メソッドを使用して半径を取得し、ローカル変数に保存することをお勧めします。- int firstCircleRadius = first.getR() など、他の円についても同様であり、ローカル変数を使用して数式を簡単に読み取れるようにします。
c)。r をプロパティ名として使用しないでください。代わりに radius を使用してください。これにより、それだけで読みやすさが向上します。
個人的には、ゲッターとセッターを使用します。ただし、実際の半径をプライベート メンバーに格納してsetR()から、メソッドを使用してダミーの radius を更新することができますr。rこれは、手動で変更できるという問題を解決しません。
private int radius;
public int r;
public void setR(int r) {
radius = r;
square(); //do you squaring and other math
this.r = r;
}
私は最近、この問題に悩まされていて、Java コミュニティから多くの反対票と怒りを受けるリスクがあるので、Java アクセスに依存しないことをお勧めします。クラス変数やメソッド (またはオブジェクト指向プログラミング言語でカプセル化するために「プライベート」と考えていたもの. すべてにパブリックを使用し、夜はよく眠ってください. ここに説明があります. 次のクラスを考えてください:
package hiddenscope;
public class Privacy {
private int privateInt = 12;
private double oneBy2pi = 0.5 / Math.PI;
private String myClassName;
public Privacy () {
privateInt = 12;
myClassName = this.getClass().getName();
print ("Constructor");
}
// Given the circumference of a circle, get its radius
public double c2r (double c) {
return c * oneBy2pi;
}
// Do not let any other class call this method
private void print (String caller) {
System.out.println ("["+caller+"] Current value of privateInt in class " +
myClassName + " is " + privateInt);
}
}
これは、上記のクラスのプライベート変数 privateInt を変更し、パッケージの外側からでもプライベートな「印刷」メソッドを呼び出すクラスです (保護されていますか?):
package userpackage;
import hiddenscope.Privacy;
import java.lang.reflect.*;
public class EditPrivate {
private int i2set;
private String fieldNameToModify, myClassName;
private Object p;
private Method printer;
public EditPrivate (Object object, String fName, int value) {
fieldNameToModify = fName;
i2set = value;
p = object;
myClassName = this.getClass().getName();
Method[] methods = p.getClass().getDeclaredMethods();
for (Method m : methods) {
if (m.getName().equals("print")) {
printer = m;
printer.setAccessible(true);
break;
}}
if (printer == null)
return;
try {
printer.invoke (p, myClassName);
} catch (IllegalAccessException ex1) { ex1.printStackTrace();
} catch (InvocationTargetException ex2) { ex2.printStackTrace();
}}
public void invadePrivacy () throws Exception {
Field[] fields = p.getClass().getDeclaredFields();
for (Field field : fields) {
String fieldName = field.getName();
if (fieldName.equals(fieldNameToModify)) {
field.setAccessible(true);
if ("int".equals(field.getType().toString())) {
try {
field.setInt(p, i2set);
if (printer != null)
printer.invoke (p, myClassName);
return;
} catch (IllegalAccessException ex) {
ex.printStackTrace();
}}}}
throw new Exception ("No such int field: " + fieldNameToModify);
}
public static void main(String[] args) {
try {
Privacy p = new Privacy();
new EditPrivate(p, "privateInt", 15).invadePrivacy();
} catch (Exception ex) {
ex.printStackTrace();
}}}
この怪我に侮辱を加えるために、これは、複数のスレッドがクラスの内部変数とプライベート変数を台無しにする場合に起こることです (oneBy2pi はプライベートであるはずです!)
package userpackage;
import java.lang.reflect.Field;
import java.util.concurrent.*;
import java.util.*;
import hiddenscope.*;
public class ThreadedPi implements Runnable {
Privacy p = new Privacy();
private int nSample = 2000, bins[] = new int[4]; // 94814117 999065 0003379, 2028859
private Field c2dField;
private boolean mutate;
public ThreadedPi () {
Field[] fields = p.getClass().getDeclaredFields();
for (Field field : fields) {
String fieldName = field.getName();
if (fieldName.equals("oneBy2pi")) {
field.setAccessible(true);
c2dField = field;
}}}
synchronized private boolean setMutationOfField (boolean master, boolean value) {
if (master)
mutate = value;
return mutate;
}
public void run () {
Random rng = new Random();
for ( ; ; ) {
if (setMutationOfField (false, true)) {
int nDigit = 2 + rng.nextInt(4);
double doublePi = 4.0 * Math.atan(1.0),
decimal = Math.pow(10, nDigit),
apprxPi = Math.round(decimal * doublePi) / decimal;
try {
c2dField.setDouble (p, 0.5/apprxPi);
} catch (IllegalAccessException ex) {
ex.printStackTrace();
}} else {
return;
}}}
public void execute () {
setMutationOfField (true, true);
new Thread (this).start();
Executed[] class2x = new Executed[nSample];
ExecutorService executor = Executors.newCachedThreadPool();
for (int i = 0 ; i < 4 ; i++)
bins[i] = 0;
for (int i = 0 ; i < nSample ; i++) {
class2x[i] = new Executed();
executor.execute(class2x[i]);
double d = class2x[i].getDiameter(-1);
if (d < 399.99) bins[0]++;
else if (d < 400) bins[1]++;
else if (d < 400.01) bins[2]++;
else bins[3]++;
//System.out.println ("d: " + d);
}
setMutationOfField (true, false);
for (int i = 0 ; i < 4 ; i++)
System.out.print("\t\t[" + (i+1) + "] " + bins[i]);
System.out.println ();
}
public static void main(String[] args) {
ThreadedPi tp = new ThreadedPi();
for (int k = 0 ; k < 5 ; k++)
tp.execute();
}
private class Executed implements Runnable {
private double d=-1, c = 2513.274123;
synchronized double getDiameter (double setter) {
if (setter < 0) {
while (d < 0) {
try {
wait();
} catch (InterruptedException ex) {
}}} else {
d = setter;
notify();
}
return d;
}
public void run () {
getDiameter (p.c2d(c));
}}}
上記のすべてのクラスは完全にコンパイルおよび実行されます。これが、Java で「プライベート」が意味を持たない理由を説明しています。守れ、スコッティ！