Кресты / Говнокод #8107 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
  16. 16
  17. 17
  18. 18
  19. 19
  20. 20
  21. 21
  22. 22
  23. 23
  24. 24
  25. 25
  26. 26
  27. 27
  28. 28
  29. 29
  30. 30
  31. 31
  32. 32
#define BREAKABLE_SECTION() for(;;)

BREAKABLE_SECTION()
{
    ::Packet * pPacket;
    res = ReadPacket(pPacket);

    if(ERR_OK == res)
    {
        res = pConstructor->PutPacket(pPacket);

        if(ERR_OK == res)
        {
            res = pConstructor->GetFrame(data);
        }else
        {
            //TODO:add handler of statuses other than err_ok
        }

        if (ERR_OK ==res)
        {
            break;
        }else
        {
            //TODO:add handler of statuses other than err_ok
        }
    }else
    {
        break;
    }
    break;
}

перешел в новый проект где предложили изучить существующий код в огромном объеме - а он весь из таких циклов

Запостил: openclgovno openclgovno, (Updated )

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

  • ava ctm # Ссылка на оригинал 0

    1. ну уж конечно BREAKABLE_SECTION() читабельнее for(;;). какой с него толк? поиск по исходникам?
    2. писать goto не кошерно, а по другому не умеют - вот и заменили бряками
    Ответить

          • ava roman-kashitsyn # Ссылка на оригинал 0 ⇈

            Спасибо, кэп, стандартную библиотеку я знаю хорошо. Такие эксепшены даром не нужны.
            Ответить

            • ava sayidandrtfm # Ссылка на оригинал 0 ⇈

              >даром не нужны
              А обычно платите?

              Ну тогда минимум 3 варианта:
              - выбрать другой инструмент;
              - _продвинуть_ в следующий стандарт идею введения исключений;
              - написать/переписать свой язык.
              Ответить

              • ava roman-kashitsyn # Ссылка на оригинал 0 ⇈

                > А обычно платите?
                да, время на внедрение, знаете ли, денег стоит
                Предложенные варианты нельзя было использовать, когда писал на си. Просто жалел, что приходится писать кучу непонятного кода, который можно было бы сократить и упростить с помощью исключений.
                Ответить

                • ava sayidandrtfm # Ссылка на оригинал 0 ⇈

                  >время на внедрение
                  т. е. изначально архитектура не предполагала наличия некого подобия механизма исключений — тогда да.
                  Ответить

                  • ava roman-kashitsyn # Ссылка на оригинал 0 ⇈

                    Изначально не предполагала. Переписывать полмиллиона строк кода промышленного сервера никто не будет.
                    Ответить

                    • ava carsten # Ссылка на оригинал 0 ⇈

                      забавно будет, если эти полмиллиона строк превратятся в 50,000 строк на каком-нибудь сверх-высоком языке
                      Ответить

    • ava guest # Ссылка на оригинал 0 ⇈

      "писать goto не кошерно" как бы вы это переписали ?
      замечу тут даже флажок не воткнуть смотрите linux/list.h
      разбивать на функции чтоб спрятать goto маразм ну и как ?
      да никак городили бы или do {} while или выеросили бы циклы
      в функции что только ухудшило бы понимание кода
      понабегут теоретики и кричат goto плохо
      static VOID CheckUnique(list_head *liblist, list_head *apilist)
{
again_scan:
	apiitem *apiel;
	list_for_each_entry(apiel, apilist, link)
	{
		libitem *libel;
		list_for_each_entry(libel, liblist, link)
		{
			apiitem *apiuniqel;
			list_for_each_entry(apiuniqel, &libel->apilist, link)
			{
				if (!strcmp(apiuniqel->szApi, apiel->szApi))
				{

					list_del(&apiel->link);
					free(apiel);
					goto again_scan;
				}
			}
		}
	}
}
      Ответить

      • ava guest # Ссылка на оригинал 0 ⇈

        "писать goto не кошерно" как бы вы это переписали ?
        замечу тут даже флажок не воткнуть смотрите linux/list.h
        разбивать на функции чтоб спрятать goto это маразм ну дак как ?
        да никак городили бы или do {} while или вынесли бы циклы
        в функции что только ухудшило бы понимание кода
        понабегут теоретики и кричат goto плохо мудаки вы а не программисты
        все те кто против goto
        static VOID CheckUnique(list_head *liblist, list_head *apilist)
{
again_scan:
	apiitem *apiel;
	list_for_each_entry(apiel, apilist, link)
	{
		libitem *libel;
		list_for_each_entry(libel, liblist, link)
		{
			apiitem *apiuniqel;
			list_for_each_entry(apiuniqel, &libel->apilist, link)
			{
				if (!strcmp(apiuniqel->szApi, apiel->szApi))
				{

					list_del(&apiel->link);
					free(apiel);
					goto again_scan;
				}
			}
		}
	}
}
        Ответить

        • ava guest # Ссылка на оригинал 0 ⇈

          ждем говнорешение от гождем говнорешение от говнопрограммистов без goto предыдущего примера
          пусть уже наконец все убедятся что те кто орут goto плохо всего
          лиш теоретики а я вам скажу что они предложат вынести циклы в функции
          потому что эти типотенты считают себя мега крутыми когда делают все
          только хуже лиш бы спрятать goto у них комлекс с детства какой то
          с психикой связанный не с проста они везде стараются спрятать
          goto в даже ущерб читаемостивнопрограммистов без goto
          Ответить

          • ava guest # Ссылка на оригинал 0 ⇈

            ждем говнорешение от говнопрограммистов без goto предыдущего примера
            пусть уже наконец все убедятся что те кто орут goto плохо всего
            лиш теоретики а я вам скажу что они предложат вынести циклы в функции
            потому что эти типотенты считают себя мега крутыми когда делают все
            только хуже лиш бы спрятать goto у них комлекс с детства какой то
            с психикой связанный не с проста они везде стараются спрятать
            goto в даже ущерб читаемости
            Ответить

            • ava guest # Ссылка на оригинал 0 ⇈

              ждем говнорешение от говнопрограммистов без goto предыдущего примера
              пусть уже наконец все убедятся что те кто орут goto плохо всего
              лиш теоретики а я вам скажу что они предложат вынести циклы в функции
              потому что эти импотенты считают себя мега крутыми когда делают все
              только хуже лиш бы спрятать goto у них комлекс с детства какой то
              с психикой связанный не с проста они везде стараются спрятать
              goto в даже ущерб читаемости
              Ответить

              • ava 3.14159265 # Ссылка на оригинал 0 ⇈

                CPPGovno он же dosGovno, LinuxGovno, WindowsGovno,
                C_Krest_Krest_Govno он же openclgovno - залогинься.
                Ответить

              • ava defecate-plusplus # Ссылка на оригинал 0 ⇈

                ПАЦАНЫ, Я СЕГОДНЯ ШЁЛ КОРОЧЕ ПО ГОВНОКОДУ И УВИДЕЛ БЫДЛОКОДЕРА В МАЙКЕ "ЦИКЛЫ И УСЛОВНЫЕ ПЕРЕХОДЫ", НУ Я ПОДСКОЧИЛ И РЕЗКО ПЕРЕЕБАЛ ЕМУ В ЩЩИ С ВЕРТУШКИ И ПОЯСНИЛ ЕГО КРИКОМ "НЕ ЛЮБЛЮ ОПЕРАЦИИ СРАВНЕНИЯ", ПОТОМУ ЧТО Я УГОРЕЛ ПО GOTO, ПАЦАНЫ ДУХ СТАРОЙ ШКОЛЫ ЖИВЁТ ТОЛЬКО В ЛИНАКСЕ, ГДЕ ЕБАШАТСЯ ПО ХАРДКОРУ, ГДЕ ИДЕОЛОГИЧЕСКИЕ ДЕВСТВЕННИКИ ЖИВУТ ЕМЁРДЖЕМ, ГЦЦ И ЕБУТ ЭКСЕПШЕНЫ В РОТ! ТОЛЬКО LINUX/LIST.H, ТОЛЬКО ХАРДКОР!!! ЮНИТИ УЛЬТРАХАРДКОР РЭД-АЙ ЛИНУКС!!! пацаны ебашьте джавотварей, шарперов, крестоблядей, питонопидарасню, угорайте на ИРЦ, любите свою Операционную Систему, пацанов и Линуса! ГОВОРИТЕ ОТКРЫТО И СМЕЛО ПРЯМО В ЛИЦО! GOTO!
                Ответить

      • ava ctm # Ссылка на оригинал 0 ⇈

        поясню свое мнение по этому вопросу:
        писать goto считают не кошерным, а по другому не умеют - вот и заменили бряками.
        как я бы написал... не знаю: зависит от обработчика ERR_OK.
        в Вашем примере goto вполне уместен.
        PS. я не теоретик:)
        Ответить

        • ava guest # Ссылка на оригинал 0 ⇈

          #include <stdio.h>
          int main() {
          int i = 0;
          nach:
          i = i + 1;
          if(i <= 10) goto pech;
          if(i > 10) goto konets;
          konets:
          printf("Konets programmi.");
          return 0;
          pech:
          printf("%d v kvadrate = %d\n", i, i * i);
          goto nach;
          }

          лучшее - враг хорошего... в общем, нет понту обсирать единоразовое использование goto в условиях безысходности. Как говорится, один раз - не пидараз... Истинное же мясо, порубленное вместе с будкой и цепью начинается тогда, когда бейсиковый стиль применяестя без разрения совести для написания всего вообще кода. Как пример, см. приведенный выше шматок.
          Ответить

  • ava defecate-plusplus # Ссылка на оригинал 0

    #define BREAK_IF_ERROR(r) if (r != ERR_OK) break;

    do {
    ::Packet * pPacket;
    res = ReadPacket(pPacket);
    BREAK_IF_ERROR(res);
    res = pConstructor->PutPacket(pPacket);
    BREAK_IF_ERROR(res);
    res = pConstructor->GetFrame(data);
    BREAK_IF_ERROR(res);
    } while(0);
    // analyze res here
    Ответить

    • ava Kirinyale # Ссылка на оригинал 0 ⇈

      Ну break; же всяко быстрее пишется, чем goto KUDA_TO_TAM;
      Правда, здесь это компенсировали дефайном...
      Ответить

  • ava guest # Ссылка на оригинал 0

    openclgovno, defecate-plusplus
    Нечего меня форсить. Я давно ушел. Комунити, к этим людям я не имею ни какого отношения. (:
    Вечно ваш CPPGovno, C_Krest_Krest_Govno, dosGovno, LinuxGovno, WindowsGovno. Пока. (:


    openclgovno, ты считаешь DirectCompute11 лучше, чем OpenCL?
    Только не говори AMD FireStream и NVIDIA CUDA, ибо это не кроссаппаратные технологии другого класса.
    Ответить

    • ava openclgovno # Ссылка на оригинал 0 ⇈

      opencl относительно новая технология, а все новое должно пройти проверку. Пока же я вижу что стандарт как бы уже давно есть, но детали отличаются у производителей железа. За примерами дальше интела и нвидии ходить не надо.
      Ответить

      • ava guest # Ссылка на оригинал 0 ⇈

        Советую посмотреть ещё на C++ AMP.
        Использование GPU, как сопроцессора - это весьма перспективная идея. Ждем стандартизации этого комитетом С++ и реализацию под GCC. ^_^
        Ответить

      • ava guest # Ссылка на оригинал 0 ⇈

        Планируется отправить спецификацию C++ AMP организации по стандартизации, чтобы технология стала “открытой спецификацией”, которую мог реализовать любой компилятор. В планах сделать эту технологию доступной на компиляторах для любых платформ.
        Ответить

    • ava 3.14159265 # Ссылка на оригинал 0 ⇈

      во-первых, кем бы ты не был - ты мудак.
      во-вторых - это копипаст с моего поста, где я не упомянул о macGovno.
      Ответить

      • ava guest # Ссылка на оригинал 0 ⇈

        >ты мудак.
        во-первых, это намекает нам на твой уровень. (:

        во-вторых - это копипаст с твоего поста, ибо мне лень набирать вручную эту стену, но заметь перестановку твоих слов.
        Ответить

          • ava guest # Ссылка на оригинал 0 ⇈

            Банально выписал диагноз из медкарты, больше ничего умней придумать просто не смог. Пришлось копипастить. (:
            Ответить

            • ava Lure Of Chaos # Ссылка на оригинал 0 ⇈

              не надо придумывать оправдания, просто вот тебе было бы слабО сиронизировать над собой, не прячась в анонимусы = )
              Ответить

              • ava guest # Ссылка на оригинал 0 ⇈

                Между прочим я не прячусь. Меня просто никто не ищет и меня это устраивает. (:
                Ответить

      • ava guest # Ссылка на оригинал 0 ⇈

        Между тем, у тебя по делу вообще ни одного слова. Ты способен лишь сраться с другими. Это твой интеллектуальный придел. Ты нигде не добьёшся успеха, в отличии от openclgovno. Мне тебя жаль. (:
        Ответить

  • ava Lure Of Chaos # Ссылка на оригинал 0

    что общее между преступниками и мудаками? они возвращаются
    Ответить

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

Помни, guest, за тобой могут следить!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