|
3 / 3 / 2
Регистрация: 21.10.2013
Сообщений: 225
|
|
.NET 4.x [Code review] ООП ошибки03.10.2017, 09:06. Показов 1343. Ответов 10
Здравствуйте! Есть программа и она рабочая. И мне для дальнейшего программирования необходимо знать насколько она правильно, оптимально и красиво написана в плане ООП. Понимаю, что никто мне ничем не обязан, однако вдруг найдутся доброходы, которые мне подскажут :"Вот тут ты каку написал! Так не пишут, а пишут вот так" - ну или примерно.
PS: Ещё раз повторюсь, только если Вам самим интересно. Если советов не будет буду и дальше в том же ключе писать. Заранее спс.
0
|
|
| 03.10.2017, 09:06 | |
|
Ответы с готовыми решениями:
10
Code Review игры Тетрис
|
|
12 / 12 / 8
Регистрация: 25.05.2014
Сообщений: 36
|
|
| 03.10.2017, 09:29 | |
|
Выложите на код github. На мой взгляд, народу будет охотнее смотреть код там, чем скачивать архив
2
|
|
|
3 / 3 / 2
Регистрация: 21.10.2013
Сообщений: 225
|
|
| 03.10.2017, 09:48 [ТС] | |
|
Спс за совет.
0
|
|
|
14145 / 9374 / 1350
Регистрация: 21.01.2016
Сообщений: 35,293
|
|
| 03.10.2017, 09:53 | |
|
ToLexx, на гитхабе вам могут комментов к коду накидать прямо с веб-страницы. Так, что такой способ всем удобен.
1
|
|
|
1 / 1 / 0
Регистрация: 22.04.2017
Сообщений: 9
|
|
| 03.10.2017, 12:44 | |
|
я дико извиняюсь, но ООП я вообще не увидел, ни инкапсуляции, ни полиморфа, ни наследования.... может я не туда смотрел, подскажите где эти реализации?
1
|
|
|
3 / 3 / 2
Регистрация: 21.10.2013
Сообщений: 225
|
|||||||||||
| 03.10.2017, 14:51 [ТС] | |||||||||||
|
Добавлено через 7 минут
Нашёл вот такой вот пример:
0
|
|||||||||||
|
.NET C#,ASP.NET MVC
594 / 507 / 224
Регистрация: 16.10.2010
Сообщений: 1,902
|
||
| 03.10.2017, 14:52 | ||
|
А насчет кода - присоеденяюсь к вышесказанному, затейте гит
0
|
||
|
3 / 3 / 2
Регистрация: 21.10.2013
Сообщений: 225
|
|
| 03.10.2017, 14:54 [ТС] | |
|
Или тут вариант, что "S" и "B" будут неизменны, как бы ими не игрались в классе и его методах, т.к. в конструкторе фактически они переименованы???
Ок всё-таки дотряхию этот git((
0
|
|
|
911 / 796 / 329
Регистрация: 08.02.2014
Сообщений: 2,391
|
|
| 03.10.2017, 14:54 | |
|
1
|
|
|
.NET C#,ASP.NET MVC
594 / 507 / 224
Регистрация: 16.10.2010
Сообщений: 1,902
|
|
| 03.10.2017, 14:58 | |
|
ToLexx, плюс в том что класс служит для описания объекта, сущности чего либо
В первом примере ваш MyClass содержит набор свойств, описывая объекты более рационально, согласитесь если заменить название MyClass на к примеру Human, сразу становиться понятно что класс описывает человека, со своим набором свойств которые уникальны для каждого. В вашем случае - имя и возраст. И обьект создаете вы вполне осмысленный. Обьект Александр, которому 26 лет Во втором варианте MyClass по сути не несет никакой смысловой нагрузки, он содержит простой метод который можно разместить в любом классе и работать он будет так же. Это так в двух словах, и пример и правда не очень хороший
1
|
|
|
Фрилансер
761 / 458 / 146
Регистрация: 06.01.2017
Сообщений: 4,856
|
|
| 04.10.2017, 18:02 | |
|
залейте лучше на гитхаб а то реально архив скачивать не охота))
0
|
|
| 04.10.2017, 18:02 | |
|
Помогаю со студенческими работами здесь
11
[Code review] Сумма нечётных от первого числа до 2 [Code review] Алгоритм Брезенхема для линии
[Code review] Даны два числа. Заменить большее число тройным произведением, меньшее - полусуммой [Code review] Дано натуральное число n, можно его представить в виде суммы трех квадратов натуральных чисел Искать еще темы с ответами Или воспользуйтесь поиском по форуму: |
|
| Опции темы | |
|
|
Новые блоги и статьи
|
|||
|
Символьное дифференцирование
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, то после закрытия окошка. . .
|