对象中的自引用是不良代码设计的证据

Question

我在我正在查看的一些代码中遇到了自我引用。

示例

TestObject selfReference = this;

是否存在需要在对象中进行自引用的好情况？这是糟糕的编码设计或风格的标志吗？

编辑：

这是一个示例，如果我使用 this ，它会抛出错误，但是当使用 selfReference 时，它​​会编译。

public class IFrame extends InternalFrame
{
    public IFrame()
    {
         addComponentListener(new java.awt.event.ComponentAdapter()
        {
            public void componentResized(java.awt.event.ComponentEvent evt) 
            {
                Window.setCurrComponent(this); //compile error
            }
            public void componentMoved(ComponentEvent evt)
            {
                Window.setCurrComponent(selfReference); //compiles correctly
            }
        });
    }
}

public class InternalFrame extends JInternalFrame
{
    protected InternalFrame selfReference = this;
}

public class Window
{
    InternalFrame currFrame;

    public static void setCurrComponent(InternalFrame iFrame)
    {
        currFrame = iFrame
    }
}

Answer 1

是的，在某些情况下，隐式自我引用可能是完全自然的。例如，考虑一个当前仅包含单个元素的循环链表。

但是，拥有一个名为 selfReference 的成员变量根本没有任何意义。

Answer 2

我会对此进行尝试。我猜测该代码的作者不知道当您想从嵌套类访问“外部 this”时可以编写 Classname.this 。

也就是说，他创建了一个这样的结构：

class Executor {
    public void execute(Example example) {

    }
}

public class Example {

    Example selfReference = this;

    class Nested {
        public void method() {
            //Oh, oh, can't do this: 
            //new Executor().execute(this);
            //It gives:
            //The method execute(Example) in the type Executor is not applicable for the arguments (Example.Nested)

            //How the hell do I invoke the executor method from here?
            //lets do something really odd.
            new Executor().execute(selfReference);

            //This is what he should have done
            new Executor().execute(Example.this);
        }
    }

}

Answer 3

没有必要。

自引用的一个完全有效的情况是一个需要某些处理程序的对象。如果对象本身实现了该处理程序接口，则对处理程序的引用就是对同一对象的引用。完全可以，恕我直言。

Answer 4

我想说，自引用是 Java 中糟糕代码的标志，特别是如果它的名称类似于“selfReference”，因为 Java 已经有一个名为“this”的标准自引用。在支持闭包的语言中，这是一个不同的故事，因为非 this 自引用可以将“this”对象保留在闭包的范围内。

Answer 5

如果没有更多上下文，代码就不会做任何有用的事情，并且可能会造成混乱。我认为它不符合“设计”或“风格”的资格。

然而，更多的背景可能会说明为什么要这样做。

Answer 6

我想说这是糟糕代码的标志：

• 每个人都知道 this 的含义，所以我看不到需要名为 selfReference 的引用，
• 除非您打算这样做稍后将该引用更改为其他内容 - 例如

TestObject selfReference = this;
//调用一些函数
selfReference = 一些其他对象； //其中 someOtherObject 也是 TestObject 的实例
//调用 someOtherObjects 上的一些函数

• ，但是，如果是这种情况，并且引用在某个时刻被重新分配给其他对象，那么它不应该被称为 selfReference - 因为它最终可能不会引用自我/这个！

Answer 7

使用更新的代码示例，您似乎在从匿名内部类引用外部对象时遇到了问题。语法是：

OuterClass.this

在您的情况下：

public void componentResized(java.awt.event.ComponentEvent evt) 
{
    Window.setCurrComponent(IFrame.this); //no more compile error
}

如果您仅使用 this ，那么您将引用 new ComponentAdapter()。

感谢您提供额外的背景信息。

Answer 8

Window.setCurrComponent(this); //编译错误
上面语句中的“this”不是 InternalFrame 对象，而是 ComponentAdapter 对象，就像您在匿名内部类中一样。

Window.setCurrComponent(selfReference); //编译正确
如果您想引用 IFrame 类的对象，上面的行是正确的语句。

Answer 9

如果您正在编写一个解析器并且具有包含相同类型的子元素的数据结构，那么您将需要具有同一类的成员变量。

Answer 10

很多时候您想要使用this。您应该在需要引用它所属的对象的情况下使用。不过，我认为没有任何理由从 this 创建对象，一个简单的 this.attribute 就足够了。