1. C# / Говнокод #16227

    +140

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    protected override void HandleImpl( DeleteTrashModel model )
{
	switch( model.TableName )
	{
		case DeleteTrashTablesEnum.AuthenticationTicket:
			this.DeleteTrash( DeleteTrashTablesEnum.AuthenticationTicket, model.DaysCount );
			break;
		case DeleteTrashTablesEnum.ResetPasswordTicket:
			this.DeleteTrash( DeleteTrashTablesEnum.ResetPasswordTicket, model.DaysCount );
			break;
		case DeleteTrashTablesEnum.Notification:
			this.DeleteTrash( DeleteTrashTablesEnum.Notification, model.DaysCount );
			break;
		case DeleteTrashTablesEnum.Event:
			this.DeleteTrash( DeleteTrashTablesEnum.Event, model.DaysCount );
			break;
	}
}

    Если есть Enum - нужен switch!

    Запостил: sickphilosopher, 25 Июня 2014

    Комментарии (8) RSS

    • ava AppleMan 25.06.2014 13:33 # +1

      Если есть Enum - нужен switch!
      А ещё если есть скобочки - нужны пробелы
      Ответить

    • ava kegdan 25.06.2014 14:20 # 0

      DeleteTrash должен быть методом model
      Ответить

    • ava Lure Of Chaos 25.06.2014 18:56 # +1

      почему нет ветки default? вот если null придет
      Ответить

      • ava wvxvw 26.06.2014 08:19 # +1

        В Сищарпе просто так нулл энуму не присвоить, кроме тото, если память не изменяет, он не любит, когда енум не полностью обработали в свитче, так что с этим, я думаю, проблем не будет.
        Ответить

        • ava kegdan 26.06.2014 09:50 # 0

          можно и дефаут поставить - все норм

          enum MyEnum
        {
            a, b, c
        }
        private static void Main(string[] args)
        {
            MyEnum a = MyEnum.b;
            switch (a)
            {
                    case MyEnum.a:
                    Console.WriteLine("suck my balls");
                    return;
                default: return;
            }
            
        }
          Ответить

          • ava bormand 26.06.2014 10:12 # 0

            Если в switch'е обработаны все ветки, а потом default, то предупреждения про unreachable code не будет?
            Ответить

      • ava kegdan 26.06.2014 09:48 # 0

        патапушта енум не может быть нулл
        Ответить

    Добавить комментарий