469,271 Members | 1,759 Online
Bytes | Developer Community
New Post

Home Posts Topics Members FAQ

Post your question to a community of 469,271 developers. It's quick & easy.

counting number of (overlapping) occurances

I have two strings S1 and S2. I want to know how many times
S2 occurs inside S1.

For instance

if S1 = "AAAA"
and S2 = "AA"

then the count is 3. Is there an easy way to do this in python?
I was trying to use the "count" function but it does not do
overlapping counts it seems.

Thanks,
--j

Mar 10 '06 #1
8 5768
"John" <we**********@yahoo.com> writes:
if S1 = "AAAA"
and S2 = "AA"

then the count is 3. Is there an easy way to do this in python?
I was trying to use the "count" function but it does not do
overlapping counts it seems.


len([1 for i in xrange(len(s1)) if s1[i:].startswith(s2)])
Mar 10 '06 #2
Thanks a lot,

This works but is a bit slow, I guess I'll have to live with it.
Any chance this could be sped up in python?

Thanks once again,
--j

Mar 10 '06 #3
"John" <we**********@yahoo.com> writes:
This works but is a bit slow, I guess I'll have to live with it.
Any chance this could be sped up in python?


Whoops, I meant to say:

len([1 for i in xrange(len(s1)) if s1.startswith(s2,i)])

That avoids creating a lot of small strings.

If s1 is large you may want to use fancy algorithms like Boyer-Moore
string search. If s2 is also large, you could have some kind of
finite state machine scan through s1 so it recognizes overlapping
occurrences of s2 in parallel. There might be some way to combine
that with Boyer-Moore.
Mar 10 '06 #4
John wrote:
This works but is a bit slow, I guess I'll have to live with it.
Any chance this could be sped up in python?


Sure, to a point. Instead of:

def countoverlap(s1, s2):
return len([1 for i in xrange(len(s1)) if s1[i:].startswith(s2)])

Try this version, which takes smaller slices (resulting in 2x-5x speed
increase when dealing with a large s1 and a small s2):

def countoverlap(s1, s2):
L = len(s2)
return len([1 for i in xrange(len(s1)-L+1) if s1[i:i+L] == s2])

And for a minor extra boost, this version eliminates the list
comprehension:

def countoverlap(s1, s2):
L = len(s2)
cnt = 0
for i in xrange(len(s1)-L+1):
if s1[i:i+L] == s2:
cnt += 1
return cnt

Finally, if the execution speed of this function is vital to your
application, create a C extension. String functions like this one are
generally excellent candidates for extensionizing.

--Ben

Mar 10 '06 #5
John <we**********@yahoo.com> wrote:
Thanks a lot,

This works but is a bit slow, I guess I'll have to live with it.
Any chance this could be sped up in python?


Sure (untested code):

def count_with_overlaps(needle, haystack):
count = 0
pos = 0
while True:
where = haystack.index(needle, pos)
if where == -1: return count
count += 1
pos = where + 1
Alex
Mar 10 '06 #6
Alex Martelli <al*****@yahoo.com> wrote:
John <we**********@yahoo.com> wrote:
Thanks a lot,

This works but is a bit slow, I guess I'll have to live with it.
Any chance this could be sped up in python?
Sure (untested code):

def count_with_overlaps(needle, haystack):
count = 0
pos = 0
while True:
where = haystack.index(needle, pos)


Oops -- 'find', not 'index' -- sorry (it WAS untested;-).
if where == -1: return count
count += 1
pos = where + 1

Alex
Mar 10 '06 #7
On Thu, 09 Mar 2006 22:01:17 -0800, John wrote:
Thanks a lot,

This works but is a bit slow, I guess I'll have to live with it.
Any chance this could be sped up in python?


I don't know. What is it?

You should always quote enough of the previous poster's message to give
context. Messages sometimes go missing in Usenet, and people won't have
the foggiest idea what you are talking about.

--
Steven.

Mar 11 '06 #8
Steven D'Aprano wrote
You should always quote enough of the previous poster's message to give
context. Messages sometimes go missing in Usenet, and people won't have
the foggiest idea what you are talking about.


one would think that given how many Pythoneers we now have working
for google, at least one of them could go fix the horridly broken posting
interface in googlegroups...

:::

for those who use googlegroups and prefer to make a least a little sense
to people read this newsgroup/mailing list using other tools, please click
"show options" and use the "reply" link at the top of the message instead
of the link with the same name at the bottom.

</F>

Mar 11 '06 #9

This discussion thread is closed

Replies have been disabled for this discussion.

Similar topics

4 posts views Thread by Travers Naran | last post: by
4 posts views Thread by shell | last post: by
1 post views Thread by jabailo | last post: by
1 post views Thread by CARIGAR | last post: by
reply views Thread by suresh191 | last post: by
By using this site, you agree to our Privacy Policy and Terms of Use.