Курсы по программированию

Формула программиста
основатель — Волосатов Евгений Витольдович
Вступай в Телеграм чат проекта | Обход блокировки
Игровой комплекс Сокобан - Введи код скидки

БД - Управление гостиницей / Модель. Удаление комнат.

  • На этом уроке мы добавим метод для удаления комнат.
    Исправим логическую ошибку метода Update в классах Room и Client.
    Подготовим функцию для теста созданного класса.
    Рекомендую эти тесты создать самостоятельно.
  • Дата отправки отчёта: 4 января 2016 г.
  • Задание выполнено: за 20 мин.
  • Чему научился: повторил
  • Что было сложным: сложно не было
  • Комментарии: Ладно, мне это сразу не понравилось, но раз уж на уроке возникла дискуссия, вставлю и свои пять копеек. Учитель слегка неправ, на мой взгляд. Свойства (они же геттеры и сеттеры) для того и существуют, что бы выполнять различные проверки и не давать доступ напрямую. А функции SetRoom, SetFloor, SetBeds, SetStep это никакие не свойства, и уж тем более не поля. Это просто функции. Я бы даже сказал просто ЛИШНИЕ функции. Лишний код. У нас уже есть для этого настоящее СВОЙСТВО выше { get; private set} , в котором мы непонятно зачем сделали private set (в вышеупомянутых функциях мы точно так же напрямую без обработки просто присваиваем значения. Правильней было бы вообще обойтись без SetRoom, SetFloor, SetBeds, SetStep, а свойства set отставить публичным. И никакой принцип инкапсуляции это бы не нарушило, ибо для того и существуют свойства! И тут же бы делали защиту от sql-инъекций.
    Прям написать вот так { get; set{ room = sql.addslashes(value);} }
  • Оценка видео-уроку:
Отчёт от 4342 за БД - Управление гостиницей / Модель. Удаление комнат.




Оцени работу

 
Сохранить страницу:

6203. Николай Миролюбов
Николай Миролюбов
ответить
→  Алекс  # БД - Управление гостиницей / Модель. Удаление комнат. / 2016-01-04 18:39

В принципе да, только придется поля еще добавлять.


1. Евгений Волосатов
Евгений Волосатов
ответить
→  Алекс  # БД - Управление гостиницей / Модель. Удаление комнат. / 2016-01-04 17:26

Спасибо, Алексей.
Нужно будет исправиться :-/



Начинаем практику по языку C#




Чтобы стать хорошим программистом — нужно писать программы. На нашем сайте очень много практических упражнений.

После заполнения формы ты будешь подписан на рассылку «C# Вебинары и Видеоуроки», у тебя появится доступ к видеоурокам и консольным задачам.

Несколько раз в неделю тебе будут приходить письма — приглашения на вебинары, информация об акциях и скидках, полезная информация по C#.

Ты в любой момент сможешь отписаться от рассылки.
Ладно, мне это сразу не понравилось, но раз уж на уроке возникла дискуссия, вставлю и свои пять копеек. Учитель слегка неправ, на мой взгляд. Свойства (они же геттеры и сеттеры) для того и существуют, что бы выполнять различные проверки и не давать доступ напрямую. А функции SetRoom, SetFloor, SetBeds, SetStep это никакие не свойства, и уж тем более не поля. Это просто функции. Я бы даже сказал просто ЛИШНИЕ функции. Лишний код. У нас уже есть для этого настоящее СВОЙСТВО выше { get; private set} , в котором мы непонятно зачем сделали private set (в вышеупомянутых функциях мы точно так же напрямую без обработки просто присваиваем значения. Правильней было бы вообще обойтись без SetRoom, SetFloor, SetBeds, SetStep, а свойства set отставить публичным. И никакой принцип инкапсуляции это бы не нарушило, ибо для того и существуют свойства! И тут же бы делали защиту от sql-инъекций. Прям написать вот так { get; set{ room = sql.addslashes(value);} }
Научился: Научился добавлять описания для каждого метода.
Трудности: Находить собственные ошибки.
Урок очень понравился, урок поучительный показывает, как можно упростить или изменить программу.