Лаунчер xLauncher от XeroXP

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

 

(он первый выложил реализацию некоторых идей, которые мне до сих пор помогают)
и тогда скажу так: сделай лучше, если сможешь)

 

Если у человека есть хорошие идеи, это не значит, что он напишет хороший код, да и не беспокойся за меня, я себе всегда делаю всё самое лучшее.

 

Народ, 1.7.x запускает?

 

Нет.

 

у меня да (так как я допилил запуск)

 

Ну молодец! И что? Зачем постоянно писать, что у тебя всё работает и что ты допилил что-то? Без того уже давно понятно, что у тебя ЧСВ OVER 1000

 

Ну раз у тебя да, я рад за тебя.

 

1) и то, что у меня все работает (а отвечал я не тебе, я вообще не понимаю, зачем ты мне пишешь?)
2) постоянно (3 раза только написал + пункт 1)
3) у всех проблемы только из-за кривости рук (если есть проблема, то ты кривой)
4) чсв у меня как у бога (1000 это мало)

 

Terminator Т-850, лучше бы выложили фиксы, а не кичились ими. Люди спасибо скажут
Вон XeroXP дорабатывал Империю, со всеми делился, теперь ему столько людей благодарны

 

Terminator Т-850,

+1 )))

 

я еврей)

 

Не знает ли кто-нибудь почему при изменении кодировки файла конфига (ANSI -> UTF-8) перестают отображаться поля авторизации.

Terminator Т-850, жаль. Пользу бы принёс, а так тупо словами "У меня работает" (а вы все лохи) никому ты не поможешь.

Pentiumnet, сейчас попробую немного пошаманить с запуском 1.7.x и кину, что получилось

 

При авторизации выдает ошибку : "Сервер авторизации недоступен", в чем проблема подскажите пожалуйста?!

 

Встретил в коде довольно забавную конструкцию (цикл while). Знает кто-нибудь что по идее должна выполнять эта функция?

Код:

private static String calculateHash(MessageDigest algorithm, String fileName) throws Exception {
    FileInputStream fis = new FileInputStream(fileName);
    BufferedInputStream bis = new BufferedInputStream(fis);
    DigestInputStream dis = new DigestInputStream(bis, algorithm);

    while (dis.read() != -1);
    byte[] hash = algorithm.digest();

    return byteArray2Hex(hash);
}

[DOUBLEPOST=1397591798,1397579748][/DOUBLEPOST]О боги, что это такое в sendAuth?

Код:

label123:
while (true) {
    while (true) {
        while (true) {
            // Код
            break label123;
        }
    }
}

[DOUBLEPOST=1397632904][/DOUBLEPOST]Для чего делать так?

Код:

if (foo()) {
    if (bar()) {
        foobar();
    }
} else {
    foobar();
}

Если можно сделать проще...

Код:

if (!foo() || bar()) {
    foobar();
}

И для чего null приводить к какому-либо типу?

Код:

g2.drawImage(this.update, 42, 536, (ImageObserver) null);

После двухдневного разбора кода сложилось впечатление, что у автора раздвоение личности. Вот кусок в котором он пишет все хорошо, использует foreach, не пишет двойных if'ов, и вот сразу же через метод он забыл про существование foreach, заменяя его for'ами и while'ами, пишет безумные условия с пустым телом, обращается к статическим переменным через this, создает массивы с одним элементом . (Зато дизайн красивый )

Но паблик на то он и паблик... хотя прочитать учебник по java тоже было бы неплохо.[DOUBLEPOST=1397638571][/DOUBLEPOST]Потихоньку правлю код. Если хотите - помогайте (GitHub)

 

просто у этого лаунчера и есть двойная личность
все функции, которые ты привел выше, кроме одной (самой первой), по-моему, не XeroXP
изначально был лаунчер без массивов и циклов (представь одну и ту же функцию несколько раз подряд с одним замененным пораметром), кроме авторизации, в которой были циклы (все это ensirius-а), выглядело это все не очень хорошо, многое можно было упростить, хотя там в принципе ничего и не было кроме авторизации)
после пришел XeroXP и оставил от старого лаунчера процентов 25 в xLauncher-е

и дизайн делал dragoy, XeroXP его натянул на лаунчер

как-то так было, точнее не могу сказать)

а массивы с одним параметром нужны, чтобы люди могли туда еще параметров закинуть[DOUBLEPOST=1397642623,1397641903][/DOUBLEPOST]

Заметил, ты убираешь излишки) только там немного кодировка нарушена, судя по комментам в xSettings

 

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

Не тот случай. Там в массиве строка лежала. Использовался этот массив только в одном месте, и StringBuilder'ом объединялся в одну строку . Легче просто дописать в строку нужную информацию.

Дык кодировка файлов - windows-1251, а гитхаб ее не хавает.
Я бы исправил на UTF-8, но при каком-либо изменении названия текстовых полей в конфиге, магическим образом эти самые поля исчезают. Пока с этой проблемой не разбирался.

UPD: Разобрался, исправил

 

скорее уж привести код в хорошее состояние (просто он читабельный, но некрасивый)
ты, наверно, империю не видел)

 

Империя это пипец, Там столько суровых русских решений!

 

Terminator Т-850, видел, допиливал, но надоело и забил на нее. Там конечно нечто... Костыли на велосипедах.
А про читабельность - из-за глупых условий и циклов код трудно читать.