----------------------------------------
"0","../logs/fa5cbfdc-facd-48a6-8cc0-9fce41045207.log","files_experiment1","True",5
01-15-all the parameters are used only numerically and cannot be null, why not use int instead of Integer?
01-14-If we want to check preconditions, we could also check whether the arguments are positive
01-10-It is unclear whether big boxes could be partially packed (e.g. 16 items in 4 big boxes). If yes, the algorithm does not work. A test for this condition is also missing.
01-27-Should be <, when small equals the number of needed smallBoxes this is OK
11-19-adding a test for boundary conditions would have spotted the above error
----------------------------------------
"1","../logs/af4373cb-82c1-4670-a8d1-83a05b06415a.log","files_experiment1","True",0
----------------------------------------
"2","../logs/9ab44703-b20e-44f1-8d61-223a75513683.log","files_experiment1","False",3
01-19-better to use <= instead of <?
01-25-Confusing that 3rd method argument is being reassigned within the method.
01-27-should be < instead of <=?
----------------------------------------
"3","../logs/b88a7d09-1983-49ca-ae83-9145ac789fa4.log","files_experiment1","True",3
01-16-Agree with Pat (would normally put a +1 or thumbsup on her comment e.g on github), we should check other arguments as well
01-15-We should rather take and return simple ints and not Integers, below there are places we are modifying their values
01-18-This entire logic is a bit convoluted and difficult to understand
----------------------------------------
"4","../logs/7fdb5f6f-4786-4d59-a2ec-1bd8e41ad348.log","files_experiment2","False",4
01-10-What does result refer to? The result of the function? The result of the line?
01-11-This should be an
01-15-What do you carry?
01-19-theres a space in "i ++" this should be "i++"
----------------------------------------
"5","../logs/dc8c79df-36ea-48db-ba54-4d56c169d2e1.log","files_experiment2","False",6
11-24-Should either the first or second array be [0] to match the name of this test method?
11-9-Assuming this is performing a deep equals check and not checking if `first` and `second` both reference the same array.
01-18-Why `null` and not zero for the default value?
01-23-Should be >= (and please add a test for this case!)
01-30-Only need to add the carry term if > 0. I don't think the carry can ever be more than 1?
01-16-Extra space after ++?
----------------------------------------
"6","../logs/9644f8a2-c4b0-479c-816b-17a591f545c7.log","files_experiment2","False",4
01-23-Should be res >= 10
01-30-Should be carry > 0
11-25-Extreme conditions are not tested at all.
11-1-Extreme conditions (numbers of different length, etc.) are not tested at all!
----------------------------------------
"7","../logs/e1258540-0724-4e89-88ca-fea12404ca71.log","files_experiment2","True",1
01-30-inconsistent style : No parenthesis - while line 23 has parenthesis.
----------------------------------------
"8","../logs/b25b9d98-6af5-4e82-a6d0-dd7ba238738b.log","files_experiment1","False",0
----------------------------------------
"9","../logs/63962c8c-7904-4bda-ad69-50916271e21c.log","files_experiment1","False",1
01-27-only need < here
----------------------------------------
"10","../logs/c3c86566-8b12-480c-9b74-2298912f016b.log","files_experiment1","True",7
01-16-You should divide 'total' by 'small', and not by 5. Otherwise it works only for one given case of big box capacity of 5.
01-19-You should check if 'big' is not null
01-25-You should use 'small' in multiplication insteado of 5
01-23-You should use 'small' in multiplication insteado of 5
01-24-You should use 'small' in multiplication instead of 5
01-18-You should check if 'big' is not null
01-26-You should remove '=' from the condition because if small boxes equal to 'total' it means that there are just enough boxes.
----------------------------------------
"11","../logs/76e9cdc4-7718-4786-9420-7a9c2eb14efd.log","files_experiment2","False",6
01-17-null should be 0 (zero).
01-18-null should be 0 (zero).
01-23-> should be >=
01-30->= should be >
11-13-Does not test numbers of different length.
11-24-Does not test a zero number.
----------------------------------------
"12","../logs/d73f0b52-69b5-4a16-aaae-a6ba437d0caf.log","files_experiment2","True",13
01-16-You are modifying the input list. You need to work on a copy of the original lists.
01-11-This should be a "or"  operation, not "and".
01-12-In case of invalid input, you should throw exception, not return an empty list.
01-20-You need to convert Integer to int.
01-21-You need to convert Integer to int.
01-30-You are adding "int" to an Integer array.
01-26-this should be if (res >= 10) or if(res>9)
01-34-Where is the last carry ? If at the end carry is 1, you need to insert it.
11-5-Wrongly creating Integer array.
11-9-Should use assertArrayEquals.
11-19-Should use assertArrayEquals.
11-30-Should use assertArrayEquals.
11-33-Many important use cases are missing. (null values, final remaning carry, etc,)
----------------------------------------
"13","../logs/946c6cf7-4986-47b3-b39b-919f0c219d76.log","files_experiment2","True",5
01-16-Using Math.max() will lead to an IndexOutOfBoundsException for the smaller list.
01-20-Since null can be assigned to one of the values, this can fail. You probably want to assign a 0 instead.
11-13-This test does not actually test with lists of different length. The method name is thus misleading and the important case remains untested.
11-24-Why is this a "zero" test?
01-31-This is not a generic carry, but only works for cases in which the sum is >100 and <200.
----------------------------------------
"14","../logs/f7431ffc-5e19-4fef-b267-3f2d89782b85.log","files_experiment1","True",2
01-19-You should check that 'big' is null as well
01-27-You should check that 'small' is null as well
----------------------------------------
"15","../logs/92ec4a9a-e783-449b-8837-e1752a8dec34.log","files_experiment2","True",2
11-24-This does not test what the name implies
11-1-Add a test for the case when two digits sum to exactly 10
----------------------------------------
"16","../logs/057d0c5c-8188-4c6d-b478-81b37436ef47.log","files_experiment1","False",0
----------------------------------------
"17","../logs/66edbf5e-0b9a-4fe3-88e8-adf59c2d182a.log","files_experiment1","False",2
01-27-Space before (
01-19-Does not guard against non-valid values (e.g., -1 big boxes)
----------------------------------------
"18","../logs/10fb34af-1fbf-4675-b870-23366100cb22.log","files_experiment1","True",2
01-18-please use math.min instead
01-25-please use a const instead of 5
----------------------------------------
"19","../logs/f3dd58a6-ac79-42a3-be92-87dfaf47d25e.log","files_experiment1","True",2
01-15-replace Integer with int to avoid null pointer exceptions here
01-27-should be just less <, the equal = is a valid case
----------------------------------------
"20","../logs/6689743a-cecc-4917-a7a6-d0e2ae05921e.log","files_experiment2","True",1
01-19-this expression goes before the value of carry changes
----------------------------------------
"21","../logs/aac86644-6fce-42d1-bf75-21339cab23b8.log","files_experiment1","False",4
01-16-Add a constant for the big box capacity and use it here instead of hardcoding capacity.
01-25-Use big box capacity constant. Also, better introduce local variable instead of modifying input paramter.
01-15-Use int instead of Integer to avoid auto-boxing. Also make method name more explicit something like computeNeededSmallBoxesNumber.
01-17-Initialize bigBoxesWeCanUse with maxBigBoxes to simplify following if.
----------------------------------------
"22","../logs/23f0f32e-d9dd-47f9-bf47-a01f0327bfc3.log","files_experiment2","True",2
01-24-Carry should equal res / 10
11-28-Can we add a test for lists of different sizes?
----------------------------------------
"23","../logs/4c653f7e-9781-42de-944a-86620fac92fe.log","files_experiment2","True",3
01-13-inputs shouldn't be modified, create a copy for reverse array number
01-14-same
01-17-should return 0, not null for NPE
----------------------------------------
"24","../logs/297d1961-7f1e-48cd-90a8-98d0e7535ee7.log","files_experiment2","True",2
01-12-check if first or second is null => nullpointerexception
11-31-add a test where a list is null
----------------------------------------
"25","../logs/b3638a43-8613-47eb-8f6b-82a0a0224b13.log","files_experiment1","False",1
01-27-the condition should be `small < total` because small == total is accepted
----------------------------------------
"26","../logs/5c278fed-ec80-4aa9-9afb-663c84143d09.log","files_experiment1","True",2
11-20-add unit-test for the case we need only big bozex, i.e. when expected result = 0
01-27-should be: if small < total
----------------------------------------
"27","../logs/b170bcac-4ea9-4a59-9504-9607fe27d608.log","files_experiment1","True",1
01-19-Cast maxBigBoxes to a float, otherwise the logic fails
----------------------------------------
"28","../logs/27108b2c-8f17-42f9-a168-683697c5dadf.log","files_experiment1","True",1
01-27-You should test for condition small is less than total to return -1
----------------------------------------
"29","../logs/36b80936-831f-4e73-8e5a-899022740e27.log","files_experiment2","True",1
11-17-result should be 6,1,7 or test case should be set to failure
----------------------------------------
"30","../logs/28b95d84-db09-4352-a68a-e58744ff64e0.log","files_experiment2","False",5
01-18-why set it to null? It will give NPE later. You should refactor the for and exit the first time one of the 2 arrays is finished
11-4-give ex a better name please
11-14-same here
11-25-same here
01-23-I guess this should be >=, otherwise it's gonna fail on 9+1, or 8+2,etc.
----------------------------------------
"31","../logs/8ad40ff3-45d2-4f71-8e42-8cf21f3a71ef.log","files_experiment2","False",9
01-16-for(int i=0
01-20-int res
01-22-comment this line,i.e., //carry=0
01-26-else{carry=0
11-1-import java.org.pack.ExcerciseSumArray
11-16-List<Integer> second = Arrays.asList(0,7,8)
11-26-List<Integer> first= Arrays.asList(0)
11-27-List<Integer> second= Arrays.asList(0)
11-28-List<Integer> result= Arrays.asList(0)
----------------------------------------
"32","../logs/8f42fa00-e396-48f5-bee4-a635fc6e3329.log","files_experiment2","False",4
01-17-why `null` and not 0 ?
01-1-imports ?
01-20-this can produce null pointer exception when firstValue or secondValue are unboxed. Use Optional.ofNullable or somtehing like this
11-32-there is no test for number greater then 100. however this logic is not working for this kind of numbers
----------------------------------------
"33","../logs/b916adac-9705-4c2f-9b4e-34b26abcc16b.log","files_experiment2","True",1
01-13-reverse could be problematic
----------------------------------------
"34","../logs/d656a799-fdfc-4ec8-b44d-6f1dc182fb98.log","files_experiment1","False",1
01-27-Here it should be < rather than <=
----------------------------------------
"35","../logs/ab59eb51-4619-40ee-845c-071da9ec4187.log","files_experiment1","False",1
01-27-Should be <, not <=
----------------------------------------
"36","../logs/5d9898a8-262d-49d9-a01a-0d6e37ff355d.log","files_experiment2","True",3
01-24-line 24: carry = res - res% 10
11-13-Test arrays have same length, contrary to the test name
11-24-strange name for this test
----------------------------------------
"37","../logs/5a010087-b687-49fa-ae40-8f5acff84211.log","files_experiment1","True",2
01-27-Should be <.  Equal is ok.
11-20-Add a test for having exactly enough small boxes.  calculate(5,3,20)
----------------------------------------
"38","../logs/62c7ced8-9f36-403e-954a-fb9d5b1adead.log","files_experiment2","False",5
01-9-naming of variables
01-13-what is the point of the reverse?
01-16-This should be more explicit
01-22-carry is initialized twice
01-3-The examples are not clear since they are too contrived. More examples are necessary to determine if the intent is to treat 3,4 as 34 and the answer would be 1, 3, 2 or if it would be 12, 12
----------------------------------------
"39","../logs/b222a12f-4d59-4cc0-babf-fe47cf800947.log","files_experiment2","False",4
01-17-OutOfBounds Exception will be thrown
01-18-OutOfBounds will be thrown
01-20-Need to move after line 26, Must concat strings and parse to integer
01-25-res = result %10
----------------------------------------
"40","../logs/033b99d5-c200-411d-87bc-97e09e923760.log","files_experiment1","True",2
01-27-Should be small < total, if small == total, we have enough boxes
01-15-Check that all parameters are greater or equal 0
----------------------------------------
"41","../logs/0717faba-dc45-4b15-95bb-93735fec2ac9.log","files_experiment2","False",3
01-18-if firstValue goes to null, then it gives a NullPointerException in the next line
01-23-this should be >=10 otherwise if fails. For example when we have 65 + 5. Create a test for this please!
11-32-tests looks good to me. Maybe you should change their names though
----------------------------------------
"42","../logs/da272199-3938-4639-bc92-18735bc92c5c.log","files_experiment1","False",2
01-15-Use `int` instead of `Integer` when you don't want to deal with the possibility that it might be `null` (plus it might perform slightly better)
01-27-Should this be `if (small < total)`?
----------------------------------------
"43","../logs/439aff7c-d5b5-47e3-9feb-e63084931bca.log","files_experiment2","False",4
01-24-carry might not always be equal to 1, instead assign carry = res / 10
01-18-instead of defaulting to null if not present, use zero.
01-17-Instead of defaulting to null if not present, use zero.
11-31-I suggest adding a unit test to excersize adding together two arrays of different lengths to ensure that scenraio is covered.
----------------------------------------
"44","../logs/4667b03b-8e3e-4d17-8880-c041c1bfb568.log","files_experiment2","True",5
11-13-Looks like this test doesn't do what you intend it to do. Try making either 'first' or 'second' a longer array
11-24-You might want to revisit this one as well. Did you intend to use 0 as an input here?
01-18-I'm a bit unfamiliar with Java, but wouldn't you want the else condition to be 0 instead of 'null'?
01-30-You shouldn't need to append 'carry' if it equals 0.
01-19-Add in some validation here to ensure there are no integers in the list with more than one diget
----------------------------------------
"45","../logs/d720ed46-2ed4-4fdb-8604-0928d651626c.log","files_experiment1","True",2
01-19-Check if big is null
01-27-Check if small is null
----------------------------------------
"46","../logs/c175d7de-444d-4a3f-a2a8-0109118e08c5.log","files_experiment2","False",3
01-16-This should be min(). Using max() will cause an out-of-bounds error.
01-20-If firstValue or secondValue is null, this line will fail. I would replace this and the preceeding two lines with a more-thorough if/else chain that will handle each case of iterating past one of the lists's length.
11-1-This test does not cover edge cases like adding lists of different length, which would expose the problem I commented in the implementation. Please add additional tests covering that case, such as the one in the documentation for that method.
----------------------------------------
"47","../logs/4bbfb514-4e4b-447e-a671-68fe04f92292.log","files_experiment1","False",4
01-16-total /5 does not necessary result in int, explicit conversion
01-19-<= big
01-25-do not overwrite inputs
01-27-total > small
----------------------------------------
"48","../logs/02298781-4acb-4db8-842b-aa8692672064.log","files_experiment2","True",4
11-22-Test name does not correspond to the test case. This test looks redundant since it checks the same case in the previous test (e.g., two digit numbers, carry at each step, etc.).
11-11-Test name does not correspond to the test case.
01-19-This throws null pointer exception when firstValue or secondValue is null as the result of the assignments in lines 17 and 18. 0 should be used instead of null when i is greater or equal to the size of the number.
01-29-This should be carry > 0. Otherwise it adds a 0 to the result when carry is 0.
----------------------------------------
"49","../logs/457aa7ef-a164-489a-bded-93bcb441cbfe.log","files_experiment1","False",0
----------------------------------------
"50","../logs/1b4b15c1-027a-4d23-8ab7-554a8a24d04e.log","files_experiment2","False",7
01-17-0 instead of null, otherwise addition will unbox the null Integer and cause NPE
01-18-0 instead of null, otherwise addition will unbox the null Integer and cause NPE
01-23-change to res >= 10, otherwise if digit sum is 10, carry is 0 instead of 1.
01-30-`carry > 0` or `carry == 1` instead, otherwise additional 0 is added in front of the sum
01-16-Style error: i++ without space
11-13-Doesn't test what the method says, thus lets a bug slip through
11-24-Not exactly sure what it intends to test, but there are no zeroes. All 3 tests are pretty much the same. Not exactly a "bug" but bad tests let bugs through so in my interpretation bad test == bug
----------------------------------------
"51","../logs/70417919-2a46-42c6-9c8c-8c33676bf10b.log","files_experiment1","False",2
11-8-expected value should be 0
01-15-missing input validation, probably should expect all values to be non-negative
----------------------------------------
"52","../logs/4fb2ba2f-dde8-462f-9aed-500ecf3848c4.log","files_experiment2","False",4
01-23-if (res >= 10) ...
01-30-if (carry > 0) ...
11-13-wrong name
01-9-code works only when both input lists contain only positive numbers
----------------------------------------
"53","../logs/30a24383-3f4c-4f2c-9a97-4bc6437efa43.log","files_experiment1","False",3
01-27-should be < not <=, if small == total, there are enough small boxes
01-25-If there are 0 small boxes, but 3 big and total 15, then total will become zero here, causing a -1 to return but should return 0
11-5-add test cases where some input values are 0
----------------------------------------
"54","../logs/4ea31cd0-d711-4e0f-bb2b-09d0d071660e.log","files_experiment2","True",4
01-16-you have to start backwards in these lists, in the case that they are of unequal length
01-20-sure you can add null to an integer?
11-2-test cases only test equal length lists! --> add other cases
01-30-these 2 lines should be removed? carry has already been added in line 20, now you are adding an extra digit to the list
----------------------------------------
"55","../logs/a8954390-4f8b-4a3d-9fe4-3d1f89fa4cc6.log","files_experiment1","False",3
01-27-small == total should Not return - 1
11-20-add test where no small boxes are needed
01-17-what about negative input values?
----------------------------------------
"56","../logs/10c3e80e-0a07-4792-a3ff-9c8a0015ce35.log","files_experiment1","True",8
01-19-Use ternary i.e. Integer bigBoxesWeCanUse =maxBigBoxes > big ? maxBigBoxes : big
01-25-better to no change the reference for input argument. Define a new variable here
01-27-ditto on changing to turnery
11-6-Rename test to testCalculateSmallBox_fewBigBoxes
01-15-Method name is to generic, As well as variable names.
11-14-rename to testCalculateBoxes_notEnoughBox
11-8-nit: Don't need to have a variable defined
11-16-ditto
----------------------------------------
"57","../logs/d75737a0-9d63-439b-a131-a0484fd3ca29.log","files_experiment2","True",5
01-16-increment should be i++ now there is a white space between i and ++
01-20-carry will always be "0" at this point
01-19-Summing null with an integer might throw an exception in Java, not sure, but good to check.
01-24-I wonder if each array element is a number between 0-9. If not carry should not be hardcoded as 1.
01-30-Should be >
----------------------------------------
"58","../logs/4c1fe7ed-271a-4a95-abd3-e09c2e642af3.log","files_experiment1","True",3
01-16-Use float type instead, to avoid rounding errors. Also, check for null for the other parameters as well
01-15-Check for negative parameter values
01-27-This should not be equal. Consider the case (1, 0, 1).
----------------------------------------
"59","../logs/4b9ab916-eece-471a-b9b5-767bbc947131.log","files_experiment1","False",2
01-27-Use < and not <=.
01-19-Use <= and not <
----------------------------------------
"60","../logs/019a7e45-e9e4-408b-aa26-92ecc2f417a6.log","files_experiment2","True",7
01-17-Dont set null in else part set 0 (Zero)
01-18-same here Zero in else part
01-27-Don't add  in list if res is 0 (zero) . Add if condition
01-29-add  Collections.reverse(result)
01-33-remove it
01-9-return List interface not tightly arraylist , if require to use linkedlist then we can easily move
11-7-Mockito to do Mock and then use when then
----------------------------------------
"61","../logs/9b995ecc-ce92-4abb-96de-1f7bcde82250.log","files_experiment1","False",2
01-27-should be <
01-25-doesn't need to be * 5
----------------------------------------
"62","../logs/9cd94363-845a-4d17-9a0a-bf0fa2a2f350.log","files_experiment1","True",1
01-27-Change the operator from '<=' to '<'. This code returns -1 when you have the exact number of small boxes needed.
----------------------------------------
"63","../logs/42c1fc61-e82f-4e0e-b199-d8c993c8ccf6.log","files_experiment2","False",5
01-20-This line throws a NullPointerException
11-10-This passes the test case
11-22-This also passes the test case
11-32-This also passes the test case
01-1-There should be an import statement before line 1: import java.util.*
----------------------------------------
"64","../logs/6599757f-1192-4479-a72c-36d8c3a1552d.log","files_experiment2","True",1
01-14-This is a bug. The check shoud be =, otherwise it will avoid summing the first element of the list.
----------------------------------------
"65","../logs/a5f37f18-c8b5-43c8-81b2-54902dd403f7.log","files_experiment1","True",3
01-17-Following Pat's comment, you should check for null for all the other parameters.
01-27-What happens if the number of small boxes is the same as the current total? Wouldn't that be a valid as well?
01-18-Also review if the parameters are non-negative numbers
----------------------------------------
"66","../logs/497120bb-67ca-4898-9ef7-b902a8cf7821.log","files_experiment1","False",1
01-27-Use small < tota -> case 0 small boxes, 3 bix and total = 15
----------------------------------------
"67","../logs/a67414c1-a8c4-4345-9094-76f8db0fbe60.log","files_experiment2","False",4
01-20-can't add nulls
01-22-resetting inside the loop
01-23-what if it equals 10
01-24-should this be add?
----------------------------------------
"68","../logs/bf96c1d3-fdc0-4c56-83b2-fd8ab82a0b2e.log","files_experiment2","False",4
01-20-nullpointerexception when lists have different size
11-16-test name states different length for numbers, implementation does not match
11-24-idem, test name does not match implementation
01-23-> must be >=
----------------------------------------
"69","../logs/527211c5-43e6-4877-8a94-315a21a3aadd.log","files_experiment2","True",3
01-20-Potentially adding null - problem?
11-16-Both first and second number are two-digit numbers.
11-28-Test data doesn't produce a sum of zero.
----------------------------------------
"70","../logs/936b2be0-76f2-4948-acf5-bef79d96c03c.log","files_experiment2","False",1
01-30-Missing parenthesises to be consistent in the overall style
----------------------------------------
"71","../logs/3284448e-a07c-4a26-a731-525adf2957c5.log","files_experiment2","False",5
01-18-iterator i will be out of index if both arrays are of different size.
01-17-Check must be applied if i (iterator) is greator than the size of array size, then first value and second value could be zero
11-3-This test will be pass, but this couldn't verifys if array size is different
11-14-This test names as verify different length but array being passed is of same length
11-24-As the name of test states, there is no test being performed that do the zero test
----------------------------------------
"72","../logs/a7f15eaf-7fc0-4c69-bf6b-1cae4bebb15b.log","files_experiment2","False",4
01-17-use int instead of Integer. replace null with 0 to prevent null pointer exception.
01-18-use int instead of Integer. replace null with 0 to prevent null pointer exception.
01-27-use new Integer(res) instead of res
01-31-use new Integer(carry) instead of carry
----------------------------------------
"73","../logs/ab2aa8ea-d6f8-4016-b34a-0fcb1df59897.log","files_experiment1","False",1
01-27-= should be removed from <=
----------------------------------------
"74","../logs/6bf5eaf6-b971-4f6d-b75b-46e15bf82c7d.log","files_experiment2","False",3
01-23->= otherwise it might fail in some cases. And add a test for this case
01-30-this is always true right?
11-4-Add a test case for the >=
----------------------------------------
"75","../logs/10aec582-3f4a-497f-bb12-440ea4ceceff.log","files_experiment1","True",2
01-27-If small == total then we have enough small boxes, so the comparison should be strictly less than (<)
01-19-_If_ you really need to check that 'total' is null in row 16, then you should do the same check for 'small' and 'big'
----------------------------------------
"76","../logs/8430e230-a76f-49e6-911e-3a2fab1c9874.log","files_experiment1","True",3
01-16-big and small should also be checked for null
01-15-big and small should also be checked for null
01-27-should be small < total
----------------------------------------
"77","../logs/56450a5e-e25d-4c77-8faf-7e8b41a3d7c6.log","files_experiment2","False",6
01-20-If firstValue or secondValue is null, it cannot be added to int
11-5-Object ex need not be created under every method. Instead use @Before annotation
11-25-The test method name and the actual execution dont seem to match
11-8-Similarly, better to create the constant Lists "first" and "second" outside the method during TEST initializtion
11-22-The test case should cover error scenarios as well
01-25-Should carry be - carry = res / 10
----------------------------------------
"78","../logs/d7914fa8-65bd-4a05-aebf-60c2b1117580.log","files_experiment2","False",5
01-18-why null here? It's gonna raise a NPE the next line
01-23-res >= 10. otherwise it's gonna fail in some cases
01-13-not sure that this is optimzed. Maybe you could design the algorithm in a way that it starts from the end of the 2 numbers
11-24-sumZeroTest?
11-13-the numbers have equal length
----------------------------------------
"79","../logs/b9b145bc-9e73-4570-a8f6-8f3cdfdfffd8.log","files_experiment2","True",1
01-20-you use a variable carry that you havent given a value yet before but after.
----------------------------------------
"80","../logs/1748919d-9195-4e95-9b1c-2d3b4a27fe01.log","files_experiment2","False",4
01-33-not sure if you can reverse an empty list if input are null
01-30-it should be > not >=, because leading zero is not needed
01-16-why is second statement in ()?
11-24-why is it called sumZero? I can't see major differences between test cases
----------------------------------------
"81","../logs/03475c7d-8fd3-4bab-ab9b-a61af848394d.log","files_experiment2","True",3
01-30->0, carry is always >=0
11-13-why the name is sumNumbersDifferentLength? The numbers have the same length
11-24-again, the name of the test is wrong
----------------------------------------
"82","../logs/d7b0a57c-69e3-47f3-9804-0185ad271df2.log","files_experiment2","True",4
01-9-ArrayList => List
01-19-null cases should be handled - use of Optional would be better tough
11-15-as the method name suggests first and second should actually have different length
11-27-as the method name suggest either first or second should be zero
----------------------------------------
"83","../logs/3d8530db-1b29-456e-9059-8bd184c2ef09.log","files_experiment2","True",1
01-17-Here and line 18, should be 0, not null
----------------------------------------
"84","../logs/9de03bce-67c7-4261-a465-b4f5550906c2.log","files_experiment1","True",3
01-15-You should check for negative values in the argument
01-22-You should check if 'big' is null
01-27-you should check if 'small' is null
----------------------------------------
"85","../logs/f99ccb2c-1f9a-45e4-8105-6b7cabe3970d.log","files_experiment1","False",2
01-27-If small equals to total we can use all small boxes, therefore for small = total returing -1 is not correct
01-15-Inputs are not sanitize and this code wont work if incorrect numbers will be passed. For calculate(1,2,2) will be returned -8
----------------------------------------
"86","../logs/330ee04f-2047-447e-8a9f-53f11d3166d7.log","files_experiment2","False",0
----------------------------------------
"87","../logs/a0a6ecd5-7137-4d28-a3d4-dfe7391e113f.log","files_experiment1","True",1
01-27-should be <
----------------------------------------
"88","../logs/8c6d954d-01ed-40e2-aa4e-cf6ffaf3fe9a.log","files_experiment1","True",0
----------------------------------------
"89","../logs/9c131297-fbfc-4326-8825-7dd6bff2af83.log","files_experiment1","False",0
----------------------------------------
"90","../logs/1efe4e9e-98d5-420f-a678-a3bae8c6eac1.log","files_experiment2","False",3
01-10-There is no check if the input parameters firstNumber and secondNumber are NULL, in which case there'll be a NPE
01-23-here we need "res >= 10"
11-9-we cannot check for equality here, because "result" and the actual result of "getSum()" are not the same list, but ists with the same values
----------------------------------------
"91","../logs/976be29a-25b4-4c4b-86e3-751e5536b4ac.log","files_experiment1","True",1
01-15-Likewise, we should check whether the best is null. Better, pass int values instead of boxed integers.
