1. Си / Говнокод #6695

    +147

    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
    case 5:
	   { 
	    char podr[20];
            int r=1;
	    puts("Vvedite nazvanie podrazdelenia");
	    scanf("%s",podr);
	    for(i=0;i<n;i++)
			if(!strcmp(podr,s[i].podr))
			{ r=0;
			  for(int j=i;j<n;j++) s[j]=s[j+1];
			  n--; 
                          i--; // костыль
		    }
	    if(r) puts("ERROR");
	    getch();
	    } break;

    Опять костыль для лабы друга.

    Запостил: dark-al, 17 Мая 2011

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

    • ava SIGSEGV 17.05.2011 19:48 # +1

      И в чем заключается костыль? Или автор только что с паскаля "слез"?
      Ответить

      • ava dark-al 17.05.2011 19:57 # −1

        На паскале только в школе писал. В данном случае я назвал костылем, то что добавил в код (чтобы он естественно работал). Университетские лабы ничто иное как говнокод.
        Ответить

        • ava Lure Of Chaos 17.05.2011 20:38 # +6

          Университетские лабы ничто иное как унылое говно
          Ответить

        • ava Fail 17.05.2011 20:53 # 0

          а вот если по массиву бежать с другой стороны, то "костыль" бы не понадобился. так что проблема не только в университетских лабах....
          Ответить

          • ava dark-al 17.05.2011 21:03 # 0

            Мне дали лабу, сказали сделай чтобы работало. Да, костыль можно убрать, но это займет время. А оно мне надо? :)
            Ответить

    • ava guest 17.05.2011 22:48 # 0

      за хуёвое форматирование кода надо расстреливать...
      Ответить

    • ava TarasB 18.05.2011 09:52 # 0

      > scanf("%s",podr);

      Кстати, эта процедура проверяет буфер на переполнение?
      Ответить

      • ava koodeer 18.05.2011 13:51 # 0

        Нет конечно. Переполнение буфера здесь обеспечено ;)
        Ответить

        • ava TarasB 18.05.2011 13:54 # 0

          А зачем тогда вообще нужна эта процедура?
          Ответить

          • ava guest 18.05.2011 14:31 # +3

            Для переполнения буфера, очевидно.
            Ответить

          • ava TheCalligrapher 18.05.2011 16:58 # 0

            Какая "процедура"? 'scanf'? Классический 'scanf' предоставляет средства защиты от переполнения буфера, только надо не забывать ими пользоваться. В данном случае автор этого не сделал. Так что спрашивать надо зачем нужен этот автор, а не зачем нужна эта "процедура".
            Ответить

    • ava TarasB 18.05.2011 09:54 # 0

      А, да, за такое удаление элемента из массива надо расстреливать.
      Ответить

      • ava Vindicar 18.05.2011 13:52 # +2

        За "за X надо расстреливать" надо расстреливать!


        Куда вы меня тащи~
        Ответить

        • ava TarasB 18.05.2011 13:55 # 0

          Не, ну серьёзно, ну почему для удаления элементов из массива берут такой тупорылый алгоритм, который хуже пузырька? И даже то, что для этого говноалгоритма приходится на ходу менять счётчик и границы цикла, не смущает макак.
          Ответить

          • ava Vindicar 18.05.2011 14:06 # 0

            Я что-то не соображу, какая тут альтернатива. Отсортировать по признаку "содержит-не содержит", а потом обрезать конец?
            Ответить

            • ava TarasB 18.05.2011 14:36 # 0

              > Я что-то не соображу
              Пипец, я для детей статью писал.
              http://forum.pascal.net.ru/index.php?showtopic=26642
              Вообще да, ты примерно прав.
              Ответить

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