1. C++ / Говнокод #3350

    +1001

    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
    19. 19
    20. 20
    21. 21
    22. 22
    struct Node{
    char* d;
    Node *left;
    Node *right;
};

Node* search_insert(Node *root, char* d){
     /*.........*/
    Node *pnew = new Node;
    pnew->d = d
     /*.........*/
}
int main(){
    /*.........*/
    Node *root=NULL;
    char* r=new char;
    for (int i = 0; i<10; i++){
            scanf("%s", r);
            root=search_insert(root, b[i]);
    }
     /*.........*/
}

    Вот такой тупой недочет стоил мне туевой хучи времени. Не единожды стирал все и писал заново. Просветление пришло на третий день.
    Указатель.... поле в структуре - указатель... указатель в который я раз за разом лезу в цикле...

    Запостил: kir_rik, 30 Мая 2010

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

    • ava kir_rik 30.05.2010 15:00 # 0

      upd: упс, опечаточтка при посте: root=search_insert(root, r);
      Ответить

    • ava guest 30.05.2010 15:28 # −4

      вот такой тупой недодроч...
      Ответить

    • ava absolut 30.05.2010 15:30 # 0

      Зачем динамическое выделение char* r=new char; почему нельзя было обойтись просто char r; ?
      Чтение строки (%s) в память, выделенную под символ до добра не доведет.
      Ответить

      • ava nil 30.05.2010 15:39 # 0

        Стек не резиновый:))))
        Ответить

        • ava guest 30.05.2010 15:50 # −1

          ...а силиконовый и вазелиновый:)
          Ответить

        • ava absolut 30.05.2010 17:42 # +2

          Тут дело скорее не в стеке, а в том, что человек узнал о наличии динамического выделения памяти и использует это там где надо и не надо.
          Ответить

      • ava kir_rik 30.05.2010 16:17 # −1

        Зачем динамическое выделение char* r=new char; почему нельзя было обойтись просто char r;
        Где gcc, а где и mingw
        Ответить

        • ava guest 30.05.2010 17:35 # −2

          ты долбоёб их даже не отличишь
          Ответить

        • ava absolut 30.05.2010 17:49 # +2

          >Где gcc, а где и mingw
          Что Вы этим хотели сказать?
          Ответить

          • ava kir_rik 30.05.2010 19:14 # −2

            У меня при попытке заполнения char* переменной без предварительного new, программа вываливается. Разные компиляторы по разному воспринимают одинаковый код.
            Ответить

            • ava guest 30.05.2010 20:01 # +2

              а зачем *? просто char не катит уже?
              Ответить

              • ava guest 30.05.2010 20:09 # 0

                Затем, что школота узнала про указатели и теперь пихает их куда попало
                Ответить

                • ava kir_rik 30.05.2010 21:07 # +2

                  Забавно... Позавчера народилось новое поколение людей, которые будут, брызгая слюнями, орать в интернетах "Школота!"
                  (с)Башорг
                  Ответить

                  • ava ISith 31.05.2010 01:56 # 0

                    ЫЫЫ, у нас -- вчера, но учитывая этот твой пост, любой код, который ты напишешь в ближайший год, можно смело сюда постить, а вот среди школоты есть челы, котроые реально уже дохера умеюд, по крайней мере относительно написания троянов =)
                    Ответить

                    • ava kir_rik 31.05.2010 01:59 # +1

                      О да, большой скилл - открыть прогу которая генерит трояны и нажать generate
                      Ответить

            • ava guest 30.05.2010 20:44 # +4

              ой какой ужас, злой компилятор не дает писать в невыделенную память.. ой-ой-ой...
              Ответить

            • ava guest 30.05.2010 20:50 # 0

              Стандарт один, если нет кода вызывающего Undefined Behaviour, то все номрмальные компиляторы воспринимают одинаковый код по одинаковому.
              Ответить

            • ava absolut 30.05.2010 22:43 # +3

              Наличие указателя еще не говорит о наличии свободной памяти адресуемой этим указателем. Ошибка распространенная для начинающего программиста. Читайте мат.часть.
              Ответить

              • ava kir_rik 30.05.2010 23:36 # 0

                Ну потому и пишу new. Противоречивые какие-то каменты.
                Ответить

                • ava absolut 30.05.2010 23:45 # +2

                  Это не "каменты" противоречивые. А подход к реализации. Надо знать что делаешь, а не пытаться исправить код до псевдорабочего состояния путем перебора вариантов, ссылаясь на то, что "Разные компиляторы по разному воспринимают одинаковый код."
                  Ответить

                • ava UncleAli 30.05.2010 23:56 # +5

                  Вы в 18-й строке пытаетесь прочитать строку в однобайтовый буфер, в который поместится лишь пустая строка (этот самый байт зохавает терминатор строки). Таким образом, налицо решето.
                  И, да, вы вырезали слишком много контекста, без него назначение кода слегка замутняется.
                  Ответить

                  • ava guest 31.05.2010 09:26 # +1

                    > И, да, вы вырезали слишком много контекста, без него назначение кода слегка замутняется.

                    для чего велосипедят связанные списки - это такая тайна
                    Ответить

                    • ava absolut 31.05.2010 11:23 # +4

                      На данной стадии изучения языка это даже полезно.
                      Ответить

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