Ветки фан-рефакторинга по JS

Question

[spmbt]

Из темы «Весёлые (сосисочные) рефакторинги»
предлагаю выделить обсуждение и организацию одной или больше веток по JS сюда, чтобы согласовать все открытые вопросы и не рассылать по 10+ писем каждому участнику.
Отметившиеся в теме с указанием языка JS на 00:00 15 мар 2013 — такие участники:
kabzon
medved13
termi
vooD
spmbt

werdender
maxatwork
ariser
ahmpro
meettya
firestarter
Athari
kafeman
BrainInTheBox

Столько человек может хватить на 2 параллельных рефакторинга: по 4-6 человек каждый. На первую группу материал вроде бы есть, который подал vooD. Какие будут правила? Если предложенные с 1 ведущим и игрой «в тёмную», когда только ведущий видит код каждого шага, то кто будет ведущим? vooD, берёте на себя такое дело? Или возьмётся быть ведущим первый предложивший, kabzon?

Если будут другие правила, например, с открытым кодом, выкладываемым пошагово в Гитхаб, то, возможно, и юмора не будет, и другие форки сделают из понравившегося варианта — получится разброд. Для пользы дела был бы хорош открытый вариант, а для фана — закрытый. На каком остановимся?

Впрочем, ничто не мешает найти 3 образца кодов, 2 сделать закрытыми, а 1 открытый и попробовать оба варианта.

Ведущему по классике (схеме ilichme ) нужно каждый шаг отписывать в Гитхабе, но код туда не выкладывать до конца. Каждые день-два интересоваться, кто возьмётся следующим, кто отказался, и следить за заранее подготовленной очередью. Тут и работы, и внимательности много надо.

Для открытой схемы ведущий тоже нужен, и ему тоже есть работа — принимать пулл-реквесты. Но плюс будет в том, что следить за ходом могут все и, наверное, включаться в рефакторинг вне очереди тоже могут. Но пропадёт эффект испорченного телефона. Если не выберем ведущего первого типа, то придётся выбирать второго типа, которым может быть в том числе участником, и сам рефакторить код.

P.S.: vooD мне показал репозиторий с кодом. Если другим показал тоже, то предлагаю его поставить на открытый вариант рефакторинга.

UPD: (0:55) С первым кодом определились, на него по мере интереса переходят желающие. На мой взгляд, надо срочно подыскать мозговым штурмом ещё пару хороших вариантов кода. Чтобы:
1) было, что рефакторить;
2) код бы работал и мог проверяться;
3) был бы сам по себе интересен — кусок либы, плагина, игрушка, инструмент.

Comments

[kafeman]

Гайдлайны будут?

[Silver_Clash]

Неплохо бы правила подробнее описать. Потому что либо я туго соображаю, либо просто время позднее :)

Answer 1

[Vladislav Klimanov]

Ну раз исходник для рефакторинга уже был показан сообществу, то на закрытый вариант он не годится, т.к. не получится испорченного телефона.

делимся на группы, можно к примеру так(3x5):
I:
kabzon
medved13
termi
vooD
spmbt

II:
werdender
maxatwork
ariser
ahmpro
meettya

III:
firestarter
Athari
kafeman
BrainInTheBox
Silver_Clash

группа выбирает ведущего(либо самовыдвиженец)

закрытый вариант игры(aka испорченный телефон):
ведущий исходник отправляет следующему человеку в группе
у игрока есть 2 дня на то, чтобы прислать код обратно
если в течении 2х дней кода нет, то код с предыдущей итерации отправляется следующему в списке, а не приславший выбывает из игры
ведущий после каждой отправки исходника отписывается о отправке
ведущий хранит у себя исходники всех игроков, чтобы в последствии сделать diff
игрок может отказаться править код, тогда ход переходит следующему в списке
игрок может посчитать, что код его устраивает, в этом случае ход опять-таки переход к следующему

открытый вариант игры:
имеется общий репозиторий, в который пул-реквестами игроками вносятся изменения
ведущий их принимает
/* буду рад, если кто-то дополнит */

Comments

[ariser]

Полагаю, что участвовать таки хотят все. Т. е. проще выбрать открытый вариант, при котором и ведущий сможет коммитить свою версию.
Однако ведущий одной группы при «закрытом» варианте может рефакторить в другой — так тоже никто не уйдёт обиженным.

[spmbt]

Я бы участвовал в любой группе, и лучше бы чувствовал 1 из 5, а не из 15. Все на один проект — слишком много, особенно, толпой. Вижу большой проблемой то, что тестов нет. Впорос тестирования выпадает на ведущего, чтобы он смержил, проверил, а потом или исправил ошибки или отверг реквест. Но это же огромная работа на ведущего, особенно, при 15 участниках. При том, что прямая задача рефакторящего — проверить эквивалентность преобразований.

