update GenServer #15

Zabalkanskiy · 2017-08-27T08:24:48Z

No description provided.

rtrv · 2017-08-27T09:03:57Z

mix-otp/genserver.md

+
+# {{ page.title }}
+
+{% include toc.html %}


Эту строку нужно убрать, у нас нет этого меню

rtrv · 2017-08-27T09:04:11Z

mix-otp/genserver.md

+
+{% include toc.html %}
+
+{% include mix-otp-preface.html %}


Это тоже нужно убрать

rtrv · 2017-08-27T09:04:39Z

mix-otp/genserver.md

+
+{% include mix-otp-preface.html %}
+
+В [предыдущей главе](/getting-started/mix-otp/agent.html) мы использовали агенты(agents) чтобы представить наши корзины(buckets). В первой главе мы указали имя каждой корзины, поэтому мы можем сделать следующее:


Перед открывающимися скобками должны быть пробелы

rtrv · 2017-08-28T09:08:40Z

mix-otp/genserver.md

+OK
+```
+
+Поскольку агенты процессы, каждая корзина имеет идентификатор процесса (PID), но не имеет названия. Мы узнали об регистрации имени [в главе Процессы](/getting-started/processes.html) и вы могли бы решить эту проблему с помощью такой регистрации. Например, мы могли бы создать корзину так:


Между "агенты" и "процессы" в начале нужно тире
"о регистрации" вместо "об"

rtrv · 2017-08-28T09:10:19Z

mix-otp/genserver.md

+Тем не менее, это ужасная идея! Имена Процессов в Эликсире должны быть атомами, это означает, что мы должны преобразовать имя корзины (часто получаемое из внешнего клиента) в атомы, **мы никогда не
+должны преобразовывать ввод пользователя в атомы**. Сборщик мусора не уничтожает атомы.Генерация атомов из пользовательского ввода может привести к исчерпанию Системной памяти!
+
+На практике скорее всего, вы достигнете предела максимального количества атомов для Виртуальной Машины Эрланга и  ваша система упадет в любом случае.


"скорее всего" выделяется запятыми с обеих сторон
"и ваша" два пробела, перед "и" нужна запятая

rtrv · 2017-08-28T09:42:51Z

mix-otp/genserver.md

+
+## `call`, `cast` или `info`?
+
+До сих пор мы воспользовались тремя коллбэками: `handle_call/3`, `handle_cast/2` и `handle_info/2`. Вот что мы должны учитывать при


В конце этой строки перевод каретки лишний
"Вот, что мы должны" нужна запятая

rtrv · 2017-08-28T09:45:17Z

mix-otp/genserver.md

+
+1. `handle_call/3` должен быть использован для синхронных запросов. Он должен быть выбором по умолчанию в ожидании ответа сервера и является полезным механизмом противодавления.
+
+2. `handle_cast/2` должен быть использован для асинхронных запросов, когда ты не заботишься об ответе. Бросок не дает гарантии что сервер получил сообщение и, по этой причине, его следует использовать с осторожностью. Например, функция `create/2`, которую мы определили в этой главе должна использовать `call/2`. Мы использовали `cast/2` для обучающих целей.


"которую мы определили в этой главе должна" запятая перед "должна"

rtrv · 2017-08-28T09:46:24Z

mix-otp/genserver.md

+
+2. `handle_cast/2` должен быть использован для асинхронных запросов, когда ты не заботишься об ответе. Бросок не дает гарантии что сервер получил сообщение и, по этой причине, его следует использовать с осторожностью. Например, функция `create/2`, которую мы определили в этой главе должна использовать `call/2`. Мы использовали `cast/2` для обучающих целей.
+
+3. `handle_info/2` должен использоваться для всех других сообщений, которые сервер может получать и не передаются через `GenServer.call/2` или `GenServer.cast/2`, включая регулярные сообщения `send/2`. Мониторинг cообщений `:DOWN` как раз такой пример.


":DOWN как" тут тире перед "как", подлежащее и сказуемое - существительные

rtrv · 2017-08-28T09:49:44Z

mix-otp/genserver.md

+
+Ранее мы узнали о связях в [главе Процессы](/getting-started/processes.html). Теперь, с полным завершением реестра, вам может быть интересно: когда мы должны использовать мониторы и когда следует использовать ссылки?
+
+Ссылки двунаправлены. Если вы связываете два процесса и один из них выйдет из строя, другая сторона тоже выйдет из строя (если только это не улавливает флаг trapping exits true). Монитор является однонаправленным: процесс мониторинга будет только получать уведомления о контролируемом. Другими словами: используйте ссылки, когда вы хотите получить связанные сбои, и мониторьте за процессами, когда вы просто хотите получать информацию о сбоях, выходах и т.д.


"два процесса и один из них" запятая перед "и"
"если только это не улавливает флаг trapping exits true" -> "если только не установлен флаг trapping exits true"
"мониторьте за процессами" -> "мониторьте процессы"

rtrv · 2017-08-28T09:52:07Z

mix-otp/genserver.md

+ref = Process.monitor(pid)
+```
+
+Это плохая идея, так как мы не хотим, чтобы реестр падал при сбое корзины! Мы обычно избегаем непосредственного создания новых процессов, вместо этого мы делегируем эту ответственность супервизорам. Как мы увидим в следующей главе, супервизеры полагаются на ссылки, и это объясняет, почему API-интерфейсы на основе ссылок (`spawn_link`, `start_linkи` т. д.) Настолько распространены в Эликсире и <abbr title="Open Telecom Platform">OTP</abbr>.


