How to print the output only 1 time using python?

Question

The size of the tuples in the list is 39.

Here is my codes:

def joinphrase(joinph,i):
    length = len(joinph)
    res = ['%s %s' % (joinph[i][0], joinph[i + 1][0])
    for i in range(length)
       if i < length - 1 and joinph[i][2] == 'B-NP' and joinph[i + 1][2] == 'I-NP']
           return res 

def sentjoinphrase(joinph):
    return [joinphrase(joinph, i) for i in range(len(joinph))]


example = test_sents[0]
print (sentjoinphrase(example))

I want to join two strings from the different tuples with a condition and print the join output. But the output printed 39 times according to the size of the tuples in the list.

How to print the output only 1 time?

Could you show us the tuple you are referring to?

David Greydanus
– David Greydanus

2015-04-28 15:10:48 +00:00
Commented Apr 28, 2015 at 15:10 — David Greydanus
– David Greydanus, Commented Apr 28, 2015 at 15:10

Tui Popenoe · Accepted Answer · 2015-04-28 15:16:33Z

You have numerous other problems, but for your print statement, it looks like you are creating a list with

def sentjoinphrase(joinph):
       return [joinphrase(joinph, i) for i in range(len(joinph))]

that is the length of your initial phrase (for i in range(len(joinph)) Then in your joinphrase function, you overwrite the value of the argument i with

for i in range(length) # Where length is equal to the length of joinph

Because this is a list comprehension, it looks at only the internal i variable, and not the passed argument. This looks like it is just calling joinphrase the number of times the length of the phrase, in this case 39, because the passed argument is unecessary. So if you are getting the correct value back just from joinphrase (just multiple times), why don't you remove the argument i like so:

def joinphrase(phrase):
    phrase_len = len(phrase)
    return ['%s %s' % (phrase[i][0], joinph[i + 1][0])
       for i in range(phrase_len)
       if i < phrase_len - 1 and phrase[i][2] == 'B-NP' and phrase[i + 1][2] == 'I-NP']

Then you no longer need to call sentjoinphrase as the argument is superfluous, and you can just call joinphrase directly

Collectives™ on Stack Overflow

How to print the output only 1 time using python?

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related