http://imageshack.us/scaled/thumb/713/rbox2djsfun01.jpg (<==пример очевидных багов)

Пытался запустить эту игру в браузере — в первую очередь в глаза бросается, что с фонами один большой глюк. Потом, эффект водяного потока неполноценный — пара колец вырывается в правую часть игрового поля, а остальные нет. И пара последних колец у меня застряли внизу при 1 попытке пройти этот уровень.



Вопрос, как же проверять, если уже 3 очевидных глюка? Все ведь согласны, что рефакторинг с привнесением багов — это не рефакторинг?

[Vladislav Klimanov]

в личку постучались, попросили опубликовать.
Crystal_HMR:

Не могу писать на хабре, напишу сюда: хорошо бы, ради продолжения игры после опубликования кода, каждой из групп — передать код одной группы к другой. Игра затянется, но поиск «идеального» кода станет интереснее :)
Т.е. каждая группа должна обработать три исходника.

[Vladislav Klimanov]

Кто в какую группу попадет совершенно не важно.
Ведущий ведет игру не в той группе, где он рефакторит, чтобы дать ведущим самим поиграть.
На 3 группы требуется 3 исходника, на крайний случай можно использовать один исходник на 2 группы.
Чтобы играть в закрытую нужно, чтобы исходник вначале видел только ведущий и тот, кто предложил этот код. И соответственно эти люди работать с этим кодом не будут.
Ведущий, а лучше тот, кто предложил исходник, должен покрыть покрыть код тестами.
На данный момент ищутся:
3 человека, которые готовы предложить код на игру и 3 ведущих.
Предложенные исходники должны быть покрыты тестами и не содержать очевидных багов.

[vooD]

1. Тестов нет — и это суть говнокода.
2. Игра расчитана на ios safari (iphone 4+). Там глюков с бекграундом нет. Если есть желание пофиксить для десктопа — не вижу преград.
3. Поток работает только на пол экрана — это физика игры. Так и задумывалось.
4. Да, иногда кольца застревают внизу, НО, опять же, если играть на телефоне то там работает акселерометр и колца можно выкатить наклоном телефона. Как это эмулировать на десктопе не знаю. Буду рад совету.
5. Я готов быть ведущим и принимать пул реквсеты хоть от 15 игроков. Однако пока тишина.

Answer 2

[Silver_Clash]

Я бы тоже поучаствовал! Возьмете?

Comments

[spmbt]

Да, не вопрос. Только надо подумать, как разделиться на 2-3 группы (или больше), набрать подходящие исходные коды и по какой схеме рефакторить.

Answer 3

[vooD]

Я готов быть ведущим. Вот игра о которой я говорил. Лучше всего смотреть на iphone safari. github.com/vood/box2d-javascript-fun
По сути файл для рефакторинга — это js/game.js. Легче играть в открытую — через пул реквесты. Тогда можно обсуждать изменения, коментируя непосредственно код. Игра же в темную, интереснее, хотя не такая прозрачная. По какому пути идти — решать обществу.

Comments

[spmbt]

ОК, пусть эта ветка по факту будет открытая и участники определятся, кто будет в ней, в какой очереди, а Вы — ведущий с естественными правами админа репы. А на ПК эта игра работать будет? А то ведь проверять её надо.

[Silver_Clash]

Пора спать пожалуй, а то я сообщениями промахиваюсь уже :)

[vooD]

Работает без проблем в webkit браузерах

Answer 4

[spmbt]

ищутся: 3 человека, которые готовы предложить код на игру и 3 ведущих.
Предложенные исходники должны быть покрыты тестами и не содержать очевидных багов.

Поддерживаю.
За эти дня 3 — ни одного нового предложения кода. на vooD-репо — те же 100 коммитов.

От себя могу предложить порефакторить юзерскрипт для Хабра — Habr Percentage Ring (коды). Он небольшой, меньше 100 строк, и вполне обозримый для нескольких часов. По нему был отзыв с перечислением массы внутренних кодовых недостатков, но, тем не менее, он работает и может быть протестирован очень просто, в браузере, не отходя от сайта. Можно попробовать как рефакторить, так и преобразовать функциональность с задержки на 3 секунды в непрерывную поддержку стилей — кликнул на стрелке, а положение эллипса не нарушилось. Ноесть и богатое поле для простого рефакторинга — понятные имена, понятные режимы.
Есть желающие записаться.

Answer 5

[Алексей Блышко]

Подписался на участие. Это может быть забавным.

Answer 6

[spmbt]

Но что-то мёртвая тишина. На следующий раз надо иметь готвый движок с текстами, тестами и правилами.