131

I have to create a function which takes a string, and it should return true or false based on whether the input consists of a repeated character sequence. The length of the given string is always greater than 1 and the character sequence must have at least one repetition.

"aa" // true(entirely contains two strings "a")
"aaa" //true(entirely contains three string "a")
"abcabcabc" //true(entirely containas three strings "abc")

"aba" //false(At least there should be two same substrings and nothing more)
"ababa" //false("ab" exists twice but "a" is extra so false)

I have created the below function:

function check(str){
  if(!(str.length && str.length - 1)) return false;
  let temp = '';
  for(let i = 0;i<=str.length/2;i++){
    temp += str[i]
    //console.log(str.replace(new RegExp(temp,"g"),''))
    if(!str.replace(new RegExp(temp,"g"),'')) return true;
  }
  return false;
}

console.log(check('aa')) //true
console.log(check('aaa')) //true
console.log(check('abcabcabc')) //true
console.log(check('aba')) //false
console.log(check('ababa')) //false

Checking of this is part of the real problem. I can't afford a non-efficient solution like this. First of all, it's looping through half of the string.

The second problem is that it is using replace() in each loop which makes it slow. Is there a better solution regarding performance?

12
  • 19
    This link may be useful to you. I always find geekforgeeks as a good source for algorithm problems - geeksforgeeks.org/…
    – Leron
    Apr 24, 2019 at 6:10
  • 9
    Do you mind if I borrow this and make it a coding challenge on the Programming Golf exchange site?
    – ouflak
    Apr 24, 2019 at 14:23
  • 7
    @ouflak you can do that.
    – Maheer Ali
    Apr 24, 2019 at 14:34
  • 12
    In case your curious, codegolf.stackexchange.com/questions/184682/…
    – ouflak
    Apr 24, 2019 at 14:55
  • 24
    @Shidersz Using Neural networks for this feels a bit like using a cannon to shoot a mosquito.
    – JAD
    Apr 25, 2019 at 9:30

13 Answers 13

190

There’s a nifty little theorem about strings like these.

A string consists of the same pattern repeated multiple times if and only if the string is a nontrivial rotation of itself.

Here, a rotation means deleting some number of characters from the front of the string and moving them to the back. For example, the string hello could be rotated to form any of these strings:

hello (the trivial rotation)
elloh 
llohe 
lohel 
ohell 

To see why this works, first, assume that a string consists of k repeated copies of a string w. Then deleting the first copy of the repeated pattern (w) from the front of the string and tacking it onto the back will give back the same string. The reverse direction is a bit trickier to prove, but the idea is that if you rotate a string and get back what you started with, you can apply that rotation repeatedly to tile the string with multiple copies of the same pattern (that pattern being the string you needed to move to the end to do the rotation).

Now the question is how to check whether this is the case. For that, there’s another beautiful theorem we can use:

If x and y are strings of the same length, then x is a rotation of y if and only if x is a substring of yy.

As an example, we can see that lohel is a rotation of hello as follows:

hellohello
   ^^^^^

In our case, we know that every string x will always be a substring of xx (it’ll appear twice, once at each copy of x). So basically we just need to check if our string x is a substring of xx without allowing it to match at the first or halfway character. Here’s a one-liner for that:

function check(str) {
    return (str + str).indexOf(str, 1) !== str.length;
}

Assuming indexOf is implemented using a fast string matching algorithm, this will run in time O(n), where n is the length of the input string.

20
  • 13
    Very nice! I've added it to the jsPerf benchmark page.
    – user42723
    Apr 25, 2019 at 2:23
  • 10
    @user42723 Cool! Looks like it's really, really fast. Apr 25, 2019 at 3:20
  • 5
    FYI: I had a hard time believing that sentence until I reversed the wording: "A string is a nontrivial rotation of itself if and only if it consists of the same pattern repeated multiple times". Go figure. Apr 25, 2019 at 7:58
  • 11
    Do you have references to those theorems?
    – HRK44
    Apr 25, 2019 at 12:40
  • 4
    I think the first statement is the same as "Lemma 2.3: If x and a rotation of x are equal, then x is a repetition" at doi.org/10.1016/j.tcs.2008.04.020 . See also: stackoverflow.com/a/2553533/1462295
    – BurnsBA
    Apr 25, 2019 at 18:32
