1

我正在构建一个 Android 应用程序，并设置了一个基本 DBObject 类来简化表交互。DBObject 构造函数设置一个游标并将其传递给虚拟方法 loadRecord，子类将覆盖该方法以加载其自己的属性。根据 loadRecord 末尾的 Log.i 调用，属性都是正确的，但是一旦调用方法取回对象，所有数据成员都具有类声明中定义的默认值，而不是加载的值（字符串为 null ，浮点数为 0.00 等）。

public abstract class DBObject {
....

    public DBObject(DBHelper dbh, String table_name, int _id){
        // loads an existing DBObject from the database by id
        this.dbh = dbh;
        this.TABLE_NAME = table_name;
        this.id = _id;
        setupDbr();
        String sql = String.format("select * from %s where _id=%d", TABLE_NAME, id);
        Cursor cur = dbr.rawQuery(sql, null);
        if (cur.getCount() == 0){
            throw new RowDoesNotExistException(id);
        } else if (cur.getCount() > 1){
            throw new RowDoesNotExistException("Record is not unique.", id);
        }
        cur.moveToFirst();
        loadRecord(cur);
    }
....
abstract void loadRecord(Cursor cur);

此构造函数调用重写的 loadRecord，它在子类中如下所示：

public class Expense extends DBObject{

    public String date = null;
    public String supplier = null;
    public String description = null;
    public float amount = 0.00f;
    public boolean receipt = false;

    public Expense(DBHelper dbh, String _id){
        super(dbh, DBHelper.EXPENSE_TABLE, _id);
...
    @Override
    void loadRecord(Cursor cur){
        id = cur.getInt(0);
        date = cur.getString(1);
        supplier = cur.getString(2);
        description = cur.getString(3);
        amount = cur.getFloat(4);
        receipt = (cur.getInt(5) == 1);
        Log.i("net.bradmont.reimburseit", String.format("%d, %s, %s, %s", id, date, supplier, description));
    }

在 DBObject 构造函数中设置的成员保持原样。我对此完全不知所措......

提前致谢！

4

1 回答 1

2

问题是你有实例变量初始化器：

public String date = null;
public String supplier = null;
public String description = null;
public float amount = 0.00f;
public boolean receipt = false;

初始化器将在超类构造函数执行之后、子类构造函数的主体执行之前执行。这是在构造函数中调用重写方法的问题之一——这通常是一个非常糟糕的主意。

您可以通过不使用实例变量初始化程序来解决此特定问题：

public String date;
public String supplier;
public String description;
public float amount;
public boolean receipt;

现在在超类构造函数完成和子类构造函数执行之间不会有额外的赋值。然而，这并不是一个很好的解决方案——最好重新设计它以避免在超类构造函数中调用被覆盖的方法。

我也会尝试摆脱公共字段......将它们设为私有，以避免暴露你的类的实现细节。

有关如何执行构造函数的详细信息，请参阅JLS 的第 12.5 节。

于 2012-10-08T22:32:43.537 回答