rust-lang-ru / rust_book_ru

The Rust Programming Language на русском языке
http://rurust.github.io/rust_book_ru/
184 stars 22 forks source link

Edit method-syntax.md and fix #38 #39

Closed ghost closed 8 years ago

ghost commented 8 years ago

Review on Reviewable

mkpankov commented 8 years ago

Review status: 0 of 1 files reviewed at latest revision, 2 unresolved discussions.


_src/method-syntax.md, line 19 [r1] (raw file):_ Об эти запятые спотыкаешься. Я бы не добавлял.

Кстати, если уж править - "наверное"


_src/method-syntax.md, line 227 [r1] (raw file):_ Тот же вопрос, что и в другом PR - почему тут нужна запятая?


_Comments from the review on Reviewable.io_

mkpankov commented 8 years ago

Reviewed 1 of 1 files at r1. Review status: all files reviewed at latest revision, 2 unresolved discussions.


_Comments from the review on Reviewable.io_

ghost commented 8 years ago

Review status: all files reviewed at latest revision, 2 unresolved discussions.


_src/method-syntax.md, line 19 [r1] (raw file):_ Пожалуй, здесь «наверное» будет более созвучно, другой разницы между вариантами нет. А запятыми здесь выделяется вводное слово «наверно(е)»: в данном предложении оно не имеет значения образа действия (и, следовательно, не является обстоятельством), а несёт только добавочное значение.


_src/method-syntax.md, line 227 [r1] (raw file):_ Запятая требуется для выделения придаточного, предлагаю обратиться ко второму PR.


_Comments from the review on Reviewable.io_

grishy commented 8 years ago

Reviewed 1 of 1 files at r1. Review status: all files reviewed at latest revision, 2 unresolved discussions.


_Comments from the review on Reviewable.io_

mkpankov commented 8 years ago

Review status: all files reviewed at latest revision, 2 unresolved discussions.


_src/method-syntax.md, line 19 [r1] (raw file):_ Суть моего беспокойства в том, что это вводное предложение содержит 4 запятые и по сути ничего не говорит. Я думаю, это следствие перегрузки вводными словами - стоит оставить либо "к счастью", либо "как вы уже догадались". С другой стороны, "наверное" всё равно не в кассу и портит всю элегантность.

Предлагаю радикально сократить до "К счастью, это возможно!".


_src/method-syntax.md, line 227 [r1] (raw file):_ Хорошо, согласен - непривычно просто. Не замечал в других книгах запятых в похожих местах.


_Comments from the review on Reviewable.io_

mkpankov commented 8 years ago

Reviewed 1 of 1 files at r2. Review status: all files reviewed at latest revision, all discussions resolved.


_Comments from the review on Reviewable.io_

mkpankov commented 8 years ago

Спасибо за работу!


Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


_Comments from the review on Reviewable.io_