|
63 / 46 / 11
Регистрация: 27.12.2017
Сообщений: 1,484
|
|||||||||||
Оценка кода28.01.2020, 22:43. Показов 6548. Ответов 123
Метки нет (Все метки)
Здравствуйте, прошу вас всех оценить мой код , по возможности дать советы что так , а что не так и как это исправить либо в каком направлении смотреть , это моя первая попытка создать что-то не ради забавы , а чтобы использовать это в реальной жизни. Сразу скажу что еще не все доделал что хотел бы. Это декодер asn1.der , данные передаются в decode() в виде BLOB конвертированного в массив знаков.
header Кликните здесь для просмотра всего текста
cpp Кликните здесь для просмотра всего текста
0
|
|||||||||||
| 28.01.2020, 22:43 | |
|
Ответы с готовыми решениями:
123
Оценка кода
|
|
Just Do It!
|
||
| 09.02.2020, 15:32 | ||
|
Что у вас получится: смартпоинтер исчезнет? я поэтому и написал выше что можно строить иерархию классов так что смарты не нужны, но ясень что они очень удобны, чтобы писать гуанокод.
0
|
||
|
8973 / 4319 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
|
|||||||
| 09.02.2020, 15:39 | |||||||
|
должно быть так:
0
|
|||||||
|
19500 / 10105 / 2461
Регистрация: 30.01.2014
Сообщений: 17,816
|
||
| 09.02.2020, 15:42 | ||
![]() За многословность, за отсутствие предметной области. Было бы желание. Плюс ко всему ты зачем-то сразу привел решение проблемы, хотя нужно, чтобы автор сам его предложил.
0
|
||
|
8973 / 4319 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
|
||
| 09.02.2020, 15:44 | ||
|
есть такая новичково-джуниорская ошибка: когда они начинают думать о гипотетических проблемах, вместо того, что бы думать о реальных проблемах конкретного кода.
0
|
||
|
19500 / 10105 / 2461
Регистрация: 30.01.2014
Сообщений: 17,816
|
||
| 09.02.2020, 15:46 | ||
|
В тех случаях, когда не целесообразно писать руками по такой обертке на каждый чих (вспомним о бритве Оккама), существует библиотечное, обобщенное решение, которое реализует RAII: смартпойнтеры.
1
|
||
|
8973 / 4319 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
|
|||
| 09.02.2020, 15:50 | |||
|
вот как раз таки можешь начать с многословности и отсутствия предметной области. заметь: весьма наглядное.
0
|
|||
|
19500 / 10105 / 2461
Регистрация: 30.01.2014
Сообщений: 17,816
|
|||
| 09.02.2020, 15:55 | |||
![]() Если бы я сам привел решение, то не было бы обучающего эффекта. Но ладно, раз ты так хочешь, я самоустраняюсь из темы и предоставляю ТС усваивать информацию по твоему, лучшему курсу. ![]() Если у тебя есть какие-то вопросы лично ко мне, прошу задавать их в личку.
0
|
|||
|
Неэпический
|
||||||
| 09.02.2020, 16:01 | ||||||
|
hoggy,
0
|
||||||
|
63 / 46 / 11
Регистрация: 27.12.2017
Сообщений: 1,484
|
|
| 09.02.2020, 16:02 [ТС] | |
|
DrOffset, так а что насчёт bad_alloc? И как смартпоинтеры может определить что указатель кказвыет именно на массив?
0
|
|
| 09.02.2020, 16:09 | |
|
Не по теме: В реальном коде где Вы видели что бы конструкторе был ввод/вывод?
0
|
|
|
8973 / 4319 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
|
||||
| 09.02.2020, 16:22 | ||||
|
раз уж ты хотел, что бы рассматривали эксепшены вообще, а не конкретно: неоднозначную ситуацию с bad_alloc. как ты умыдряешься сохранять спокойствие среди засилия тупизны на этом форуме? помнишь, что ты мне ответил? ты ответил мне: "я сохраняю спокойствие, потому что мне всё равно о чём думают новички, и что из них получится в будущем". (ц) по памяти. с тех пор прошло не мало времени. и сейчас я понимаю: мне тоже всё равно. у меня нет цели кого-то учить.
0
|
||||
|
Неэпический
|
||
| 09.02.2020, 16:23 | ||
unique_ptr<int> - на один элементunique_ptr<int[]> - на массив элементов
0
|
||
|
63 / 46 / 11
Регистрация: 27.12.2017
Сообщений: 1,484
|
||||||
| 09.02.2020, 16:29 [ТС] | ||||||
|
Croessmah, не совсем понимаю как работает тогда удаление?
0
|
||||||
| 09.02.2020, 16:35 [ТС] | |
|
Не по теме: тема слегка уже переходит во флуд, слишком много сообщений которые не несут никакой пользы , тяжко потом будет кому-то читать
0
|
|
|
2784 / 1937 / 570
Регистрация: 05.06.2014
Сообщений: 5,602
|
||
| 09.02.2020, 16:36 | ||
|
0
|
||
|
Mental handicap
1246 / 624 / 171
Регистрация: 24.11.2015
Сообщений: 2,429
|
|||||||
| 09.02.2020, 16:40 | |||||||
0
|
|||||||
|
1550 / 877 / 179
Регистрация: 05.12.2015
Сообщений: 2,555
|
|||
| 10.02.2020, 14:59 | |||
|
На выходных было не до форума. А тут нафлудили-то!
Теперь, допустим, твой сын первый раз проехал на велосипеде и не упал. Он спрашивает тебя: "Папа, а я хорошо еду?". Ты его отправишь читать наставление по тренировкам спортсменов-велосипедистов? Сам же писал:
0
|
|||
| 10.02.2020, 14:59 | |
|
Помогаю со студенческими работами здесь
120
оценка кода Оценка стиля кода Оценка кода динамического двумерного массива
Оценка стиля написания кода Искать еще темы с ответами Или воспользуйтесь поиском по форуму: |
|
Новые блоги и статьи
|
|||
|
Символьное дифференцирование
igorrr37 13.02.2026
/ *
Логарифм записывается как: (x-2)log(x^2+2) - означает логарифм (x^2+2) по основанию (x-2).
Унарный минус обозначается как !
*/
#include <iostream>
#include <stack>
#include <cctype>. . .
|
Камера Toupcam IUA500KMA
Eddy_Em 12.02.2026
Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
|
И ясному Солнцу
zbw 12.02.2026
И ясному Солнцу,
и светлой Луне.
В мире
покоя нет
и люди
не могут жить в тишине.
А жить им немного лет.
|
«Знание-Сила»
zbw 12.02.2026
«Знание-Сила»
«Время-Деньги»
«Деньги -Пуля»
|
|
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров
8Observer8 12.02.2026
Содержание блога
Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .
|
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image)
8Observer8 11.02.2026
Содержание блога
Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .
|
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image
8Observer8 10.02.2026
Содержание блога
Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .
|
Установка Qt-версии Lazarus IDE в Debian Trixie Xfce
volvo 10.02.2026
В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .
|