Javascript-форум (https://javascript.ru/forum/)
-   Ваши сайты и скрипты (https://javascript.ru/forum/project/)
-   -   Прошу оценить код моей javascript игры ) (https://javascript.ru/forum/project/48802-proshu-ocenit-kod-moejj-javascript-igry.html)

colmer 17.07.2014 15:30
Прошу оценить код моей javascript игры )
 
Написал небольшую игру про драконов. Все первоначальные задумки реализовал. Прошу прокомментировать мой код) Приятно было бы услышать конструктивную критику... Использовал только клиентскую часть.

http://fish.jollybee.ru/ - Сама "игра"
https://github.com/colmer/gamedev/bl...ter/index.html Ветка

рони 17.07.2014 15:35
colmer,
а как увидеть вживую демо? и какие правила

colmer 17.07.2014 15:55
Добавил в ссылку) Правила просты, выбираешь имя, оружие и аксессуар. и бьешься с драконом. Можно использовать аксессуар 1 раз за бой. Вероятность попадения 50/50 %

рони 17.07.2014 16:20
colmer,
Цитата:
Вы победили!
а где сразиться ещё? :)

colmer 17.07.2014 17:00
Добавил) А по поводу кода что скажите?)))

рони 17.07.2014 17:11
colmer,
многовато кода --- лог после добавления самбы прокручивался чтоб показать -- и по сравнению с картинками выглядит куцо - в самом логе непонятно немного -- возможно так -- или промазал или нанёс такой то урон - а то наносил урон наносил -- а никакого толку - и визуализация -- где жизнь героя полоской или часть фона на сердце

colmer 17.07.2014 18:28
Обыграл немного по другому, добавил жизней игроку, логам добавил время и выделил урон.

рони 18.07.2014 02:34
colmer,
location.reload(); -- может без перезагрузки ? ломает снова вводить ник )))

bes 18.07.2014 08:49
Цитата:
Сообщение от colmer
Правила просты, выбираешь имя, оружие и аксессуар. и бьешься с драконом. Можно использовать аксессуар 1 раз за бой. Вероятность попадения 50/50 %
не понял смысл игры

colmer 18.07.2014 13:32
Цитата:
Сообщение от рони (Сообщение 321599)
colmer,
location.reload(); -- может без перезагрузки ? ломает снова вводить ник )))
Готово)))

Цитата:
Сообщение от bes (Сообщение 321603)
не понял смысл игры
Пока что просто поучить Javascript )

Может пару советов по организации кода?))) Может засунуть все в методы объектов?)))

рони 18.07.2014 14:07
colmer,
рамка при фокусе в инпут
.welcome input {
width: 400px;
margin: 20px 0 0 50px;
height: 30px;
font-size: 30px;
line-height: 30px;
border-radius: 10px;
outline: none;
}

рони 18.07.2014 14:09
colmer,
как вариант ...
$( document ).ready(function() {

    $(".chuse").click( function() {
    		$(".welcome").fadeOut( function() {
    			$(".weapons").fadeIn();
    		} );
    });
    $("input").on("input",function() {
       var name = $.trim(this.value);
       this.value = name ;
       player.nickname = name;
       $(".chuse").prop({"disabled" : !name}).text(name ? "Начать" : "Выбрать").css({"fontSize" : name ? "12px" : "24px"}).stop(true, true).animate({"fontSize": name ? "24px" : "12px"},600)
    })



    var inventory = function (chose) {

colmer 18.07.2014 15:55
Цитата:
Сообщение от рони (Сообщение 321642)
colmer,
как вариант ...
$( document ).ready(function() {

    $(".chuse").click( function() {
    		$(".welcome").fadeOut( function() {
    			$(".weapons").fadeIn();
    		} );
    });
    $("input").on("input",function() {
       var name = $.trim(this.value);
       this.value = name ;
       player.nickname = name;
       $(".chuse").prop({"disabled" : !name}).text(name ? "Начать" : "Выбрать").css({"fontSize" : name ? "12px" : "24px"}).stop(true, true).animate({"fontSize": name ? "24px" : "12px"},600)
    })



    var inventory = function (chose) {
Если .css({"fontSize" : name ? "12px" : "24px"}) оставить, получается бешенная кнопка))) При каждом нажатии уменьшает и увеличивает текст, оставил :

$(".chuse").prop({"disabled" : !name}).text(name ? "В бой" : "Ввести").stop(true, true).animate({"fontSize": name ? "24px" : "12px"},600);
:)


Часовой пояс GMT +3, время: 05:11.