Luciano,

Did you check your typing?
Did you put it into a standard codemodule and not a worksheet codemodule?
Did you copy the code into a codemodule in the correct workbook?

It worked for me - let me know if you can't get it to work, and I will send
you a working version, if you gmail account is not a spoof.

Bernie

"Luciano Paulino da Silva" wrote in message
...
Dear Bernie,
I tried a lot to run the code but it is not working. Have you some
idea about what could it be happening?
I just receive the error: #NAME! for all cells in the interval.
Thanks in advance,
Luciano


On 15 abr, 19:29, "Bernie Deitrick" deitbe @ consumer dot org wrote:
Luciano,

Same drill: Enter

=REPEATS(A1) into cells A2:B??? using Ctrl-Shift-Enter.

HTH,
Bernie
MS Excel MVP

Option Explicit

Function Repeats(strBig As String) As Variant
Dim FoundRpts() As String
Dim RptCount As Integer
Dim i As Integer
Dim j As Integer
Dim k As Integer
Dim x As Integer
Dim RptExists As Boolean

RptCount = 1
ReDim FoundRpts(1 To 2, 1 To 2)
For i = 1 To Len(strBig) - 1
For j = 2 To Len(strBig) - i + 1
x = isRpt(Mid(strBig, i, j), strBig, i + 1)
If x 0 Then
If RptCount = 1 Then
FoundRpts(1, 2) = Mid(strBig, i, j)
FoundRpts(2, 2) = x
RptCount = 2
Else
RptExists = False
For k = 2 To UBound(FoundRpts, 2)
If FoundRpts(1, k) = Mid(strBig, i, j) Then
RptExists = True
End If
Next k
If Not RptExists Then
ReDim Preserve FoundRpts(1 To 2, 1 To RptCount + 1)
FoundRpts(1, RptCount + 1) = Mid(strBig, i, j)
FoundRpts(2, RptCount + 1) = x
MsgBox FoundRpts(1, RptCount + 1) & " " & x
RptCount = RptCount + 1
End If
End If
End If
Next j
Next i

FoundRpts(1, 1) = "Repeats found:"
FoundRpts(2, 1) = RptCount - 1
Repeats = Application.Transpose(FoundRpts)

End Function
Function isRpt(strRpt As String, strPar As String, i As Integer) As
Integer
isRpt = 0
If InStr(i, strPar, strRpt) 0 Then
isRpt = (Len(Mid(strPar, i, Len(strPar))) - _
Len(Replace(Mid(strPar, i, Len(strPar)), strRpt, ""))) / _
Len(strRpt) + 1
End If
End Function

"LucianoPaulinodaSilva" wrote in
...

It is exactly that I want.
Thank you,
Luciano

On 15 abr, 12:35,LucianoPaulinodaSilva
wrote:
Dear Bernie,
Yes, you are correct that there are other repeats in this string.
Thanks in advance,
Luciano

On 15 abr, 09:49, "Bernie Deitrick" deitbe @ consumer dot org wrote:

OK for for palindromes, but the string that you posted

AABBAABB

has the repeats

AA
AAB
AABB
AB
ABB
BB

not just AA and BB...

If that is what you actually want, then I will modify the code to do
so.

HTH,
Bernie
MS Excel MVP

"LucianoPaulinodaSilva" wrote in
...

OK!
It is the following:
My first problem that was related to palindromes detection your
suggestion was absolutely perfect.
Now, my second problem is relate to detect repeats (sequences of
letters or numbers which are repeating at least twice within a
string)
in the same sequences. Of course that some repeats are also
palindromes. But in this case the listing and counting must be only
of
repeats consisting of two or more letters. For example, the string
that you listed bellow has only one repeat (AA) that should be
listed
in this case. It appears 2 times in the string.
Didi you understand?
Thanks in advance,
Luciano

On 14 abr, 23:34, "Bernie Deitrick" deitbe @ consumer dot org
wrote:
Luciano,

I'm not clear on what you want - only those that appear more than
once?
What do you want with the ABBA and the AABBAA and the BAAB, all of
which
only appear once?

Bernie

"LucianoPaulinodaSilva" wrote in
...
Bernie,
It is possible that I have not explained very well. At this
moment,
I`m needing a code that detect the repeats for a givem string
instead
to detect palindromes. For example, the string AABBAABB has two
repeats (AA and BB), and they appear 2 times.
Do you understand it now?
Thanks in advance,
Luciano

On 14 abr, 21:16, "Bernie Deitrick" deitbe @ consumer dot org
wrote:

Luciano,

Change the code to that given below to include the count of the
number of
repeats, then select A2:B??? before entering =PALINDROMES(A1)
using
Ctrl-Shift-Enter.

HTH,
Bernie
MS Excel MVP

Function Palindromes(strBig As String) As Variant
Dim FoundPals() As String
Dim PalCount As Integer
Dim i As Integer
Dim j As Integer
Dim k As Integer
Dim PalExists As Boolean

PalCount = 1
ReDim FoundPals(1 To 2, 1 To 2)
For i = 1 To Len(strBig) - 1
For j = 2 To Len(strBig) - i + 1
If isPal(Mid(strBig, i, j)) Then
If PalCount = 1 Then
FoundPals(1, 2) = Mid(strBig, i, j)
FoundPals(2, 2) = 1
PalCount = 2
Else
PalExists = False
For k = 2 To UBound(FoundPals, 2)
If FoundPals(1, k) = Mid(strBig, i, j) Then
PalExists = True
FoundPals(2, k) = FoundPals(2, k) + 1
End If
Next k
If Not PalExists Then
ReDim Preserve FoundPals(1 To 2, 1 To PalCount + 1)
FoundPals(1, PalCount + 1) = Mid(strBig, i, j)
FoundPals(2, PalCount + 1) = 1
PalCount = PalCount + 1
End If
End If
End If
Next j
Next i

FoundPals(1, 1) = "Palindromes found:"
FoundPals(2, 1) = PalCount - 1
Palindromes = Application.Transpose(FoundPals)

End Function
Function isPal(strPal As String) As Boolean
Dim i As Integer
Dim strTemp As String
isPal = False
For i = Len(strPal) To 1 Step -1
strTemp = strTemp & Mid(strPal, i, 1)
Next i
isPal = (strPal = strTemp)
End Function

"LucianoPaulinodaSilva" wrote
in
...
Bernie,
Yes, it is working fine. Thank you very much.
During some situations it is very slow, but I understand that
there
are a lot of possibilities to test.
Do you know how some code could be used to check about repeats
using
the same strategy?
Thanks in advance,
Luciano