refactor: rename ArrayModel::get() to at()

This avoids conflicts with the get() method in Object and better
self-documents its purpose.

Author: danielzting

levels/bubble_sort.gd

@@ -22,7 +22,7 @@ func _init(array).(array):
     pass
 
 func next(action):
-    if array.get(_index) > array.get(_index + 1):
+    if array.at(_index) > array.at(_index + 1):
         if action != null and action != ACTIONS.SWAP:
             return emit_signal("mistake")
         array.swap(_index, _index + 1)

levels/insertion_sort.gd

@@ -22,7 +22,7 @@ func _init(array).(array):
     pass
 
 func next(action):
-    if array.get(_index - 1) > array.get(_index):
+    if array.at(_index - 1) > array.at(_index):
         if action != null and action != ACTIONS.SWAP:
             return emit_signal("mistake")
         array.swap(_index - 1, _index)

levels/merge_sort.gd

@@ -32,7 +32,7 @@ func next(action):
         if action != null and action != ACTIONS.LEFT:
             return emit_signal("mistake")
         _left += 1
-    elif array.get(_left) <= array.get(_right):
+    elif array.at(_left) <= array.at(_right):
         if action != null and action != ACTIONS.LEFT:
             return emit_signal("mistake")
         _left += 1

levels/selection_sort.gd

@@ -22,7 +22,7 @@ func _init(array).(array):
     pass
 
 func next(action):
-    if array.get(_base) > array.get(_index):
+    if array.at(_base) > array.at(_index):
         if action != null and action != ACTIONS.SWAP:
             return emit_signal("mistake")
         array.swap(_base, _index)

models/array_model.gd

@@ -1,8 +1,3 @@
-"""
-A plain old one-dimensional random access array wrapper around the
-built-in class.
-"""
-
 class_name ArrayModel
 extends Reference
 
@@ -16,7 +11,7 @@ func _init(size=16):
     array.shuffle()
     self.size = size
 
-func get(i):
+func at(i):
     """Retrieve the value of the element at index i."""
     return array[i]
 

project.godot

@@ -45,6 +45,11 @@ _global_script_classes=[ {
 "path": "res://levels/merge_sort.gd"
 }, {
 "base": "ComparisonSort",
+"class": "QuickSort",
+"language": "GDScript",
+"path": "res://levels/quick_sort.gd"
+}, {
+"base": "ComparisonSort",
 "class": "SelectionSort",
 "language": "GDScript",
 "path": "res://levels/selection_sort.gd"
@@ -57,6 +62,7 @@ _global_script_class_icons={
 "ComparisonSort": "",
 "InsertionSort": "",
 "MergeSort": "",
+"QuickSort": "",
 "SelectionSort": ""
 }
 

scripts/levels.gd

@@ -5,6 +5,7 @@ var levels = [
     InsertionSort,
     SelectionSort,
     MergeSort,
+    QuickSort,
 ]
 var level: ComparisonSort
 
@@ -27,7 +28,7 @@ func _ready():
     bottom_button.focus_neighbour_bottom = top_button.get_path()
     # If no last played level, autofocus first level
     if GlobalScene.get_param("level") == null:
-        top_button.grab_focus()
+        bottom_button.grab_focus()
 
 func _on_Button_focus_changed():
     """Initialize the preview section."""

views/array_view.gd

@@ -25,7 +25,7 @@ func _process(delta):
     for i in range(level.array.size):
         rects[i].rect_scale.y = -1 # HACK: Override scale to bypass weird behavior
         rects[i].color = level.get_effect(i)
-        rects[i].rect_size.y = rect_size.y * level.array.get(i) / level.array.size
+        rects[i].rect_size.y = rect_size.y * level.array.at(i) / level.array.size
 
 func _on_Level_mistake():
     """Flash the border red on mistakes."""