Осуществите рефакторинг для скорости: преобразуйте в дату

Я работаю на меня, я - работающий не по найму кодер, и в результате у меня нет роскоши обзоров кода или способности улучшиться основанный на одноранговом узле, программирующем. Я собираюсь использовать это в качестве осуществления, чтобы видеть, могло ли сообщество StackOverflow помочь рассмотреть простой метод, который я записал;

    internal static DateTime CONVERT_To_DateTime(int binDate)
    {
        //   3/10/2008 = 1822556159
        //   2/10/2008 = 1822523391
        //   1/10/2008 = 1822490623
        //   30/09/2008 = 1822392319
        //   29/09/2008 = 1822359551

        //   September 30th 2008
        //   1822392319 = 0x6c9f7fff
        //   0x6c = 108 = 2008 (based on 1900 start date)
        //   0x9 = 9 = September
        //   0xf7fff - take top 5 bits = 0x1e = 30

        //   October 1st 2008
        //   1822490623 = 0x6ca0ffff
        //   0 x6c = 108 = 2008
        //   0 xa = 10 = October
        //   0x0ffff  - take top 5 bits = 0x01 = 1

        //   OR using Binary (used by this function)
        //   a = 1822556159 (3/10/2008)
        //   1101100 1010 00011 111111111111111

        //   b = 1822523391 (2/10/2008)
        //   1101100 1010 00010 111111111111111

        //   c = 1822490623 (1/10/2008)
        //   1101100 1010 00001 111111111111111

        //   D = 1822392319 (30/09/2008)
        //   1101100 1001 11110 111111111111111

        //   Excess 111111 are probably used for time/seconds which
        //   we do not care for at the current time

        var BaseYear = 1900;

        //  Dump the long date to binary
        var strBinary = Convert.ToString(binDate);

        //  Calculate the year
        var strBYear = strBinary.Substring(0, 7);
        var iYear = Convert.ToInt32(strBYear, 2) + BaseYear;

        //  Calculate the month
        var strBMonth = strBinary.Substring(7, 4);
        var iMonth = Convert.ToInt32(strBMonth, 2);

        //  Calculate the day
        var strBDay = strBinary.Substring(11, 5);
        var iDay = Convert.ToInt32(strBDay, 2);

        // ensure that month and day have two digits
        var strDay = iDay < 10 ? "0" + iDay : iDay.ToString();
        var strMonth = iMonth < 10 ? "0" + iMonth : iMonth.ToString();

        //  Build the final date
        var convertedDate = iYear + strMonth + strDay;

        return DateTime.ParseExact(convertedDate, "yyyyMMdd", null);
    }

Это - метод, который берет числовое представление даты и преобразовывает его в Тип данных datetime. Я хотел бы, чтобы метод был рассмотрен для достигания самого быстрого времени выполнения, потому что он выполняется в цикле.

Любые комментарии к методу ценятся, поскольку это будет осуществлением для меня. я с нетерпением жду некоторых ответов.

5
задан Phillis 3 February 2010 в 14:52
поделиться

4 ответа

Вы выполняете манипуляции со строками. Это настоящий убийца производительности при использовании в узких циклах.

    static DateTime ToDateTime(int value)
    {
        var year = (int)((value & 0xff000000) >> 24);
        var month = (value & 0xf00000) >> 20;
        var day = (value & (0xf8000)) >> 15;

        return new DateTime(1900 + year, month, day);
    }

Вот как вы это делаете. Сначала возьмите 1822490623 и преобразуйте его в двоичный формат:

0110 1100 1010 0000 1111 1111 1111 1111

Это маска для года:

f    f    0    0    0    0    0    0 

Это для месяца:

0    0    f    0    0    0    0    0

И это для дня:

0    0    0    f    8    0    0    0

Значение «года» необходимо сдвинуть вправо на 6 * 4 бит, «месяц» - на 5 * 4 , а «день» - на 3 * 4 + 3 бит .

4
ответ дан 18 December 2019 в 10:44
поделиться

Добро пожаловать в сообщество, Филлис. :)

Антон прав, ваши манипуляции со строками будут медленными. Поскольку похоже, что вы используете параметр как битовое поле, я предлагаю изучить различные (гораздо более быстрые) битовые операторы: <<, >>, &, | и ~. Похоже, вы пытаетесь выполнять двоичные манипуляции, поэтому используйте операторы, созданные для этого.

Например. (непроверено, сразу без манжеты):

Вы начинаете со значения 0x6c9f7fff . Байт старшего разряда составляет год. Чтобы замаскировать все, что не является годом, сделайте что-то вроде:

int year = ((binDate & 0xFF000000) >> 24) + BaseYear;

Аналогично, следующие 4 бита - это месяц, поэтому:

int month = (binDate & 0x00F00000) >> 20;
int date = (binDate & 0x000F8000) >> 15;
return new DateTime(year, month, date);
4
ответ дан 18 December 2019 в 10:44
поделиться

Вместо преобразования в строку, затем в целые числа, а затем в строка, затем на сегодняшний день, просто получите целые числа путем сдвига и маскирования и создайте значение DateTime непосредственно из целочисленных значений:

binDate >>= 15;
int day = binDate & 31;
binDate >>= 5;
int month = binDate & 15;
binDate >>= 8;
int year = binDate + 1900;
return new DateTime(year, month, day);
10
ответ дан 18 December 2019 в 10:44
поделиться

Я предлагаю вам найти код C / C ++ который выполняет аналогичную работу; затем перенесите его на C #

-2
ответ дан 18 December 2019 в 10:44
поделиться
Другие вопросы по тегам:

Похожие вопросы: