【沈陽吉大計算機培訓(xùn)網(wǎng)-java課程】Java程序中代碼常見的十種錯誤

導(dǎo)讀：每一個程序員在編寫代碼的過程中都免不了出現(xiàn)錯誤或是小的失誤，這些小的錯誤和失誤往往使得程序員還得返工。那么，如何才能盡量避免這些錯誤的發(fā)生呢？筆者總結(jié)只有在日常的編寫代碼中總結(jié)出經(jīng)驗，在這篇文章中，筆者列出了10個Java編程中常見的錯誤，你可以把這些錯誤添加到你的代碼審查的檢查列表中，這樣在經(jīng)過代碼審查后，你可以確信你的代碼中不再存在這類錯誤了。

一、常見錯誤1：多次拷貝字符串

測試所不能發(fā)現(xiàn)的一個錯誤是生成不可變(immutable)對象的多份拷貝。不可變對象是不可改變的，因此不需要拷貝它。最常用的不可變對象是String。

如果你必須改變一個String對象的內(nèi)容，你應(yīng)該使用StringBuffer。下面的代碼會正常工作：

Strings=newString("Texthere");

但是，這段代碼性能差，而且沒有必要這么復(fù)雜。你還可以用以下的方式來重寫上面的代碼：

Stringtemp="Texthere";Strings=newString(temp);

但是這段代碼包含額外的String，并非完全必要。更好的代碼為：

Strings="Texthere";二、常見錯誤2：沒有克隆(clone)返回的對象

封裝(encapsulation)是面向?qū)ο缶幊痰闹匾拍?。不幸的是，Java為不小心打破封裝提供了方便Java允許返回私有數(shù)據(jù)的引用(reference)。下面的代碼揭示了這一點：

importjava.awt.Dimension;

/***//***Exampleclass.Thexandyvaluesshouldnever*benegative.*/

publicclassExample…{

privateDimensiond=newDimension(0,0);

publicExample()…{}

/***//***Setborder="1"Heightandwidth.Bothborder="1"Heightandwidthmustbenonnegative*oranexceptionisthrown.*/

publicsynchronizedvoidsetValues(intborder="1"Height,intwidth)throwsIllegalArgumentException…{

if(border="1"Height<0||width<0)

thrownewIllegalArgumentException();

d.border="1"height=border="1"Height;

d.width=width;

}

publicsynchronizedDimensiongetValues()…{

//Ooops!Breaksencapsulation

returnd;

}}

Example類保證了它所存儲的border="1"Height和width值永遠非負數(shù)，試圖使用setValues()方法來設(shè)置負值會觸發(fā)異常。不幸的是，由于getValues()返回d的引用，而不是d的拷貝，你可以編寫如下的破壞性代碼：

Exampleex=newExample();

Dimensiond=ex.getValues();

d.border="1"height=-5;d.width=-10;

現(xiàn)在，Example對象擁有負值了!如果getValues()的調(diào)用者永遠也不設(shè)置返回的Dimension對象的width和border="1"Height值，那么僅憑測試是不可能檢測到這類的錯誤。

不幸的是，隨著時間的推移，客戶代碼可能會改變返回的Dimension對象的值，這個時候，追尋錯誤的根源是件枯燥且費時的事情，尤其是在多線程環(huán)境中。

更好的方式是讓getValues()返回拷貝：

publicsynchronizedDimensiongetValues()…{

returnnewDimension(d.x,d.y);}

現(xiàn)在，Example對象的內(nèi)部狀態(tài)就安全了。調(diào)用者可以根據(jù)需要改變它所得到的拷貝的狀態(tài)，但是要修改Example對象的內(nèi)部狀態(tài)，必須通過setValues()才可以。三、常見錯誤3：不必要的克隆

我們現(xiàn)在知道了get方法應(yīng)該返回內(nèi)部數(shù)據(jù)對象的拷貝，而不是引用。但是，事情沒有絕對：/***//***Exampleclass.Thevalueshouldnever*benegative.*/

publicclassExample…{

privateIntegeri=newInteger(0);

publicExample()…{}

/***//***Setx.xmustbenonnegative*oranexceptionwillbethrown*/

publicsynchronizedvoidsetValues(intx)throwsIllegalArgumentException…{

if(x<0)

thrownewIllegalArgumentException();

i=newInteger(x);

}

publicsynchronizedIntegergetValue()…{

//Wecan"tcloneIntegerssowemakeacopythisway.

returnnewInteger(i.intValue());

}}

這段代碼是安全的，但是就象在錯誤1#那樣，又作了多余的工作。Integer對象，就象String對象那樣，一旦被創(chuàng)建就是不可變的。因此，返回內(nèi)部Integer對象，而不是它的拷貝，也是安全的。

方法getValue()應(yīng)該被寫為：

publicsynchronizedIntegergetValue()…{

//"i"isimmutable,soitissafetoreturnitinsteadofacopy.

returni;}

Java程序比C++程序包含更多的不可變對象。JDK所提供的若干不可變類包括：

