Код из Delphi в С++!?

Question

[s14e]

Есть простой код на delphi, суть - шифрует серийный номер диска уже проверенной и рабочей функцией:

const
  SEED_SIZE = 16;
  SEED:  array[0..SEED_SIZE - 1] of Byte = (
    $51, $2F, $F6, $AE, $46, $8D, $DE, $5D, $D5, $73, $0A, $6D, $12, $56, $83, $78
  );

  function xordata(InStr: RawByteString; InSize: Integer): RawByteString;
  var
    i, j: Integer;
    bt: byte;
  begin
    SetLength(Result, InSize);
    j := 0;
    for i := 0 to InSize - 1 do
    begin
      if j = 16 then j := 0;
      bt := Byte(InStr[i + 1]) xor SEED[j];
      bt := bt xor i;
      Byte(Result[i + 1]) := bt;
      inc(j);
    end;
  end;

xordata(SERIAL_NUMBER, SERIAL_NUMBER_LENGTH);

Нужно данное решение грамотно перевести на С++. Господа знатоки прошу помочь, как грамотно и кратко написать такой код?

Comments

[d'Ivan]

тип RawByteString не описан

[s14e]

Роман Мирр:

InStr: RawByteString; - Обычный AnsiString, строка с серийным номером который потом в функции ксорим.

[Mercury13]

Это стандартный тип юникодных Delphi, примерно эквивалентный AnsiString D7.

Answer 1

[Mercury13]

InStr/InSize — немного избыточно по дельфийским меркам и можно преобразоватть в const char* data / size_t length.

Результат — ну std::string, разумеется. Или std::vector<char>. Что вместо SetLength — читай доку.

InStr[i + 1] → data[i]. Остальные массивы нумеруются с нуля, всё как и было.

Inc(j) → ++j.

Мне кажется, этот код механически переписывался с Си на Паскаль и переписать обратно будет несложно.

Ещё тут магическое число 16 — догадайся, на что заменять. Кстати, не стоит размазывать по коду две строки, связанные с j.

Comments

[s14e]

unsigned char key[16] = {
0x51, 0x2F, 0xF6, 0xAE, 0x46, 0x8D, 0xDE, 0x5D, 0xD5, 0x73, 0x0A, 0x6D, 0x12, 0x56, 0x83, 0x78
};

std::string xordata(std::string InStr, int InSize)
{
std::string result(InSize, 0);
int i; int j;
unsigned char bt;
j = 0;

for (i = 0; i < InSize; ++i)
{
if (j == 16) j = 0;
bt = unsigned char(InStr[i] ^ key[j]);
bt = bt ^ i;
result[i] = bt;
++j;
}
return result;
}

Result есть, тот который был необходим, Mercury13, пожалуйста проверьте есть ли ошибки в коде? Как то оптимизировать можно ли, если вообще нужно.. Спасибо.

[Mercury13]

s14e: Что, код настолько критичен, что нужна оптимизация?
1. const char* InStr — этого хватит. На худой конец const std::string& InStr.
2. result[i] = unsigned char(InStr[i] ^ key[ i & KEY_LEN_MASK] ^ i);
KEY_LEN_MASK = 15, разумеется.