Debug Labs typowe błędy rysowania

def on_drag(event):
    # 'znana' logika: im wyżej na ekranie, tym WIĘCEJ Y
    if event.y > start_y:
        status.config(text=„Moving UP”)  # nieprawidłowe dla Canvas!

def on_drag(event):
    if event.y > start_y:
        status.config(text=„Poruszamy się w DÓŁ”)  # Prawdziwe: Y Wznosi się w dół

def on_press(event):
    pass  # zapomnieli zapisać event.x, event.y

def on_drag(event):
    canvas.create_line(start_x, start_y, event.x, event.y)  #start_x nieokreślone

def on_press(event):
    global start_x, start_y
    start_x, start_y = event.x, event.y

canvas.bind("<Motion>", on_drag)   # wyzwala się przy KAŻDYM ruchu myszy

canvas.bind("<B1-Motion>", on_drag)   # tylko przy wciśniętym lewym przycisku

def on_drag(event):
    canvas.create_oval(
        event.x - 2, event.y - 2, event.x + 2, event.y + 2,
        fill="black",
    )

def on_drag(event):
    canvas.create_line(last_x, last_y, event.x, event.y, width=3, capstyle=tk.ROUND)
    # last_x, last_y aktualizowane po każdym odcinku

def on_drag(event):
    canvas.delete(preview_id)
    preview_id = canvas.create_rectangle(start_x, start_y, event.x, event.y)

def on_drag(event):
    canvas.coords(preview_id, start_x, start_y, event.x, event.y)

def on_press(event):
    canvas.create_rectangle(event.x, event.y, event.x, event.y, dash=(4, 2))
    # id nie zapisany — preview_id nigdzie się nie pojawił

def on_drag(event):
    canvas.coords(preview_id, ...)  # NameError: preview_id nie jest zdefiniowana

def on_press(event):
    global preview_id
    preview_id = canvas.create_rectangle(event.x, event.y, event.x, event.y, dash=(4, 2))

def get_bounds():
    return start_x, start_y, current_x, current_y  # z wyłączeniem kierunku

left, top, right, bottom = get_bounds()
if left > right:
    raise ValueError(„to niemożliwe”)  # ale przeciąganie w górę-i-lewo właśnie to daje!

left, top, right, bottom = normalize_bounds(start_x, start_y, current_x, current_y)
# jest teraz left <= right oraz top <= bottom gwarantowane, niezależnie od kierunku

shapes = []
item_id = canvas.create_rectangle(10, 10, 50, 50)
shapes.append("rectangle")
print(shapes[item_id])   #IndexError jeśli item_id więcej niż len(shapes) - 1

shapes_by_id = {}
item_id = canvas.create_rectangle(10, 10, 50, 50)
shapes_by_id[item_id] = "rectangle"   # słownik — id jako KLUCZ, a nie indeks

def undo(self):
    item_id = self.undo_stack.pop()   # zakłada JEDEN id na akcję
    self.canvas.delete(item_id)

def undo(self):
    shapes = self.undo_stack.pop()   # lista postacie JEDNEGO działania
    del self.document[len(self.document) - len(shapes):]
    self.render_document()

def _commit_action(self, shapes):
    self.document.extend(shapes)
    self.undo_stack.append(shapes)
    # zapomniałem self.redo_stack.clear()

def _commit_action(self, shapes):
    self.document.extend(shapes)
    self.undo_stack.append(shapes)
    self.redo_stack.clear()

def clear_canvas(self):
    self.canvas.delete("all")
    self.document.clear()
    #undo_stack i redo_stack pozostają nietknięte!

def clear_canvas(self):
    self.document.clear()
    self.undo_stack.clear()
    self.redo_stack.clear()
    self.render_document()

def choose_custom_color(self):
    _rgb, hex_color = colorchooser.askcolor()
    self.set_color(hex_color)   # nie testowałem hex_color na None!

def choose_custom_color(self):
    _rgb, hex_color = colorchooser.askcolor()
    if hex_color is not None:
        self.set_color(hex_color)

def save_document(self):
    path_str = filedialog.asksaveasfilename()
    Path(path_str).write_text(...)   # nie sprawdziłem pustego sznurka!

def save_document(self):
    path_str = filedialog.asksaveasfilename()
    if not path_str:
        return
    Path(path_str).write_text(...)

def load_from_path(self, path):
    data = json.loads(path.read_text(encoding="utf-8"))
    self.document = [Shape.from_dict(item) for item in data["items"]]
    #forgot self.render_document()!

def load_from_path(self, path):
    data = json.loads(path.read_text(encoding="utf-8"))
    shapes = [Shape.from_dict(item) for item in data["items"]]
    self.document = shapes
    self.undo_stack.clear()
    self.redo_stack.clear()
    self.render_document()   # Bez tej linii ekran nie będzie wiedział o pobraniu
    # (pełna wersja z uszkodzonym przetwarzaniem plików znajduje się w sekcji 18.30)

def load_from_path(self, path):
    data = json.loads(path.read_text(encoding="utf-8"))
    self.document = [Shape.from_dict(i) for i in data["items"]]  # już zastąpiono!
    self.undo_stack.clear()
    self.render_document()   # ... I tylko tutaj wpadamy w zły kolor

def load_from_path(self, path):
    try:
        data = json.loads(path.read_text(encoding="utf-8"))
        shapes = [Shape.from_dict(i) for i in data["items"]]
        for shape in shapes:
            self.canvas.winfo_rgb(shape.color)
    except (json.JSONDecodeError, KeyError, TypeError, ValueError, tk.TclError) as exc:
        messagebox.showerror(„Plik nie mógł zostać otwarty”, str(exc))
        return
    self.document = shapes
    self.render_document()

#standby: jeśli rozciągniesz okno, stare kształty będą się z nim zbliżać
canvas.grid(row=1, column=0, sticky="nsew")
# ... Ale nikt nie przelicza istniejących współrzędnych create_line/rectangle/oval

# Fair Model: Powiększone okno daje WIĘCEJ MIEJSCA na nowe kształty,
# a nie rozciąga już istniejące — jeśli potrzebny ostatni, współrzędne
# musiałaby być przeliczana ręcznie przy zmianie rozmiaru.