·Boolean

·Byte

·Character

·Class

·Double

·Float

·Integer

·Long

·Short

·String·大部分的Exception的子類四、常見錯誤4：自編代碼來拷貝數(shù)組

Java允許你克隆數(shù)組，但是開發(fā)者通常會錯誤地編寫如下的代碼，問題在于如下的循環(huán)用三行做的事情，如果采用Object的clone方法用一行就可以完成：

publicclassExample…{

privateint[]copy;

/***//***Saveacopyof"data"."data"cannotbenull.*/

publicvoidsaveCopy(int[]data)…{

copy=newint[data.length];

for(inti=0;i

copy[i]=data[i];

}}

這段代碼是正確的，但卻不必要地復(fù)雜。saveCopy()的一個更好的實現(xiàn)是：

voidsaveCopy(int[]data)…{

try…{

copy=(int[])data.clone();

}catch(CloneNotSupportedExceptione)…{

//Can"tgethere.

}}

如果你經(jīng)?？寺?shù)組，編寫如下的一個工具方法會是個好主意：

staticint[]cloneArray(int[]data)…{

try…{

return(int[])data.clone();

}catch(CloneNotSupportedExceptione)…{

//Can"tgethere.

}}

這樣的話，我們的saveCopy看起來就更簡潔了：

voidsaveCopy(int[]data)…{

copy=cloneArray(data);}五、常見錯誤5：拷貝錯誤的數(shù)據(jù)

有時候程序員知道必須返回一個拷貝，但是卻不小心拷貝了錯誤的數(shù)據(jù)。由于僅僅做了部分的數(shù)據(jù)拷貝工作，下面的代碼與程序員的意圖有偏差：

importjava.awt.Dimension;

/***//***Exampleclass.Theborder="1"Heightandwidthvaluesshouldnever*be

negative.*/

publicclassExample…{

staticfinalpublicintTOTAL_VALUES=10;

privateDimension[]d=newDimension[TOTAL_VALUES];

publicExample()…{}

/***//***Setborder="1"Heightandwidth.Bothborder="1"Heightandwidthmustbenonnegative*oranexceptionwillbethrown.*/

publicsynchronizedvoidsetValues(intindex,intborder="1"Height,intwidth)throwsIllegalArgumentException…{

if(border="1"Height<0||width<0)

thrownewIllegalArgumentException();

if(d[index]==null)

d[index]=newDimension();

d[index].border="1"height=border="1"Height;

d[index].width=width;

}

publicsynchronizedDimension[]getValues()

throwsCloneNotSupportedException…{

return(Dimension[])d.clone();

}}

這兒的問題在于getValues()方法僅僅克隆了數(shù)組，而沒有克隆數(shù)組中包含的Dimension對象，因此，雖然調(diào)用者無法改變內(nèi)部的數(shù)組使其元素指向不同的Dimension對象，但是調(diào)用者卻可以改變內(nèi)部的數(shù)組元素(也就是Dimension對象)的內(nèi)容。方法getValues()的更好版本為：

publicsynchronizedDimension[]getValues()throwsCloneNotSupportedException…{

Dimension[]copy=(Dimension[])d.clone();

for(inti=0;i

//NOTE:Dimensionisn"tcloneable.

if(d!=null)

copy[i]=newDimension(d[i].border="1"Height,d[i].width);

}

returncopy;}

在克隆原子類型數(shù)據(jù)的多維數(shù)組的時候，也會犯類似的錯誤。原子類型包括int,float等。簡單的克隆int型的一維數(shù)組是正確的，如下所示：publicvoidstore(int[]data)throwsCloneNotSupportedException…{

this.data=(int[])data.clone();

//OK}

拷貝int型的二維數(shù)組更復(fù)雜些。Java沒有int型的二維數(shù)組，因此一個int型的二維數(shù)組實際上是一個這樣的一維數(shù)組：它的類型為int[]。簡單的克隆int[][]型的數(shù)組會犯與上面例子中g(shù)etValues()方法第一版本同樣的錯誤，因此應(yīng)該避免這么做。下面的例子演示了在克隆int型二維數(shù)組時錯誤的和正確的做法：publicvoidwrongStore(int[][]data)throwsCloneNotSupportedException…{

this.data=(int[][])data.clone();//NotOK!

}

publicvoidrightStore(int[][]data)…{

//OK!

this.data=(int[][])data.clone();

for(inti=0;i

if(data!=null)

this.data[i]=(int[])data[i].clone();

}}

咨詢

地址：沈陽市和平區(qū)三好街文萃路云頂大廈四樓

熱點推薦：沈陽高級框架技術(shù)培訓(xùn)課程

沈陽5月Java高級框架技術(shù)培訓(xùn)課程

沈陽MCSA資格認證培訓(xùn)

沈陽吉大UG軟件設(shè)計課程

>>閱讀更多關(guān)于沈陽JAVA課程的資訊信息請訪問：沈陽（JAVA課程）

或者直接訪問sy.houxue/xuexiao-28032/