68

You can do it by a capturing group and backreference. Just check it's the repetition of the first captured value.

function check(str) {
  return /^(.+)\1+$/.test(str)
}

console.log(check('aa')) //true
console.log(check('aaa')) //true
console.log(check('abcabcabc')) //true
console.log(check('aba')) //false
console.log(check('ababa')) //false

In the above RegExp:

  1. ^ and $ stands for start and end anchors to predict the position.
  2. (.+) captures any pattern and captures the value(except \n).
  3. \1 is backreference of first captured value and \1+ would check for repetition of captured value.

Regex explanation here

For RegExp debugging use: https://regex101.com/r/pqlAuP/1/debugger

Performance : https://jsperf.com/reegx-and-loop/13

16
  • 2
    Can you explain to us what this line is doing return /^(.+)\1+$/.test(str) Apr 24, 2019 at 6:09
  • 34
    Also what is the complexity of this solution? I'm not absolutely sure but it doesn't seem to be much faster than the one the OP has.
    – Leron
    Apr 24, 2019 at 6:13
  • 8
    @PranavCBalan I'm not good at algorithms, that's why I write in the comments section. However I have several things to mention - the OP already has a working solution so he is asking for one that will give him better performance and you haven't explained how your solution will outperform his. Shorter doesn't mean faster. Also, from the link you gave: If you use normal (TCS:no backreference, concatenation,alternation,Kleene star) regexp and regexp is already compiled then it's O(n). but as you wrote you are using backreference so is it still O(n)?
    – Leron
    Apr 24, 2019 at 6:23
  • 5
    You can use [\s\S] instead of . if you need to match newline characters in the same way as other characters. The dot character doesn't match on newlines; the alternative searches for all white-space and non-whitespace characters, which means that newlines are included in the match. (Note that this is faster than the more intuitive (.|[\r\n]).) However, if the string definitely doesn't contain newlines, then the simple . will be fastest. Note this will be a lot simpler if the dotall flag is implemented.
    – HappyDog
    Apr 24, 2019 at 8:19
  • 2
    Isn't /^(.+?)\1+$/ a little faster? (12 steps vs 20 steps) Apr 25, 2019 at 8:39
30

Perhaps the fastest algorithmic approach is building a Z-function in linear time:

The Z-function for this string is an array of length n where the i-th element is equal to the greatest number of characters starting from the position i that coincide with the first characters of s.

In other words, z[i] is the length of the longest common prefix between s and the suffix of s starting at i.

C++ implementation for reference:

vector<int> z_function(string s) {
    int n = (int) s.length();
    vector<int> z(n);
    for (int i = 1, l = 0, r = 0; i < n; ++i) {
        if (i <= r)
            z[i] = min (r - i + 1, z[i - l]);
        while (i + z[i] < n && s[z[i]] == s[i + z[i]])
            ++z[i];
        if (i + z[i] - 1 > r)
            l = i, r = i + z[i] - 1;
    }
    return z;
}

JavaScript implementation
Added optimizations - building a half of z-array and early exit

function z_function(s) {
  var n = s.length;
  var z = Array(n).fill(0);
  var i, l, r;
  //for our task we need only a half of z-array
  for (i = 1, l = 0, r = 0; i <= n/2; ++i) {
    if (i <= r)
      z[i] = Math.min(r - i + 1, z[i - l]);
    while (i + z[i] < n && s[z[i]] == s[i + z[i]])
      ++z[i];

      //we can check condition and return here
     if (z[i] + i === n && n % i === 0) return true;
    
    if (i + z[i] - 1 > r)
      l = i, r = i + z[i] - 1;
  }
  return false; 
  //return z.some((zi, i) => (i + zi) === n && n % i === 0);
}
console.log(z_function("abacabacabac"));
console.log(z_function("abcab"));

Then you need to check indexes i that divide n. If you find such i that i+z[i]=n then the string s can be compressed to the length i and you can return true.

For example, for

string s= 'abacabacabac'  with length n=12`

z-array is

(0, 0, 1, 0, 8, 0, 1, 0, 4, 0, 1, 0)

and we can find that for

i=4
i+z[i] = 4 + 8 = 12 = n
and
n % i = 12 % 4 = 0`

