NameError: имя 'высота' не определено

Question

NameError: имя 'высота' не определено

1

У меня есть этот код gps, который получает данные gps и записывает в журнал:

#!/usr/bin/python

from systemd import journal
import gps
import time
import threading
import datetime

# Listen on port 2947 (gpsd) of localhost
session = gps.gps("localhost", "2947")
session.stream(gps.WATCH_ENABLE | gps.WATCH_NEWSTYLE)

while True:
    try:
        report = session.next() # Wait for a 'TPV' report and display 
the current time

        # To see all report data, uncomment the line below
        #print report

        if report['class'] == 'TPV':
            if hasattr(report, 'time'):
                timestamp = (time.time()*1000)
                #print timestamp

            if hasattr(report, 'lat'):
                latitude = report.lat
                #print latitude

            if hasattr(report, 'lon'):
                longitude = report.lon
                #print longitude    

            if hasattr(report, 'alt'):
                altitude = report.alt
                #print altitude

        else:
            timestamp = (time.time()*1000)
            latitude = 0
            longitude = 0 
            altitude = 0

        journal.send(
        channel = 'gps',
        priority = journal.Priority.INFO,
        timestamp = "%f" % (timestamp),
        latitude = "%f" % (latitude),
        longitude = "%f" % (longitude), 
        altitude = "%f" % (altitude), 
        )

except KeyError:
    pass
except KeyboardInterrupt:
    quit()
except StopIteration:
    session = None
    print "GPSD has terminated"

И я получаю эту ошибку:

Traceback (most recent call last):
  File "gps-messi.py", line 57, in <module>
    altitude = "%f" % (altitude),
NameError: name 'altitude' is not defined

Смешная часть - это код, который отлично работает отлично, и иногда он дает мне эту ошибку. Я не могу понять, что я должен сделать, чтобы он работал правильно все время. Это имеет какое-то отношение к ней?

Sohil Mehta 28 авг. 2018, в 16:55

Источник

1

Вы должны всегда проверять, определена ли переменная, прежде чем пытаться отформатировать строку. У вас нет гарантии, что переменная будет определена, когда вы попытаетесь получить ее из библиотеки.
scrappedcola 28 авг. 2018, в 14:45
2

Возможно, это опечатка? if hasatr(report, 'alt'): должно быть hasattr ?
Matt Cremeens 28 авг. 2018, в 14:47
1

Кстати, ваше else принадлежит только последнему, if .
Psytho 28 авг. 2018, в 14:49
1

Почему вы перезаписываете ранее указанные метки времени, широты, долготы и высоты, если нет атрибута speed ? Похоже, с этим кодом много проблем с отступами.
chepner 28 авг. 2018, в 14:49
0

И почему вы игнорируя report.time , если он присутствует?
chepner 28 авг. 2018, в 14:52
0

hasattr - это просто оболочка для getattr которая ловит исключение. Просто используйте getattr(report, 'time', time.time()*1000) и т. Д.
chepner 28 авг. 2018, в 14:55
0

@MattCremeens Извините, это была опечатка здесь. Но код в порядке. В коде это только hasattr.
Sohil Mehta 28 авг. 2018, в 14:58
0

@Psytho Это была ошибка. Я сделал редактирование. Посмотрите на код еще раз.
Sohil Mehta 28 авг. 2018, в 14:58
0

@chepner Да, я сожалею об этом. Это ошибка здесь на форуме. Но это все нормально на моем Пи. Я не использую время GPS, потому что мне нужно только регистрировать системное время. Следовательно, время. Время
Sohil Mehta 28 авг. 2018, в 15:01

Показать ещё 7 комментариев

Теги:

python

5 ответов

3

Кажется, что отчет из сеанса не имеет атрибута с именем "высота" в зависимости от случаев. Если атрибут "alt" существует, он создает переменную высоту, позволяющую коду работать. Если атрибут "alt" не существует, код не будет работать. Инициируйте переменные перед операторами if, и он будет работать нормально.

Wesgur 28 авг. 2018, в 11:48

2

Я согласен с @wesgur вы инициируя alititude в if statemtent, но бывают ситуации, когда, что if выражение условия не будут выполнены, и программа не имеет определение altitude

vash_the_stampede 28 авг. 2018, в 12:41

1

Переместить блок

    timestamp = (time.time()*1000)
    latitude = 0
    longitude = 0 
    altitude = 0

before if report['class'] == 'TPV': branch. Таким образом, вы всегда будете инициализированы четырьмя переменными.

Psytho 28 авг. 2018, в 13:07

0

Так что лучше? Инициализация переменных перед оператором if или глобальная инициализация переменных?
Sohil Mehta 28 авг. 2018, в 15:16
0

Что делать, if report['class'] == 'TPV' равен true, но нет атрибута "alt" ? altitude остается неинициализированной в вашем текущем коде, как вы уже видели.
Psytho 28 авг. 2018, в 15:18
0

