This method is throwing an IndexOutOfBoundsException, and I don’t understand why as I’ve guarded against it.<\/p>\n
private static boolean firstLoop(int custNo, LinkedList stock, LinkedList custs, Random generator, String colour, int col, Tracking tracking) {\n\n if ((stock.get(tracking.getLast(col)) != null) && (stock.get(tracking.getLast(col)).getLength() >= custs.get(custNo).getLength())) {\n\n stock.get(tracking.getLast(col)).length = Cut.cut(stock.get(tracking.getLast(col)).getLength(), custs.get(custNo).getLength(), colour);\n if (stock.get(tracking.getLast(col)).length < 5) {\n\n stock.remove(tracking.getLast(col)); \/\/**CAUSES EXCEPTION**\n\n tracking.add();\/\/ recycle\n\n }\n return true;\n } else {\n for (int j = tracking.getLast(col) + 1; j < stock.size(); j++) {\n\n if ((stock.get(j).getLength() >= custs.get(custNo).getLength())) {\n \/\/ pipe is long enough, cut away the desired length\n stock.get(j).setLength(Cut.cut(stock.get(j).getLength(), custs.get(custNo).getLength(), colour));\n\n tracking.setLast(col, j);\n\n return true;\n }\n }\n\n \/\/ no suitable pipes available, order new one of correct colour with\n \/\/ random length 100-200 then cut from it, add to arraylist\n Pipe temp2 = new Pipe(col, generator.nextInt(101) + 100);\n temp2.setLength(Cut.cut(temp2.length, custs.get(custNo).length, colour));\n stock.add(temp2);\n tracking.setLast(col, stock.size() - 1);\n return false;\n }\n\n}\n<\/code><\/pre>\nI’ve found that the marked line is the one that causes the exception (the program runs perfectly when it’s commented out). However, I’m confused because tracking.getLast(col) works perfectly in the lines above it, and the remove function is not inside an iterator or a loop. Here is the Tracking class:<\/p>\n
public class Tracking {\n\nstatic int lastR=0;\nstatic int lastG=0;\nstatic int lastY=0;\n\n\npublic void setLast(int col, int last){\n if(col==0){\n lastR=last;\n }else if(col==1){\n lastG=last;\n }else if(col==2){\n lastY=last;\n }else{\n System.out.println(\"Colour does not exist\");\n }\n}\n\npublic static int getLast(int col){\n if(col==0){\n System.out.println(lastR+\" red\");\n return lastR;\n }else if(col==1){\n System.out.println(lastG+\" green\");\n return lastG;\n }else{\n System.out.println(lastY+\" yellow\");\n return lastY;\n }\n}\n<\/code><\/pre>\nAnd this is the use of the method that throws the error:<\/p>\n
if ((yellowStock.get(Tracking.getLast(2)) != null) && (yellowStock.get(Tracking.getLast(2)).getLength() >= custs.get(i).getLength())) {\n\n firstLoop(i, yellowStock, custs, generator, \"yellow\", 2, recycle);\n} \n<\/code><\/pre>\nException in thread “main” java.lang.IndexOutOfBoundsException: Index: 3, Size: 3 at java.util.LinkedList.checkElementIndex(Unknown Source) at java.util.LinkedList.get(Unknown Source) at NextFit.next(NextFit.java:26) at Main.main(Main.java:58)<\/p>\n
Thank you very much guys.<\/p>\n
\n- \n
The problem is as follows:<\/p>\n
LinkedList.remove()<\/code> comes in two flavours:<\/p>\n\nLinkedList.remove(int index)<\/code>, which removes the element at the specified index<\/li>\nLinkedList.remove(Object o)<\/code>, which removes the specifed element<\/li>\n<\/ol>\nFurther, tracking.getLast(int col)<\/code> returns int<\/code>, so when you call<\/p>\nstock.remove(tracking.getLast(col));\n<\/code><\/pre>\nthe version of remove()<\/code> that is called is number 1, not number 2 as you wanted.<\/p>\nTry calling stock.remove(col);<\/code> or whatever makes sense.<\/p>\nAs an aside, your coding style could use some improvement, especially the use of static fields and methods in Tracking<\/code> instead of instance<\/em> fields and methods.<\/p>\n<\/li>\n<\/ol>\nOriginally posted 2013-12-02 21:01:23. <\/small><\/p>","protected":false},"excerpt":{"rendered":"
This method is throwing an IndexOutOfBoundsException, and I don’t understand why as I’ve guarded against it. private static boolean firstLoop(int custNo, LinkedList stock, LinkedList custs, Random generator, String colour, int col, Tracking tracking) { if ((stock.get(tracking.getLast(col)) != null) && (stock.get(tracking.getLast(col)).getLength() >= custs.get(custNo).getLength())) { stock.get(tracking.getLast(col)).length = Cut.cut(stock.get(tracking.getLast(col)).getLength(), custs.get(custNo).getLength(), colour); if (stock.get(tracking.getLast(col)).length < 5) { stock.remove(tracking.getLast(col)); \/\/**CAUSES […]<\/p>\n","protected":false},"author":1,"featured_media":0,"comment_status":"closed","ping_status":"closed","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[1],"tags":[],"class_list":["post-1852","post","type-post","status-publish","format-standard","hentry","category-uncategorized"],"_links":{"self":[{"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/posts\/1852","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/comments?post=1852"}],"version-history":[{"count":0,"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/posts\/1852\/revisions"}],"wp:attachment":[{"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/media?parent=1852"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/categories?post=1852"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/tags?post=1852"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}