-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
deepDuplicateArray #28
Comments
Hi, I'm pretty new to github and would like to try my hand at this issue. During my computer science studies I worked a lot with JavaScript / Typescript and I think I can fix this issue. I would be happy if I would be assigned to that. Best regards |
@TheSoniix Sounds great, we look forward to seeing your PR! |
How should I deal with objects in an array? Can i ignore that case, because we want a solution for subarrays? We have not implemented a function of duplicating objects yet. The duplicateArray function would only saves the references of the objects. |
Good point! #29 proposes duplication and deep duplication for an object. If you would like to work on that as well, please let me know so that I can assign you to that PR. I think both methods will need to use each other (for example, arrays within an object would use the |
Of course! I will try it |
We should have a method that duplicates an array and all subarrays, like this:
We encourage using the existing
duplicateArray
method and recursion to create thedeepDuplicateArray
method.The text was updated successfully, but these errors were encountered: