Member 14194093 Ответов: 2

Почему он возвращает false в подпрограмме


c = (3,3)
d = (3,5)

bondy = [4]
bondx = [3]

def adjacentnode(nodea ,nodeb):
    if nodea[0] == nodeb[0] and nodea[1] == nodeb[1]+2 and nodea[1]+1 in bondy:
        adjacent = True
    elif nodea[0] == nodeb[0] and nodea[1] == nodeb[1]-2 and nodea[1]-1 in bondy:
        adjacent = True
    elif nodea[1] == nodeb[1] and nodea[0] == nodeb[0]+2 and nodeb[0]+1 in bondx:
        adjacent = True
    elif nodea[1] == nodeb[1] and nodea[0] == nodeb[0]-2 and nodeb[0]-1 in bondx:
        adjacent = True
    else:
        adjacent = False
    return adjacent


print (adjacentnode((c),(d)))


Что я уже пробовал:

конечно, c и d смежны и должны возвращать true

2 Ответов

Рейтинг:
2

Patrice T

Сначала измените свой код для печати частичных результатов выполнения условий

def adjacentnode(nodea ,nodeb):
    print ("First consition")
    print (nodea[0] == nodeb[0])
    print (nodea[1] == nodeb[1]+2)
    print (nodea[1]+1 in bondy)
    ... # and so on for other conditions
    if nodea[0] == nodeb[0] and nodea[1] == nodeb[1]+2 and nodea[1]+1 in bondy:
        adjacent = True
    elif nodea[0] == nodeb[0] and nodea[1] == nodeb[1]-2 and nodea[1]-1 in bondy:
        adjacent = True
    elif nodea[1] == nodeb[1] and nodea[0] == nodeb[0]+2 and nodeb[0]+1 in bondx:
        adjacent = True
    elif nodea[1] == nodeb[1] and nodea[0] == nodeb[0]-2 and nodeb[0]-1 in bondx:
        adjacent = True
    else:
        adjacent = False
    return adjacent

и он покажет вам, где код терпит неудачу.
Цитата:
конечно, c и d смежны и должны возвращать true

Ваш код ведет себя не так, как вы ожидаете, или вы не понимаете, почему !

Существует почти универсальное решение: запускайте свой код на отладчике шаг за шагом, проверяйте переменные.
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не знает, что должен делать ваш код, он не находит ошибок, он просто помогает вам, показывая, что происходит. Когда код не делает того, что ожидается, вы близки к ошибке.
Чтобы увидеть, что делает ваш код: просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.

Отладчик - Википедия, свободная энциклопедия[^]

Освоение отладки в Visual Studio 2010 - руководство для начинающих[^]
Базовая отладка с помощью Visual Studio 2010 - YouTube[^]

27.3. ПДБ — отладчика Python — питон 3.6.1 документации[^]
Отладка в Python | Python покоряет Вселенную[^]
pdb – интерактивный отладчик - Python модуль недели[^]

Отладчик здесь только для того, чтобы показать вам, что делает ваш код, и ваша задача-сравнить его с тем, что он должен делать.


Рейтинг:
0

Richard MacCutchan

Добавьте следующие две строки в начале вашей функции, и вы сразу увидите, что не так:

print("nodea[0]:", nodea[0], "nodeb[0]:", nodeb[0])
print("nodea[1]:", nodea[1], "nodeb[1]+2:", nodeb[1]+2)


Member 14194093

разве это не имеет значения, так как он перейдет в блок if с -2

Member 14194093

деф adjacentnode(узел ,узел B):
если nodea[0] == nodeb[0] и nodea[1] == nodeb[1]+2:
рядом = правда
elif nodea[0] == nodeb[0] и nodea[1] == nodeb[1]-2:
рядом = правда
elif nodea[1] == nodeb[1] и nodea[0] == nodeb[0]+2 :
рядом = правда
elif nodea[1] == nodeb[1] и nodea[0] == nodeb[0]-2:
рядом = правда
еще:
рядом = ложь
возвращение прилегающих

только при добавлении nodea[1]+1 В оператор bondy: он перестает работать

Richard MacCutchan

Но ваш код имеет nodea[1]-1 in bondy что является ложью. Я понятия не имею, какую реальную проблему вы пытаетесь решить здесь, но если ваши правила не будут специфичны для значений в двух списках, у вас всегда будут проблемы.