内部�c�让我厌恶Java

TreeNode — Thu, 15 Jun 2006 14:37:00 GMT

��h��较一下这两段功能大致相同的代码。抛开语法的差别不论，你觉得哪�U�风格好�Q�你愿意�l�护哪一�D�代码？

        b.addSelectionListener( new SelectionAdapter()
        {
            public void widgetSelected( SelectionEvent e )
            {
                Runnable longJob = new Runnable()
                {
                    boolean    done    = false;
                    int        id;

                    public void run()
                    {
                        Thread thread = new Thread( new Runnable()
                        {
                            public void run()
                            {
                                id = nextId[0]++;
                                display.syncExec( new Runnable()
                                {
                                    public void run()
                                    {
                                        if ( text.isDisposed() )
                                            return;
                                        text
                                                .append( "\nStart long running task "
                                                        + id );
                                    }
                                } );
                                for ( int i = 0; i < 100000; i++ )
                                {
                                    if ( display.isDisposed() )
                                        return;
                                    System.out
                                            .println( "do task that takes a long time in a separate thread "
                                                    + id );
                                }
                                if ( display.isDisposed() )
                                    return;
                                display.syncExec( new Runnable()
                                {
                                    public void run()
                                    {
                                        if ( text.isDisposed() )
                                            return;
                                        text
                                                .append( "\nCompleted long running task "
                                                        + id );
                                    }
                                } );
                                done = true;
                                display.wake();
                            }
                        } );
                        thread.start();
                        while ( !done && !shell.isDisposed() )
                        {
                            if ( !display.readAndDispatch() )
                                display.sleep();
                        }
                    }
                };
                BusyIndicator.showWhile( display, longJob );
            }
        } );

另外一�U�：

        delegate void NotifyStartDelegate( int threadId );
        delegate void NotifyFinishDelegate( int threadId );

        btnInvoke.Click += BtnInvokeClick;

        void BtnInvokeClick(object sender, System.EventArgs e)
        {
            text.Text = "invoke long running job";

            Cursor = Cursors.WaitCursor;
            Thread thread = new Thread( new ThreadStart(ThreadProc) );
            thread.Start();
        }

        private void ThreadProc()
        {
            int threadId = nextId ++;
            bool done = false;

            if ( IsDisposed )
                return;

            Invoke( new NotifyStartDelegate(notifyThreadStart), new object[] { threadId } );
            for ( int i=0; i<100000; i++ )
            {
                if ( IsDisposed )
                    return;
                Console.WriteLine( "do task that takes a long time in a separate thread " + threadId );
            }

            if ( IsDisposed )
                return;
            Invoke( new NotifyFinishDelegate(notifyThreadFinish), new object[] { threadId } );
            done = true;
        }

        private void notifyThreadStart( int threadId )
        {
            text.Text += "\r\nStart long task " + threadId;
            threadCount ++;
        }

        private void notifyThreadFinish( int threadId )
        {
            text.Text += "\r\nCompleted long running task " + threadId;
            threadCount --;
            if ( threadCount == 0 )
                Cursor = Cursors.Default;
        }

        private int nextId = 0;
        private int threadCount = 0;

我在另一个地方也抱怨过�Q�在所有我了解的语�a��Ҏ��里面，没有一�U�像Java内部�c�M��栯��我觉得反感——甚臛_��了恶心的地步。大多作B/S�pȝ��的Java�E? 序员可能不会有这��L��感觉�Q�因为那个领域基本上很少会用到这个概��c��可是在C/S�Q�不��用Swing�q�是SWT�Q�内部类都是�l�不�q�去的一座大山。在阅读Eclipse站点上许多代码示例以后，我终于有了痛苦到——一点也不夸张——想要作呕的地步。上面第一�D�代码就是让我感到窝心的代码之一�Q�仅仅是其中之一�Q�还不是最丑陋的）。我惻I��Java 语言的发明者大概从来就没写�q�桌面程序；他根本也不打��ؓ�q�个领域的程序员提供一�U�比较好的事件回调机制�?br />
内部�c�L��什么问题呢�Q�首先，你愿意在�ȝ��代码逻辑之前�Q�先�׃��好几分钟��L��清楚“这个括号到底是和哪个配对”这�U�蠢问题吗？你不妨回头看看第一�D늚�代码�Q�想惌��D�其实相当简单的�E�序�Q�是不是真的值得用这么多括号去考验你的智力�?br />
内部�c�L��对封装的严重破坏。它对外部类的�Q何私有变量都有完全的讉K��权限——如果你�H�然发现某个变量的内容不对劲了，你不能仅仅在setXXX里面加个�? 点就指望能捕获到错误�Q�真正的元凶可能是内部类里面的哪一句呢。如果内部类都非常简单，那倒也没什么。可是当内部�cȝ��来实��C��件的时候，你就没法指望它一直那么简单了�?br />
内部�c�L��试的盲区。TDD��L��_��要测试，��试�Q�所有包含逻辑的类都应当通过充分的测试。可是内部类怎么��试�Q�只要想惛_��能知道，大多数内部类是根本没法测试的�Q�它和外部类实在是耦合的太紧密了。匿名内部类的问题更严重——它是绝�Ҏ��法测试的。你怎么��试一个连名字都没有的�Ҏ��Q?br />
不管有意无意�Q�内部类在（臛_��我看到的�Q�实践中事实上鼓�׃��不好的编�E�风根{��就是说�Q�它�q�背了DRY�Q�Don't Repeat Yourself�Q�的原则。比如，button.addSelectionListener后面几乎��L��跟着SelectionAdapter+括号+ widgetSelected原型�?一堆括��P��Display.asyncExec后面��L��要写上new Runnble()�Q�void run�Q�）�Q�括��P��{�等。千��一律的东西�Q�可是又不得不写。而且�Q�几乎没有什么好的办法可以改�q�！因�ؓ语法的规则要求你必须�q�样做。每天写�q�些无聊的东西，你的话会不会烦？哦，工具是有的。可是工具只负责生成代码�Q�以后的�l�护�q�是要你来做——不是么�Q?br />

TreeNode 2006-06-15 22:37 发表评论

TreeNode — Sat, 03 Jun 2006 04:46:00 GMT

摘要: 本文试图透过SWT面向对象的封装，深入研究界面框架的构造中最复杂也可以说是最有趣的问题之一——即界面框架如何以面向对象的方式实现操作�pȝ��底层的消息机制�?nbsp; 阅读全文

TreeNode 2006-06-03 12:46 发表评论

www.亚洲在线,精品亚洲综合,国产区视频在线

内部�c�让我厌恶Java