"Настолько" с маленькой буквы
"ОТП" ранее было русскими, тут "OTP". Думаю, стоит везде использовать один вариант, и в случае кириллицы расшифровку также давать на русском, "открытая телекоммуникационная платформа"

"start_linkи" тут русская "и" затесалась

sobolevn · 2017-08-28T10:09:56Z

@rtrv титаническая работа!

rtrv · 2017-08-28T11:11:00Z

@Zabalkanskiy спасибо за перевод, отличная работа, только мелочи поправить. Пожалуйста, отписывайся в #6 когда начинаешь перевод какого-то раздела, чтобы не возникло коллизий

rtrv · 2017-09-06T15:22:26Z

mix-otp/genserver.md

-{% include mix-otp-preface.html %}
-
-В [предыдущей главе](/getting-started/mix-otp/agent.html) мы использовали агенты(agents) чтобы представить наши корзины(buckets). В первой главе мы указали имя каждой корзины, поэтому мы можем сделать следующее:
+В [предыдущей главе](/getting-started/mix-otp/agent.html) мы использовали агенты (agents) чтобы представить наши корзины (buckets). В первой главе мы указали имя каждой корзины, поэтому мы можем сделать следующее:


Вероятно забыл указать в прошлый раз, перед "чтобы" сразу после закрывающейся скобки нужна запятая

rtrv · 2017-09-06T15:22:31Z

mix-otp/genserver.md

@@ -22,7 +18,7 @@ GET shopping milk
 OK
 ```

-Поскольку агенты процессы, каждая корзина имеет идентификатор процесса (PID), но не имеет названия. Мы узнали об регистрации имени [в главе Процессы](/getting-started/processes.html) и вы могли бы решить эту проблему с помощью такой регистрации. Например, мы могли бы создать корзину так:
+Поскольку - агенты процессы, каждая корзина имеет идентификатор процесса (PID), но не имеет названия. Мы узнали об регистрации имени [в главе Процессы](/getting-started/processes.html) и вы могли бы решить эту проблему с помощью такой регистрации. Например, мы могли бы создать корзину так:


Тут тире между "агенты" и "процессы"

"об регистрации" -> "о регистрации"

rtrv · 2017-09-06T15:24:31Z

mix-otp/genserver.md


-Вместо того, чтобы злоупотреблять регистрацией имени объекта, мы создадим процесс регистрации, который содержит карту(map), которая связывает имя корзины с процессом корзины.
+Вместо того, чтобы злоупотреблять регистрацией имени объекта, мы создадим процесс регистрации, который содержит мэп, который связывает имя корзины с процессом корзины.


Обсуждали потом в комментах, что map всё-таки переводим как "словарь", дальше не пишу по этому поводу

rtrv · 2017-09-06T15:26:16Z

mix-otp/genserver.md

@@ -196,7 +191,7 @@ iex> flush()

 Обратите внимание, `Process.monitor(pid)` возвращает уникальную ссылку, что позволяет нам мониторить выходящие сообщения процесса. После того, как мы остановили агента, мы можем  при помощи `flush/0` посмотреть все консольные сообщения и обратите внимание, что `:DOWN` сообщение прибыло, с точной ссылкой, возвращенной монитором, указывающией, что процесс завершен корзиной с причиной `:normal`.

-Давайте переопределим коллбэки сервера, чтобы исправить баг и пройдем тест. Вопервых, мы изменим состояние GenServer используя два словаря: один, будет содержать `name ‐> pid` и другой, содержит `ref ‐> name`. Затем нам нужно мониторить корзины при помощи `handle_cast/2`, а также реализовать функцию коллбэков `handle_info/2`  для обработки мониторинга Сообщений. Полное выполнение коллбэков сервера показано ниже:
+Давайте переопределим коллбэки сервера, чтобы исправить баг и пройдем тест. Во-первых, мы изменим состояние GenServer используя два словаря: один будет содержать `name ‐> pid` а другой `ref ‐> name`. Затем нам нужно мониторить корзины при помощи `handle_cast/2`, а также реализовать функцию коллбэков `handle_info/2`  для обработки мониторинга Сообщений. Полное выполнение коллбэков сервера показано ниже:


"один будет содержать name ‐> pid а другой ref ‐> name" -> "один будет содержать name ‐> pid, а другой - ref ‐> name". Не хватает запятой и тире

rtrv · 2017-09-06T15:27:00Z

mix-otp/genserver.md

@@ -234,20 +229,19 @@ def handle_info(_msg, state) do
 end
 ```

