zacharyrperales / hackerrank_c_solutions Goto Github PK

The memory management portion of this problem is difficult to fully comprehend and solve. Assert statements and unit tests may assist in narrowing the scope of the entire challenge to just the send_all_acceptable_packages function temporarily so that memory management solutions can be explored and more closely examined.

The preliminary unit tests do not follow best coding practices.

1. There is repeated code that needs to be refactored into reusable units.
2. Only one scenario is covered for the send_all_acceptable_packages function. All logical scenarios should be covered for full coverage.

Instead of a pointer to the first element of the array being stored for the source and target office package arrays as they initially contain, which would be in the heap(?), in the send_all_acceptable_packages function, those values become NULL in the debugger after reallocating and moving memory in the aforementioned function. I believe that this is due to the newly created moved memory being stored on the stack and therefore not being accessible to the debugger or something to that effect. This does not appear to affect program operations but it's not ideal.

This process needs to be clarified and/or resolved.