Да, я понял это. Но где я должен инициализировать? до или лучше?
Sohil Mehta 28 авг. 2018, в 15:27
0

Я не думаю, что в этом случае будет какая-то разница.
Psytho 28 авг. 2018, в 15:31
0

ОК, круто. Большое спасибо Психо.
Sohil Mehta 28 авг. 2018, в 15:32

Показать ещё 3 комментария

0

Большое спасибо всем. Вот как выглядит мой код сейчас. Я тестировал его, и это кажется прекрасным. Но в случае, если я все еще что-то сползаю, может кто-то просто проверить отступы строки journal.send? Если то, что я сделал правильно (я изменил имена некоторых переменных и добавил скорость):

#!/usr/bin/python

from systemd import journal
import gps
import time
import threading
import datetime

# Listen on port 2947 (gpsd) of localhost
session = gps.gps("localhost", "2947")
session.stream(gps.WATCH_ENABLE | gps.WATCH_NEWSTYLE)

while True:
    try:
        report = session.next() # Wait for a 'TPV' report and display the current time  
        # To see all report data, uncomment the line below
        #print report

        timestamp = 0        
        latitude = 0
        longitude = 0 
        altitude = 0
        speed = 0

        if report['class'] == 'TPV':

            if hasattr(report, 'time'):
                timestamp = (time.time()*1000)
                #print timestamp

            if hasattr(report, 'lat'):
                latitude = report.lat
                #print latitude

            if hasattr(report, 'lon'):
                longitude = report.lon
                #print longitude  

            if hasattr(report, 'alt'):
                altitude = report.alt
                #print altitude

            if hasattr(report, 'speed'):
                speed = report.speed
                #print speed

        journal.send(
        channel = 'gps',
        priority = journal.Priority.INFO,
        timestamp = "%f" % (timestamp),
        lat_deg = "%f" % (latitude),
        lon_deg = "%f" % (longitude), 
        alt_m = "%f" % (altitude), 
        speed_mps = "%f" % (speed),
        )


    except KeyError:
        pass
    except KeyboardInterrupt:
        quit()
    except StopIteration:
        session = None
        print "GPSD has terminated"

Sohil Mehta 28 авг. 2018, в 13:24

Ещё вопросы

Вы должны всегда проверять, определена ли переменная, прежде чем пытаться отформатировать строку. У вас нет гарантии, что переменная будет определена, когда вы попытаетесь получить ее из библиотеки.
Возможно, это опечатка? if hasatr(report, 'alt'): должно быть hasattr ?
Кстати, ваше else принадлежит только последнему, if .
Почему вы перезаписываете ранее указанные метки времени, широты, долготы и высоты, если нет атрибута speed ? Похоже, с этим кодом много проблем с отступами.
И почему вы игнорируя report.time , если он присутствует?
hasattr - это просто оболочка для getattr которая ловит исключение. Просто используйте getattr(report, 'time', time.time()*1000) и т. Д.
@MattCremeens Извините, это была опечатка здесь. Но код в порядке. В коде это только hasattr.
@Psytho Это была ошибка. Я сделал редактирование. Посмотрите на код еще раз.
@chepner Да, я сожалею об этом. Это ошибка здесь на форуме. Но это все нормально на моем Пи. Я не использую время GPS, потому что мне нужно только регистрировать системное время. Следовательно, время. Время
Так что лучше? Инициализация переменных перед оператором if или глобальная инициализация переменных?
Что делать, if report['class'] == 'TPV' равен true, но нет атрибута "alt" ? altitude остается неинициализированной в вашем текущем коде, как вы уже видели.
Да, я понял это. Но где я должен инициализировать? до или лучше?
Я не думаю, что в этом случае будет какая-то разница.

Matt Cremeens · Accepted Answer · 2018-08-28T12-14-00.000Z

Я предполагаю, что if hasattr(report, 'alt'): не разрешает True, тогда

if hasattr(report, 'alt'):
    altitude = report.alt

не будет присваивать что-либо altitude и там, когда вы доберетесь досюда

journal.send(
    channel = 'gps',
    priority = journal.Priority.INFO,
    timestamp = "%f" % (timestamp),
    latitude = "%f" % (latitude),
    longitude = "%f" % (longitude), 
    altitude = "%f" % (altitude), 
    )

вы по существу используете altitude прежде чем она будет назначена. Чтобы этого избежать, вы можете инициализировать altitude до некоторой величины, прежде чем перейти к этой точке с помощью

altitude=0

Это позволит убедиться, что оно назначено до его использования, и вы не должны получать эту ошибку.

Большое спасибо Мэтт. Так что лучше? Инициализация переменных перед оператором if или глобальная инициализация переменных?
Я думаю, что достаточно инициализировать перед оператором if. Может быть, сразу после назначения report .
Фактически, вы, вероятно, должны сделать это для всех переменных, используемых в journal.send .
Рад помочь. Я конечно получил много помощи здесь. Всего наилучшего.