Skip to content
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

Enable deletion #13

Open
SethPoulsen opened this issue May 26, 2020 · 2 comments
Open

Enable deletion #13

SethPoulsen opened this issue May 26, 2020 · 2 comments

Comments

@SethPoulsen
Copy link
Owner

I assume people will want to be able to delete nodes and pointers somehow, even when in the mode where nodes that are left dangling are still drawn on the screen.

We need to come up with what the UI should be like, then implement it.

@jasonxia17
Copy link
Collaborator

If a deleted node has pointers pointing to it, what should we do with those pointers after the deletion? I think we should somehow indicate that those pointers are not null; rather, they are pointing to some garbage that we no longer have control over (because that represents what would actually happen in C++ in this scenario).

Figuring out how to undo a delete operation might also be tricky, since we need to restore the pointers that previously pointed to the deleted node.

@SethPoulsen
Copy link
Owner Author

Those are great points. I think part of the design here will end up depending on

  1. the semantics of the language that the students are using along with the tool
  2. if we want to restrict operations in order to teach something

Right now its looking like we are going to end up targeting students have mostly a Java background, so that might make the design a little different.

For now I think I'm going to wait on this until we have a more clear idea of the exact tasks we are going to be asking students to do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants