Перейти к содержанию

[NPC]CashCoins


SupR1g

Рекомендуемые сообщения

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

Я тебя не пытался "зацепить", я лишь желал увидеть свои ошибки или возможные варианты усовершенствования/модернизирования скрипта.

Ссылка на комментарий
Поделиться на другие сайты

я лишь желал увидеть свои ошибки или возможные варианты усовершенствования/модернизирования скрипта.

Да пожалуйста

1) 18 и 72 строчки

2)

input .@Amount;
if( .@Amount < 1 || .@Amount > 10 ){
бла-бла}

Меняем на

input(.@Amount, 1, 10);

Хотя негоже не оставлять юзверю шанса передумать в последний момент. Я бы сделал 0 для выхода.

3) Во втором кейсе тоже можно убрать лишнюю проверку

input(.@Amount, 1, (countitem(.@Coin)>10?10:countitem(.@Coin)));

4) В конце


break;
case 4:
break;

можно заменить на

                case 4:
break;

  • Upvote 1
Ссылка на комментарий
Поделиться на другие сайты

я лишь желал увидеть свои ошибки или возможные варианты усовершенствования/модернизирования скрипта.

Да пожалуйста

1) 18 и 72 строчки

2)

input .@Amount;
if( .@Amount < 1 || .@Amount > 10 ){
бла-бла}

Меняем на

input(.@Amount, 1, 10);

Хотя негоже не оставлять юзверю шанса передумать в последний момент. Я бы сделал 0 для выхода.

3) Во втором кейсе тоже можно убрать лишнюю проверку

input(.@Amount, 1, (countitem(.@Coin)>10?10:countitem(.@Coin)));

4) В конце


break;
case 4:
break;

можно заменить на

                case 4:
break;

Спасибо)

Ссылка на комментарий
Поделиться на другие сайты

1.

Зачем писать полное название самому

[ ^0000FF"+countitem(8903)+"^000000 ]x ^FF0000Gold Coins.^000000";

если можно опять же обойтись

[ ^0000FF"+countitem(8903)+"^000000 ]x ^FF0000" + getitemname(8903) + ".^000000";

2.

Лучше добавить сколько именно не хватает коинов

Ссылка на комментарий
Поделиться на другие сайты

1.

Зачем писать полное название самому

[ ^0000FF"+countitem(8903)+"^000000 ]x ^FF0000Gold Coins.^000000";

если можно опять же обойтись

[ ^0000FF"+countitem(8903)+"^000000 ]x ^FF0000" + getitemname(8903) + ".^000000";

Не особо важный момент. Но спасибо.

2.

Лучше добавить сколько именно не хватает коинов

Зачем? Если есть это:

mes "У Вас есть сейчас:";

mes "- [ ^0000FF"+countitem(8903)+"^000000 ]x ^FF0000Gold Coins.^000000";

mes "- [ ^0000FF"+countitem(8904)+"^000000 ]x ^FF0000Silver Coins.^000000";

mes "- [ ^0000FF"+countitem(8905)+"^000000 ]x ^FF0000Bronze Coins.^000000";

mes "- [ ^0000FF"+countitem(673)+"^000000 ]x ^FF0000Steel Coins.^000000";

Ссылка на комментарий
Поделиться на другие сайты

1.Удобность

2.Игрокам не придеться считать сколько не хватает

1. Пусть будет так.

2. Не хватает до чего?До монеты? о_О Я не могу понять, поясни.

Ссылка на комментарий
Поделиться на другие сайты

OFFTOP: зачем писать оранжевым цветом? Все сообщения выглядят вырвиглазно и читаются с трудом =\ Глаза реагируют ппц =(

Ссылка на комментарий
Поделиться на другие сайты

OFFTOP: зачем писать оранжевым цветом? Все сообщения выглядят вырвиглазно и читаются с трудом =\ Глаза реагируют ппц =(

Привычка, если очень трудно читать, выделяй мышкой мой текст :P

Изменено пользователем SupR1g
Ссылка на комментарий
Поделиться на другие сайты

OFFTOP: зачем писать оранжевым цветом? Все сообщения выглядят вырвиглазно и читаются с трудом =\ Глаза реагируют ппц =(

Привычка, если очень трудно читать, выделяй мышкой мой текст :P

Если ты хочешь что бы тебя услышали, то будь добр пиши нормальным цветом. Потому что ради тебя никто твой текст мышкой выделять не будет.

Спасибо.

Ссылка на комментарий
Поделиться на другие сайты

OFFTOP: зачем писать оранжевым цветом? Все сообщения выглядят вырвиглазно и читаются с трудом =\ Глаза реагируют ппц =(

Привычка, если очень трудно читать, выделяй мышкой мой текст :P

Если ты хочешь что бы тебя услышали, то будь добр пиши нормальным цветом. Потому что ради тебя никто твой текст мышкой выделять не будет.

Спасибо.

Кому интересно, тот не будет придираться из-за такой мелочи, а просто прочитает. Но я тебя услышал. Спасибо.

Ссылка на комментарий
Поделиться на другие сайты

Кому интересно, тот не будет придираться из-за такой мелочи, а просто прочитает. Но я тебя услышал. Спасибо.

Не туда логика пошла. Это не придирки, а банальное неудобство чтения. Лично мной воспринимается как неуважение к собеседнику. Ну а помогать\советовать человеку, который элементарно не уважает тебя - как-то.. не хочется, да?

P.S. Оффтоп пошел, пора возвращаться в основное русло темы.

Ссылка на комментарий
Поделиться на другие сайты

Кому интересно, тот не будет придираться из-за такой мелочи, а просто прочитает. Но я тебя услышал. Спасибо.

Не туда логика пошла. Это не придирки, а банальное неудобство чтения. Лично мной воспринимается как неуважение к собеседнику. Ну а помогать\советовать человеку, который элементарно не уважает тебя - как-то.. не хочется, да?

P.S. Оффтоп пошел, пора возвращаться в основное русло темы.

Мы поняли друг друга.

  • Upvote 1
Ссылка на комментарий
Поделиться на другие сайты

×
×
  • Создать...
Яндекс.Метрика