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

[alexandrage]

Лооол. Рили такая есть проблема? Пока с таким не сталкивался....

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

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

 

[Dereku]

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

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

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

 

[OsipXD]

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

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);
        }
    }
}

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

 

[xDark]

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

 

[Dereku]

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

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

 

[alexandrage]

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

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);
        }
    }
}

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

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

 

[OsipXD]

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

Смысл в том, что нет магических чисел, вследствие чего код более понятен.
@Dereku, закрепил

 

[alexandrage]

Смысл в том, что нет магических чисел, вследствие чего код более понятен.
@Dereku, закрепил

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

 

[OsipXD]

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

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

 

[alexandrage]

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

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

 

[xDark]

Определение версии nms почти без проблем или "для тех, кто столкнулся с какулей от md_5 при определении версии nms из версии сервера":

Java 8 Stream:

public static String getNmsVersion() {
    String nmsPackage = "net.minecraft.server";
    String pckg = Arrays.stream(Package.getPackages())
            .filter(pk -> pk.getName().startsWith(nmsPackage))
            .map(pk -> pk.getName()).findFirst().orElse(null);

    if (pckg == null) {
        throw new RuntimeException("Failed to recognize nms version");
    }

    return pckg.substring(nmsPackage.length() + 1);
}

Java 7:

public static String getNmsVersion() {
    String nmsPackage = "net.minecraft.server";
    for (Package pack : Package.getPackages()) {
        if (pack.getName().startsWith(nmsPackage)) {
            return pack.getName().substring(nmsPackage.length() + 1);
        }
    }
    throw new RuntimeException("Failed to recognize nms version");
}

xDark:

//Кто нибудь проверьте на правильность
public static String getNmsVersion() {
    return Bukkit.getServer().getClass().getName()
            .replace("org.bukkit.craftbukkit.", "")
            .replace(".CraftServer", "");
}

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

 

[slavik123123123]

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

 

[Dereku]

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

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

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

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

 

[slavik123123123]

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

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

 

[alexandrage]

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

Ты не понял, это было кккомбо против OsipXD .

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

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

 

[Dereku]

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

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

 

[OsipXD]

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

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

 

[alexandrage]

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

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

 

[OsipXD]

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

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

 

[alexandrage]

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

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