Vectors #2074
Vectors #2074
Conversation
timeutills V1
github-actions
bot
added
the
pr: new extension
|
there is no block for deleting vectors? |
|
also please run !format |
|
!format |
Yes, just stop using a vector |
|
!format |
|
!format |
|
!format |
|
@fath11 there is now |
| @@ -0,0 +1,169 @@ | |||
| (function (Scratch) { | |||
There was a problem hiding this comment.
you're missing the header data
There was a problem hiding this comment.
< >, Imma re-add that Ig
| opcode: "vectorCreate", | ||
| blockType: Scratch.BlockType.COMMAND, | ||
| text: Scratch.translate( | ||
| "create vector with ID [ID], direction [DIRECTION] and magnitude [MAGNITUDE]" |
There was a problem hiding this comment.
you have inconsistent capitalization across the blocks, the blocks should be written starting with lowercase. they are NOT sentences.
| } | ||
|
|
||
| vectorCreate(args) { | ||
| const id = args.ID; |
There was a problem hiding this comment.
you don't cast anything, this is poor practice
There was a problem hiding this comment.
aren't they numbers, you can not put strings in there, and if you could, who would?
| } | ||
|
|
||
| // Vectors storage | ||
| const ids = []; |
|
|
||
| // Vectors storage | ||
| const ids = []; | ||
| const directions = {}; |
There was a problem hiding this comment.
these should be Object.create(null); or a Map
|
|
||
| vectorMag(args) { | ||
| const id = args.ID; | ||
| return magnitudes[id] ?? Scratch.translate("vector not found"); |
There was a problem hiding this comment.
you should return an empty string or 0 here
There was a problem hiding this comment.
do I want people to wonder why their shit isn't working? if you said yes then you are wrong, the answer is no.
There was a problem hiding this comment.
do I want people to wonder why their shit isn't working? if you said yes then you are wrong, the answer is no.
@GarboMuffin said to chill the language
|
|
||
| vectorDir(args) { | ||
| const id = args.ID; | ||
| return directions[id] ?? Scratch.translate("vector not found"); |
|
Maps are < >, it does what it needs to do very well |
|
!format |
|
it would be wonderful if you would be more reasonable with your language on this issue tracker |
|
it already does what it needs to do, I don't want to spend multiple hours rewriting the entire extension |
Vectors is a extension for Vectors
Blocks: Create vector with ID,direction and magnitude: creates a vector, vector direction, returns direction of the vector based on ID, vector magnitude, returns magnitude of the vector based on ID, Change Magnitude and direction, changes magnitude and direction of the vector based on ID, and List of IDs, lists all elements in IDs