Size: a a a

2020 June 14

KK

Kirill Kaymakov in pro.cxx.holywars
Потому что тогда правильно:
/// <summary>
/// This method must return status
/// If status is true - return true.
/// If status is false - return false
/// </summary>
/// <returns>
/// If status is true - true
/// If status is false - false
/// </returns>
/// <exception>
/// This method doesn't throw any exception
/// </exception>
public boolean GetStatus()
{
   if (status == true)
  {
          // If status is true
          // this method must return true
          return true;
  }
  else
  {
           // If status is false
          // this method must return false
          return true;
  }
}
источник

CD

Constantine Drozdov in pro.cxx.holywars
Kirill Kaymakov
Потому что тогда правильно:
/// <summary>
/// This method must return status
/// If status is true - return true.
/// If status is false - return false
/// </summary>
/// <returns>
/// If status is true - true
/// If status is false - false
/// </returns>
/// <exception>
/// This method doesn't throw any exception
/// </exception>
public boolean GetStatus()
{
   if (status == true)
  {
          // If status is true
          // this method must return true
          return true;
  }
  else
  {
           // If status is false
          // this method must return false
          return true;
  }
}
источник

CD

Constantine Drozdov in pro.cxx.holywars
Нужны UML диаграммы!
источник

KK

Kirill Kaymakov in pro.cxx.holywars
А еще 10 менеджеров над головой, которые обсуждают их правильность, да!
источник

CD

Constantine Drozdov in pro.cxx.holywars
KPI по количеству методов возвращающих true!
источник

KK

Kirill Kaymakov in pro.cxx.holywars
Кость, ты же лид, как там еще больше доки написать?
источник

CD

Constantine Drozdov in pro.cxx.holywars
Я лид курильщика
источник

CD

Constantine Drozdov in pro.cxx.holywars
Не пишу доки
источник

KK

Kirill Kaymakov in pro.cxx.holywars
Блин
источник

KK

Kirill Kaymakov in pro.cxx.holywars
Constantine Drozdov
KPI по количеству методов возвращающих true!
На самом деле это было бы смешно, если бы не было так печально
источник

CC

Chris Calvin in pro.cxx.holywars
Kirill Kaymakov
Потому что тогда правильно:
/// <summary>
/// This method must return status
/// If status is true - return true.
/// If status is false - return false
/// </summary>
/// <returns>
/// If status is true - true
/// If status is false - false
/// </returns>
/// <exception>
/// This method doesn't throw any exception
/// </exception>
public boolean GetStatus()
{
   if (status == true)
  {
          // If status is true
          // this method must return true
          return true;
  }
  else
  {
           // If status is false
          // this method must return false
          return true;
  }
}
Не описаны гарантии исключений еще как минимум
источник

CC

Chris Calvin in pro.cxx.holywars
Не доказана потокобезопасность метода по формальным правилам из мемори модели языка
источник

CC

Chris Calvin in pro.cxx.holywars
Там еще до сотни строк можно напидорасить, да)
источник

KK

Kirill Kaymakov in pro.cxx.holywars
Chris Calvin
Не описаны гарантии исключений еще как минимум
Во, добавил
источник

CC

Chris Calvin in pro.cxx.holywars
Плюс метаинформация о том, кто писал, когда писал, начиная с какой версии, какие ревизии кода были и тд
источник

KK

Kirill Kaymakov in pro.cxx.holywars
Chris Calvin
Плюс метаинформация о том, кто писал, когда писал, начиная с какой версии, какие ревизии кода были и тд
Это в шапке
источник

CC

Chris Calvin in pro.cxx.holywars
Kirill Kaymakov
Во, добавил
Имею ввиду - описать что этот метод не меняет гарантий обработки исключительных  ситаций  приложения аля:
1. Нет гарантий - когда есть исключительная ситуация происходит хуй пойми что
2. Нестрогие гарантии - когда есть исключительная ситуация приложение сохраняет какое-то допустимое состояние
3. Строгие гарантии - в случае возникновения исключительной ситуации возвращается к предыдущему согласованному состоянию
источник

IL

Ilya L in pro.cxx.holywars
Chris Calvin
Имею ввиду - описать что этот метод не меняет гарантий обработки исключительных  ситаций  приложения аля:
1. Нет гарантий - когда есть исключительная ситуация происходит хуй пойми что
2. Нестрогие гарантии - когда есть исключительная ситуация приложение сохраняет какое-то допустимое состояние
3. Строгие гарантии - в случае возникновения исключительной ситуации возвращается к предыдущему согласованному состоянию
О так вот как это правильно описать!
источник

IL

Ilya L in pro.cxx.holywars
Надо будет тоже к себе утащить...
источник

CC

Chris Calvin in pro.cxx.holywars
Ну это если прям кошерно :)
источник