Форум программистов, компьютерный форум CyberForum.ru

Сказали некрасиво пишу - C++

Восстановить пароль Регистрация
 
 
Рейтинг: Рейтинг темы: голосов - 45, средняя оценка - 4.98
Demelurg
0 / 0 / 0
Регистрация: 12.10.2013
Сообщений: 28
12.10.2013, 11:08     Сказали некрасиво пишу #1
изучаю с++ почти 2 месяц , пишу програму она работает но препод говорит что иногда не красиво пишу ((( пишу так

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
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
class Time
{
private:
    int hour;
    int minute;
    int second;
    int tmp_sec;
    int tmp_min;
public:
    Time():hour(0),minute(0),second(0) 
    {}
    Time(int h , int m , int s):hour(h),minute(m),second(s)
    {}
    void Out_put()const
    {
        cout<<setw(3)<<" Time = "<<setw(3)<<hour<<" : "
            <<setw(3)<<minute<<" : "<<second<<endl;
    }
 
    Time sum_Time(Time);
};
 
Time Time::sum_Time(Time other_time)
   {
       Time tmp;
       tmp_sec = 60;
       tmp_min = 60;
 
       tmp.hour = hour + other_time.hour ;
 
       tmp.second = second + other_time.second ;
       tmp.minute = minute + other_time.minute ;
        
           if ( tmp.second >= 60 )
               {
                   tmp_sec *= tmp.second / 60 ;
                   tmp.minute += tmp_sec / 60 ;
                   tmp.second -= tmp_sec ;
               }
 
               if ( tmp.minute >= 60 )
                   {
                       tmp_min *= tmp.minute / 60 ;
                       tmp.hour += tmp_min / 60 ;
                       tmp.minute -= tmp_min ;
                   }
      return tmp;
   }
   
int main()
{
    Time first_Time(22,155,177);
    Time second_Time(12,39,89);
 
    first_Time.Out_put();
    second_Time.Out_put();
 
    cout<<endl<<endl<<" You Three = ";
    Time three_Time = first_Time.sum_Time(second_Time);
    three_Time.Out_put();
 
    cout<<endl<<endl;
 
    return 0;
}
не красиво потомучто не подписую что делает та или иная функция в коментариях ???
Лучшие ответы (1)
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
12.10.2013, 11:08     Сказали некрасиво пишу
Посмотрите здесь:

пишу ed, отмена изменений C++
Пишу ОС, нужен доступ к экрану C++
C++ Вопрос новичка - пишу калькулятор
Пишу консольное приложение C++
C++ пишу калькулятор. подскажите
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
kventin_zhuk
БНТУ ФИТР
 Аватар для kventin_zhuk
214 / 154 / 15
Регистрация: 26.12.2012
Сообщений: 382
12.10.2013, 17:33     Сказали некрасиво пишу #21
Demelurg, именно. и тогда конструктор может выглядеть уже так:
C++
1
2
3
4
Time(int hour, int minute, int second)
{
     // тело конструктора
}
а то при вызове из кода конструктора параметры h m s - не очень информативно

и еще я не ставлю пробел перед запятой в параметрах функций/методов
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
ct0r
C++/Haskell
 Аватар для ct0r
1549 / 568 / 39
Регистрация: 19.08.2012
Сообщений: 1,174
Завершенные тесты: 1
12.10.2013, 17:36     Сказали некрасиво пишу #22
Придираться конечно можно. Но в целом мне кажется норм.
К чему я бы придрался, если б имел например жуткое желание завалить:
про tmp_min, tmp_sec уже сказали
про именование private уже сказали (я бы только ставил _ в конце, а не в начале)
нет проверки на допустимость передаваемых в конструктор данных (иначе зачем знаковый тип?)
про смесь стилей в sum_Time уже сказали (в том числе это может быть константная функция-член, принимающая константную ссылку)
фигурные скобки я бы не сдвигал, так как такой стиль конечно есть и имеет право на существование, но очень мало кем используется.

private в данном случае я бы не стал переносить в конец. Публичный интерфейс это конечно хорошо, но иногда один взгляд на самые важные private-члены позволяет сразу очень многое узнать о классе.

Цитата Сообщение от kventin_zhuk Посмотреть сообщение
перед private-полями ставлю нижнее подчеркивание - удобно потом в конструктор параметры передавать
Чем удобно? По-моему вообще без разницы.
C++
1
Time(int hour, int minute, int second): hour(hour), minute(minute), second(second)
В стандарте такая запись вполне определена и всегда работает.
Demelurg
0 / 0 / 0
Регистрация: 12.10.2013
Сообщений: 28
12.10.2013, 17:38  [ТС]     Сказали некрасиво пишу #23
про пробел ето уже кому как удобней , а на счет информативности и на счет подчеркивание - реально кульный совет спасиб.
dimakozyr
 Аватар для dimakozyr
157 / 24 / 1
Регистрация: 17.08.2013
Сообщений: 461
12.10.2013, 17:39     Сказали некрасиво пишу #24
Часто пишут, что, например, нужно писать в классе сначала public часть, а потом private.
Много таких вещей можно узнать из книги "Совершенный код" Макконнелла
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
12.10.2013, 17:40     Сказали некрасиво пишу #25
Цитата Сообщение от kventin_zhuk Посмотреть сообщение
перед private-полями ставлю нижнее подчеркивание - удобно потом в конструктор параметры передавать
Это не смущает?
Миниатюры
Сказали некрасиво пишу  
castaway
Эксперт С++
4844 / 2983 / 367
Регистрация: 10.11.2010
Сообщений: 11,021
Записей в блоге: 10
Завершенные тесты: 1
12.10.2013, 17:41     Сказали некрасиво пишу #26
Цитата Сообщение от dimakozyr Посмотреть сообщение
Часто пишут, что, например, нужно писать в классе сначала public часть, а потом private.
Бессмысленно. Тогда уж заместо class писать struct и не писать вначале public:
Demelurg
0 / 0 / 0
Регистрация: 12.10.2013
Сообщений: 28
12.10.2013, 17:41  [ТС]     Сказали некрасиво пишу #27
Цитата Сообщение от ct0r Посмотреть сообщение
нет проверки на допустимость передаваемых в конструктор данных (иначе зачем знаковый тип?)
извините не понял ? проверка какие дание ввожу ?
Цитата Сообщение от ct0r Посмотреть сообщение
фигурные скобки я бы не сдвигал, так как такой стиль конечно есть и имеет право на существование, но очень мало кем используется.
скобки которые после конструктора ?
dimakozyr
 Аватар для dimakozyr
157 / 24 / 1
Регистрация: 17.08.2013
Сообщений: 461
12.10.2013, 17:42     Сказали некрасиво пишу #28
Цитата Сообщение от castaway Посмотреть сообщение
Бессмысленно. Тогда уж заместо class писать struct и не писать вначале public:
Ну да, я так и делаю Но если класс, то это делается, чтобы сначала было видно по public методам, для чего класс нужен
Убежденный
Системный программист
 Аватар для Убежденный
14197 / 6212 / 985
Регистрация: 02.05.2013
Сообщений: 10,351
Завершенные тесты: 1
12.10.2013, 17:43     Сказали некрасиво пишу #29
Цитата Сообщение от kventin_zhuk Посмотреть сообщение
И перед private-полями ставлю нижнее подчеркивание - удобно потом в конструктор параметры передавать
В C++ имена, начинающиеся с подчеркивания, зарезервированы.

C++03, 17.4.3.1.2 Global names
Certain sets of names and function signatures are always reserved to the implementation:

- Each name that contains a double underscore (_ _) or begins with an underscore followed by an upper-case letter (2.11) is reserved to the implementation for any use.

- Each name that begins with an underscore is reserved to the implementation for use as a name in the
global namespace.
Demelurg
0 / 0 / 0
Регистрация: 12.10.2013
Сообщений: 28
12.10.2013, 17:43  [ТС]     Сказали некрасиво пишу #30
Цитата Сообщение от alsav22 Посмотреть сообщение
Это не смущает?
а вот етого я не знал н тоесть знал что есть зарезирвирование имена но про подчеркивание не знал.
kventin_zhuk
БНТУ ФИТР
 Аватар для kventin_zhuk
214 / 154 / 15
Регистрация: 26.12.2012
Сообщений: 382
12.10.2013, 17:44     Сказали некрасиво пишу #31
alsav22, это откуда вообще?) Всегда так делал - и неоднократно видел как делают другие. Это же все сугубо в моем классе.
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
12.10.2013, 17:47     Сказали некрасиво пишу #32
Цитата Сообщение от kventin_zhuk Посмотреть сообщение
alsav22, это откуда вообще?)
29 пост.
Цитата Сообщение от kventin_zhuk Посмотреть сообщение
Всегда так делал - и неоднократно видел как делают другие.
Делать, вообще, всё можно. Тут тема другая.
kventin_zhuk
БНТУ ФИТР
 Аватар для kventin_zhuk
214 / 154 / 15
Регистрация: 26.12.2012
Сообщений: 382
12.10.2013, 17:54     Сказали некрасиво пишу #33
ct0r, это если использовать список инициализации. А вот если теле конструктора обращаться и к тому и к другому? this будет нужно использовать.

Добавлено через 6 минут
Убежденный, Ну первое понятно - а второе: зарезервированы для использования в качестве имени в глобальном пространстве имен. Не тот случай же.
ct0r
C++/Haskell
 Аватар для ct0r
1549 / 568 / 39
Регистрация: 19.08.2012
Сообщений: 1,174
Завершенные тесты: 1
12.10.2013, 17:55     Сказали некрасиво пишу #34
Цитата Сообщение от Demelurg Посмотреть сообщение
извините не понял ? проверка какие дание ввожу ?
скобки которые после конструктора ?
Если я передам в конструктор отрицательные числа, то что будет?
Да, и которые после if.

castaway
Я бы все-таки идеологически немного различал структуру и класс. Обычно пишу структуру, когда на первом месте - доступ и модификация данных, а класс - когда действия над ними.

kventin_zhuk
Ну это уже само собой. Я думал, что ты только про список инициализации говоришь.
Demelurg
0 / 0 / 0
Регистрация: 12.10.2013
Сообщений: 28
12.10.2013, 17:57  [ТС]     Сказали некрасиво пишу #35
извините ребят я уже заплутался , спасиб много полезного узнал , иду дальше учить есле что то вас ищо потревожу .
castaway
Эксперт С++
4844 / 2983 / 367
Регистрация: 10.11.2010
Сообщений: 11,021
Записей в блоге: 10
Завершенные тесты: 1
12.10.2013, 17:59     Сказали некрасиво пишу #36
Цитата Сообщение от ct0r Посмотреть сообщение
Я бы все-таки идеологически немного различал структуру и класс.
А что их различать? У них разница только в public: и private: по-умолчанию. Если хочешь больше писанины то следуй своему варианту, я не против.
Phantom
Эксперт C++
 Аватар для Phantom
3159 / 841 / 15
Регистрация: 29.12.2008
Сообщений: 952
12.10.2013, 18:00     Сказали некрасиво пишу #37
Как на мой вкус, то переменные убого названы. Есть camelCase aka "верблюжья нотация", есть under_score - принципиальные схемы именования. Почитай про венгерскую нотацию. У тебя же какой-то средний вариант, смотрится не очень. Конечно, кажется мелочью, но на самом деле в будущем, на реальной работе, могут потребовать (корпоративный стиль).

Второе, что не понравилось - тот факт, что какая-то часть методов класса реализована при его объявлении, а метод sum_Time вынесен. Я бы вынес объявление класса в .h-файл, его реализацию в .cpp-файл, и точку входа main - в свой файл. Это традиционный подход, но даже если ты его не захочешь придерживаться, то хотя бы в своем коде должно быть какое-то однообразие. Выносишь методы - выноси все, реализуешь их при объявлении - реализуй все.

И не брезгуй пробелами и адекватным форматированием (вон на 34 и 41 строках if-ы на разном уровне), для этого есть Ctrl + K, Ctrl + F в Visual Studio, например.

Так что в целом соглашусь с преподавателем - неэстетичный код.
Demelurg
0 / 0 / 0
Регистрация: 12.10.2013
Сообщений: 28
12.10.2013, 18:02  [ТС]     Сказали некрасиво пишу #38
а Лафоре говорит что структура ток для даных а клас для даных + методы(функции).и по ООП лутше использовать класы чем структуры или я не прав ?
castaway
Эксперт С++
4844 / 2983 / 367
Регистрация: 10.11.2010
Сообщений: 11,021
Записей в блоге: 10
Завершенные тесты: 1
12.10.2013, 18:03     Сказали некрасиво пишу #39
Цитата Сообщение от Demelurg Посмотреть сообщение
а Лафоре говорит что структура ток для даных а клас для даных + методы(функции).и по ООП лутше использовать класы чем структуры или я не прав ?
Или. Вместе с Лафоре.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
12.10.2013, 18:05     Сказали некрасиво пишу
Еще ссылки по теме:

C++ Пишу транслятор и интерпритатор
Сказали сделать такое C++
Ядрышко ОСи пишу C++

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

Или воспользуйтесь поиском по форуму:
kventin_zhuk
БНТУ ФИТР
 Аватар для kventin_zhuk
214 / 154 / 15
Регистрация: 26.12.2012
Сообщений: 382
12.10.2013, 18:05     Сказали некрасиво пишу #40
Demelurg, В плюсах стркутура считай тот же класс. Но вот в си - обычные) создатели плюсов оставили зарезервированное слово для совместимости и расширили их возможности
Yandex
Объявления
12.10.2013, 18:05     Сказали некрасиво пишу
Ответ Создать тему
Опции темы

Текущее время: 20:07. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru