Программа для шифрования файлов в один контейнер. Оцените реализацию

D_Sergeevich · Регистрация: 06.02.2014

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

Приветствую.
Написал свою первую программу. Суть такая: Выбираются файлы, добавляются в контейнер(zip) и потом этот контейнер шифруется алгоритмом AES. Ключ и вектор это хэш(MD5) пароля.
Хочу что бы Вы как опытные в этом деле оценили реализацию, указали на ошибки и т.д. Особенно интересует реализация "CryptoClass" Интерфейс да, немного туповатый.

@anutka1201 · 27.02.2014, 12:28

Привет, в целом все неплохо, некоторые замечания:
- Достаточно долго соображала в первой функции, как мне открыть диалог бокс для добавления файла, возможно нужно сделать кнопку "добавить файлы"?
- Ну и когда просят вводить пароль хорошо бы, чтобы по нажатии кнопки "Enter" нажималась кнопка ок.
- MD5 слаба функция, или в этой реализации это не существенно?

Добавлено через 2 минуты
Ну и конечно хорошо бы используя ООП переписать)

D_Sergeevich · 27.02.2014, 12:38 **[ТС]**

Спасибо.
По поводу удобства интерфейса - можно не обращать на него внимания.
А что вы имеете введу переписать используя ООП? по конкретней можно?

D_Sergeevich · 28.02.2014, 14:31 **[ТС]**

апнусь

@Grishaco · 28.02.2014, 17:39

ZipException, при попытке распаковать в директорию, где есть файлы.

Добавлено через 25 минут
Вот могу поделится классом определяющим сложность пароля

Кликните здесь для просмотра всего текста

C#

public class PasswordStrength
    {
        private const string badPass = "Week; Use letters & numbers";
        private const string goodPass = "Medium; Use special charecters";
        private const string sameAsUsername = "Password is the same as username.";
        private const string shortPass = "Too Short Password";
        private const string strongPass = "Strong Password";
 
        public string passwordStrength(string password, string username)
        {
            int score = 0;
 
            //password < 4
            if (password.Length < 4)
            {
                return shortPass;
            }
 
            //password == username
            if (password.ToLower() == username.ToLower())
                return sameAsUsername;
 
            //password length
            score += password.Length*4;
            score += (checkRepetition(1, password).Length - password.Length)*1;
            score += (checkRepetition(2, password).Length - password.Length)*1;
            score += (checkRepetition(3, password).Length - password.Length)*1;
            score += (checkRepetition(4, password).Length - password.Length)*1;
 
            var regex = new Regex("/(.*[0 - 9].*[0 - 9].*[0 - 9])/");
 
            //password has 3 numbers
            if (regex.IsMatch(password))
                score += 5;
 
            regex = new Regex("/(.*[!,@,#,$,%,^,&,*,?,_,~].*[!,@,#,$,%,^,&,*,?,_,~])/");
            //password has 2 sybols
            if (regex.IsMatch(password))
                score += 5;
 
            regex = new Regex("/([a - z].*[A - Z])|([A - Z].*[a - z])/");
            //password has Upper and Lower chars
            if (regex.IsMatch(password))
                score += 10;
 
            regex = new Regex("/([a - zA - Z])/");
            var regex1 = new Regex("/([0 - 9])/");
            //password has number and chars
            if (regex.IsMatch(password) && regex1.IsMatch(password))
                score += 15;
 
            regex = new Regex("/([!,@,#,$,%,^,&,*,?,_,~])/");
            regex1 = new Regex("/([0-9])/");
 
            //password has number and symbol
            if (regex.IsMatch(password) && regex1.IsMatch(password))
                score += 15;
 
            regex = new Regex("/([!,@,#,$,%,^,&,*,?,_,~])/");
            regex1 = new Regex("/([a-zA-Z])/");
 
            //password has char and symbol
            if (regex.IsMatch(password) && regex1.IsMatch(password))
                score += 15;
 
            regex = new Regex(@"/^\w +$/");
            regex1 = new Regex(@"/^\d +$/");
            //password is just a nubers or chars
            if (regex.IsMatch(password) || regex1.IsMatch(password))
                score -= 10;
 
            //verifing 0 < score < 100
            if (score < 0) score = 0;
            if (score > 100) score = 100;
 
            if (score < 34) return badPass;
            if (score < 68) return goodPass;
            return strongPass;
        }
 
        public int passwordStrengthPercent(string password, string username)
        {
            int score = 0;
 
            //password < 4
            if (password.Length < 4)
            {
                return 0;
            }
 
            //password == username
            if (password.ToLower() == username.ToLower()) return 0;
 
            //password length
            score += password.Length*4;
            score += (checkRepetition(1, password).Length - password.Length)*1;
            score += (checkRepetition(2, password).Length - password.Length)*1;
            score += (checkRepetition(3, password).Length - password.Length)*1;
            score += (checkRepetition(4, password).Length - password.Length)*1;
 
            var regex = new Regex("/(.*[0 - 9].*[0 - 9].*[0 - 9])/");
 
            //password has 3 numbers
            if (regex.IsMatch(password))
                score += 5;
 
            regex = new Regex("/(.*[!,@,#,$,%,^,&,*,?,_,~].*[!,@,#,$,%,^,&,*,?,_,~])/");
            //password has 2 sybols
            if (regex.IsMatch(password))
                score += 5;
 
            regex = new Regex("/([a - z].*[A - Z])|([A - Z].*[a - z])/");
            //password has Upper and Lower chars
            if (regex.IsMatch(password))
                score += 10;
 
            regex = new Regex("/([a - zA - Z])/");
            var regex1 = new Regex("/([0 - 9])/");
            //password has number and chars
            if (regex.IsMatch(password) && regex1.IsMatch(password))
                score += 15;
 
            regex = new Regex("/([!,@,#,$,%,^,&,*,?,_,~])/");
            regex1 = new Regex("/([0-9])/");
 
            //password has number and symbol
            if (regex.IsMatch(password) && regex1.IsMatch(password))
                score += 15;
 
            regex = new Regex("/([!,@,#,$,%,^,&,*,?,_,~])/");
            regex1 = new Regex("/([a-zA-Z])/");
 
            //password has char and symbol
            if (regex.IsMatch(password) && regex1.IsMatch(password))
                score += 15;
 
            regex = new Regex(@"/^\w +$/");
            regex1 = new Regex(@"/^\d +$/");
            //password is just a nubers or chars
            if (regex.IsMatch(password) || regex1.IsMatch(password))
                score -= 10;
 
            if (score > 100) return 100;
            return (score);
        }
 
        private string checkRepetition(int pLen, string str)
        {
            string res = "";
            for (int i = 0; i < str.Length; i++)
            {
                bool repeated = true;
                for (int j = 0; j < pLen && (j + i + pLen) < str.Length; j++)
                {
                    repeated = repeated && (str[j + i] == str[j + i + pLen]);
                    if (j < pLen) repeated = false;
                    if (repeated)
                    {
                        i += pLen - 1;
                        repeated = false;
                    }
                    else
                    {
                        res += str[i];
                    }
                }
            }
            return res;
        }
    }

