Как правильно поступить с классом Random?

Question

[Петр]

Добрый день.
Написал обычный консольный генератор, но столкнулся с такой проблемой, что повторно используя программу, длинна пароля суммируется.
К примеру ввожу длину пароля в два символа, происходит генерация, далее повторив операцию но указав три символа итоговый пароль вырастает до пяти символов, причем два из них используются из прошлого запроса.
Вот исходный код:

using System;

namespace PasswordGenerator
{
    class Program
    {
        static void Main(string[] args)
        {

            string Pass = "";
            string rnd_psw = "1234567890QWERTYUIOPASDFGHJKLMNBVCXZ";
            int pass_lenght;
            Random rnd = new Random();

            Console.WriteLine("\t\t\t\t###Типа дофига крутой генератор паролей!!###");

            while(true)
            {

                Console.Write("Введите длину пароля:");
                pass_lenght = Convert.ToInt32(Console.ReadLine());

                
                for (int i = 0; i < pass_lenght; i++)
                {
                    Pass = Pass + rnd_psw[rnd.Next(0, rnd_psw.Length)];
                }

                Console.WriteLine("Ваш пароль:{0}", Pass);
                Console.WriteLine("Повторить? y/n");
                if (Console.ReadKey(true).Key != ConsoleKey.Y)
                    break;
            }
        }


       
    }
}

Я сделал вот что, после строчки кода:

for (int i = 0; i < pass_lenght; i++)
                {
                    Pass = Pass + rnd_psw[rnd.Next(0, rnd_psw.Length)];
                }
                Console.WriteLine("Ваш пароль:{0}", Pass);

Дописал Pass = null; и получил это:

for (int i = 0; i < pass_lenght; i++)
                {
                    Pass = Pass + rnd_psw[rnd.Next(0, rnd_psw.Length)];
                }
                Console.WriteLine("Ваш пароль:{0}", Pass);
                Pass = null;

Верно ли я сделал или можно иначе поступить?

Answer 1

[Зелимхан Бельтоев]

Непонятно зачем вам Pass выносить за цикл. К тому же использовать неизменяемый тип string для этой задачи - не лучшее решение.

Стиль именования переменных не критикую (а мог бы!):

var password = new StringBuilder(pass_lenght);

for (int i = 0; i < pass_lenght; i++)
{
    password.Append(rnd_psw[rnd.Next(0, rnd_psw.Length)]);
}

Console.WriteLine("Ваш пароль:{0}", password);

Comments

[Петр]

То есть Pass=null; можно в цикл вставить?

[Зелимхан Бельтоев]

Иван Фролов: нет. Не нужно явно обнулять так ссылку. К тому же, если так хочется использовать string, просто перенесите строку string Pass = ""; перед циклом for (...) - и проблема будет решена без Pass=null;