ho scritto questo codice in C#
per crittografare una stringa con una chiave:Aiutami con la crittografia XOR
private static int Bin2Dec(string num)
{
int _num = 0;
for (int i = 0; i < num.Length; i++)
_num += (int)Math.Pow(2, num.Length - i - 1) * int.Parse(num[i].ToString());
return _num;
}
private static string Dec2Bin(int num)
{
if (num < 2) return num.ToString();
return Dec2Bin(num/2) + (num % 2).ToString();
}
public static string StrXor(string str, string key)
{
string _str = "";
string _key = "";
string _xorStr = "";
string _temp = "";
for (int i = 0; i < str.Length; i++)
{
_temp = Dec2Bin(str[i]);
for (int j = 0; j < 8 - _temp.Length + 1; j++)
_temp = '0' + _temp;
_str += _temp;
}
for (int i = 0; i < key.Length; i++)
{
_temp = Dec2Bin(key[i]);
for (int j = 0; j < 8 - _temp.Length + 1; j++)
_temp = '0' + _temp;
_key += _temp;
}
while (_key.Length < _str.Length) _key += _key;
if (_key.Length > _str.Length) _key = _key.Substring(0, _str.Length);
for (int i = 0; i < _str.Length; i++)
if (_str[i] == _key[i]) { _xorStr += '0'; } else { _xorStr += '1'; }
_str = "";
for (int i = 0; i < _xorStr.Length; i += 8)
{
char _chr = (char)0;
_chr = (char)Bin2Dec(_xorStr.Substring(i, 8)); //ERROR : (Index and length must refer to a location within the string. Parameter name: length)
_str += _chr;
}
return _str;
}
Il problema è che ottengo sempre l'errore quando voglio decifrare un testo encryted con questo codice:
string enc_text = ENCRYPT.XORENC("abc","a"); // enc_text = " ♥☻"
string dec_text = ENCRYPT.XORENC(enc_text,"a"); // ArgumentOutOfRangeException
Eventuali indizi?
Tutto quello che posso dire è eh? :) Forse è un esercizio educativo, ma non è necessario convertire i caratteri in stringhe, xorli manualmente e poi riconvertirli in una stringa. Come dimostrato dalle funzioni Dec2Bin e Bin2Dec, il char può essere convertito da e verso gli init con cast, quindi prendi char da entrambe le stringhe, applica l'operatore '^' xor e metti in una nuova stringa. – tyranid
Sarebbe utile se si specifica quale errore si sta ottenendo :) –
Inoltre, è possibile utilizzare StringBuilders anziché Stringhe. Le stringhe sono immutabili (non possono essere modificate), quindi cose come _str + = _temp; genera una nuova stringa ogni volta, il che rende questo metodo inutilmente pesante/costoso. Utilizzare un StringBuilder e .Append (temp). –