You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Names (types, variables, properties, Properties, functions,...) should be sufficiently descriptive and specific, and should avoid non-standard abbreviations. For example: ...
There are dozens of occurrences of the t abbreviation, like:
derive: t=>t.launcherProperty,
Sometimes t is a launcher, sometimes t is a field. All of these occur in the context of a relatively unusual use of DynamicProperty, and t doesn't make understanding that any easier.
The text was updated successfully, but these errors were encountered:
For code review #32 ...
From the PhET coding conventions:
There are dozens of occurrences of the
t
abbreviation, like:Sometimes
t
is a launcher, sometimest
is a field. All of these occur in the context of a relatively unusual use of DynamicProperty, andt
doesn't make understanding that any easier.The text was updated successfully, but these errors were encountered: