Twoje komentarze
Александр,
был рад помочь, спасибо за проделанную работу, купил рекомендованную Вами книгу (кстати, советую обновить ссылку на 402- странице, т.к. сейчас есть обновленная версия данной книги: http://shtonda.blogspot.com/2016/04/refactoring-patterns-joshua-kerievsky.html).
С уважением,
Дмитрий
Александр, добрый день,
По первому пункту, честно говоря, не понятно в чем сложность, куки же сайт хранит? Конечно, сайт Ваш и решения тут Вы принимаете сами, как пользователь, я высказал свое мнение.
Желаю удачи в дальнейшем развитии проекта.
С уважением,
Дмитрий
Александр, добрый день,
спасибо за оперативность, подтверждаю исправления, дальнейшие комментарии относятся к версии книги v2018-2.11:
- страница 229 Предложение "В идеале, этот хотелось бы поместить прямо в служебный класс...". После слова "этот" пропущено слово "код".
- страница 249 Предложение "В этом случае запрос движется по цепи, пока не найдётся обработчик, могущий его обработать.". Немного режет взгляд "могущий его обработать", я бы перефразировал, например, так: "В этом случае запрос движется по цепи, пока не найдётся обработчик, имеющий код (условия) обработки данного запроса.".
- страница 267 Предложение "Хорошие программы обычно структурированы в виде слоёв.". Как вариант, для интересующихся можно дать ссылки на паттерны MVC/MVP MVVM... (кстати, может имеет смысл добавить их в книгу?).
- страница 341 Предложение "Реализуйте их так, чтобы после каждого изменения состояния они слали оповещения...". Аналогично комментарию к странице 187.
- страница 346 Предложение "Паттерн Состояние невозможно рассматривать в отрыве от концепции машины состояний, также известной...". Ссылка "машины состояний" на английском языке, было бы логичней приводить ссылки на языке издания (в данном случае: https://ru.wikipedia.org/wiki/%D0%9A%D0%BE%D0%BD%D0%B5%D1%87%D0%BD%D1%8B%D0%B9_%D0%B0%D0%B2%D1%82%D0%BE%D0%BC%D0%B0%D1%82).
- страница 359 Предложение "...все методы интерфейса состояния должны быть реализованы по всех классах состояний.". Опечатка, нужно "...реализованы ВО всех...".
- страница 377 Предложение "В нашем примере шаги открытия и закрытия документов будут отличаться для всех подклассов, поэтому останутся абстрактными. А вот одинаковый для всех типов документов код обработки данных переедет в базовый класс.". Комментарий: если честно, не понял данный фрагмент и поэтому не согласен, на мой взгляд, "шаги открытия и закрытия документов" как раз должны быть одинаковыми, а вот "код обработки данных" как раз уникален для каждого типа документа, и поэтому всё должно быть как раз наоборот, т.е. "шаги открытия и закрытия документов" должны быть в базовом классе (с точки зрения C# данные методы можно будет сделать виртуальными, если всё же потребуется их переопределять), а вот "код обработки данных", ввиду его различия в каждой реализации, должен быть абстрактным методом с уникальной реализацией в каждом подклассе. Если что-то другое имелось ввиду, возможно нужно перефразировать, т.к. сейчас выглядит не понятно, буду рад Вашему комментарию.
- страница 378 Предложение "Это опциональные шаги, которые выглядят как обычные методы, но не содержат никакого кода.". На мой взгляд, здесь имеет смысл дополнить, чтобы не потерять нить рассуждения, начатую в конце страницы 376, например, так: "...но не содержат никакого кода (по своей сути, это абстрактные методы базового класса не требующие реализации в конкретном контексте)."
- страница 378 Предложение "...хук даёт подклассам дополнительные точки «вклинивания» в ход шаблонного метода.". Опечатка, нужно "...в КОД шаблонного...".
- страница 380 Диаграмма. Если вспомнить старую добрую Warcraft, то ресурсы как раз собирали орки, т.е. имеет смысл поменять местами заголовки производных классов на диаграмме (и кстати, раз на 381 странице Вы пишете, что "монстры вообще не будут заниматься строительством", то, также имеет смысл убрать у них на диаграмме производного класса метод "buildStructures()") ;)
- страница 402 Предложение "Распечатайте шпаргалки по паттернам и...". Ссылка "шпаргалки по паттернам" на английском языке, комментарий аналогичен к странице 346.
С уважением,
Дмитрий
Александр, добрый день,
решение подтверждаю, спасибо за оперативность.
Чтобы не плодить новую тему, обращу здесь Ваше внимание еще не некоторые моменты:
1) при первом входе на сайт, либо по нажатию логотипа из интерфейса форума, всегда открывается английская страница сайта, что несколько неудобно, я же уже регистрировался, соответственно система должна помнить, что мой выбор - русский язык, можно же это хранить в куках, либо в локальном хранилище?
2) небольшая ошибка верстки (а может так задумано? Хотя смотрится не очень), поползла сетка:
3) на мой взгляд, было бы полезным отображать текущую версию книги (т.к. некоторые пользователи скачавшие ранее Вашу книгу, и вернувшиеся на сайт через какое-то время, будут проинформированы о наличии/отсутствии обновлений):
С уважением,
Дмитрий
Александр,
верю, что старались :)
Разобрался, причина подобного поведения в блокировщике рекламы Adblock (соответственно, выпадает элемент вёрстки со всеми вытекающими):
Надеюсь, информация будет полезна.
С уважением,
Дмитрий
Customer support service by UserEcho
Александр,
Тогда добавлю мой сценарий в копилку Вашего будущего исследования. Обычно я начинаю поиск нужной информации с поисковых систем (скорее всего это и будет самый массовый сценарий), соответственно, то, что мне нравится, добавляю в избранное. Ну а далее два варианта, 1) либо я кликаю по пункту избранного и по ссылке попадаю на сохраненную ранее русскую страницу, либо 2) я начинаю вбивать адрес в адресной строке (браузеры сейчас стали достаточно умными и уже после набора "re" у меня подставляется данный сайт), а вот тогда я попадаю на английский вариант сайта, что не есть юзер-френдли.
Относительно приведенного "плохого сценария", ну и что, сохранилась английская кука, затем, он когда-то зашел на сайт повторно, и выбрал русскую страницу, что мешает пересохранить куку? Теперь относительно фразы: "все последующие его посещения, например, из яндекса будут перебрасывать его на английскую версию", вообще не понимаю, как это может быть связано? То как работает сайт с поисковыми системами зависит только от его SEO-оптимизации (https://ru.wikipedia.org/wiki/%D0%9F%D0%BE%D0%B8%D1%81%D0%BA%D0%BE%D0%B2%D0%B0%D1%8F_%D0%BE%D0%BF%D1%82%D0%B8%D0%BC%D0%B8%D0%B7%D0%B0%D1%86%D0%B8%D1%8F), если пользователь ищет сайт набирая контекст поиска на английском и страница корректно оптимизирована под этот язык, то и переход из поисковой системы будет на английскую версию страницы, то же самое и с русским, да и с любым другим языком. Поэтому, на мой взгляд, данные опасения не имеют под собой почвы.
С уважением,
Дмитрий