請比較一下這兩段功能大致相同的代碼。拋開語法的差別不論，你覺得哪種風格好？你愿意維護哪一段代碼？
????????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?);
????????????}
????????}?);
另外一種：
????????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;
我在另一個地方也抱怨過：在所有我了解的語言特性里面，沒有一種像Java內部類一樣讓我覺得反感——甚至到了惡心的地步。大多作B/S系統的Java程
序員可能不會有這樣的感覺，因為那個領域基本上很少會用到這個概念。可是在C/S，不管用Swing還是SWT，內部類都是繞不過去的一座大山。在閱讀Eclipse站點上許多代碼示例以后，我終于有了痛苦到——一點也不夸張——想要作嘔的地步。上面第一段代碼就是讓我感到窩心的代碼之一（僅僅是其中之一，還不是最丑陋的）。我想，Java
語言的發明者大概從來就沒寫過桌面程序；他根本也不打算為這個領域的程序員提供一種比較好的事件回調機制。
內部類有什么問題呢？首先，你愿意在去看代碼邏輯之前，先花上好幾分鐘去搞清楚“這個括號到底是和哪個配對”這種蠢問題嗎？你不妨回頭看看第一段的代碼，想想這段其實相當簡單的程序，是不是真的值得用這么多括號去考驗你的智力。
內部類是對封裝的嚴重破壞。它對外部類的任何私有變量都有完全的訪問權限——如果你突然發現某個變量的內容不對勁了，你不能僅僅在setXXX里面加個斷
點就指望能捕獲到錯誤；真正的元兇可能是內部類里面的哪一句呢。如果內部類都非常簡單，那倒也沒什么。可是當內部類用來實現事件的時候，你就沒法指望它一
直那么簡單了。
內部類是測試的盲區。TDD總是說，要測試，測試，所有包含邏輯的類都應當通過充分的測試。可是內部類怎么測試？只要想想就能知道，大多數內部類是根本沒法測試的，它和外部類實在是耦合的太緊密了。匿名內部類的問題更嚴重——它是絕對無法測試的。你怎么測試一個連名字都沒有的方法？
不管有意無意，內部類在（至少我看到的）實踐中事實上鼓勵了不好的編程風格。就是說，它違背了DRY（Don't Repeat
Yourself）的原則。比如，button.addSelectionListener后面幾乎總是跟著SelectionAdapter+括號+
widgetSelected原型再+一堆括號；Display.asyncExec后面總是要寫上new Runnble()，void
run（），括號，等等。千篇一律的東西，可是又不得不寫。而且，幾乎沒有什么好的辦法可以改進！因為語法的規則要求你必須這樣做。每天寫這些無聊的東
西，你的話會不會煩？哦，工具是有的。可是工具只負責生成代碼，以后的維護還是要你來做——不是么？