Во-первых, вы не можете завершить любую логику ПОСЛЕ оператора возврата.Таким образом, все ваше состояние после return [numerator/gcd +"|"+ denominator/gcd];
бессмысленно. Вы должны завершить «скрытие» или «показ» вашего тега span до вас return
.
Во-вторых, если я понимаю, чтовы пытаетесь сделать правильно, почему бы вам просто не условно hide()
промежуток, еслиесть gcf 1?
то есть.
if(gcd === 1){
$('.span-to-hide').hide();
}else{
$('.span-to-hide').show();
}
Наконец, способ, которым вы возвращаете свой результат, отличается от всего, что я когда-либо видел.Не уверен, если вы делаете это по причине или нет ... но подумайте над пересмотром.
Редактировать Я признаю, что у меня небольшие проблемы с попыткой понять, что вы пытаетесь сделать, поэтому я перестану пытаться ответить на этот вопрос.Но я добавлю небольшой самородок, который может быть вам полезен в будущем ...
Если вы пытаетесь нацелиться на определенный элемент и у вас есть доступ к html,для вас почти всегда имеет смысл дать этому элементу id
.id
уникально по соглашению, поэтому нет причин пытаться нацеливаться на родителей ... или любой другой материалТочно нацеливайтесь на то, что вам нужно, структурируя ваш html таким способом, который будет наиболее простым для того, что вы пытаетесь сделать.Вы разработчик, вы на месте водителя.Не усложняйте свою работу.
Во-вторых , чтобы получить лучшее качество ответов на SO, вам следует структурировать свои вопросы немного более тщательно.Включите полные примеры, чтобы люди могли фактически выполнить ваш код или приблизиться к возможности повторить ваш сценарий.В любом случае, я надеюсь, что вы найдете свое решение раньше, чем позже.Продолжайте кодировать!Становится легче / тяжелее лол.