|
0 / 0 / 1
Регистрация: 12.10.2013
Сообщений: 32
|
||||||
Сказали некрасиво пишу12.10.2013, 11:08. Показов 12668. Ответов 160
Метки нет (Все метки)
изучаю с++ почти 2 месяц , пишу програму она работает но препод говорит что иногда не красиво пишу ((( пишу так
0
|
||||||
| 12.10.2013, 11:08 | |
|
Ответы с готовыми решениями:
160
RichTextBox курсор некрасиво мерцает При загрузке картинка некрасиво мигает
|
|
БНТУ ФИТР
215 / 155 / 42
Регистрация: 26.12.2012
Сообщений: 382
|
||||||
| 12.10.2013, 17:33 | ||||||
|
Demelurg, именно. и тогда конструктор может выглядеть уже так:
![]() и еще я не ставлю пробел перед запятой в параметрах функций/методов
0
|
||||||
|
Игогошка!
1801 / 708 / 44
Регистрация: 19.08.2012
Сообщений: 1,367
|
|||||||
| 12.10.2013, 17:36 | |||||||
|
Придираться конечно можно. Но в целом мне кажется норм.
К чему я бы придрался, если б имел например жуткое желание завалить: про tmp_min, tmp_sec уже сказали про именование private уже сказали (я бы только ставил _ в конце, а не в начале) нет проверки на допустимость передаваемых в конструктор данных (иначе зачем знаковый тип?) про смесь стилей в sum_Time уже сказали (в том числе это может быть константная функция-член, принимающая константную ссылку) фигурные скобки я бы не сдвигал, так как такой стиль конечно есть и имеет право на существование, но очень мало кем используется. private в данном случае я бы не стал переносить в конец. Публичный интерфейс это конечно хорошо, но иногда один взгляд на самые важные private-члены позволяет сразу очень многое узнать о классе.
0
|
|||||||
|
0 / 0 / 1
Регистрация: 12.10.2013
Сообщений: 32
|
|
| 12.10.2013, 17:38 [ТС] | |
|
про пробел ето уже кому как удобней , а на счет информативности и на счет подчеркивание - реально кульный совет спасиб.
0
|
|
|
162 / 29 / 3
Регистрация: 17.08.2013
Сообщений: 560
|
|
| 12.10.2013, 17:39 | |
|
Часто пишут, что, например, нужно писать в классе сначала public часть, а потом private.
Много таких вещей можно узнать из книги "Совершенный код" Макконнелла
0
|
|
|
5500 / 4895 / 831
Регистрация: 04.06.2011
Сообщений: 13,587
|
|
| 12.10.2013, 17:40 | |
|
1
|
|
|
0 / 0 / 1
Регистрация: 12.10.2013
Сообщений: 32
|
|||
| 12.10.2013, 17:41 [ТС] | |||
|
0
|
|||
|
162 / 29 / 3
Регистрация: 17.08.2013
Сообщений: 560
|
||
| 12.10.2013, 17:42 | ||
Но если класс, то это делается, чтобы сначала было видно по public методам, для чего класс нужен
0
|
||
|
Ушел с форума
|
|||
| 12.10.2013, 17:43 | |||
|
C++03, 17.4.3.1.2 Global names
0
|
|||
|
0 / 0 / 1
Регистрация: 12.10.2013
Сообщений: 32
|
||
| 12.10.2013, 17:43 [ТС] | ||
|
0
|
||
|
БНТУ ФИТР
215 / 155 / 42
Регистрация: 26.12.2012
Сообщений: 382
|
|
| 12.10.2013, 17:44 | |
|
alsav22, это откуда вообще?) Всегда так делал - и неоднократно видел как делают другие. Это же все сугубо в моем классе.
0
|
|
|
5500 / 4895 / 831
Регистрация: 04.06.2011
Сообщений: 13,587
|
|||
| 12.10.2013, 17:47 | |||
|
0
|
|||
|
БНТУ ФИТР
215 / 155 / 42
Регистрация: 26.12.2012
Сообщений: 382
|
|
| 12.10.2013, 17:54 | |
|
ct0r, это если использовать список инициализации. А вот если теле конструктора обращаться и к тому и к другому? this будет нужно использовать.
Добавлено через 6 минут Убежденный, Ну первое понятно - а второе: зарезервированы для использования в качестве имени в глобальном пространстве имен. Не тот случай же.
0
|
|
|
Игогошка!
1801 / 708 / 44
Регистрация: 19.08.2012
Сообщений: 1,367
|
||
| 12.10.2013, 17:55 | ||
|
Да, и которые после if. castaway Я бы все-таки идеологически немного различал структуру и класс. Обычно пишу структуру, когда на первом месте - доступ и модификация данных, а класс - когда действия над ними. kventin_zhuk Ну это уже само собой. Я думал, что ты только про список инициализации говоришь.
1
|
||
|
0 / 0 / 1
Регистрация: 12.10.2013
Сообщений: 32
|
|
| 12.10.2013, 17:57 [ТС] | |
|
извините ребят я уже заплутался , спасиб много полезного узнал , иду дальше учить есле что то вас ищо потревожу
.
0
|
|
|
3189 / 869 / 39
Регистрация: 29.12.2008
Сообщений: 951
|
|
| 12.10.2013, 18:00 | |
|
Как на мой вкус, то переменные убого названы. Есть camelCase aka "верблюжья нотация", есть under_score - принципиальные схемы именования. Почитай про венгерскую нотацию. У тебя же какой-то средний вариант, смотрится не очень. Конечно, кажется мелочью, но на самом деле в будущем, на реальной работе, могут потребовать (корпоративный стиль).
Второе, что не понравилось - тот факт, что какая-то часть методов класса реализована при его объявлении, а метод sum_Time вынесен. Я бы вынес объявление класса в .h-файл, его реализацию в .cpp-файл, и точку входа main - в свой файл. Это традиционный подход, но даже если ты его не захочешь придерживаться, то хотя бы в своем коде должно быть какое-то однообразие. Выносишь методы - выноси все, реализуешь их при объявлении - реализуй все. И не брезгуй пробелами и адекватным форматированием (вон на 34 и 41 строках if-ы на разном уровне), для этого есть Ctrl + K, Ctrl + F в Visual Studio, например. ![]() Так что в целом соглашусь с преподавателем - неэстетичный код.
1
|
|
|
0 / 0 / 1
Регистрация: 12.10.2013
Сообщений: 32
|
|
| 12.10.2013, 18:02 [ТС] | |
|
а Лафоре говорит что структура ток для даных а клас для даных + методы(функции).и по ООП лутше использовать класы чем структуры или я не прав ?
0
|
|
|
БНТУ ФИТР
215 / 155 / 42
Регистрация: 26.12.2012
Сообщений: 382
|
|
| 12.10.2013, 18:05 | |
|
Demelurg, В плюсах стркутура считай тот же класс. Но вот в си - обычные) создатели плюсов оставили зарезервированное слово для совместимости и расширили их возможности
0
|
|
| 12.10.2013, 18:05 | |
|
Помогаю со студенческими работами здесь
40
LaTeX - Как настроить стиль вывода формул? Формулы выводятся некрасиво .NET сказали нет? Сказали сделать такое
phpBB На работе мне сказали сделать таблицу Искать еще темы с ответами Или воспользуйтесь поиском по форуму: |
|
Новые блоги и статьи
|
|||
|
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога
Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
|
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование
. \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json>
Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом.
# Check if. . .
|
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так:
https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347
Основана на STM32F303RBT6.
На борту пять. . .
|
Камера 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. Пошагово создадим проект для загрузки изображения. . .
|