331 / 283 / 78
Регистрация: 02.08.2016
Сообщений: 1,008
|
||||||
1 | ||||||
Прошу оценить код на C++25.02.2017, 18:58. Показов 1153. Ответов 10
Метки нет (Все метки)
Сильно не пинайте, только начал изучать C++, написал вот такое кот. Что не так в этом коде? Перечислите, какие есть недостатки. Предложите свой вариант рефакторинга.
0
|
25.02.2017, 18:58 | |
Ответы с готовыми решениями:
10
Прошу оценить свой уровень Прошу оценить стиль программы Калькулятор для начинающих, прошу оценить Я не прошу писать мне код, я прошу подсказать мне, что за структура требуется в задании |
Неэпический
|
|
25.02.2017, 19:03 | 2 |
Можно убрать if.
Плюс ко всему, Вы выделяли память с помощью new[], поэтому освобождать нужно с помощью delete[]. Динамический полиморфизм?
1
|
331 / 283 / 78
Регистрация: 02.08.2016
Сообщений: 1,008
|
|
25.02.2017, 19:55 [ТС] | 3 |
Спасибо.
Вообще говоря, это задание из вакансии Яндекса https://yandex.ru/jobs/vacanci... yamaps_nn/ и мне было интересно, как его решат более опытные программисты, чем я, но похоже оценивать чужой код тут не любят Вот моё решение: 1 Убрать пустой #include 2 использовать наследование и функцию draw сделать виртуальной 3 Использовать средства C++ для чтения файлов и вынести чтение за пределы класса Feature, ну или хотя бы назвать метод чуть более очевидно readFromFile 4 Использовать современные средства для управления памятью, вместо new delete.
0
|
Любитель чаепитий
|
|
25.02.2017, 20:17 | 5 |
Ага. А откуда возьмутся функции ака fread, fopen? Не из воздуха жеж.
А разве удаление нулевого указателя корректно? Добавлено через 10 минут А всё, вспомнил. Там, вроде, внутри проверка есть.
0
|
-16 / 14 / 3
Регистрация: 22.01.2013
Сообщений: 154
|
|
25.02.2017, 20:20 | 7 |
Ты - буратино.
0
|
331 / 283 / 78
Регистрация: 02.08.2016
Сообщений: 1,008
|
|
25.02.2017, 20:36 [ТС] | 8 |
Ну так тогда нужно приинклудить stdio.h
Да Добавлено через 14 минут Он там есть, просто они забыли экранировать символы < > и браузер распознал его как тег...
0
|
Игогошка!
1801 / 708 / 44
Регистрация: 19.08.2012
Сообщений: 1,367
|
|
26.02.2017, 01:59 | 9 |
Тут все просто. Алгоритм такой, очевидно:
1) Врубаемся в то, что делает код. 2) Включаем телепатию и просекаем, что должен делать код на самом деле. 3) Полностью стираем текущее поделие школьника-наркомана (в данном случае это явно так) и пишем свое. 4) Отправляем в яндекс. Ждем ответа. 5) Идем работать не в яндекс. 6) ... 7) Профит!
8
|
Падаван С++
447 / 261 / 89
Регистрация: 11.11.2014
Сообщений: 916
|
||||||
26.02.2017, 10:13 | 10 | |||||
DevAlone, ну допустим я бы применил паттерн стратегия для реализации рисования объектов, что то типо такого
мне еще жутко не нравится метод read но как то его перерабатывать лень, но я чисто показать суть, что можно в этом коде что то и поменять, на правильность не претендую, но как по мне выглядит получше
1
|
Неэпический
|
||||||
26.02.2017, 10:19 | 11 | |||||
1
|
26.02.2017, 10:19 | |
26.02.2017, 10:19 | |
Помогаю со студенческими работами здесь
11
Просьба оценить код Оценить код Java для начинающих , прошу оценить код Прошу оценить код класса парсинга данных Искать еще темы с ответами Или воспользуйтесь поиском по форуму: |