Пользовательский код кодировки Base32 C#

Question

Я написал следующий код для кодирования bitarray в пользовательскую строку кодировки base32. Моя идея заключается в том, что пользователь может смешать порядок массива base32 в соответствии с требованиями и может добавлять похожие символы, такие как I и 1 и т. Д.

Мое намерение задать вопрос: правильно ли написан код или не хватает основы. Насколько я знаю, он генерирует результат в соответствии с требованием, однако я хочу просто проверить код здесь. Если есть недостатки, дайте мне знать.

У пользователя будет строка, которая должна быть закодирована base32. Поэтому в его функции он назвал бы это таким образом.

BitArray ba = new BitArray(Encoding.UTF8.GetBytes(CustomString)); 
GenerateBase32FromString(ba); 

Теперь GenerateBase32FromString как ниже

static string GenerateStringFromKey(BitArray ba) 
{ 
    try 
    { 
     // user will modify the order as per requirement. 
     char[] cProdKeyPoss = "ABGCD1EF2HI3KL4MN5PQ6RS7TV8WX9YZ".ToCharArray(); 
     StringBuilder finalstring = new StringBuilder(); 
     // add zero value bits to round off to multiple of 5 
     //ba.Length = ba.Length + (5-(ba.Length % 5)); 
     // remove bits to round off to multiple of 5 
     ba.Length = ba.Length - (ba.Length % 5); 
     Console.WriteLine("ba.length = " + ba.Length.ToString()); 

     for (int i = 0; i < ba.Length; i = i + 5) 
     { 
      int[] bitvalue = { 0, 0, 0, 0, 0 }; 

      if (ba.Get(i) == true) 
       bitvalue[0] = 1; 

      if (ba.Get(i + 1) == true) 
       bitvalue[1] = 1; 

      if (ba.Get(i + 2) == true) 
       bitvalue[2] = 1; 

      if (ba.Get(i + 3) == true) 
       bitvalue[3] = 1; 

      if (ba.Get(i + 4) == true) 
       bitvalue[4] = 1; 

      int temp = (16 * bitvalue[0]) + (8 * bitvalue[1]) + (4 * bitvalue[2]) + (2 * bitvalue[3]) + (bitvalue[4]); 
      finalstring.Append(cProdKeyPoss[temp].ToString()); 
     } 
     Console.WriteLine(finalstring.ToString()); 
     return finalstring.ToString(); 
    } 
    catch (Exception ex) 
    { 
     Console.WriteLine(ex.Message); 
     return null; 
    } 
} 

Я держал оба варианта, где я буду рубить биты раунда к кратному 5 или добавит дополнительные нулевые биты значение, чтобы сделать его множественным из 5.

Answer 1

несколько предложений:

• вы не имеют ни малейшего представления о «дополнения» - так что вы не можете
• Передайте ключевые символы в качестве параметра для метода. Вы можете иметь перегрузку для «по умолчанию», если вы хотите
• дать либо StringBuilder емкость, чтобы начать с, или создать обугленного массив нужной длины и создать строку непосредственно из этого
• Ловля Exception почти всегда плохая идея, и это, безусловно, здесь. Любое исключение было бы связано с ошибкой в ​​этом коде, поэтому просто позвольте этому пузыриться в стеке

• Сравнение с «истинным» всегда выглядит вонючим ко мне, и я лично ставил if/for/while/etc в телах блоков даже для отдельных операторов, так что я бы

  if (ba.Get(i)) 
  { 
      bitValue[0] = 1; 
  } 
  

• Там нет никакого реального смысла в массив битов, чтобы начать с. Почему бы просто не добавить значение, начинающееся с 0?

  if (ba.Get(i)) 
  { 
      temp += 16; 
  } 
  // etc 
  

• Повторные код, аналогичный предлагает цикл:

  int temp = 0; 
  for (int j = 0; j < 5; j++) 
  { 
      if (ba.Get(i + j)) 
      { 
       // This could just be "1 << j" if you don't mind the 
       // results being different to your current code 
       temp += 1 << (4 - j); 
      } 
  } 
  

• методы библиотеки не должны писать в консоль
• Не называйте ToString, когда вы получили правильный характер - просто вызовите Append(char) (или установите значение в массиве char результата).