so s might be represented as substring of length 4 repeated three times.

6
  • 3
    return z.some((zi, i) => (i + zi) === n && n % i === 0) Apr 24, 2019 at 12:27
  • 2
    Thanks for adding JavaScript stuff to Salman A and Pranav C Balan
    – MBo
    Apr 24, 2019 at 18:05
  • 1
    Alternate approach by avoiding an additional iteration const check = (s) => { let n = s.length; let z = Array(n).fill(0); for (let i = 1, l = 0, r = 0; i < n; ++i) { if (i <= r) z[i] = Math.min(r - i + 1, z[i - l]); while (i + z[i] < n && s[z[i]] == s[i + z[i]]) ++z[i]; // check condition here and return if (z[i] + i === n && n % i === 0) return true; if (i + z[i] - 1 > r) l = i, r = i + z[i] - 1; } // or return false return false; } Apr 24, 2019 at 18:14
  • 2
    Using the z-function is a good idea, but it is 'information -heavy', it contains a lot of information that is never used. Apr 25, 2019 at 7:37
  • @Axel Podehl Nevertheless, it treats string in O(n) time (each char is used at most two times). In any case we must check every char, so there is no theoretically faster algorithm (while optimized in-built methods might outperform). Also in the last edit I limited calculation by 1/2 of string length.
    – MBo
    Apr 25, 2019 at 7:43
23

I read the answer of gnasher729 and implemented it. The idea is that if there are any repetitions, then there must be (also) a prime number of repetitions.

function* primeFactors (n) {
    for (var k = 2; k*k <= n; k++) {
        if (n % k == 0) {
            yield k
            do {n /= k} while (n % k == 0)
        }
    }
    if (n > 1) yield n
}

function check (str) {
    var n = str.length
    primeloop:
    for (var p of primeFactors(n)) {
        var l = n/p
        var s = str.substring(0, l)
        for (var j=1; j<p; j++) {
            if (s != str.substring(l*j, l*(j+1))) continue primeloop
        }
        return true
    }
    return false
}

A slightly different algorithm is this:

function check (str) {
    var n = str.length
    for (var p of primeFactors(n)) {
        var l = n/p
        if (str.substring(0, n-l) == str.substring(l)) return true
    }
    return false
}

I've updated the jsPerf page that contains the algorithms used on this page.

3
  • This seems really fast since it skips unnecessary checks. Apr 25, 2019 at 2:51
  • 1
    Very nice, only I think I would check that the first letter reoccurs at the specified location before making the substring calls.
    – Ben Voigt
    Apr 27, 2019 at 5:05
  • For people stumbling on function* for the first time like me, it's for declaring a generator, not a regular function. See MDN Apr 29, 2019 at 15:44
17

Assume the string S has length N and is made of duplicates of the substring s, then the length of s divides N. For example, if S has length 15, then the substring has length 1, 3, or 5.

Let S be made of (p*q) copies of s. Then S is also made of p copies of (s, repeated q times). We have therefore two cases: If N is prime or 1, then S can only be made of copies of the substring of length 1. If N is composite, then we only need to check substrings s of length N / p for primes p dividing the length of S.

So determine N = the length of S, then find all its prime factors in time O (sqrt (N)). If there is only one factor N, check if S is the same string repeated N times, otherwise for each prime factor p, check if S consists of p repeations of the first N / p characters.

2
  • I haven't checked the other solutions, but this seems very fast. You can leave out the "If there is only one factor N, check ..., otherwise" part for simplicity, as this is not a special case. Would be nice to see a Javascript implementation that can be run in jsPerf next to the other implementations.
    – user42723
    Apr 24, 2019 at 16:59
  • 1
    I've now implemented this in my answer
    – user42723
    Apr 25, 2019 at 1:22
10

I think a recursive function might be very fast as well. The first observation is that the maximum repeated pattern length is half as long as the total string. And we could just test all possible repeated pattern lengths: 1, 2, 3, ..., str.length/2

The recursive function isRepeating(p,str) tests if this pattern is repeated in str.

If str is longer than the pattern, the recursion requires the first part (same length as p) to be a repetition as well as the remainder of str. So str is effectively broken up into pieces of length p.length.

If the tested pattern and str are of equal size, recursion ends here, successfully.

If the length is different (happens for "aba" and pattern "ab") or if the pieces are different, then false is returned, propagating up the recursion.

function check(str)
{
  if( str.length==1 ) return true; // trivial case
  for( var i=1;i<=str.length/2;i++ ) { // biggest possible repeated pattern has length/2 characters

    if( str.length%i!=0 ) continue; // pattern of size i doesn't fit
    
    var p = str.substring(0, i);
    if( isRepeating(p,str) ) return true;
  }
  return false;
}


function isRepeating(p, str)
{
  if( str.length>p.length ) { // maybe more than 2 occurences

    var left = str.substring(0,p.length);
    var right = str.substring(p.length, str.length);
    return left===p && isRepeating(p,right);
  }
  return str===p; 
}

console.log(check('aa')) //true
console.log(check('aaa')) //true 
console.log(check('abcabcabc')) //true
console.log(check('aba')) //false
console.log(check('ababa')) //false

Performance: https://jsperf.com/reegx-and-loop/13

9
  • 1
    Would it be faster to check if( str===p.repeat(str.length/i) ) return true; instead of using a recursive function? Apr 24, 2019 at 14:24
  • 1
    Don't put console.logs in jsperf tests, prepare the functions inside the globals section, also prepare the test strings in the globals section (sorry, cannot edit the jsperf)
    – Salman A
    Apr 24, 2019 at 15:17
  • @Salman - good point. I just modified the jsperf from my predecessor (Pranav C), first time I used jsperf, cool tool. Apr 24, 2019 at 15:23
  • @SalmanA : updated : jsperf.com/regex-and-loop/1 ... thanks for the info... even I'm not familiar with it(Jsperf) ... thanks for the information Apr 24, 2019 at 15:47
  • Hi Salman, thanks a lot for jsperf.com/reegx-and-loop/10 - yes, that new perf test makes much more sense. The setup of functions should go into the preparation code. Apr 24, 2019 at 16:03
7

Wrote this in Python. I know it is not the platform, but it did take 30 mins of time. P.S.=> PYTHON

def checkString(string):
    gap = 1 
    index= 0
    while index < len(string)/2:
        value  = [string[i:i+gap] for i in range(0,len(string),gap) ]

        x = [string[:gap]==eachVal for eachVal in value]

        if all(x):
            print("THEY ARE  EQUAL")
            break 

        gap = gap+1
        index= index+1 

checkString("aaeaaeaaeaae")
6

My approach is similar to gnasher729, in that it uses the potential length of the substring as the main focus, but it is less math-y and process intensive:

L: Length of original string

S: Potential lengths of valid sub-strings

Loop S from (integer part of) L/2 to 1. If L/S is an integer check your original string against the fist S characters of the original string repeated L/S times.

The reason for looping from L/2 backwards and not from 1 onwards is to get the largest possible substring. If you want the smallest possible substring loop from 1 to L/2. Example: "abababab" has both "ab" and "abab" as possible substrings. Which of the two would be faster if you only care about a true/false result depends on the type of strings/substrings this will be applied to.

5

The following Mathematica code almost detects if the list is repeated at least once. If the string is repeated at least once, it returns true, but it might also return true if the string is a linear combination of repeated strings.

IsRepeatedQ[list_] := Module[{n = Length@list},
   Round@N@Sum[list[[i]] Exp[2 Pi I i/n], {i, n}] == 0
];

This code looks for the "full-length" contribution, which must be zero in a repeating string, but the string accbbd is also considered repeated, as it is a sum of the two repeated strings ababab and 012012.

The idea is to use Fast Fourier Transform, and look for the frequency spectra. By looking at other frequencies, one should be able to detect this strange scenario as well.

4

The basic idea here is to examine any potential substring, beginning at length 1 and stopping at half of the original string's length. We only look at substring lengths that divide the original string length evenly (i.e. str.length % substring.length == 0).

This implementation looks at the first character of each possible substring iteration before moving to the second character, which might save time if the substrings are expected to be long. If no mismatch is found after examining the entire substring, then we return true.

We return false when we run out of potential substrings to check.

function check(str) {
  const len = str.length;
  for (let subl = 1; subl <= len/2; ++subl) {
    if ((len % subl != 0) || str[0] != str[subl])
      continue;
    
    let i = 1;
    for (; i < subl; ++i)
    {
      let j = 0;
      for (; j < len; j += subl)
        if (str[i] != str[j + i])
          break;
      if (j != len)
        break;
    }
    
    if (i == subl)
      return true;
  }
  return false;
}

console.log(check('aa')) //true
console.log(check('aaa')) //true
console.log(check('abcabcabc')) //true
console.log(check('aba')) //false
console.log(check('ababa')) //false

0

It's been more than a year since this question was posted but I used length of string and object form to validate whether it is true or false.

const check = (str) => {
  let count = 0;
  let obj = {};
  if (str.length < 2) return false;
  
  for(let i = 0; i < str.length; i++) {
    if (!obj[str[i]]) {
       count+=1;
      obj[str[i]] = 0;
    };
    obj[str[i]] = obj[str[i]] + 1;
  };
  
  if (Object.values(obj).every(item => item === 1)) {
    return false
  };
  
  if ([...str].length%count === 0) {
    return true
  } else {
    return false
  };
};

console.log(check("abcabcabcac")) // false
console.log(check("aaa")) // true
console.log(check("acaca")) // false
console.log(check("aa")) // true
console.log(check("abc")) // false
console.log(check("aabc")) // false

-1

I'm not familiar with JavaScript, so I don't know how fast this is going to be, but here is a linear time solution (assuming reasonable builtin implementation) using only builtins. I'll describe the algorithm in pseudocode.

function check(str) {
    t = str + str;
    find all overlapping occurrences of str in t;
    for each occurrence at position i
        if (i > 0 && i < str.length && str.length % i == 0)
            return true;  // str is a repetition of its first i characters
    return false;
}

The idea is similar to MBo's answer. For each i that divides the length, str is a repetition of its first i characters if and only if it remains the same after shifting for i characters.

It comes to my mind that such a builtin may be unavailable or inefficient. In this case, it is always possible to implement the KMP algorithm manually, which takes about the same amount of code as the algorithm in MBo's answer.

3
  • The OP wants to know whether repetition exists. The second line of (the body of) your function counts the number of repetitions - that's the bit that needs to be explained. E.g. "abcabcabc" has 3 repetitions of "abc", but how did your second line work out whether it had any repetitions?
    – Lawrence
    Apr 24, 2019 at 12:05
  • @Lawrence I don't understand your question. This algorithm is based on the idea that the string is a repetition of its substring if and only if for some divisor of its length i, s[0:n-i] == s[i:n], or equivalently, s == s[i:n] + s[0:i]. Why does the second line need to work out whether it had any repetitions? Apr 25, 2019 at 0:20
  • Let me see if I understand your algorithm. First, you append str to itself to form t, then scan t to try to find str inside t. Okay, this can work (I've retracted my downvote). It's not linear in strlen(str), though. Say str is of length L. Then at each position p=0,1,2,..., checking whether str[0..L-1] == t[p..p+L-1] takes O(L) time. You need to do O(L) checks as you go through the values of p, so it's O(L^2).
    – Lawrence
    Apr 25, 2019 at 4:52
-10

One of the simple ideas is to replace the string with the substring of "" and if any text exist then it is false, else it is true.

'ababababa'.replace(/ab/gi,'')
"a" // return false
'abababab'.replace(/ab/gi,'')
 ""// return true

4
  • yes, for abc or unicorn wouldn't user will check with /abc/ or /unicorn/ , sorry if i am missing your context Apr 24, 2019 at 8:10
  • 3
    The question could be clearer, but what it's asking for is a way of deciding whether the string is completely made up of 2 or more repetitions of any other string. It is not searching for a specific substring.
    – HappyDog
    Apr 24, 2019 at 8:26
  • 2
    I've added some clarification to the question, which should make it clearer now.
    – HappyDog
    Apr 24, 2019 at 9:55
  • @Vinod if you are already going to use regex you should anchor your match and use test. No reason to modify the string just to validate some condition.
    – Marie
    Apr 24, 2019 at 11:55

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service, privacy policy and cookie policy

Not the answer you're looking for? Browse other questions tagged or ask your own question.