Varable parsing error with python
Dave Angel
davea at davea.name
Tue Feb 10 15:40:45 EST 2015
On 02/10/2015 01:24 PM, Denis McMahon wrote:
> On Tue, 10 Feb 2015 18:39:42 +1100, Chris Angelico wrote:
>
>> On Tue, Feb 10, 2015 at 6:30 PM, OmPs <torque.india at gmail.com> wrote:
>>> def _getPackgeVersion(xmlfile, p):
>>> package = str(p)
>>> if isinstance(fpmdict["application"]["package"], list):
>>> for i in fpmdict["application"]["package"]:
>>> if i["@name"] == p:
>>> _pkgVersion = i["version"]
>>> else:
>>> _pkgversion = fpmdict["application"]["package"]["version"]
>>> return _pkgVersion
>>
>> One of your branches doesn't have a return statement in it, so Python
>> just returns None. You may want to unindent that return statement one
>> level.
>
> Even if he unindents the return, it will still return None if it doesn't
> find an i such that i["@name"] == p, as _pkgVersion only gets set if such
> a match is found.
But if _pkgVersion weren't set, he'd be getting UnboundLocalError
exception, not a None value.
>
--
DaveA
More information about the Python-list
mailing list