Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix description hash fetch #142

Closed
wants to merge 6 commits into from
Closed

Fix description hash fetch #142

wants to merge 6 commits into from

Conversation

usernaimandrey
Copy link
Contributor

No description provided.


Метод fetch принемает вторым параметром значение по умалчанию:
То есть если мы запрашиваем значение по несуществующему ключу через метод `[]` (не забываем в Ruby - все есть объект и соответственно `[]` обычный метод объекта Hash), он молча возвращает `nil`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Скобки это зло) Вот полезняха https://ru.hexlet.io/courses/awesome-text

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Скобки это зло) Вот полезняха https://ru.hexlet.io/courses/awesome-text

Спасибо за ссылку
Почему зло, потому что их содеожимое игнорируется?)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Скобки указывают на то, что предложение составлено неверно. В хорошо составленных текстах скобки не нужны. Прогони текст через https://glvrd.ru/

https://maximilyahov.ru/blog/all/skobki/

В целом там и по другим частям текста тоже много всякого. Главный совет тут, читать книгу "пиши сокращай"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Прогнал текст через https://glvrd.ru/ и от корректировал

@fey fey requested a review from mokevnin December 10, 2022 13:44
@fey
Copy link
Contributor

fey commented Dec 28, 2022

@mokevnin посмотришь изменения?

@ssssank
Copy link
Contributor

ssssank commented Jan 11, 2023

@mokevnin ждём ревью: от скобок избавились, текст причесали

@@ -2,45 +2,49 @@

name: Метод fetch и извлечение значений из хеша
theory: |

В Ruby помимо стандартного способа извлечения значения из хеша существует метод fetch. В чем же принципиальное отличие?
В Ruby значения из hash по ключу извлекает метод `[]` и `fetch`. Посмотрим в чем отличие.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тут нужно оставить вопрос.

Метод fetch принемает вторым параметром значение по умалчанию:
Если запрашивать значение по несуществующему ключу, метод `[]` молча возвращает `nil`.

Не забываем в Ruby - все объект и `[]` метод объекта Hash.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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


Не забываем в Ruby - все объект и `[]` метод объекта Hash.

Метод `fetch` же действует иначе - если ключа нет, получим ошибку КеуError
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Должен быть пример


Метод `fetch` же действует иначе - если ключа нет, получим ошибку КеуError

Метод `fetch` принимает вторым параметром значение по умалчанию:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тут нужно дописать текст, в котором раскрыть как это значение используется

puts env.fetch(:user, nil) # -> nil
puts env.fetch(:user, 'guest') # -> 'guest'
```

И даже блок или lambda:
И даже блок или `lambda`:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучше по русски: лямбда-функция

```
Если переданного ключа нет в hash, вызываем блок, передав в него первым аргументом ключ, а вторым hash, иначе вернуть исходный hash.

Сдесь понадобится метод `yield`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Орфографическая ошибка. Давайте текст еще через ворд прогоним


Сдесь понадобится метод `yield`.

**[yield](https://apidock.com/ruby/Proc/yield)**
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ссылки нужно либо прямо в тексте делать, либо выносить в tips

@usernaimandrey
Copy link
Contributor Author

От корректировал @mokevnin посмотри пожалуста

@usernaimandrey
Copy link
Contributor Author

@mokevnin посмотри пожалуста

@fey
Copy link
Contributor

fey commented Jan 17, 2023

@usernaimandrey можешь плз ребейзнуть и убрать мердж коммит?
Посмотри в закрепе internship там должны быть несколько постов про то как актуализировать ветку
Обычно это делается командой git pull --rebase upstream main и делается алиас. git up upstream main.
Это нужно, чтобы история коммитов была ровная, чоткая без таких коммитов и лишних историй :)
Если нужна будет помощь - пингани в чатике, помогу.

@usernaimandrey
Copy link
Contributor Author

@usernaimandrey можешь плз ребейзнуть и убрать мердж коммит? Посмотри в закрепе internship там должны быть несколько постов про то как актуализировать ветку Обычно это делается командой git pull --rebase upstream main и делается алиас. git up upstream main. Это нужно, чтобы история коммитов была ровная, чоткая без таких коммитов и лишних историй :) Если нужна будет помощь - пингани в чатике, помогу.

хорошо сделаю

@usernaimandrey usernaimandrey closed this by deleting the head repository Jan 17, 2023
@usernaimandrey
Copy link
Contributor Author

@usernaimandrey можешь плз ребейзнуть и убрать мердж коммит? Посмотри в закрепе internship там должны быть несколько постов про то как актуализировать ветку Обычно это делается командой git pull --rebase upstream main и делается алиас. git up upstream main. Это нужно, чтобы история коммитов была ровная, чоткая без таких коммитов и лишних историй :) Если нужна будет помощь - пингани в чатике, помогу.

Закрыл этот пр т.к. напортачил с ребэйзом сделал другой

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants