Говнокод #3835 — C++ — Говнокод.ру

C++ / Говнокод #3835

+148
1. 1
```
Block *** Cells;
```
Лаконичная семантика C++.
Глядишь на код -- и всё понятно: Cells - это указатель массива указателей на массивы указателей объектов класса Block.
А устно с первого раза хрен выговоришь.

Запостил: maniac, 30 Июля 2010

Tweet
Комментарии (37) RSS
- absolut 30.07.2010 22:33 # +5
  
  Встречаются иногда тройные указатели в реальной жизни программиста. Ничего не поделать.
  Ответить
- ch 30.07.2010 22:35 # +5
  
  где здесь с++?
  где здесь говнокод?
  И вообще это можно интерпретировать как сишный указатель на трехмерный массив.
  Ответить
  - Kornew 30.07.2010 22:42 # −3
    
    C++ подразумевается, в том, что Block - это класс (но аффтар не соизволил этого продемонстрировать)
    
    зы: аффтор осуждает непонятные ему конструкции языка, которым он по всей видимости не очень владеет...
    (предположительно перешёл с паскаля или с какого-то языка, в котором нету явной работы с памятью)
    Ответить
    - maniac 30.07.2010 22:49 # 0
      
      Угу, с паскаля, но без фанатизма. ФП - вещь полезная.
      >>указателей объектов класса Block.
      Всё-таки соизволил. Словестно.
      Ответить
    - TarasB 31.07.2010 15:48 # 0
      
      > предположительно перешёл с паскаля или с какого-то языка, в котором нету явной работы с памятью
      
      facepalm.jpg
      Ответить
      
      bugmenot 26.08.2010 18:03 # +2
      
      я бы даже сказал facepalm.pas
      Ответить
  - maniac 30.07.2010 22:47 # −1
    
    Во-первых, двумерный массив, ежели на то пошло.
    Во-вторых, это пока ещё указатели на указатели и так далее, пока это всё не инициализировано. А каждый указатель на указатель массивов указателей объектов может быть инициализирован своей длиной. И получится, например, треугольный массив указателей, который до трёхмерного не дотягивает примерно полтора измерения.
    Ответить
    - ch 30.07.2010 22:52 # 0
      
      Стоп-стоп-стоп. Чушь.
      Cells[x][y][z]
      До трех умеешь считать?
      Ответить
      
      maniac 30.07.2010 22:57 # 0
      
      Массив объектов? Любопытно. Но, как я уже описал, Cells - это указатель массива указателей на массивы указателей объектов класса Block и никак иначе.
      Ответить
      
      ch 30.07.2010 23:00 # 0
      
      это одно и тоже, марш книжки читать прежде чем разводить демагогию про то, какие тебе массивы мерещатся
      Ответить
      
      maniac 30.07.2010 23:21 # +3
      
      Мне не массивы мерещатся, а двухголовые страусы с фиолетовой шерстью и тяжелым бластером наперевес. А ещё они разговаривают со мной голосами имперских штурмовиков. В основном опохмелиться спрашивают.
      Ответить
      
      Kornew 30.07.2010 23:02 # −1
      
      ch, не в том фишка.
      для создания двумерной матрицы(NxM) объектов нужно выполнить такие действия:
      
      Block *** Cells = (Block ***) new Block[row_count]; for (int i = 0; i < row_count; i++) { Cells[i] = (Block **) new Block[col_count]; for (int j = 0; j < col_count; j++) Cells[i][j] = new Block(); } Cells[1][4]->SomeMethod(); //вариант, демонстрирующий все 3 измерения: Cells[1][4][0].SomeMethod();
      Ответить
      
      maniac 30.07.2010 23:16 # 0
      
      Оно самое. Плюс для каждого объекта можно вызвать свой собственный конструктор/конструктор с другими параметрами/не вызывать конструктора.
      Ответить
      
      ch 30.07.2010 23:43 # 0
      
      Ага, а потом гулять по куче безымянных полуинициализированных объектов как по минному полю. Так пишут только сишники и только с целью дальше в цикле все инициализировать и работать как с массивом.
      Ответить
      
      Kornew 31.07.2010 00:21 # 0
      
      обеспечение помехозащищённости - обязанность программиста;
      
      если пишет не дурак, то там будут проверки:
      
      if (Cells[i][k]) //проверяем был ли создан объект
      {
      // работаем с классом:
      // если инициализация провалилась
      // посыпятся либо ошибки(по Сишному),
      // либо Эксепшены(по Цпп"шному)
      }
      
      так что это "минное поле" - не представляет особой угрозы...
      
      а если пишет кто-то без опыта, то там даже с одиночным объектом могут быть проблемы ;)
      Ответить
      
      ch 31.07.2010 04:38 # +1
      
      угу, угу, только тогда указатели нужно еще инициализировать нулями, да еще лишние нулевые указатели в стеке, если случайно скопировать такую переменню, а потом для каждой копии вызвать delete, то будет краш. короче есть вещи которые не нужно делать, и это как раз тот случай.
      Ответить
      
      Kornew 31.07.2010 07:16 # −2
      
      1% new инициализирует всегда нулями (ну, б если его не переопределять аллоком). т.е. в любом случае, если не выделить память под объект - то ссылка будет указывать на NULL
      
      2% не обязательно вызывать delete для каждого элемента массива;
      delete[] с этим прекрассно справляется
      (правда вот на счёт вызовов деструкторов хз как там будет)
      Ответить
      
      Altravert 31.07.2010 07:31 # +2
      
      Че??!!11 То есть new char[123] отдаст строку забитую нулями? И delete[] вызовет delete для всех объектов на которые ссылаются указатели, содержавшиеся в массиве? Сслылка может указывать на 0?
      Да вы ебанулись, парни(с)
      Ответить
      
      Kornew 31.07.2010 11:50 # −1
      
      #include <stdio.h> int main(int argc, char* argv[]) { int count = 123; char *str = new char[count]; for (int i = 0; i < count; i++) printf("%d", str[i]); scanf("%c", &count); return 0; }
      
      у меня почемуто всё в нули выводит...
      
      NULL == (long) 0;
      Ответить
      
      absolut 31.07.2010 11:55 # +4
      
      Теперь попробуйте отключить отладку.
      А вообще, вы что-нибудь слышали об неинициализированной памяти ?
      Ответить
      
      Kornew 31.07.2010 17:12 # −2
      
      перекомпилил в релизе (если имелось ввиду отключить Дебаг режим) - результат тот же...
      
      "об неинициализированной памяти"
      я не слышал, но решил прогуглить: узнал новое про неинициализированны указатели и опасность их появления (спасибо)
      
      но оное представляет опасность, если код пишет какой-то имбицыл, который вовсе не понимает что делает
      
      зы: надеюсь вы поведаете еще что-то интересное (желательно с пруфлинком)
      Ответить
      
      absolut 31.07.2010 18:01 # +3
      
      Это не тот сайт, по которому стоит программирование изучать.
      Читайте лучше книжки хорошие.
      Ответить
      
      Kornew 31.07.2010 18:12 # −1
      
      в книгах ~50..70% лишнего текста, который объясняет и так понятные вещи...
      
      (когда-то решил прочитать трупастрауса - руководство по С++
      - полезной инфы почти не получил)
      
      так что в книги я не верю =(
      лучше уж какие-то сухие мануалы, без лишнего текста, типа:
      
      (проблема):
      (причины возникновения);
      (способы борьбы);
      (примечания);
      Ответить
      
      Altravert 31.07.2010 18:55 # +2
      
      Ты не найдешь таких (если уровень пункта "проблема" выше того же страутрупа и всяхуйняза21день++). Читай книжки, а из них делай выводы, это важнее. А этот сайт и правда не для таких поисков.
      Ответить
      
      Kornew 31.07.2010 18:24 # 0
      
      зы: любой ресурс пригоден для изучения языка, если на оном говорится: "как нужно" или "как Не нужно" делать ;-)
      
      всё зависит от умения анализировать информацию
      ----------
      
      говонокод.ру - это не только развлекательный ресурс;
      
      тут можно узнать полезную инфу:
      - ошибки других (дабы не делать свои)
      - правки в коментах
      - анализ кода на гавно
      - интересные идеи реализации
      т.п.
      Ответить
      
      absolut 31.07.2010 19:59 # +1
      
      >(когда-то решил прочитать трупастрауса - руководство по С++
      >- полезной инфы почти не получил) ...
      >...всё зависит от умения анализировать информацию
      Странно видеть две этих фразы от одного человека :)
      Ответить
      
      Kornew 31.07.2010 21:05 # −2
      
      честно говоря, я решил прочитать эту книгу уже после 2 лет выполнения практических заданий на
      С++
      
      по этому там почти ничего нового не узнал;
      Ответить
      
      ch 31.07.2010 14:10 # 0
      
      Мне просто пофиг на детали глупых действий.
      Вот например такие массивы используют потому итерирование по массиву бытрее, чем по контейнеру. Но если там половина нулевых указателей, нафиг это надо? Чтобы писать while(!++Cells); да?
      Ответить
      
      Kornew 31.07.2010 16:38 # 0
      
      честно, я даже представления не имею, зачем использовать двумерный массив объектов, часть из которых не инициализируется...
      ------
      мои посты были о том, что всякого рода ошибок можно избежать...
      
      ну а вообще, лучше все эти ошибки присекать на этапе проектирования, дабы потом не приходилось делать заплатки...
      Ответить
      
      ch 31.07.2010 22:16 # 0
      
      все еще не
      Ошибок также можно избегать до их появления, не применяя опасные идеи и техники, в конце концов если что не так, то сам программист за это и отвечает как ты уже сказал. К тому же ты уже успел выстрелить себе в ногу, предположив что что-то всегда инициализируется нулями при распределении памяти.
      Ответить
      
      Kornew 31.07.2010 17:26 # 0
      
      >> для создания двумерной матрицы(NxM) объектов нужно выполнить такие действия:
      
      а чего минусы? тут что-то не так?..
      Ответить
      
      absolut 31.07.2010 20:32 # +3
      
      Для создания двумерной матрицы достаточно одного цикла.
      А вот приведение типа :
      (Block ***) new Block[row_count];
      - это просто здец!
      Ответить
      
      Kornew 31.07.2010 21:24 # 0
      
      Ой мамочкиии... спасибо ;-)
      
      ну вот, 2 года верил препу, что это правильный способ выделения памяти
      а сейчас получив замечание - попробовал иначе, и познал истину
      - всётаки, что не говорите, но тут можно получить некие знания... вот как сейчас ;)
      
      new Block**[row_count];
      
      пошёл думать, как тут 1 циклом обойтись...
      Ответить
      
      Kornew 31.07.2010 21:39 # 0
      
      c 1 циклом можно создать двумерный массив Ссылок на объект...
      
      а двумерный массив объектов(с вызовом конструктора) - через вложенный цикл(как мну писал), либо через 2 отдельных цикла;
      
      иль я снова не прав?(
      Ответить
- Altravert 31.07.2010 07:07 # 0
  
  maniac, ты главное &&&i не пытайся сделать))
  Не говнокод, полюбому. Цивильное обсуждение.
  Ответить
  - maniac 31.07.2010 09:53 # −1
    
    Пхыхыхы :-D
    Пусть и не говнокод, зато документацию нужно читать, даже если очень лень читать конструкции букафф типа "Cells - это указатель массива указателей на массивы указателей объектов класса Block". А то тут уже кому-то трехмерные массивы мерещатся.
    Ответить
    - Altravert 01.08.2010 20:06 # +1
      
      А как тебе "массив массивов указателей"? Первое что в голову пришло.
      Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только во вторник, пятницу или субботу
Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

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

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

Говнокод: по колено в коде.

C++ / Говнокод #3835

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

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