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

@Demelurg · Регистрация: 12.10.2013

Студворк — интернет-сервис помощи студентам

изучаю с++ почти 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;
}

не красиво потомучто не подписую что делает та или иная функция в коментариях ???

alsav22 · 12.10.2013, 11:09

Сообщение от Demelurg

но препод говорит что иногда не красиво пишу

Сообщение от Demelurg

не красиво потомучто не подписую что делает та или иная функция в коментариях ???

А у преподавателя спросить?

@Demelurg · 12.10.2013, 11:15 **[ТС]**

а у него не хотел спрашивать.

alsav22 · 12.10.2013, 11:29

Сообщение от Demelurg

а у него не хотел спрашивать.

У него как раз и нужно. Кто может знать, что ему не нравится? Коментарии, по-моему, не относятся к красоте написания кода. Обычно это относится к форматированию.

@Demelurg · 12.10.2013, 11:34 **[ТС]**

ну видимо всетаки спрошу а на ваш взгляд все норм ?

alsav22 · 12.10.2013, 11:36

Ничего особо некрасивого не вижу, кое-где пробелы не помешали бы (для удобочитаемости):

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
66
67
68
69
70
71
72
#include <iostream> 
#include <iomanip>
using namespace std; 
 
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;
}

@ninja2 · 12.10.2013, 11:38

Demelurg, На мой взгляд так ты красиво пишешь. Пошли препода куда подальше, не обращай внимания.

Скажи ему как хочу так и пишу, то уже не ваше дело. Как писать это выбор лично каждого, ну если в команде, то тогда уже вместе решают как что писать.

@OhMyGodSoLong · 12.10.2013, 11:38

Сообщение от Demelurg

не красиво потомучто не подписую что делает та или иная функция в коментариях ???

Лучше называйте функции так, чтобы их предназначение было очевидно из названия.

@Demelurg · 12.10.2013, 11:39 **[ТС]**

ясно всем спасибки.

Tulosba · 12.10.2013, 11:54

Что замечено:
1. Зачем tmp_sec, tmp_min в качестве членов класса?
2. Класс надо бы вынести в отдельный модуль.
3. Передавать объект, если он не меняется, лучше по константной ссылке.
4. Приватные данные я бы поместил в конец определения класса.

@Anton_Kretov · 12.10.2013, 11:58

Я бы советовал делать объявление переменных одного типа в одной строке. Пример: int a, b, c, d;

alsav22 · 12.10.2013, 12:00

А я бы не советовал...

@Dimka-novitsek · 12.10.2013, 13:21

Да нормальный код. По моему, читается даже лучше, чем примеры из учебника.

@castaway · 12.10.2013, 13:29

Не по теме:

Сообщение от Dimka-novitsek

По моему, читается даже лучше, чем примеры из учебника.

Я бы даже сказал, что читается лучше чем в 99% постах раздела "С++ для начинающих" )

@cosmic · 12.10.2013, 15:12

Я бы сказал что ты больше 2х месяцев занимаешься С++

@Matan! · 12.10.2013, 15:50

Нормально всё читается.

Сообщение от Anton_Kretov

Я бы советовал делать объявление переменных одного типа в одной строке. Пример: int a, b, c, d;

Это спорно.Нам говорили,что хорошим стилем является объявление каждой переменной в отдельной строке.Кто как хочет,так и объявляет.Лишь бы винегрета не было в коде.

@Avazart · 12.10.2013, 16:04

C++
1
sum_Time()

Може лучше

C++
1
sumTime()

или

C++
1
sum_time()

Что-то одно как разделитель слов.

Непонятно назначение :

C++
1
2
int tmp_sec;
int tmp_min;

Если они временные почему не объявлять их локально, если константы то почему их не объявлять как константы ?

@Demelurg · 12.10.2013, 17:06 **[ТС]**

1.да можно и не вносить их в клас ( tmp_sec, tmp_min ) но начал так писать и чтоб не поплутать оставил.
2. в Лафоре на тот момент не говорили как делать файл ".h" да и для маленькой проги не хотел создавать.
3.согласен но недавно начал учить класы да и метод который выводит походу const ?

