Создание генератора

@Acc3di · Регистрация: 21.10.2018

Студворк — интернет-сервис помощи студентам

Добрый день!
Учу C# Windows Forms для поступления в университет. Создаю простенький генератор паролей, он работает, но выводит по 1 символу. Сижу и не могу увидеть свою ошибку. Помогите пожалуйста.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
private void BtnCreatePass_Click(object sender, EventArgs e)
        {
            
            string PassLetter = "qwertyuiopasdfghjklzxcvbnm";
            if (chbNum.Checked == true)
            {
                PassLetter += "1234567890"; // Цифры
            }
            if (chbLetZ.Checked == true)
            {
                PassLetter += PassLetter.ToUpper(); // Заглавные
            }
            if (chbLetP.Checked == true)
            {
                PassLetter += PassLetter; // Прописные
            }
            if (chbSpec.Checked)
            {
                PassLetter += "!@#$%^&*?";
            }
            int kol = (int)numericUpDown3.Value; // Выбор количества символов
            string result = "";
            Random rand = new Random();
            int Len = PassLetter.Length;
            for (int i = 0; i < kol; i++);
            result += PassLetter[rnd.Next(100)];
                tbPass.Text += result;
        }

@Defences · 05.08.2019, 00:44

Acc3di, вы закрываете блок for. Из-за точки с запятой.

Этот код

C#
1
2
3
for (int i = 0; i < kol; i++); // эта точка с запятой
            result += PassLetter[rnd.Next(100)];
                tbPass.Text += result;

Выглядит вот так

C#
1
2
3
4
5
for (int i = 0; i < kol; i++)
{
}
            result += PassLetter[rnd.Next(100)];
                tbPass.Text += result;

Уберите ;

C#
1
2
3
for (int i = 0; i < kol; i++)
            result += PassLetter[rnd.Next(100)];
                tbPass.Text += result;

Добавлено через 2 минуты
К тому же вы объявили

C#
1
 Random rand = new Random();

А используете какой-то другой rnd

C#
1
PassLetter[rnd.Next(100)];

Так же вы указали длину, а используете почему-то 100, будет крит. ошибка если выпадет 98, а у вас всего 40 длина...

C#
1
 result += PassLetter[rnd.Next(100)];

C#
1
 result += PassLetter[rnd.Next(Len)];

Добавлено через 5 минут
И вот тут у вас может быть ошибка.

Представим что выбрали все chb

C#
1
2
3
4
5
6
7
8
9
 string PassLetter = "qwertyuiopasdfghjklzxcvbnm";
            if (chbNum.Checked)
                PassLetter += "1234567890"; // строка имеет вид qwertyuiopasdfghjklzxcvbnm1234567890
            if (chbLetZ.Checked)
                PassLetter += PassLetter.ToUpper(); // строка имеет вид qwertyuiopasdfghjklzxcvbnm1234567890QWERTYUIOPASDFGHJKLZXCVBNM1234567890
            if (chbLetP.Checked)
                PassLetter += PassLetter; // строка имеет вид qwertyuiopasdfghjklzxcvbnm1234567890QWERTYUIOPASDFGHJKLZXCVBNM1234567890qwertyuiopasdfghjklzxcvbnm1234567890QWERTYUIOPASDFGHJKLZXCVBNM1234567890
            if (chbSpec.Checked)
                PassLetter += "!@#$%^&*?"; // строка имеет вид qwertyuiopasdfghjklzxcvbnm1234567890QWERTYUIOPASDFGHJKLZXCVBNM1234567890qwertyuiopasdfghjklzxcvbnm1234567890QWERTYUIOPASDFGHJKLZXCVBNM1234567890!@#$%^&*?

@Diamante · 05.08.2019, 00:50

Acc3di, не надо привязываться к формам или чему бы то ни было еще, codebehind это злое злище!
напиши отдельный метод генерации, получающий на вход все необходимое и выдающий результат, так что бы его можно было хоть в консоли использовать хоть в формах, wpf,asp, да хоть wcf

@Defences · 05.08.2019, 01:00

Acc3di, если брать ваш код, то примерно вот так он должен выглядить

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }
 
        private readonly  Random random = new Random();
 
        private void BtnCreatePass_Click(object sender, EventArgs e)
        {
            string passSeed = GetPassSeed();
            if (passSeed == null)
            {
                MessageBox.Show("Нужно выбрать хотя бы 1 вариант.");
                return;
            }
 
            int count = (int) nudCount.Value;
            int length = passSeed.Length;
            string result = "";
 
            for (int i = 0; i < count; i++)
                result += passSeed[random.Next(length)];
 
            tbPass.Text = result;
        }
 
        private string GetPassSeed()
        {
            string PassLetter = "";
            if (chbNum.Checked)
                PassLetter += "1234567890"; // Цифры
            if (chbLetZ.Checked)
                PassLetter += "QWERTYUIOPASDFGHJKLZXCVBNM"; // Заглавные
            if (chbLetP.Checked)
                PassLetter += "qwertyuiopasdfghjklzxcvbnm"; // Прописные
            if (chbSpec.Checked)
                PassLetter += "!@#$%^&*?";
 
            if (PassLetter == "")
                return null;
 
            return PassLetter;
        }
    }

