Комбинаторика: раскритиковать код

@Denis Ivanov · Регистрация: 28.10.2013

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

Добрый день, только начинаю учить c#. Задача - поиск пароля, где psw[1..n] - пароль, случайно написанный,
а alp[1..k] - алфавит. Комбинаторика говорит о размещении, смысл ясен, но хочется чтобы более смыслящие поругали мой бездарный код.

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
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
    class Operation
    {
        
        // вычисляем количество элементов массива
        public int Len(double k, double n){
            double x = Math.Pow(k, n);
            return Convert.ToInt32(x);
 
        }
 
       
        string[] pwd;
        string[] alp;
        public int z = 0;
 
 
        public string[] Pwd { 
            get { return pwd; }
            set { pwd = value; }
        }
 
        public string[] Alp
        {
            get { return alp; }
            set { alp = value; }
        }
 
 
 
        // рекурсивный перебор вариантов
        public void Fun(int k,int n)
        {
            if (k == n)
                foreach (string x in pwd)
                    Console.WriteLine(x);
            else
            {
                z++;
                for (int i = z; i <= k; i++)
                {
                    for (int j = 0; j < alp.Length; j++)
                        pwd[i].Insert(i,alp[j]);
                        if (Match("123",pwd[i])) break;
                        else
                    Fun(k + 1, n);
 
                }
                   
            }
        }
 
        public bool Match(string a, string b)
        {
            if (a == b) return true;
            else
                return false;
            
            
        }
 
 
 
    }

@titans2011 · 26.09.2014, 17:58

Ну не вдаваясь в логику, можно немного порефакторить ваш код. Например вместо этого:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
string[] pwd;
        string[] alp;
        public int z = 0;
 
 
        public string[] Pwd { 
            get { return pwd; }
            set { pwd = value; }
        }
 
        public string[] Alp
        {
            get { return alp; }
            set { alp = value; }
        }

Написать так:

C#
1
2
3
        public int z = 0; 
        public string[] Pwd { get;set; }
        public string[] Alp { get;set; }

Добавлено через 5 минут
Вот эта функция зачем она нужна?

C#
1
2
3
4
5
6
7
8
public bool Match(string a, string b)
        {
            if (a == b) return true;
            else
                return false;
            
            
        }

Вы знаете, что стринг это ссылочный тип данных, и в данном конкретном случае вы сравниваете адреса памяти?
В таком случае тут вместо этого:

C#
1
2
3
4
5
for (int j = 0; j < alp.Length; j++)
                        pwd[i].Insert(i,alp[j]);
                        if (Match("123",pwd[i])) break;
                        else
                    Fun(k + 1, n);

Лучше написать так:

C#
1
2
3
4
5
6
for (int j = 0; j < alp.Length; j++)
{
        pwd[i].Insert(i,alp[j]);
        if (pwd[i].Equalse("123")) break;
        else Fun(k + 1, n);
}

Добавлено через 40 секунд
И не экономньте на скобках. Отсутствие фигурных скобок приводит к ухудшению читаемости кода!

@XRoy · 26.09.2014, 18:48

Сообщение от titans2011

Вот эта функция зачем она нужна?

C#
1
2
3
4
5
6
public bool Match(string a, string b)
    {
     if (a == b) return true;
       else
           return false;
}

Вы знаете, что стринг это ссылочный тип данных, и в данном конкретном случае вы сравниваете адреса памяти?

Он то ссылочный, но == перегружен для него, так что сравниваются значения. Это ведь не Java.

@Denis Ivanov · 27.09.2014, 13:00 **[ТС]**

Спасибо всем за критику, так правда лучше

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .
Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .

@Denis Ivanov 2 / 2 / 2 Регистрация: 28.10.2013 Сообщений: 39
	27.09.2014, 13:00 [ТС]
	Спасибо всем за критику, так правда лучше 0

Комбинаторика: раскритиковать код

Решение