Стартап [WIP] Code Snippets - Полезные куски кода и некоторые неочевидные вещи. [+ Ликбез]

Она еще не релизная, все может изменится. Да и все это делается проще без всяких сетов арайсов и прочей лабуды.

Код:

    static {
        for(Package pack : Package.getPackages()) {
            String name = pack.getName();
            if(name.startsWith("net.minecraft.server")) {
                version = pack.getName().substring(21);
                break;
            }
        }
    }

 

С contains улыбнулся, спасибо, настроение++.

 

Только, ИМХО, лучше обойтись без магических чисел и сделать что-то типа:

Код:

public static String getNmsVersion() {
    String nmsPackage = "net.minecraft.server";
    for(Package pack : Package.getPackages()) {
        String name = pack.getName();
        if(name.startsWith(nmsPackage)) {
            int versionIndex = nmsPackage.length() + 1;
            return pack.getName().substring(versionIndex);
        }
    }
}

[DOUBLEPOST=1496867242,1496866319][/DOUBLEPOST]@Dereku, приведи свой пост к какому-то окончательному варианту, чтобы я его добавил в шапку. Или так оставишь?

 

Bukkit.getServer().getClass().getName().replace("org.bukkit.craftbukkit", "net.minecraft.server"); ???

 

Допилил все три варианта.

 

А смысл? Длинна же не динамическая.

 

Смысл в том, что нет магических чисел, вследствие чего код более понятен.[DOUBLEPOST=1496911671,1496911507][/DOUBLEPOST]@Dereku, закрепил

 

Он и так понятен был, всем кроме тебя).

 

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

 

Погоди, сейчас фаталити будет .
String s = Bukkit.getServer().getClass().getName();
s.substring(23, s.length()-12);

 

Bukkit.getServer().getClass().getPackage().getNane().replace("org.bukkit.craftbukkit", "net.minecraft.server");

 

А такое чтоли больше не работает?
Bukkit.getServer().getClass().getPackage().getName().replace(".", ",").split(",")[3];

 

Суть в том чтобы получить v1_12_R1, а не net.minecraft.server.v1_12_R1.

А зачем реплейсить?

 

Я хз. Нашел этот метод 2 года назад и юзаю все еще. Да и в других плагинах вроде также почемуто делают.[DOUBLEPOST=1496917430,1496917361][/DOUBLEPOST]https://www.spigotmc.org/wiki/nms-on-different-versions-without-reflection/
Вот тут это было.

 

Ты не понял, это было кккомбо против OsipXD .[DOUBLEPOST=1496922387,1496921881][/DOUBLEPOST]

Не осилили такое split("\\."), вот и закостылили.

 

Ну я так же подумал.

 

Ну всё. Теперь я потрачу годы на то чтобы разобраться в этом коде.
А теперь серьезно. Ты правда считаешь, что я не мог бы понять тот код без доп. переменной? Правда не понимаешь почему магические числа это плохо? Или прикидываешься?

 

Рили не понимаю, в чем касяк резать строку постоянной длинны статическими цифрами. Вот если бы там была неопределенная переменная, но это не так.

 

Проблема в стиле кода. Я в любом случае написал бы с переменной, чтобы даже не требовалось думать откуда взялась цифра 21 и не нужно было её считать вручную. А так это получается сразу два антипаттерна: хардкод и магические числа.
Да, в этом случае можно было просто проигнорировать, потому что кусок кода маленький и простой, но мне проще в любом коде соблюдать один стиль и не делать поблажек для конкретных кусков.

 

С тобой все ясно.