I need to find the longest common substring between one string and a list of others. This would be very easy to do naively - find all the substrings of the first string, then check via index() on the others. However this is O(mnk) which is bad, m'kay. So I managed to find an algorithm that uses a matrix to find the longest substring in O(nk). [ m = number of substrings in first string, n = number of strings to look in, k = avg length of strings ].
Unfortunately the algorithm requires looking at each individual character in the strings one by one. Sadly this is very slow in perl, despite perl being perfect for most string matching needs. I hope this is fixed in perl 6. Anyway, I decided the best way to go would be to prototype in Inline::C and then migrate to XS. It turns out I really hate programming in C. All the array indexes being dangerous and lack of being able to print datastructures easily, and the malloc/free nightmare, and ugly for() loops.
Despite all that, I came up with this, which appears to work pretty well:
#!/usr/bin/perl -wAnd results in:
use Inline C; use Benchmark; use strict;
my $string = "hello"; my @to_match = qw(aloha helen noddy);
timethese(1_000_000, { match => sub { my $max_substr = do_match($string, \@to_match); if ($max_substr ne 'hel') { die "Bad match: $max_substr"; } #else { print "good match\n" } }, });
#print "Found: $max_substr\n";
__END__ __C__
#define mtrx(x,y) ((x) + ((y) * match_len))
SV * do_match( char * fixed, AV * list ) { I32 i; I32 list_len = -1; SV** to_match = NULL; I32 longest = 0; I32 answer_end = 0; const I32 fixed_len = strlen(fixed); SV * result = newSV(0);
if (result == NULL) return NULL;
/* length of the array */ list_len = av_len(list);
if (list_len < 0) return NULL;
for (i = 0; i <= list_len; i++) { I32 match_len; char *ptr; char *matrix; I32 x, y; int longer = 0; /* get the i'th entry out of the array */ to_match = av_fetch(list, i, 0); if (!to_match) break;
/* get a char* out of the SV (sets match_len as side effect) */ ptr = SvPV(*to_match, match_len); /* allocate a long string, and use arithmetic to treat it like a matrix */ matrix = calloc(match_len * fixed_len, sizeof(char)); if (matrix == NULL) return NULL;
answer_end = 0;
/* NB: using 1,1 as root of matrix so we have a zero'd border to prevent segfaults when fetching matrix[x-1,y-1] */ for (x = 1; x <= fixed_len; x++) { for (y = 1; y <= match_len; y++) { if (fixed[x-1] == ptr[y-1]) { matrix[mtrx(x,y)] = 1 + matrix[mtrx(x-1,y-1)]; if (matrix[mtrx(x,y)] > longest) { longest = matrix[mtrx(x,y)]; answer_end = y; longer = 1; } } } }
/* if this one was longer than previous ones, we store it in the SV */ if (longer) { /* null terminate it first! */ ptr[answer_end] = 0; sv_setpvn(result, &ptr[answer_end - longest], longest); }
free(matrix); }
return result; }
$ perl common_substring.pl Benchmark: timing 1000000 iterations of match... match: 13 wallclock secs (10.04 usr + 0.03 sys = 10.07 CPU) @ 99304.87/s (n=1000000)Now I need to run it on some real data (tens of thousands of strings) to see how well it scales!
Re:And that works really well...
Matts on 2004-09-29T20:01:41
s/char/wchar/:-) (I know this doesn't really work) Re:And that works really well...
bart on 2004-09-30T08:33:58
It should do just as well on UTF8, no? Except for its probable confusion on string length vs. number of bytes..Re:And that works really well...
Matts on 2004-09-30T09:43:27
In a multibyte string you get potential clashes between characters with different surrogate bytes but similar character bytes, meaning invalid results.Re:And that works really well...
Elian on 2004-09-30T12:22:15
Nope. UTF-8 is particularly bad. In addition to potentially having combining characters in the match, which is a Unicode issue, you can run into lots of cases where the second (and third, and fourth, and fifth...) byte of two multibyte characters are identical but the characters aren't the same because the first byte isn't.
This technique only works on fixed-width characters in sets with no combining characters (In which case it works really well) or in cases where you can ensure that you're only using the subset of characters that don't allow combining characters.Re:And that works really well...
Matts on 2004-09-30T12:56:25
Though to be fair it would be easy to fix for UTF-8. You just need to use a NEXT() macro (or something - NEXT() is what libxml uses internally) instead of iterating through an array.Re:And that works really well...
Elian on 2004-09-30T13:09:50
Oh, sure -- it's not that big a deal, and you could certainly handle it straightforwardly enough. It'd be a bit of a pain, though.Re:And that works really well...
waitman on 2005-11-27T17:35:32
I had an idea that seems to work on big strings. Kind of like Edward Scissor Hands cutting out the "bad stuff" in a bunch and leaving only the "good stuff". If you have two strings a and b, chop up a into pieces that have *characters* in common with b, and vice-versa. then chop up all the pieces. Like salad or a block of ice.
Then it is quick to find the real matches. Should work with other things besides 8 bit text.
I did it in Python, here-> http://www.livejournal.com/users/bramcohen/22069.html
Re:Dynamic programming? hmm!
Matts on 2004-09-29T20:08:40
I'm mostly just parroting the page I read - I have no idea what's "dynamic" about this technique. Maybe that's a consequence of programming perl all the time - everything is dynamic anyway!
However there's a number of papers out there on longest common sequence detection, that use similar techniques and algorithms (and all use GTCA in their examples). I guess you'd have to keep a corpus of spam around to determine similarity using this technique though.Re:Dynamic programming? hmm!
jmason on 2004-09-29T20:24:02
oh btw -- here's a good one:
http://cbrg.inf.ethz.ch/bio-recipes/NigerianPrince/code.html
'Phylogenetic tree of the "Nigerian Prince" email scam';) protocol analysis URL
jmason on 2004-09-30T19:34:25
btw, here's the URL for that presentation:
http://www.baselineresearch.net/PI/PI-Toorcon.pdf
You can also stop scanning the tail end of a word if the length of longest match at the current point, plus the length of the rest of the word is smaller than the current longest known match. (I think you can make that check every time you get a mismatch, since when the character matches you are extending a possible match that there was room enough for.
Both of these will help with the scaling - as soon as you've found a reasonable match you'll start skipping that much of the tail end of most words.
I have a feeling that there may be some way of extending the Boyer-Moore algorithm (see, for example, Mastering Algorithms with Perl, page 373) to provide additional speedups by letting you skip multiple characters as you scan through a word. At the very least, you could keep a list of where characters occurs. Then, when you are looking for a match starting at pos x, check the character at pos x+longest - it that character does not occur in the target word you can immediately discard all starting positions from x..(x+longest) as candidates for a match. (There might be a shorter or equal match in there somewhere, but you don't care about them.) If that character does occur in the target, you might be able to use a bitmask that lists all of the locations that it occurs. Now work backwards, shifting the bitmask and anding it against the bitmask for the character in the previous location. When the bitmask result goes to 0, you've eliminated (or proven) the original position as a match of length longest+1. If it was a match, extend it as much as possible and record it as the new longest match. If it wasn't, the position following the one that failed is the new starting position candidate (and you have a bit mask to test out with, too). The basic Boyer-Moore algorithm doesn't work directly because it was designed to find a match for one entire string anywhere within another string. Since this problem is accepting any sub-string of the match target, you need to modify the approach.)
Re:All common substrings
Matts on 2004-10-07T07:57:17
Creating arrays in C is easy:You can just return the AV * and the builtin perl typemap will turn it into a list for you.AV * foo = newAV();
av_push(foo, newSVpv(thestring, 0));