2015-09-30 214 views
1

我對計算機科學有點新,我寫了一個使用了很多變量的項目。我擁有主方法中的所有代碼,並且在主方法的開始處,我已聲明瞭大部分變量。我覺得這很醜陋。我不想改變我的整個代碼並且混淆了範圍。有沒有辦法宣佈這些相同的變量,並讓它看起來更有效率/專業?聲明這些變量最正確的方式是什麼?

int numPlain; 
int numPepperoni; 
int pizzasOrdered = 0; 
int slicesOrdered = 0; 
int charmsOrdered = 0; 
int totalSlices = 0; 
int totalCharms = 0; 
int totalPlain = 0; 
int totalPepperoni = 0; 
double plainCost = 0.0; 
double pepperoniCost = 0.0; 
double sliceCost = 0.0; 
double pieCost = 0.0; 
double charmCost = 0.0; 
boolean flag = true; 
double totalCost = 0.00; 
double paymentAmt = 0.0; 
int numPies = 0; 
+1

開始尋找類,即獨立的類,比薩,澆頭,等這你開始學習抽象和課堂互動的方式 – Pradheep

+0

這是非常完美的,沒有太多可以改變。像其他人所說的,你可以將他們分成不同的班級。 – bmarkham

回答

2

可以嘗試內聯你的一些變量聲明/初始化這樣的:

int numPlain, numPepperoni; 
int pizzasOrdered = 0, slicesOrdered = 0, charmsOrdered = 0; 
int totalSlices = 0, totalCharms = 0, totalPlain = 0, totalPepperoni = 0; 
double plainCost = 0.0, pepperoniCost = 0.0, sliceCost = 0.0, pieCost = 0.0, charmCost = 0.0; 
boolean flag = true; 
double totalCost = 0.00; 
double paymentAmt = 0.0; 
int numPies = 0; 

這會使代碼看起來這是關係到少一點冗長,還組在一起變量彼此。

+2

我仍然認爲他有太多的領域。一個好的代碼檢查工具會抱怨這一點。他可能需要單獨的課程:) – TheLostMind

0

我的大學老教授總是稱呼這樣的:

int  numPlain; 
int  numPepperoni; 
int  pizzasOrdered = 0; 
int  slicesOrdered = 0; 
int  charmsOrdered = 0; 
int  totalSlices  = 0; 
int  totalCharms  = 0; 
int  totalPlain  = 0; 
int  totalPepperoni = 0; 

double plainCost  = 0.0; 
double pepperoniCost = 0.0; 
double sliceCost  = 0.0; 
double pieCost   = 0.0; 
double charmCost  = 0.0; 

boolean flag   = true; 

double totalCost  = 0.00; 
double paymentAmt  = 0.0; 

int  numPies   = 0; 

這一切都偏愛真的,雖然。主鍵只是可讀性。隨着你的進步,你也不會在單個函數中擁有許多原始變量。

編輯:還要注意,你可以組你的變量一起按類型,或通過關聯(通過其功能分組變量一起)

相關問題