[RESOLVED] loops for less times

[angelica]

Hey,

The code below only loops for 3 times. I am assuming it should loop for six times right? Can someone tell me why please?

 Private Sub RandomGenerator1()
        Dim AscChr As String, ChrAsc As String
        Dim counter As Integer = 0
        txtRandom1.Text = ""
        For counter = 0 To 5

            Randomize()
            AscChr = Abs(Int(64 - 122) * Rnd() - 64)
            ChrAsc = Chr(AscChr)
            txtRandom1.Text += String.Format(ChrAsc)
            counter += 1
        Next
    End Sub

[Bulldog]

You dont need the line

counter += 1

as this is effectively incrementing the counter value twice

[angelica]

heya,

Got confused with the C# loop. Many thanks

[jmcilhinney]

You're trying to create a random, 6-character string, correct?

Dim rng As New Random
Dim min As Integer = Convert.ToInt32("A"c)
Dim max As Integer = Convert.ToInt32("z"c) + 1
Dim chars(5) As Char
 
For index As Integer = 0 To chars.GetUpperBound(0)
    chars(index) = Convert.ToChar(rng.Next(min, max))
Next
 
MessageBox.Show(New String(chars))

Always use the Random class to generate random numbers, never Randomize() and Rnd().

[angelica]

Ok ,

Thanks for the advice. Can you pls tell me in why