Взято отсюда и переделано на c#

D_Sergeevich · 28.02.2014, 17:41 **[ТС]**

Да там сборка кривая, если с заменой то оно не заменяет, а если без то исключение

Вот могу поделится классом определяющим сложность пароля

Да этот по лучше чем тот что я нашел. Спасибо ))

@anutka1201 3 / 3 / 1 Регистрация: 05.12.2013 Сообщений: 22
	27.02.2014, 12:28	2
	Привет, в целом все неплохо, некоторые замечания: - Достаточно долго соображала в первой функции, как мне открыть диалог бокс для добавления файла, возможно нужно сделать кнопку "добавить файлы"? - Ну и когда просят вводить пароль хорошо бы, чтобы по нажатии кнопки "Enter" нажималась кнопка ок. - MD5 слаба функция, или в этой реализации это не существенно? Добавлено через 2 минуты Ну и конечно хорошо бы используя ООП переписать) 0

D_Sergeevich 29 / 29 / 10 Регистрация: 06.02.2014 Сообщений: 156
	27.02.2014, 12:38 [ТС]	3
	Спасибо. По поводу удобства интерфейса - можно не обращать на него внимания. А что вы имеете введу переписать используя ООП? по конкретней можно? 0

D_Sergeevich 29 / 29 / 10 Регистрация: 06.02.2014 Сообщений: 156
	28.02.2014, 14:31 [ТС]	4
	апнусь 0

D_Sergeevich 29 / 29 / 10 Регистрация: 06.02.2014 Сообщений: 156
	28.02.2014, 17:41 [ТС]	6
	Да там сборка кривая, если с заменой то оно не заменяет, а если без то исключение Вот могу поделится классом определяющим сложность пароля Да этот по лучше чем тот что я нашел. Спасибо )) 0