2 回答

TA貢獻1847條經驗 獲得超7個贊
在 a 達到 colAvgs 值之前,余數不會變為 0,之后嘗試訪問 convbases[a] 會崩潰。
您的 while 應考慮到 a 不能高于或等于 colAvgs。
int size = colAvgs.length;
for (int a =0;a<size;a++)
{
remainder = colAvgs[a];
while(remainder>0 && a<size)
{
remainder = remainder%2;
if ((remainder%2)==0|| (remainder%2)==1)
{
convbases[a] = Integer.toString(remainder);
a++;
}//end if
}//end while
}//end for
if(remainder>0) //you know it finished due to a being too big.
else //this is what you want to happen
但是,我認為重要的是要注意,最好不要修改 for 循環內的循環變量,因為這是 for 工作的一部分。您不應該a++在 while 循環內執行此操作。當您確切知道循環將運行的迭代次數時,可以使用 for 循環。你這里沒有這些信息。因此,您應該更改循環條件(在本例a<size中是啟動 for 循環時),或者改用 while 循環。

TA貢獻2041條經驗 獲得超4個贊
因為它a在while (remainder > 0)循環內使用并增加,而無需檢查以確保a仍然存在< convbases.length。(順便說一句,它convbases在一個地方,但convBases在另一個地方,這應該會阻止該代碼編譯;Java 區分大小寫。)
public String[] streamChars(int[] colAvgs)
{
String avgString = Arrays.toString(colAvgs);
String convBases[] = avgString.split(",");
int remainder;
for (int a =0;a<colAvgs.length;a++)
{
remainder = colAvgs[a];
while(remainder>0)
{
remainder = remainder%2;
if ((remainder%2)==0|| (remainder%2)==1)
{
convbases[a] = Integer.toString(remainder);
// ^^^^^^^^^^^^------------------------------------- using it
a++;
// ^^^^--------------------------------------------- incrementing it
}//end if
}//end while
}//end for
return convbases;
}//end streamChars
添加回答
舉報