Добавлено через 5 минут
А если брать во внимание что написал Diamante, то так

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
  public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }
        
 
        private void BtnCreatePass_Click(object sender, EventArgs e)
        {
            string result = Generator.Password((int)nudCount.Value,chbNum.Checked, chbLetP.Checked, chbLetZ.Checked, chbSpec.Checked);
            if (result == null)
            {
                MessageBox.Show("Нужно выбрать хотя бы 1 вариант.");
                return;
            }
            tbPass.Text = result;
        }
        
    }
 
    internal class Generator
    {
        private static readonly Random random = new Random();
        public static string Password(int count, bool isNum, bool isLow, bool isUpper, bool isSpec)
        {
            string PassLetter = "";
            if (isNum)
                PassLetter += "1234567890"; // Цифры
            if (isUpper)
                PassLetter += "QWERTYUIOPASDFGHJKLZXCVBNM"; // Заглавные
            if (isLow)
                PassLetter += "qwertyuiopasdfghjklzxcvbnm"; // Прописные
            if (isSpec)
                PassLetter += "!@#$%^&*?";
 
            if (PassLetter == "")
                return null;
 
            string result = "";
            for (int i = 0; i < count; i++)
                result += PassLetter[random.Next(PassLetter.Length)];
 
            return result;
 
        }
    }

@Diamante · 05.08.2019, 02:02

именно

Добавлено через 43 минуты
коли с этим разобрались, переделай-ка генератор с конкатенации на StringBuilder, думаю будет полезным опыт оптимизации решения

@Defences · 05.08.2019, 02:19

Diamante, может хоть чуть-чуть оставим написание кода ТС'у

@Diamante · 05.08.2019, 02:39

Defences, так мы же ничего не писали, как минимум я...в остальном, имхо, правильный пинок, хочет учиться - пусть учится, о fw узнает, может свои тесты проведет

@AndreyVorobey · 05.08.2019, 11:04

учитывая базовые требования к паролю, который должен содержать разные комбинации символов, то можно все свести к такому решению )

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
using System;
using System.Linq;
 
namespace ConsoleApp96
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine(string.Join("", GenerationPassword(10)));
        }
 
        public static char[] GenerationPassword(int LenPassword)
        {
            Random Rand = new Random();
 
            return Enumerable.Range(0, LenPassword).Select(c => (char)Rand.Next(0x20, 0x7e)).Where(c => !char.IsControl(c)).ToArray();
        }
    }
}

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .
SDL3 для Web (WebAssembly): Работа со звуком через SDL3_mixer 8Observer8 08.02.2026 Содержание блога Пошагово создадим проект для загрузки звукового файла и воспроизведения звука с помощью библиотеки SDL3_mixer. Звук будет воспроизводиться по клику мышки по холсту на Desktop и по. . .	SDL3 для Web (WebAssembly): Основы отладки веб-приложений на SDL3 по USB и Wi-Fi, запущенных в браузере мобильных устройств 8Observer8 07.02.2026 Содержание блога Браузер Chrome имеет средства для отладки мобильных веб-приложений по USB. В этой пошаговой инструкции ограничимся работой с консолью. Вывод в консоль - это часть процесса. . .	SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .

@Diamante 3566 / 2507 / 1174 Регистрация: 14.08.2016 Сообщений: 8,219
	05.08.2019, 00:50
	Acc3di, не надо привязываться к формам или чему бы то ни было еще, codebehind это злое злище! напиши отдельный метод генерации, получающий на вход все необходимое и выдающий результат, так что бы его можно было хоть в консоли использовать хоть в формах, wpf,asp, да хоть wcf 1

@Diamante 3566 / 2507 / 1174 Регистрация: 14.08.2016 Сообщений: 8,219
	05.08.2019, 02:02
	именно Добавлено через 43 минуты коли с этим разобрались, переделай-ка генератор с конкатенации на StringBuilder, думаю будет полезным опыт оптимизации решения 1

@Defences 215 / 149 / 48 Регистрация: 28.12.2016 Сообщений: 716
	05.08.2019, 02:19
	Diamante, может хоть чуть-чуть оставим написание кода ТС'у 1

@Diamante 3566 / 2507 / 1174 Регистрация: 14.08.2016 Сообщений: 8,219
	05.08.2019, 02:39
	Defences, так мы же ничего не писали, как минимум я...в остальном, имхо, правильный пинок, хочет учиться - пусть учится, о fw узнает, может свои тесты проведет 1