3 / 3 / 2
Регистрация: 21.10.2013
Сообщений: 225
.NET 4.x

[Code review] ООП ошибки

03.10.2017, 09:06. Показов 1343. Ответов 10

Студворк — интернет-сервис помощи студентам
Здравствуйте! Есть программа и она рабочая. И мне для дальнейшего программирования необходимо знать насколько она правильно, оптимально и красиво написана в плане ООП. Понимаю, что никто мне ничем не обязан, однако вдруг найдутся доброходы, которые мне подскажут :"Вот тут ты каку написал! Так не пишут, а пишут вот так" - ну или примерно.

PS: Ещё раз повторюсь, только если Вам самим интересно. Если советов не будет буду и дальше в том же ключе писать.

Заранее спс.
Вложения
Тип файла: zip Allan Variation v.1.zip (3.89 Мб, 10 просмотров)
0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
03.10.2017, 09:06
Ответы с готовыми решениями:

Code Review игры Тетрис
Доброго времени суток. Я самоучка. Для практики написал тетрис в консоли. Столкнулся с тем, что нужен взгляд на код со стороны, опытным...

Односвязный кольцевой список [code review]
/* Операции: - Поиск элемента по индексу - Вставка в начало/конец - Вставка перед/после элемента x - Удаление...

Двусвязный линейный список [code review]
/* Операции: - Поиск элемента по индексу - Вставка в начало/конец - Вставка перед/после элемента x - Удаление...

10
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
Эксперт .NET
 Аватар для Usaga
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 минут
Нашёл вот такой вот пример:
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace ConsoleApplication1
{
    class MyClass
    {
        public string Name;
        public byte Age;
 
        // Создаем параметрический конструктор
        public MyClass(string s, byte b)
        {
            Name = s;
            Age = b;
        }
 
        public void reWrite()
        {
            Console.WriteLine("Имя: {0}\nВозраст: {1}", Name, Age);
        }
    }
 
    class Program
    {
        static void Main(string[] args)
        {
            MyClass ex = new MyClass("Alexandr", 26);
            ex.reWrite();
 
            Console.ReadLine();
        }
    }
}
Я не врубаюсь в чём плюс по отношению к следующему:
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace ConsoleApplication1
{
    class MyClass
    {
        public void reWrite(string Name1, byte Age1)
        {
            Console.WriteLine("Имя:" +Name1+"\nВозраст: "+Age1.ToString());
        }
    }
 
    class Program
    {
        static void Main(string[] args)
        {
            MyClass ex = new MyClass();
            ex.reWrite("Alex",26);
 
            Console.ReadLine();
        }
    }
}
итог будет один, а строк ведь меньше!
0
.NET C#,ASP.NET MVC
Эксперт .NET
 Аватар для lvlkoo
594 / 507 / 224
Регистрация: 16.10.2010
Сообщений: 1,902
03.10.2017, 14:52
Цитата Сообщение от Updater Посмотреть сообщение
но ООП я вообще не увидел
C# не бывает без ООП, в априори создавай новый проект вы уже автоматически используете ООП. Вопрос только насколько придерживаться принципов, необходимости итд и тп.

А насчет кода - присоеденяюсь к вышесказанному, затейте гит
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

Не по теме:

Цитата Сообщение от ToLexx Посмотреть сообщение
итог будет один, а строк ведь меньше!
не в строках счастье



найденный пример тоже не совсем корректен
1
.NET C#,ASP.NET MVC
Эксперт .NET
 Аватар для lvlkoo
594 / 507 / 224
Регистрация: 16.10.2010
Сообщений: 1,902
03.10.2017, 14:58
ToLexx, плюс в том что класс служит для описания объекта, сущности чего либо

В первом примере ваш MyClass содержит набор свойств, описывая объекты более рационально, согласитесь если заменить название MyClass на к примеру Human, сразу становиться понятно что класс описывает человека, со своим набором свойств которые уникальны для каждого. В вашем случае - имя и возраст. И обьект создаете вы вполне осмысленный. Обьект Александр, которому 26 лет

Во втором варианте MyClass по сути не несет никакой смысловой нагрузки, он содержит простой метод который можно разместить в любом классе и работать он будет так же.

Это так в двух словах, и пример и правда не очень хороший
1
Фрилансер
 Аватар для FateOri
761 / 458 / 146
Регистрация: 06.01.2017
Сообщений: 4,856
04.10.2017, 18:02
залейте лучше на гитхаб а то реально архив скачивать не охота))
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
04.10.2017, 18:02
Помогаю со студенческими работами здесь

[Code review] Сумма нечётных от первого числа до 2
Здравствуйте, подскажите пожалуйста данный код является корректным с точки зрения проф. программистов. using System; using...

[Code review] Алгоритм Брезенхема для линии
/* * Created by SharpDevelop. * User: 26-60-911 * Date: 13.08.2017 * Time: 20:58 * * To change this template use Tools |...

Реализация списка на базе динамического массива [code review]
using System; namespace MyCollections.Generic { public class List<T> : IPrintable, ICloneableAs<List<T>> { private T content;...

[Code review] Даны два числа. Заменить большее число тройным произведением, меньшее - полусуммой
Вести два числа а и b. Число, которое больше заменить тройным произведением, число которое меньше заменить полусуммой. double a, b, a1,...

[Code review] Дано натуральное число n, можно его представить в виде суммы трех квадратов натуральных чисел
Дано натуральное число n, можно его представить в виде суммы трех квадратов натуральных чисел. ЕСли можно,то указать тройку x^2+y^2+z^2=n....


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
11
Ответ Создать тему
Опции темы

Новые блоги и статьи
Символьное дифференцирование
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, то после закрытия окошка. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru