У меня проблема с циклом python, который дважды выполняет одну строку кода

Я пытаюсь решить проблему теории графов, используя алгоритм обхода в ширину, где граф представлен в виде сетки. Сама проблема связана с местным соревнованием по программированию, поэтому я не могу поделиться подробностями.

Ввод предоставляется мне (опять же, я не могу им поделиться) в виде txt с некоторыми числовыми значениями в каждой строке, которые представляют размер сетки, положение начальной и конечной точек в этой сетке. Кроме того, есть опасные места, которых мне нужно избегать. У меня нет проблем с разбором ввода. После этого следует код ниже. Некоторые из переменных ниже являются кортежами из-за специфического ввода, и я подумал, что кортежи могут быть хорошим представлением координат x, y на сетке.

Это мой код:

from collections import deque

GRID = 3 #size of the grid
START = (1, 1) #start position
END = (0, 0) #finish position
SAFE = False #boolean, tells if I could delete one item from noareas
noareas = [ (2, 1), (1, 2), (0, 1) ] #position of dangerous areas I have to avoid


def gridmover(grid, x, y, stop_areas):
    dx = [-1, +1, 0, 0]
    dy = [0, 0, +1, -1]
    neighbours = []

    for i in range(4):
        xx = x + dx[i]
        yy = y + dy[i]
        if xx < 0 or yy < 0: continue
        if xx >= grid or yy >= grid: continue
        if (xx, yy) in stop_areas: continue
        neighbours.append((xx,yy))

    return neighbours

def breadthfirstsearch_custom(graph, start, end, stopareas, safe):
    queue = deque([ start ])
    visited = { start }
    i = 0

    while queue:
        current = queue.popleft() #this line executes twice
        path = [ current ]
        i += 1
        print(f"{i} This is current {current} and its type: {type(current)}")

        if current == end:
            return path

        if safe is True: stopareas.pop(0)
        neighbours = gridmover(graph, current[0], current[1], stopareas)

        for neighbour in neighbours:
            if neighbour not in visited:
                queue.extend(neighbour)
                path.extend(neighbour)
                visited.add(neighbour)

    return path

print(breadthfirstsearch_custom(GRID, START, END, noareas, SAFE))

Ожидаемый результат должен быть списком кортежей, представляющих путь от начала до конца.

Это я вижу как выход:

1 This is current (1, 1) and its type: <class 'tuple'>
2 This is current 1 and its type: <class 'int'>
Traceback (most recent call last):
  File "/Users/home/repos/kisaievla_github/fiks9/kolo1/t1_plaz/playgrid.py", line 52, in <module>
    print(breadthfirstsearch_custom(GRID, START, END, noareas, PRKNO))
  File "/Users/home/repos/kisaievla_github/fiks9/kolo1/t1_plaz/playgrid.py", line 42, in breadthfirstsearch_custom
    neighbours = gridmover(graph, current[0], current[1], stopareas)
TypeError: 'int' object is not subscriptable

Я понимаю вывод выше, так как цикл while выполняется current = queue.popleft() дважды. Он извлекает кортеж из очереди (желаемое поведение), а затем из этого кортежа извлекает целое число (нежелательное поведение).

Я посылаю некоторое время, пытаясь найти проблему. Следовательно, оператор печати в моем коде. Странно то, что когда у меня разные значения для START, он ведет себя по-разному. Например, используя один и тот же код, но с разными переменными.

START = (0, 0)
END = (0, 0)

Дает это:

1 This is current (0, 0) and its type: <class 'tuple'>
[(0, 0)]

это действительно то, что я хочу, так как начало совпадает с концом, а путь к нему - это координаты.

Я действительно не знаю или не вижу решения здесь. Может дело даже не в самой петле, а в хлопке. Спасибо за ответ/комментарии заранее.

Каков ваш желаемый результат?

Himanshu Kawale 22.10.2022 20:58

Ваш код не работает, вы должны очистить его и обновить свой вопрос. Я даже не проверил вашу проблему с current = queue.popleft(). Но знайте это: Popleft возвращает элемент массива, и вы пытаетесь проиндексировать этот элемент, вот причина вашего сообщения об ошибке TypeError. Это int, а не список. Я бы посоветовал вам очистить свой код, создать переменные с понятными именами вместо кортежей с двумя значениями, использовать отдельные переменные, которые объясняют значения. Я думаю, это поможет вам запустить ваш код.

Ovski 22.10.2022 21:11

Добро пожаловать в Stack Overflow! Пожалуйста, возьмите тур . Так что это сайт вопросов и ответов, так что ваш вопрос? Ознакомьтесь с Как спросить , чтобы получить советы, например, как написать хороший заголовок. Пожалуйста, отредактируйте, чтобы внести ясность. Я сам нуб в теории графов, но если вы покажете желаемый результат, я смогу помочь. Я думаю, что queue.extend должно быть queue.append. Я попробовал это для себя, и код, по крайней мере, закончился, но я не знаю, правильный ли ответ.

wjandrea 22.10.2022 21:29

Пожалуйста, уточните вашу конкретную проблему или предоставьте дополнительную информацию, чтобы выделить именно то, что вам нужно. Как сейчас написано, трудно точно сказать, о чем вы спрашиваете.

Community 23.10.2022 05:16
Почему в Python есть оператор "pass"?
Почему в Python есть оператор "pass"?
Оператор pass в Python - это простая концепция, которую могут быстро освоить даже новички без опыта программирования.
Некоторые методы, о которых вы не знали, что они существуют в Python
Некоторые методы, о которых вы не знали, что они существуют в Python
Python - самый известный и самый простой в изучении язык в наши дни. Имея широкий спектр применения в области машинного обучения, Data Science,...
Основы Python Часть I
Основы Python Часть I
Вы когда-нибудь задумывались, почему в программах на Python вы видите приведенный ниже код?
LeetCode - 1579. Удаление максимального числа ребер для сохранения полной проходимости графа
LeetCode - 1579. Удаление максимального числа ребер для сохранения полной проходимости графа
Алиса и Боб имеют неориентированный граф из n узлов и трех типов ребер:
Оптимизация кода с помощью тернарного оператора Python
Оптимизация кода с помощью тернарного оператора Python
И последнее, что мы хотели бы показать вам, прежде чем двигаться дальше, это
Советы по эффективной веб-разработке с помощью Python
Советы по эффективной веб-разработке с помощью Python
Как веб-разработчик, Python может стать мощным инструментом для создания эффективных и масштабируемых веб-приложений.
1
4
56
1
Перейти к ответу Данный вопрос помечен как решенный

Ответы 1

Ответ принят как подходящий

Проблема в этом коде:

        for neighbour in neighbours:
            if neighbour not in visited:
                queue.extend(neighbour)
                path.extend(neighbour)
                visited.add(neighbour)

neighbours — это список кортежей, возвращенных из gridmover. Таким образом, neighbour будет кортежем.

Когда вы делаете list.extend(tuple), это не добавляет кортеж в список. Вместо этого он расширяет список отдельными членами кортежа. Если вы сделаете:

lst = [1,2,3]
lst.extend((4,5,6))

Тогда lst заканчивается как [1,2,3,4,5,6], НЕ [1,2,3,(4,5,6)].

Простой ответ заключается в том, что вы хотите добавить, а не расширить:

            queue.append(neighbour)
            path.append(neighbour)

Другие вопросы по теме