1

Правильно ли написана функция.

Здесь должно быть тип если есть (верстак-1, дерево-2, палка-1) то выполняется функция если тоже самое только больше то выполняется функция если меньше то ничего.

Функция работает но не проверяться условия.


    $('#block_sword').click(function(){
        if(workbench=="1", wood=="2", stick=="1"){
          wood--;
          wood--;
          stick--;
          sword++;
          $('#wood_text').text(wood);
          $('#stick_text').text(stick);
          $('#sword_text').text(sword);
        }else if(workbench > "1", wood > "2", stick > "1"){
          wood--;
          wood--;
          stick--;
          sword++;
          $('#wood_text').text(wood);
          $('#stick_text').text(stick);
          $('#sword_text').text(sword);
        }else if(workbench < "1", wood < "2", stick < "1"){
          
        }
      });

2
  • Что это за запятые в условиях? Что за сравнения? Разбейте условия на мельчайшие части, смотрите их, смотрите тип, и сами условия тоже, тем же console.log().. Может хоть какая мысль появится, а то смотреть больно
    – InDevX
    21 мар 2021 в 15:13
  • 1
    @InDevX: причём не только смотреть, но и буквально боль физическая - силу facepalm не рассчитал :)
    – UModeL
    21 мар 2021 в 15:15

1 ответ 1

2

Выглядит всё это ооочень странно в целом........но если не вдаваться в суть что этот код делает, а только в условия, то можно так

$('#block_sword').click(function(){
    if (workbench < "1" && wood < "2" && stick < "1") //??? workbench < "1" || wood < "2" || stick < "1"   
        return;
    
    if (workbench >= "1" && wood >= "2" && stick >= "1") { //??? workbench >= "1" || wood >= "2" || stick >= "1"
        wood--;
        wood--;
        stick--;
        sword++;
        $('#wood_text').text(wood);
        $('#stick_text').text(stick);
        $('#sword_text').text(sword);
    }
});
4
  • 1
    Создает меч из дерева и палок x) У автора еще потенциальная ошибка с типами, "111" < "2" // true. Если хотя бы тут вместо "1", "2" записать числа, даже если прилетит строка в переменных - при сравнении она бдет преобразоваться в число и правильно работать. И... wood -= 2; 21 мар 2021 в 15:41
  • @OPTIMUSPRIME про типы я тоже вначале подумал....но потом посмотрел на wood--; wood--; stick--; sword++; и подумал что уж там то наверное в таком случае будет int)) 21 мар 2021 в 15:54
  • let x = "10"; console.log(--x); // 9 они умеют и со строкой) 21 мар 2021 в 15:59
  • @OPTIMUSPRIME ну, буде надеяться.....)) 21 мар 2021 в 16:01

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.