Python for loop counter list - counter not working

Question

Could someone please help me troubleshoot my code. I have two lists.

    A = [['2925750', ' Everything he mentioned, I could have evaluated on my own'], ['2925750', ' I do wish he could have shown us more at this point that could have set the fox apart.']]

B = ['mentioned','evaluated','fox','wish']

The goal is to append to list A the number of times any item in B is present in a sentence for A.

The result should be something like:

[(['2925750', ' Everything he mentioned, I could have evaluated on my own'], 0), (['2925750', ' I do wish he could have shown us more at this point that could have set the Equinox apart.'], 0)]

The problem is my count is zero.

Below is my code. Thank you in advance:

Y = []


##Find Matches
for sent in A:
  for sent in B:
      Y.append((sent, sum(sent.count(col) for col in B)))

Thank you.

shx2 · Accepted Answer · 2014-01-02 15:34:27Z

5

The variable sent in for sent in B overshadows the variable in for sent in A. Or, to be more exact, it assigns to the same name (same variable).

You should rename one of them.

Also, note you already iterate over B inside sum. In the inner loop, you probably meant to iterate over each of the lists in A.

for lst in A:
  for sent in lst:
      Y.append((sent, sum(sent.count(col) for col in B)))

answered Jan 2, 2014 at 15:34

shx2

64.8k17 gold badges139 silver badges166 bronze badges

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

Comments

Toalp · Accepted Answer · 2018-12-28 16:19:27Z

2

you used 'sent' twice.
You don't need a loop for B.
'A' is a list of lists.

My fix:

Y = []
A = [['2925750', ' Everything he mentioned, I could have evaluated on my own'], ['2925750', ' I do wish he could have shown us more at this point that could have set the fox apart.']]
B = ['mentioned','evaluated','fox','wish']

for sent in A:
    o = 0
    for i in sent:
        o +=sum(i.count(col) for col in B)
    Y.append((sent, o))

Y:

[(['2925750', ' Everything he mentioned, I could have evaluated on my own'], 2), (['2925750', ' I do wish he could have shown us more at this point that could have set the fox apart.'], 2)]

edited Dec 28, 2018 at 16:19

answered Jan 2, 2014 at 15:44

Toalp

3621 gold badge5 silver badges16 bronze badges

1 Comment

BlackHat Over a year ago

Worked perfectly. Thank you so much!

kylieCatt · Accepted Answer · 2014-01-02 15:35:34Z

2

You can't use sent as the iterator for both loops as it is still in scope.

answered Jan 2, 2014 at 15:35

kylieCatt

11k5 gold badges45 silver badges55 bronze badges

Collectives™ on Stack Overflow

Python for loop counter list - counter not working

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related