Познакомьтесь с пентестом веб-приложений на практике в нашем новом бесплатном курсе
public class SecutityPassword:IDisposable
{
private RijndaelManaged Transformer = new RijndaelManaged();
public SecutityPassword()
{
Transformer.IV = new byte[] { 12, 23, 34, 66, 56, 67, 78, 1, 1, 5, 22, 05, 67, 23, 89, 1 };
Transformer.Key = new byte[] { 02, 58, 94, 254, 3, 3, 3, 1, 54, 65, 34, 34, 45, 65, 77, 100, 02, 58, 94, 22, 43, 3, 53, 1, 54, 65, 34, 34, 45, 65, 77, 100 };
}
public string Coding(string SourceText)
{
try
{
byte[] toCode = Encoding.ASCII.GetBytes(SourceText);
ICryptoTransform Itrans = Transformer.CreateEncryptor(Transformer.Key, Transformer.IV);
MemoryStream buffer = new MemoryStream();
CryptoStream CryptoBuffer = new CryptoStream(buffer, Itrans, CryptoStreamMode.Write);
CryptoBuffer.Write(toCode, 0, toCode.Length);
CryptoBuffer.FlushFinalBlock();
byte[] result = buffer.ToArray();
return Convert.ToBase64String(result);
}
catch (Exception)
{
return SourceText;
}
}
public string Decoding(string SourceText)
{
try
{
byte[] tmp2 = Convert.FromBase64String(SourceText);
ICryptoTransform Idec = Transformer.CreateDecryptor(Transformer.Key, Transformer.IV);
MemoryStream buffer = new MemoryStream();
CryptoStream CryptoBuffer = new CryptoStream(buffer, Idec, CryptoStreamMode.Write);
CryptoBuffer.Write(tmp2, 0, tmp2.Length);
CryptoBuffer.FlushFinalBlock();
byte[] Decripted = buffer.ToArray();
return Encoding.ASCII.GetString(Decripted);
}
catch (Exception)
{
return SourceText;
}
}
#region IDisposable Members
public void Dispose()
{
Transformer.Clear();
}
#endregion
}Печально. Руки и ноги поотрывал бы за такой код, извините, за грубость.
Это первое, что программер должен уметь.
Для: Real-Mulik
Проблемы твоего кода:
* обработка исключений - выкинь её всю из этого класса. клиент класса должен знать, что происходит. он будет заниматься обработкой исключений
* закрытие ресурсов сразу по окончанию их работы
* внутреннее поле и IDisposable интерфейс не нужен здесь, обычно клиенты этого класса не будут шифровать, а потом сразу же дешифровать данные. Отработал методик по шифрованию/дешифрованию и закрыл всё за собой
* ключик и вектор шифрования дай клиенту задать свои
Переименовать сам класс и методы и можно продавать.
Перепиши и посмотрим
НЕЯВНО вызвать Dispose.
Одно правило забыли. Вернее даже написали о нём, но видно немножко не поняли (When an object is executing its finalization code, it should not reference other objects).
Dispose не должен вызываться из Finalize. Это не идентичные методы. Dispose работает и с управляемыми и неуправляемыми ресурсами, а Finalize только с неуправляемыми. Microsoft даже предложила шаблон для реализации вашего класса для корректного и надёжного закрытия ресурсов.
public class SomeClass : IDisposable
{
public void Dispose()
{
Dispose(true);
GC.SuppressFinalize(this);
}
public void Dispose(bool disposing)
{
if (!disposing)
{
// закрываем управляемые ресурсы
}
// закрываем неуправляемые
}
~SomeClass()
{
Dispose(false);
}
}Всё ставьте под сомнение, даже свои знания. Но не с этим был несогласен. А вот с этим выражением. В чём здесь ошибка, думаю поняли.
Программист должен писать код, который будет работать неважно 1000 раз/сек или 1 раз/неделю будет происходить сборака мусора.
Обучение наступательной кибербезопасности в игровой форме. Начать игру!