-Заметим, что нам удалось значительно изменить реализацию сервера, без изменения каких либо API клиента. Это одно из преимуществ четкого разделения на серверную и клиентскую части.
+Заметим, что нам удалось значительно изменить реализацию сервера без изменения каких либо API клиента. Это одно из преимуществ четкого разделения на серверную и клиентскую части.


"каких-либо" через дефис

update GenServer

9d36258

rtrv requested changes Aug 28, 2017

View reviewed changes

Zabalkanskiy added 2 commits September 6, 2017 16:18

Update genserver.md

e90a25c

Update genserver.md

d98f7a3

rtrv requested changes Sep 6, 2017

View reviewed changes

Update genserver.md

d433b9f

rtrv approved these changes Sep 8, 2017

View reviewed changes

rtrv merged commit 07301d1 into wunsh:master Sep 8, 2017

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

update GenServer #15

update GenServer #15

Zabalkanskiy commented Aug 27, 2017

rtrv Aug 27, 2017

rtrv Aug 27, 2017

rtrv Aug 27, 2017

rtrv Aug 28, 2017

rtrv Aug 28, 2017

rtrv Aug 28, 2017

rtrv Aug 28, 2017

rtrv Aug 28, 2017

rtrv Aug 28, 2017

rtrv Aug 28, 2017

rtrv Aug 28, 2017

sobolevn commented Aug 28, 2017

rtrv commented Aug 28, 2017

rtrv Sep 6, 2017

rtrv Sep 6, 2017

rtrv Sep 6, 2017

rtrv Sep 6, 2017

rtrv Sep 6, 2017

rtrv Sep 6, 2017


		{% include mix-otp-preface.html %}

		В [предыдущей главе](/getting-started/mix-otp/agent.html) мы использовали агенты(agents) чтобы представить наши корзины(buckets). В первой главе мы указали имя каждой корзины, поэтому мы можем сделать следующее:


		## `call`, `cast` или `info`?

		До сих пор мы воспользовались тремя коллбэками: `handle_call/3`, `handle_cast/2` и `handle_info/2`. Вот что мы должны учитывать при


		1. `handle_call/3` должен быть использован для синхронных запросов. Он должен быть выбором по умолчанию в ожидании ответа сервера и является полезным механизмом противодавления.

		2. `handle_cast/2` должен быть использован для асинхронных запросов, когда ты не заботишься об ответе. Бросок не дает гарантии что сервер получил сообщение и, по этой причине, его следует использовать с осторожностью. Например, функция `create/2`, которую мы определили в этой главе должна использовать `call/2`. Мы использовали `cast/2` для обучающих целей.


		2. `handle_cast/2` должен быть использован для асинхронных запросов, когда ты не заботишься об ответе. Бросок не дает гарантии что сервер получил сообщение и, по этой причине, его следует использовать с осторожностью. Например, функция `create/2`, которую мы определили в этой главе должна использовать `call/2`. Мы использовали `cast/2` для обучающих целей.

		3. `handle_info/2` должен использоваться для всех других сообщений, которые сервер может получать и не передаются через `GenServer.call/2` или `GenServer.cast/2`, включая регулярные сообщения `send/2`. Мониторинг cообщений `:DOWN` как раз такой пример.


		Ранее мы узнали о связях в [главе Процессы](/getting-started/processes.html). Теперь, с полным завершением реестра, вам может быть интересно: когда мы должны использовать мониторы и когда следует использовать ссылки?

		Ссылки двунаправлены. Если вы связываете два процесса и один из них выйдет из строя, другая сторона тоже выйдет из строя (если только это не улавливает флаг trapping exits true). Монитор является однонаправленным: процесс мониторинга будет только получать уведомления о контролируемом. Другими словами: используйте ссылки, когда вы хотите получить связанные сбои, и мониторьте за процессами, когда вы просто хотите получать информацию о сбоях, выходах и т.д.


		Вместо того, чтобы злоупотреблять регистрацией имени объекта, мы создадим процесс регистрации, который содержит карту(map), которая связывает имя корзины с процессом корзины.
		Вместо того, чтобы злоупотреблять регистрацией имени объекта, мы создадим процесс регистрации, который содержит мэп, который связывает имя корзины с процессом корзины.

update GenServer #15

update GenServer #15

Conversation

Zabalkanskiy commented Aug 27, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sobolevn commented Aug 28, 2017

rtrv commented Aug 28, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment