AttributeError: 'str' object has no attribute 'get'

Having an error I'm not quite sure how to solve, have a nested dictionary where the last item does not contain the asked get and gives an error, Not sure how to fix it.

Very basic code calling on

 for keys in interfaces: counters = interfaces[keys].get(u'interfaceCounters', {}) print keys, "inOctets:", counters.get(u'inOctets', {}), "outOctets:", counters.get(u'outOctets', {})

Here is the output:

 Management1 inOctets: 0 outOctets: 0 Ethernet2 inOctets: 561926946 outOctets: 49932066 Ethernet3 inOctets: 561926408 outOctets: 49932764 Ethernet1 inOctets: 319314970 outOctets: 264840242 Ethernet6 inOctets: 561897402 outOctets: 49930884 Ethernet7 inOctets: 561678430 outOctets: 50180896 Ethernet4 inOctets: 561927603 outOctets: 49931337 Ethernet5 inOctets: 561926383 outOctets: 49932935 Vlan1 inOctets: Traceback (most recent call last): File "ex1.py", line 36, in <module> print keys, "inOctets:", counters.get(u'inOctets', '0'), "outOctets:", counters.get(u'outOctets', '0') AttributeError: 'str' object has no attribute 'get'

Heres a tail end of interfaces:

u'Management1': {u'autoNegotiate': u'off', u'bandwidth': 1000000000, u'burnedInAddress': u'52:54:00:01:02:d3', u'description': u'', u'duplex': u'duplexFull', u'forwardingModel': u'routed', u'hardware': u'ethernet', u'interfaceAddress': [], u'interfaceCounters': {u'inBroadcastPkts': 0, u'inDiscards': 0, u'inMulticastPkts': 0, u'inOctets': 0, u'inUcastPkts': 0, u'inputErrorsDetail': {u'alignmentErrors': 0, u'fcsErrors': 0, u'giantFrames': 0, u'runtFrames': 0, u'rxPause': 0, u'symbolErrors': 0}, u'linkStatusChanges': 2, u'outBroadcastPkts': 0, u'outDiscards': 0, u'outMulticastPkts': 0, u'outOctets': 0, u'outUcastPkts': 0, u'outputErrorsDetail': {u'collisions': 0, u'deferredTransmissions': 0, u'lateCollisions': 0, u'txPause': 0}, u'totalInErrors': 0, u'totalOutErrors': 0}, u'interfaceStatistics': {u'inBitsRate': 0.0, u'inPktsRate': 0.0, u'outBitsRate': 0.0, u'outPktsRate': 0.0, u'updateInterval': 300.0}, u'interfaceStatus': u'disabled', u'lastStatusChangeTimestamp': 1245724638.6762877, u'lineProtocolStatus': u'down', u'mtu': 1500, u'name': u'Management1', u'physicalAddress': u'52:54:00:01:02:d3'}, u'Vlan1': {u'bandwidth': 0, u'burnedInAddress': u'52:54:00:01:37:37', u'description': u'', u'forwardingModel': u'routed', u'hardware': u'vlan', u'interfaceAddress': [{u'broadcastAddress': u'255.255.255.255', u'primaryIp': {u'address': u'10.220.88.31', u'maskLen': 24}, u'secondaryIps': {}}], u'interfaceStatus': u'connected', u'lastStatusChangeTimestamp': 1250568992.891673, u'lineProtocolStatus': u'up', u'mtu': 1500, u'name': u'Vlan1', u'physicalAddress': u'52:54:00:01:37:37'}}

2 Answers

If some of your interfaceCounters keys reference a string instead of a nested dictionary, just use exception handling to ignore those:

for keys in interfaces: counters = interfaces[keys].get(u'interfaceCounters', {}) try: print keys, "inOctets:", counters.get(u'inOctets', {}), "outOctets:", counters.get(u'outOctets', {}) except AttributeError: # counters is not a dictionary, ignore and move on pass

This is the ask forgiveness principle; if most of your entries do have the .get() method this is simply faster than the look before you leap principle, where you test if the method is available.

How about simply checking if your dictionary value does provide the method/attribute you need?

 for keys in interfaces: counters = interfaces[keys].get(u'interfaceCounters', {}) if hasattr(counters, 'get'): # Only print if counters supports `get` print keys, "inOctets:", counters.get(u'inOctets', {}), "outOctets:", counters.get(u'outOctets', {})