C++
1
2
3
4
5
void Out_put()const
    {
        cout << setw(3) << " Time = " << setw(3) << hour << " : "
             << setw(3) << minute <<" : " << second << endl;
    }

4. тоесть вы говорите что сначала надо public: f gjnjv private: ???

@kventin_zhuk · 12.10.2013, 17:17

Demelurg, норм, только не понравилось наименование методов. В константном - большая буква, в обычном - маленькие. Лично я открытые методы именую аля SumTime

И перед private-полями ставлю нижнее подчеркивание - удобно потом в конструктор параметры передавать

Дело привычки)

@Demelurg · 12.10.2013, 17:28 **[ТС]**

в класе хотел обявить выдает ошыбку а глобально не хотел обявлять их ( int tmp_sec; int tmp_min

)но да можно и локально.
kventin_zhuk а про

Сообщение от kventin_zhuk

перед private-полями ставлю нижнее подчеркивание

не понял как ?
ты емеешь ввиду типа не так

C++
1
2
3
4
5
6
private:
    int hour;
    int minute;
    int second;
    int tmp_sec;
    int tmp_min;

а так ?

C++
1
2
3
4
5
6
private:
    int _hour;
    int _minute;
    int _second;
    int _tmp_sec;
    int _tmp_min;

Новые блоги и статьи Все статьи Все блоги /
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. Пошагово создадим проект для загрузки изображения. . .

@Demelurg 0 / 0 / 1 Регистрация: 12.10.2013 Сообщений: 32
	12.10.2013, 11:15 [ТС]
	а у него не хотел спрашивать. 0

@Demelurg 0 / 0 / 1 Регистрация: 12.10.2013 Сообщений: 32
	12.10.2013, 11:34 [ТС]
	ну видимо всетаки спрошу а на ваш взгляд все норм ? 0

@ninja2 979 / 196 / 33 Регистрация: 26.09.2012 Сообщений: 2,041
	12.10.2013, 11:38
	Demelurg, На мой взгляд так ты красиво пишешь. Пошли препода куда подальше, не обращай внимания. Скажи ему как хочу так и пишу, то уже не ваше дело. Как писать это выбор лично каждого, ну если в команде, то тогда уже вместе решают как что писать. 2

@Demelurg 0 / 0 / 1 Регистрация: 12.10.2013 Сообщений: 32
	12.10.2013, 11:39 [ТС]
	ясно всем спасибки. 0

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	12.10.2013, 11:54
	Что замечено: 1. Зачем tmp_sec, tmp_min в качестве членов класса? 2. Класс надо бы вынести в отдельный модуль. 3. Передавать объект, если он не меняется, лучше по константной ссылке. 4. Приватные данные я бы поместил в конец определения класса. 0

@Anton_Kretov 89 / 51 / 15 Регистрация: 26.06.2013 Сообщений: 179
	12.10.2013, 11:58
	Я бы советовал делать объявление переменных одного типа в одной строке. Пример: int a, b, c, d; 1

alsav22 5500 / 4895 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	12.10.2013, 12:00
	Сообщение было отмечено как решение Решение А я бы не советовал... 3

@Dimka-novitsek 16 / 16 / 2 Регистрация: 26.04.2012 Сообщений: 1,091
	12.10.2013, 13:21
	Да нормальный код. По моему, читается даже лучше, чем примеры из учебника. 0

@cosmic 34 / 32 / 5 Регистрация: 29.08.2012 Сообщений: 84 Записей в блоге: 1
	12.10.2013, 15:12
	Я бы сказал что ты больше 2х месяцев занимаешься С++ 0

@kventin_zhuk БНТУ ФИТР 215 / 155 / 42 Регистрация: 26.12.2012 Сообщений: 382
	12.10.2013, 17:17
	Demelurg, норм, только не понравилось наименование методов. В константном - большая буква, в обычном - маленькие. Лично я открытые методы именую аля SumTime И перед private-полями ставлю нижнее подчеркивание - удобно потом в конструктор параметры передавать Дело привычки) 1

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

Решение