Это правильный способ разделения кода на более мелкие функции?
Не совсем, поскольку ваша check_values
функция теперь является частью глобального объекта window
.Утечка объектов в глобальное пространство - это плохо, ладно?
К сожалению, существует так много способов, которыми можно сделать, что трудно понять, с чего начать.
Есливаш код невелик, лучше всего оставить его внутри замыкания внутри вашей $(document).ready()
функции:
$(document).ready(function(){
function check_values() {
...
}
$("form#create_form").submit(function() {
...
var is_okay = check_values(...);
...
});
});