Как оптимальней?

Question

[Urukhayy]

1 вариант:

switch(a)
{
   case 0: 
   {
      b = 2;
      return 1;
   }
case 1: 
   {
      b = -1232;
      return 1;
   }
case 2: 
   {
      b = 12322;
      return 1;
   }
}

Вариант 2:

switch(a)
{
   case 0: 
   {
      b = 2;
   }
case 1: 
   {
      b = -1232;
   }
case 2: 
   {
      b = 12322;
   }
}

То бишь если один из case выполнится, другие не будут срабатывать, и второй вариант лучше?

Answer 1

[Roman K]

break

Answer 2

[Алексей Уколов]

Язык программирования не указан, но во многих второй вариант вообще ошибочный - case выполняется по очереди и каждый следующий перезаписывает предыдущий. Это называется switch fallthrough и считается плохой практикой, поскольку может повлечь за собой неожиданные проблемы при невнимательности, как в данном случае.

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

Comments

[Urukhayy]

А не является ли "не практичным" постоянно дублировать break в каждом case? Лишняя строчка. А так, если было-бы на автомате, гораздо удобнее.

[Алексей Уколов]

Urukhayy: я не понимаю, чего вы хотите. Не хотите писать break в конце case - пишите на языке, в котором этого деать не нужно.

[Roman K]

Urukhayy: Вы ставите фигурные скобки на отдельной строке и говорите о компактности? Фи.

[Urukhayy]

Roman Kitaev: Кому как удобней. Кому-то не нравится когда строчки "слипаются", и между ними должен быть "пробел".

[Urukhayy]

Roman Kitaev: ... Если дело идёт о новой ветке, естественно.

[Roman K]

Urukhayy: Давно уж негласно принято открывающуюся скобку не выносить на новую строку. Так улучшается читабельность и сохраняются лишние строки.

Answer 3

[Noobkesan]

во-первых, во втором варианте необходим break, а во-вторых, это не та оптимизация, о которой стоит беспокоится, а скорее дело вкуса.

Answer 4

[Сергей Протько]

Оптимальнее хранить данные в мэпе и выдавать по ключу.

Comments

[Urukhayy]

Я для примера привёл присваивание. На самом деле там может быть много кода в каждом case.

Answer 5

[Ai Lab]

вообще то break, а по сути без разницы, если вы не собираетесь проверять этот case три миллиарда раз в секунду, любой комп это условие сожрет и не подавится даже при сотне тысяч циклов в сек.