В чем проблема где ошибка?

Question

[Anton Почти]

var step = 0, ewe = 1;
	

function sompil (){
	step = step + ewe;
	if (step > 5) break;
	console.log(step);
    return step;
		
};

Консоль показывает > index.html:18 Uncaught ReferenceError: sompil is not defined
Без if break работает , смысл в том что бы прервать функцию когда у step будет значение 5.

Answer 1

[Дмитрий Еремин]

//объявили 2 переменные - супер
var step = 0;
var ewe = 1;

//потом объявляем функцию
var sompil = function(localStep, localEwe){
   localStep += localEwe; //Оператор += позволяет добавлять значение слева к текущему значению (справа)
   console.log(localStep);   
   if (localStep <= 5) return localStep; //если localStep меньше/равен 5, вернуть его
}  

//а теперь самое главное. Функция возвращает значение (слово return). Возвращает куда?
//нужно объявить переменную, которая будет хранить результат выполнения функци
var result;
//и присвоить ей значение
result = sompil(step, ewe);
console.log(result);

Та-дам

Comments

[Anton Почти]

Cпасибо! Я еще допишу и покажу конечный результат .

[Anton Почти]

var step = 0, dalwe = 1 ;
	



function some(){
	
	
	step = step+dalwe;
	console.log(step)

	if(step<5) {

	$('#ola').animate({

		width: '+=95'
		},
		400, function() {})

	}
	else {
		step = 5
		return

	}
	
	


};


function somewho (){
	step = step - dalwe;
	console.log(step)
	if(step>0){

	
	$('#ola').animate({
		width: '-=95'
		},
		400, function() {})
	return 
	}
	else{
		step = 1	
		return
	}
}

Дописал =)

[Дмитрий Еремин]

Женя Почти: здорово) Работает, как задумано?

[Anton Почти]

Дмитрий Еремин: somewho > если else сразу срабатывает то происходит странная вещь > else{step = 1}
в этом месте какое бы значение я бы не давал то console.log() выдает -1

[Дмитрий Еремин]

Женя Почти: а почему странная?
У вас изначально step = 0
Потом вы выполняете функцию somewho()
Внутри функции вы от step отнимаете dalwe -> 0-1
Потом просите вывести значение степа в консоль -> -1
Потом у вас начинается проверка на step>0 -> естественно, проверка не выполняется.
Вы попадаете в else.
Там step становится равным 1. И все - вы же нигде не просите сообщить вам, чему равен step

В общем, попробуйте добавить логгирование в else -> else{step = 1; console.log(step); return}

Answer 2

[Алексей]

Что за break без цикла?
Замените на return <число> (не знаю что по вашей логике должно возвращаться в этом случае).

Comments

[Anton Почти]

после пяти нажатий "step" будет ровна "5", и вот тут нужно что бы при нажатий на кнопку не выполнялось функция

[Niomin]

Женя Почти: Тогда нужен не break, а return.

[Anton Почти]

Niomin: Спасибо!

[Niomin]

Не за что. Но вообще, прочитайте где-нибудь о том, что такое инкапсуляция. Хранить состояние и какие-то внутренние данные в глобальной области видимости — дурная практика. Стоит сделать хотя бы так:

var sompil = {

  ewe: 1,

  step: 0,

  next: function() {
    this.step += this.ewe;
    if (this.step > 5) return;
    console.log(this.step);
    return this.step;
  }
}

[Anton Почти]

Niomin: Я думаю со времен я к этому приду, ну а пока я осваиваю азы javascripta. Спасибо за помощь, правда большое спасибо!