Javascript.RU

Создать новую тему Ответ
 
Опции темы Искать в теме
  #1281 (permalink)  
Старый 22.07.2015, 20:10
Аватар для cyber
I am Student
Отправить личное сообщение для cyber Посмотреть профиль Найти все сообщения от cyber
 
Регистрация: 17.12.2011
Сообщений: 4,415

Просто интересно кто из нас двоих дебил)
и так есть задача, есть пару селектов каждый зависит друг от друга, между ними есть кнопки которые как бы должны менять селекты местами.
Примерный вид моего решение такой
http://jsfiddle.net/50aq75cx/

Я беру делаю класс model, в котором базовый функционал, потом остальные элементы его наследуют и добавляю только метод для возврата запроса на сервер, так как там ангуляр, то все очень просто, я меняю только значения в скоупе)

Решение тимлида:
Написать для каждого селекта отдельный класс, и переставлять элементы на странице.

Вопрос кто из на дибил?
__________________
Цитата:
Если ограничения и условия описываются как "коробка", то хитрость в том что бы найти именно коробку... Не думайте о чем то глобальном - найдите коробку.
Ответить с цитированием
  #1282 (permalink)  
Старый 22.07.2015, 21:58
Аспирант
Посмотреть профиль Найти все сообщения от tysonfury2015
 
Регистрация: 22.07.2015
Сообщений: 68

Сообщение от cyber
Примерный вид моего решение такой
Что-то я запутался в Вашем коде. Застрял вот на этом куске:
function model () {
};

model.prototype.update = function ( data ) {
    var params = this.getParams( data );// вот тут вы получаете параметры, но где они используются? И почему в getParams подается data? Ведь в реализации, которая ниже, никакого приема аргумента не предусматривается?
    
    return new Promise ( function ( resolve, reject ) {
        request( this.getParams() );
    
        function request () {
            resolve();
            // get data from server
        }
    
    } );

};

function X () {
  this.title = "I am X";
  model.call( this );// зачем эта строчка, ведь в model ничего нет?
}

Вопросы в коде.

и, кстати,
X.prototype.getParams = function () {
    return {};
}

Какой смысл в этой ф-ции, если все что она делает -- создает и возвращает объект? Вместо нее можно было бы писать просто, {}, было бы то же самое.

Последний раз редактировалось tysonfury2015, 22.07.2015 в 22:07.
Ответить с цитированием
  #1283 (permalink)  
Старый 22.07.2015, 22:10
Аватар для cyber
I am Student
Отправить личное сообщение для cyber Посмотреть профиль Найти все сообщения от cyber
 
Регистрация: 17.12.2011
Сообщений: 4,415

tysonfury2015, это грубо воря псеводо код, я описал структуру)
Естественно, я не вставил бы код с продакшена)
А та функция, проводит магию и возвращает объект на основе внешних параметров, с которыми она шаманит)
__________________
Цитата:
Если ограничения и условия описываются как "коробка", то хитрость в том что бы найти именно коробку... Не думайте о чем то глобальном - найдите коробку.
Ответить с цитированием
  #1284 (permalink)  
Старый 22.07.2015, 22:21
Аспирант
Посмотреть профиль Найти все сообщения от tysonfury2015
 
Регистрация: 22.07.2015
Сообщений: 68

cyber,
Понятно. Что касается ваших разногласий, то тут все не так однозначно. Если, допустим, селектов на странице много, Все они имеют базовый функционал, и их все условно, можно разделить на 2 типа, то можно создать один базовый суперкласс, от него отнаследовать 2 подкласса, и ими уже штамповать селекты. Если инпута только 2, тогда отдельные классы для каждого конечно не нужны.
Ответить с цитированием
  #1285 (permalink)  
Старый 22.07.2015, 22:22
Аватар для EmperioAf
Профессор
Отправить личное сообщение для EmperioAf Посмотреть профиль Найти все сообщения от EmperioAf
 
Регистрация: 15.01.2015
Сообщений: 622

