Javascript.RU

Создать новую тему Ответ
 
Опции темы Искать в теме
  #31 (permalink)  
Старый 23.07.2015, 20:43
Профессор
Отправить личное сообщение для tsigel Посмотреть профиль Найти все сообщения от tsigel
 
Регистрация: 12.12.2012
Сообщений: 1,398

Lemme,
Дробление может быть очень хорошим, правда бывает и избытычным, но так вроде не плохо

...
promise.catch(this.onGetDataError.bind(this));
...


биндить соответстввенно надо если нужен контекст, в противном случае лучше сделать статичный метод.
Ответить с цитированием
  #32 (permalink)  
Старый 23.07.2015, 21:25
Аватар для Lemme
Профессор
Отправить личное сообщение для Lemme Посмотреть профиль Найти все сообщения от Lemme
 
Регистрация: 15.07.2015
Сообщений: 511

В 171 аргумент не нужен, я просто забыл его убрать, когда не было навигации, я так проверял переход между страницам.

num = num < 1 || typeof num !== 'number' ? 1 : num >= this.getDataFromStorage().pages.length ? this.getDataFromStorage().pages.length : num;

Почему ничего? Может я в чем то заблуждаюсь, но:

function test(num = 1){
	var numPages = 10;

	if (num < 1 || typeof num !== 'number') {
		num = 1;
	}
	else if (num >= numPages){
		num = numPages;
	}
	return num;
}

console.log(test(5)) // 5
console.log(test(100)) // 10
console.log(test('smth')) // 1

Последний раз редактировалось Lemme, 23.07.2015 в 21:34.
Ответить с цитированием
  #33 (permalink)  
Старый 23.07.2015, 21:55
Аватар для Lemme
Профессор
Отправить личное сообщение для Lemme Посмотреть профиль Найти все сообщения от Lemme
 
Регистрация: 15.07.2015
Сообщений: 511

Сообщение от Rise Посмотреть сообщение
Lemme, 159-я строка то где у вас находится посмотрите и что она принимает в качестве аргумента.
Если рассуждать так, то да, ничего 107 строка не делает, но я лишь решил перестраховаться, вдруг, потом захочется несколько изменить пагинацию. Например слушать изменение хэша.

т.е
<li><a href="#page=10">10</a></li>


В любом случае, спасибо за подсказку =)
Ответить с цитированием
Ответ



Опции темы Искать в теме
Искать в теме:

Расширенный поиск


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
Преобразование ТАБЛИЦЫ с родителями в json объекты Patr56 Общие вопросы Javascript 5 28.07.2014 12:30
Не могу распарсить JSON. gorenie jQuery 3 29.11.2013 22:26
JSON - отобразить данные таблицы oracle / mssql / mysql ecivgamer Javascript под браузер 3 14.11.2012 18:17
json данные не грузятся в windows XP но грузятся в w7 rustamaha Элементы интерфейса 2 28.11.2011 12:35
JSON или JSONP для запросов на другой сервер? Метод GET, для длинных сообщений? Kotakota jQuery 5 23.08.2011 23:12