`[]': Can't convert String into Integer (TypeError) Ruby script

Question

I'm writing a simple function in a ruby script to get the ID attribute from some JSON data.

def getObjectTypeId()
    objects = post(@server.oS.getAllTypeObjectCounts())
    myObject = objects.find_all{|xObjectCountInfo| xObjectCountInfo['ObjectInfo']['ObjectTypeID'] == 'myObjectTypeID'}
    puts myObject
    @objectTypeId = myObject['ObjectInfo']['ID']
end

The error is happening on the final line of the function. As you can see, I'm printing myObject to the console to verify that it's a hash and not an array (which is the most common issue I've seen that causes this error). Here's a snippet of the output:

{"_class"=>"XObjectCountInfo", "ObjectInfo"=>{"_class"=>"XObjectInfo", "ID"=>"10102", "ObjectTypeID"=>"myObjectTypeID"}}

What could be causing me to get the "can't convert String to Integer" type error?

Please use snake_case instead of camelCase when naming functions an variables in Ruby. This is the standard. — Dan Grahn
– Dan Grahn, Commented Dec 2, 2013 at 20:07

Arup Rakshit · Accepted Answer · 2013-12-02 20:13:19Z

3

Enumerable#find_all gives an array when used with block. Your myObject is an array of size 1, I am quite sure. You used puts method to print it, so you are seeing the only one element hash of the array. Use p myObject, you will get to see that the object myObject is an one element array. Your modified last statement @objectTypeId = myObject['ObjectInfo']['ID'], will be @objectTypeId = myObject.map{|h| h['ObjectInfo']['ID'] }

I used Enumerable#map as, #find_all returns an array of collection. This time you have only one element in the array,but any time you can have more than one element. In this case #map would be a good choice I think.

edited Dec 2, 2013 at 20:13

answered Dec 2, 2013 at 20:07

Arup Rakshit

119k30 gold badges270 silver badges328 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

mechanicalfish · Accepted Answer · 2013-12-02 20:08:09Z

0

If the exception is really raised on the last line of the method you provided, either myObject or my myObject['ObjectInfo'] is an array not a hash.

I'm suspecting the myObject as it is returned by find_all

answered Dec 2, 2013 at 20:08

mechanicalfish

12.9k3 gold badges49 silver badges42 bronze badges

Collectives™ on Stack Overflow

`[]': Can't convert String into Integer (TypeError) Ruby script

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related