Сообщение от cyber
Решение тимлида:
Написать для каждого селекта отдельный класс, и переставлять элементы на странице.

Вопрос кто из на дибил?
Для непосвящённого в JS, angular и пр. радости жизни путь тимлида выглядит намного проще.
"Simple made easy"(c)
Ну вот серьёзно, почему в дальнейшем тяжелее будет поддерживать этот проект, если повесить на onclick по кнопке функцию, в которой просто у двух элементов меняются классы местами ?!
Ответить с цитированием
  #1286 (permalink)  
Старый 22.07.2015, 22:27
Аватар для cyber
I am Student
Отправить личное сообщение для cyber Посмотреть профиль Найти все сообщения от cyber
 
Регистрация: 17.12.2011
Сообщений: 4,415

Сообщение от EmperioAf
Ну вот серьёзно, почему в дальнейшем тяжелее будет поддерживать этот проект, если повесить на onclick по кнопке функцию, в которой просто у двух элементов меняются классы местами ?!
Потому что там там еще куча фигни между собой, настранице 10 элементов связаных между собой и combox в котором список берется с сервера в зависимости от его позиции, это самое простое что есть
__________________
Цитата:
Если ограничения и условия описываются как "коробка", то хитрость в том что бы найти именно коробку... Не думайте о чем то глобальном - найдите коробку.
Ответить с цитированием
  #1287 (permalink)  
Старый 22.07.2015, 22:30
Аватар для cyber
I am Student
Отправить личное сообщение для cyber Посмотреть профиль Найти все сообщения от cyber
 
Регистрация: 17.12.2011
Сообщений: 4,415

tysonfury2015, там по разному, на некоторых страницах, 2 на некоторых 3-4, все это связанно с минимум одной таблицей, но год можно использовать везде один таким способом
__________________
Цитата:
Если ограничения и условия описываются как "коробка", то хитрость в том что бы найти именно коробку... Не думайте о чем то глобальном - найдите коробку.
Ответить с цитированием
  #1288 (permalink)  
Старый 22.07.2015, 22:32
Аспирант
Посмотреть профиль Найти все сообщения от tysonfury2015
 
Регистрация: 22.07.2015
Сообщений: 68

Сообщение от EmperioAf
у двух элементов меняются классы местами
Вообще-то, это производительность может здорово посадить, хотя если игра стоит свеч -- я не против таких решений.
Ответить с цитированием
  #1289 (permalink)  
Старый 22.07.2015, 22:36
Аватар для cyber
I am Student
Отправить личное сообщение для cyber Посмотреть профиль Найти все сообщения от cyber
 
Регистрация: 17.12.2011
Сообщений: 4,415

tysonfury2015, а потом и получается то что я сейчас разргебаю)
__________________
Цитата:
Если ограничения и условия описываются как "коробка", то хитрость в том что бы найти именно коробку... Не думайте о чем то глобальном - найдите коробку.
Ответить с цитированием
  #1290 (permalink)  
Старый 22.07.2015, 22:37
Аватар для cyber
I am Student
Отправить личное сообщение для cyber Посмотреть профиль Найти все сообщения от cyber
 
Регистрация: 17.12.2011
Сообщений: 4,415

И если бы идея была тупая то сомневаюсь что мидл ( единственный кроме меня кто знает нормально js ) поддержал бы меня)
__________________
Цитата:
Если ограничения и условия описываются как "коробка", то хитрость в том что бы найти именно коробку... Не думайте о чем то глобальном - найдите коробку.
Ответить с цитированием
Ответ



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

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


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
Смишных картинок тред megaupload Оффтопик 3448 03.07.2023 09:47
React'а тред melky Оффтопик 246 13.11.2016 08:07
Webpack'а тред nerv_ Сборка проекта, утилиты 58 07.05.2016 13:46
Смешных историй тред monolithed Оффтопик 11 04.03.2015 04:20
Смишных видео с намеком тред godofjavascript Оффтопик 4 30.12.2012 23:50