[ad_1]
\n<\/p>\n
Based on what you’re saying, I think you’re trying to get some data like this:<\/p>\n
input = [\"hello world\", \"foo bar\", \"red cat\"]\ndata_wanted = {\n \"foo\" : 1,\n \"hello\" : 0,\n \"cat\" : 2,\n \"world\" : 0,\n \"red\" : 2\n \"bar\" : 1\n}\n<\/code><\/pre>\nSo what you should be doing is adding the words<\/em> as keys to a dictionary, and have their values be the index of the substring in strlist<\/code> in which they are located.<\/p>\ndef locateWords(strlist):\nd = {}\nfor i, substr in enumerate(strlist): # gives you the index and the item itself\n for word in substr.split()\n d[word] = i\nreturn d\n<\/code><\/pre>\nIf the word occurs in more than one string in strlist<\/code>, you should change the code to the following:<\/p>\ndef locateWords(strlist):\nd = {}\nfor i, substr in enumerate(strlist):\n for word in substr.split()\n if word not in d:\n d[word] = [i]\n else:\n d[word].append(i)\nreturn d\n<\/code><\/pre>\nThis changes the values to lists, which contain the indices of the substrings in strlist<\/code> which contain that word.<\/p>\nSome of your code’s problems explained<\/h3>\n\n{}<\/code> is not a set, it’s a dictionary.<\/li>\nbreak<\/code> forces a loop to terminate immediately – you didn’t want to end the loop early because you still had data to process.<\/li>\nd.update(index)<\/code> will give you a TypeError: 'int' object is not iterable<\/code>. This method actually takes an iterable object and updates the dictionary with it. Normally you would use a list of tuples for this: [(\"foo\",1), (\"hello\",0)]<\/code>. It just adds the data to the dictionary.<\/li>\n- You don’t normally want to use
d.__setitem__<\/code> (which you typed wrong anyway). You’d just use d[key] = value<\/code>.<\/li>\n- You can iterate using a “for each” style loop instead, like my code above shows. Looping over the range means you are looping over the indices. (Not exactly a problem, but it could lead to extra bugs if you’re not careful to use the indices properly).<\/li>\n<\/ol>\n
It looks like you are coming from another programming language in which braces indicate sets and there is a keyword which ends control blocks (like if, fi<\/code>). It’s easy to confuse syntax when you’re first starting – but if you run into trouble running the code, look at the exceptions you get and search them on the web!<\/p>\nP.S.<\/em> I’m not sure why you wanted a set – if there are duplicates, you probably want to know all of their locations, not just the first or the last one or anything in between. Just my $0.02.<\/p>\n<\/p><\/div>\n