learn-gdscript icon indicating copy to clipboard operation
learn-gdscript copied to clipboard

Lesson 25, inconsistency between add_unit example call arguments and practice exercise place_unit function

Open jman904 opened this issue 1 year ago • 0 comments

Near the bottom of lesson 25 there is the following code snippet:

1 func run():
2   for cell in unit_cells:
3     var unit = unit_cels[cell]
4     add_unit(unit, cell)

I thought the arg ordering of add_unit was a little strange at first as add_unit(coordinate, unit_name) just feels more intuitive. Sure enough, in the 2nd practice example the order of the args for place_unit is in this order (i.e., place_unit(cell:Vector2, unit_type: String) ). It would probably add clarity to use the same function name, with the same argument ordering, in lesson module associated with the practice.

jman904 avatar Aug 24 '24 18:08 jman904