Почему возникает эта ошибка?

Question

[Twelfth Doctor]

Здравствуйте! Почему возникает ошибка Local variable record might be referenced before assignment на последней строке ?

def fromjson (json):
    data = json.loads(json)
    if data.type !='SRV' and data.type!='MX':
        record = f'{data.host} IN {data.type} {data.content}'
    if data.type=='MX':
        record = f'{data.host} IN MX {data.priority} {data.content}'
    if data.type=='SRV':
        record = f"_{data.service}._{data.proto}.{data.name}. IN SRV {data.priority} {data.weight} {data.port} {data.target}"
    return record

Answer 1

[longclaps]

Наши телепаты полагают, что ваша data не имеет всех тех атрибутов, которых от неё хотят ваши f-стороки.
Если у вас есть руки, то, отрубая ими кусочки f-сторок, вы сможете локализовать неисправность.
Кстати, давно не видал столь уродского кода.

Comments

[Twelfth Doctor]

А чем он уродский?

[longclaps]

verdex:

def fromjson (json):
    data = json.loads(json)
    if data.type=='MX':
         return f'{data.host} IN MX {data.priority} {data.content}'
    if data.type=='SRV':
         return f"_{data.service}._{data.proto}.{data.name}. IN SRV {data.priority} {data.weight} {data.port} {data.target}"
    return f'{data.host} IN {data.type} {data.content}'

[longclaps]

verdex: Эх, я прощёлкал:

def fromjson (json):
    data = json.loads(json) # да это же пи.дец

Что это такое - даже подумать страшно. Может, имено тут и ошибка, а я зря повёлся на слова о последней строке.

[maxfox]

data = json.loads(json)
Это просто шедевр! Как, по вашей версии, это должно работать?

[Twelfth Doctor]

maxfox: уже понял..

def fromjson (jdata):
    data = json.loads(jdata)
    if data.type=='MX':
         return f'{data.host} IN MX {data.priority} {data.content}'
    if data.type=='SRV':
         return f"_{data.service}._{data.proto}.{data.name}. IN SRV {data.priority} {data.weight} {data.port} {data.target}"
    return f'{data.host} IN {data.type} {data.content}'

Answer 2

[nonname]

Если все if'ы ложные, то что вернёт return record? Вот на это и ругается.