Мертвый код

[Gattsu]

Предлагаю тему, где можно показывать участки не нужного кода, которые можно удалить или переписать в более быстрый вариант. Так как исходники обширные, много унаследовано другими сборками. Думаю тема будет интересна дотошным людям

Класс NewCrypt.java метод appendChecksum
Данное

appendChecksum

PHP код:

<?php 

    public static void appendChecksum(byte[] raw, final int offset, final int size)

    {

        long chksum = 0;

        int count = size - 4;

        long ecx;

        int i;

        

        for (i = offset; i < count; i += 4)

        {

            ecx = raw[i] & 0xff;

            ecx |= raw[i + 1] << 8 & 0xff00;

            ecx |= raw[i + 2] << 0x10 & 0xff0000;

            ecx |= raw[i + 3] << 0x18 & 0xff000000;

            

            chksum ^= ecx;

        }

        

        ecx = raw[i] & 0xff;

        ecx |= raw[i + 1] << 8 & 0xff00;

        ecx |= raw[i + 2] << 0x10 & 0xff0000;

        ecx |= raw[i + 3] << 0x18 & 0xff000000;

        

        raw[i] = (byte) (chksum & 0xff);

        raw[i + 1] = (byte) (chksum >> 0x08 & 0xff);

        raw[i + 2] = (byte) (chksum >> 0x10 & 0xff);

        raw[i + 3] = (byte) (chksum >> 0x18 & 0xff);

    } 

Участок, после цикла, является мертвым

PHP код:

<?php 

        ecx = raw[i] & 0xff;

        ecx |= raw[i + 1] << 8 & 0xff00;

        ecx |= raw[i + 2] << 0x10 & 0xff0000;

        ecx |= raw[i + 3] << 0x18 & 0xff000000; 

[Gaikotsu]

ну большую часть таких фрагментов и само IDE обычно выделяет, предупреждая о "мертвом коде" или не используемых локальных переменных или приватных методах.

[Gattsu]

ну большую часть таких фрагментов и само IDE обычно выделяет, предупреждая о "мертвом коде" или не используемых локальных переменных или приватных методах.

Ну если создать переменную и не использовать её, то ругается
Данный участок не подсветит, только если не прогрессивный алгоритм анализа, просто есть не тривиальные места

Потом этот участок кода, есть на всех сборках, а это довольно забавно

[Daan Raven]

Ну если создать переменную и не использовать её, то ругается
Данный участок не подсветит, только если не прогрессивный алгоритм анализа, просто есть не тривиальные места

Потом этот участок кода, есть на всех сборках, а это довольно забавно

Eclipse не подсвечивает, а IntelliJ IDEA подсвечивает. Вроде NetBeans тоже подсвечивает, но не факт (не проверял).
Таким образом можно найти множество кода на подобии:
[src='java']Player player = inst.getPlayer();
if (player == null)
return;
// тут какой-то код
if (какое-то условие) {
// тут какой-то код, player`у значение больше никакое не присваивается
if (player == null)
return;
// тут какой-то код
}[/src]

[Donatte]

http://habrahabr.ru/post/198938/
http://www.ucdetector.org/

Полезно почитать.

[Gattsu]

ну чего же тогда у всех кривые сборки, ни одной нормальной?

тоже полезно почитать, в этой книги описываются эти алгоритмы и оптимизации
http://www.ozon.ru/context/detail/id/3829076/

[Ashtone Grey]

ну чего же тогда у всех кривые сборки, ни одной нормальной?

Ну потому что их разработкой должна заниматься целая команда человек из 50, а не 1 программист-самоучка по 5 часов в неделю на выходных, как это обычно бывает.

[JavaMan]

Не волнуйтесь. JVM ничуть не глупее вас, и может не просто найти мертвый код, но даже убрать его из памяти. Эта оптимизация называется Dead Code Elimination. Конечно всегда надо убирать излишний код, но практического смысла это несет немного.

[Gattsu]

ну ума то не прибавится. От прочитанного складывается вывод, что люди то и программировать не умеют и базовых алгоритмов не знают. Кто-то из присутствующих программировал на языках без сборщика мусора, с ручным управление памяти?
Интересно просто
С++

PHP код:

<?php 

char* test() {

  return "test";

}



test(); 

[Smiler]

С++

PHP код:

<?php 

char* test() {

  return "test";

}



test();