don't allow override of turtle id

[m0ose]

It turns out that it turtle list is sorted by id, and if you set the id it will mess some internal things, like turtle removal.

Maybe the turtle id should be private, or have a setter that gives you a warning to avoid shooting yourself in the foot?

[backspaces]

I agree! "Private", as per
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Classes/Private_class_fields
..looks pretty useful. Have you tried it?

I'm surprised that all the browsers support it, and so quickly.

[m0ose]

I haven't used the private class fields. It looks well supported, https://caniuse.com/mdn-javascript_classes_private_class_fields

[backspaces]

This is more difficult than it seemed. Turns out use of private variables is class based but AS makes heavy use of prototypal methods. So this basically depends on: #83

[m0ose]

Could it be renamed to something less common like __id__ ?

…

On Wed, Oct 4, 2023, 9:53 AM Owen Densmore ***@***.***> wrote: This is more difficult than it seemed. Turns out use of private variables is class based but AS makes heavy use of prototypal methods. So this basically depends on: #83 <#83> — Reply to this email directly, view it on GitHub <#78 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AADADQRUJODL37PYQY553XLX5WBA3AVCNFSM6AAAAAA4DOQXQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONBXGE4TGOBUHA> . You are receiving this because you authored the thread.Message ID: ***@***.***>