我正在嘗試創建一個函數，該函數將數組的所有元素與第二個數組的所有元素進行比較，并將返回所有可能的匹配項，如果未找到匹配項，則返回消息。當我嘗試實現代碼時，我得到一個索引越界錯誤。內部 for 循環可能在外部 for 循環完成運行之前達到最大值。如何修改它以防止發生這種情況？Stocks[] stockList3 = new Stocks[3];stockList3[0] = new Stocks("a", 2, 1, "Buy");stockList3[1] = new Stocks("a", 3, 1, "Buy");stockList3[2] = new Stocks("a", 4, 1, "Buy");Stocks[] stockList4 = new Stocks[3];stockList4[0] = new Stocks("a", 2, 1, "Buy");stockList4[1] = new Stocks("a", 5, 1, "Buy");stockList4[2] = new Stocks("a", 4, 1, "Buy");public void matching(Stocks[] array1, Stocks[] array2) { for (int i = 0; i < array1.length; i++) { for (int j = 0; i < array2.length; j++) { if (array1[i].stockPrice == array2[j].stockPrice) { System.out.println("It's a match at $" + array1[i].stockPrice); } System.out.println("still searching..."); } System.out.println("first loop test..."); }}
2 回答
三國紛爭
TA貢獻1804條經驗 獲得超7個贊
如何代替兩個，使用集合來存儲數組之一的存在？for loopsSetstockPrices
public static List<Stocks> matching(Stocks[] one, Stocks[] two) {
Set<Integer> stockPrices = Arrays.stream(one)
.map(stock -> stock.stockPrice)
.collect(Collectors.toSet());
return Arrays.stream(two)
.filter(stock -> stockPrices.contains(stock.stockPrice))
.collect(Collectors.toList());
}
它使用 O（n） 附加內存（其中 n 為 1.length）和 O（n + m） 性能時間（其中 m 為 2.length）。
人到中年有點甜
TA貢獻1895條經驗 獲得超7個贊
在你的j循環中，你說的不是i<array2.lengthj<array2.length
public void matching ( Stocks[] array1, Stocks[] array2){
for (int i=0; i<array1.length;i++){
for (int j=0;
j<array2.length; //this j was an i
j++){
if (array1[i].stockPrice == array2[j].stockPrice){
System.out.println("It's a match at $" + array1[i].stockPrice);
}
System.out.println("still searching...");
}
System.out.println("first loop test...");
}
}
添加回答
舉報
0/150
提交
取消