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

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
 
Рейтинг: Рейтинг темы: голосов - 45, средняя оценка - 4.98
Demelurg
0 / 0 / 0
Регистрация: 12.10.2013
Сообщений: 28
#1

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

12.10.2013, 11:08. Просмотров 5793. Ответов 160
Метки нет (Все метки)

изучаю с++ почти 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)
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Jupiter
Каратель
Эксперт С++
6550 / 3970 / 226
Регистрация: 26.03.2010
Сообщений: 9,273
Записей в блоге: 1
Завершенные тесты: 2
12.10.2013, 23:24     Сказали некрасиво пишу #101
Цитата Сообщение от castaway Посмотреть сообщение
По-твоему через класс это будет выглядеть лучше?
ответ нет, пока этот тип остается POD-ом

Цитата Сообщение от castaway Посмотреть сообщение
Кто все эти люди?
в obj-c реализация методов принципиально выносится в отдельный файл
castaway
Эксперт С++
4876 / 3015 / 370
Регистрация: 10.11.2010
Сообщений: 11,075
Записей в блоге: 10
Завершенные тесты: 1
12.10.2013, 23:28     Сказали некрасиво пишу #102

Не по теме:

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


Цитата Сообщение от Jupiter Посмотреть сообщение
ответ нет, пока этот тип остается POD-ом
Ну так в этой теме никто и не различал POD от не-POD типа.

Добавлено через 3 минуты

Не по теме:

Цитата Сообщение от Jupiter Посмотреть сообщение
в obj-c реализация методов приниципиально выносится в отдельный файл
Я думаю не стоит сюда приплетать Obj-C.

ninja2
230 / 186 / 7
Регистрация: 26.09.2012
Сообщений: 2,018
Завершенные тесты: 1
12.10.2013, 23:32     Сказали некрасиво пишу #103
Да нашо еще думать как что обозначить? Да только над важными переменными можно подумать как их называть, локальные переменные можно просто называть i1, i2, i3, i4 и т.д. Важные переменные нужно называть ну что бы они читались и сами за себя говорили.

Вообще как то не задумывался над этим, да этих стилей по названию полно. Можно например первую букву давать переменной по типу, например для int i_a; или char c_b; string s_a; ну типо сразу видно какой тип в переменной.

Я думаю достаточно просто делить переменные на локальные и важные. Тем более важных переменных на 1к строк кода будет штук 10.
Avazart
7063 / 5240 / 262
Регистрация: 10.12.2010
Сообщений: 23,052
Записей в блоге: 17
12.10.2013, 23:34     Сказали некрасиво пишу #104
Цитата Сообщение от ninja2 Посмотреть сообщение
. Можно например первую букву давать переменной по типу, например для int i_a; или char c_b; string s_a; ну типо сразу видно какой тип в переменной.
А вот этого делать не рекомендуют ...
Особенно если Вы пишете не в IDE, заманаетесь переименовывать, если вдруг придется изменить тип переменной.
castaway
Эксперт С++
4876 / 3015 / 370
Регистрация: 10.11.2010
Сообщений: 11,075
Записей в блоге: 10
Завершенные тесты: 1
12.10.2013, 23:36     Сказали некрасиво пишу #105
Цитата Сообщение от ninja2 Посмотреть сообщение
Можно например первую букву давать переменной по типу, например для int i_a; или char c_b; string s_a; ну типо сразу видно какой тип в переменной.
На самом деле.., не стоит. Локальных переменных не должно быть много, поэтому в них не сложно разобраться. Префикс для них, на мой взгляд, будет лишним.
ninja2
230 / 186 / 7
Регистрация: 26.09.2012
Сообщений: 2,018
Завершенные тесты: 1
12.10.2013, 23:37     Сказали некрасиво пишу #106
Цитата Сообщение от Avazart Посмотреть сообщение
А вот этого делать не рекомендуют ...
Нам пофигу на рекомендации, есть такой стиль, хотим и пользуемся.
Avazart
7063 / 5240 / 262
Регистрация: 10.12.2010
Сообщений: 23,052
Записей в блоге: 17
12.10.2013, 23:38     Сказали некрасиво пишу #107
Цитата Сообщение от ninja2 Посмотреть сообщение
локальные переменные можно просто называть i1, i2, i3, i4 и т.д.
Если у вас их четыре и более в одном месте вероятно не стоит...

Добавлено через 52 секунды
Цитата Сообщение от ninja2 Посмотреть сообщение
Нам пофигу на рекомендации, есть такой стиль, если хотим, то пользуемся.
Вам пофиг на потраченное Вами время ?
Croessmah
Модератор
Эксперт CЭксперт С++
12878 / 7264 / 810
Регистрация: 27.09.2012
Сообщений: 17,953
Записей в блоге: 2
Завершенные тесты: 1
12.10.2013, 23:38     Сказали некрасиво пишу #108
Цитата Сообщение от castaway Посмотреть сообщение
Ну так в этой теме никто и не различал POD от не-POD типа.
Ну лично я для большинства POD использую именно struct
Цитата Сообщение от ninja2 Посмотреть сообщение
Да только над важными переменными можно подумать как их называть, локальные переменные можно просто называть i1, i2, i3, i4 и т.д.
Вы достали. Идите почитайте книжки по проектированию и улучшению кода!!!
castaway
Эксперт С++
4876 / 3015 / 370
Регистрация: 10.11.2010
Сообщений: 11,075
Записей в блоге: 10
Завершенные тесты: 1
12.10.2013, 23:38     Сказали некрасиво пишу #109
Цитата Сообщение от ninja2 Посмотреть сообщение
Нам пофигу на рекомендации, есть такой стиль, хотим и пользуемся.
Стиль - это тоже навязанная рекомендация)
ninja2
230 / 186 / 7
Регистрация: 26.09.2012
Сообщений: 2,018
Завершенные тесты: 1
12.10.2013, 23:39     Сказали некрасиво пишу #110
Цитата Сообщение от castaway Посмотреть сообщение
На самом деле.., не стоит.
Это такой стиль есть, он даже название имеет свое, не помню как называется, он для всех переменных применяется.
castaway
12.10.2013, 23:40
  #111

Не по теме:

Цитата Сообщение от Croessmah Посмотреть сообщение
Ну лично я для большинства POD использую именно struct
Тут скорее наоборот)

Avazart
7063 / 5240 / 262
Регистрация: 10.12.2010
Сообщений: 23,052
Записей в блоге: 17
12.10.2013, 23:40     Сказали некрасиво пишу #112
Цитата Сообщение от ninja2 Посмотреть сообщение
Это такой стиль есть, он даже название имеет свое, не помню как называется, он для всех переменных применяется.
Глупый )
Почитайте Дъюхерста.

У всех рекомендаций, есть свои логические доводы, в данном случае я не вижу доводов "за".
ninja2
230 / 186 / 7
Регистрация: 26.09.2012
Сообщений: 2,018
Завершенные тесты: 1
12.10.2013, 23:44     Сказали некрасиво пишу #113
Цитата Сообщение от Croessmah Посмотреть сообщение
Вы достали. Идите почитайте книжки по проектированию и улучшению кода!!!
Это и есть одна из рекомендаций, Нужно все локальные переменные одинаково называть, повторно использовать одни и те же названия.

Добавлено через 2 минуты
Цитата Сообщение от Avazart Посмотреть сообщение
Глупый )
Почитайте Дъюхерста.
Ну ладно он свое мнение выскажет, так что мне сразу его советам следовать. А мне мб нравится мнение автора который советует так называть как я описал?

Как говориться на вкус и цвет товарищей нет, кто как хочет так и [cut].
Avazart
7063 / 5240 / 262
Регистрация: 10.12.2010
Сообщений: 23,052
Записей в блоге: 17
12.10.2013, 23:44     Сказали некрасиво пишу #114
Цитата Сообщение от ninja2 Посмотреть сообщение
Это и есть одна из рекомендаций, Нужно все локальные переменные одинаково называть, повторно использовать одни и те же названия.
Да, но не одни и те же названия для переменных обозначающих разные вещи...
ninja2
230 / 186 / 7
Регистрация: 26.09.2012
Сообщений: 2,018
Завершенные тесты: 1
12.10.2013, 23:48     Сказали некрасиво пишу #115
Цитата Сообщение от Avazart Посмотреть сообщение
Да, но не одни и те же названия для переменных обозначающих разные вещи...
Ну ясное дело, ну полюбом же у вас есть уже свои мелкие переменные которые используются в коде каждый раз и вы их называете постоянно одними и теми же именами, например для строки как правило буквой s называем для цикла обычно i используем, для двойного цикла i и j, если что то обмениваем то записываем в temp, и т.д.. Просто нужно присмотреться и вообще стремиться повторять переменные в разных локальных областях видимости. Это один из советов, да все так и делают как правило.
ct0r
Игогошка!
1762 / 664 / 42
Регистрация: 19.08.2012
Сообщений: 1,261
Завершенные тесты: 1
12.10.2013, 23:50     Сказали некрасиво пишу #116
ninja2
Венгерская нотация - жуткое зрелище. Устаревшее наследие WinAPI. Советую отвлечься от книг типа "программирование в винде для чайников" и почитать что-нибудь нормальное.
ninja2
230 / 186 / 7
Регистрация: 26.09.2012
Сообщений: 2,018
Завершенные тесты: 1
12.10.2013, 23:53     Сказали некрасиво пишу #117
ct0r, Да да, ну это я к тому привел, что бы товариши не приходили к какому нить одному стилю, просто не нужно навязывать какой нить один стиль, который якобы правильный, каждый стиль будет правильным.
Как вас учили конспекты писать? Правильно пишите так что бы вы смогли потом прочитать что написали. Так же и код, нужно писать так что бы потом его можно было прочитать.
castaway
Эксперт С++
4876 / 3015 / 370
Регистрация: 10.11.2010
Сообщений: 11,075
Записей в блоге: 10
Завершенные тесты: 1
12.10.2013, 23:54     Сказали некрасиво пишу #118
ninja2, ну ты же не хочешь сказать, что локальные переменные для цикла i и j стоит обозначить как i_i и i_j !? Это ведь просто глупо. Тут префиксы явно не нужны.
ninja2
230 / 186 / 7
Регистрация: 26.09.2012
Сообщений: 2,018
Завершенные тесты: 1
12.10.2013, 23:56     Сказали некрасиво пишу #119
Цитата Сообщение от castaway Посмотреть сообщение
ну ты же не хочешь сказать, что локальные переменные для цикла i и j стоит обозначить как i_i и i_j !? Это ведь просто глупо. Тут префиксы явно не нужны.
Ты просто не привык, через месяц два будешь так называть, привыкнешь, будет тебе казаться что просто i j называть глупо.
Ладно это просто мое ИМХО. Не буду навязывать.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
12.10.2013, 23:58     Сказали некрасиво пишу
Еще ссылки по теме:

Сказали сделать такое C++
C++ Пишу примитивный никому не нужный контейнер
Ядрышко ОСи пишу C++
C++ Пишу велосипед (shared_ptr)
Пишу итератор произвольного доступа C++

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

Или воспользуйтесь поиском по форуму:
Croessmah
12.10.2013, 23:58     Сказали некрасиво пишу
  #120

Не по теме:

Цитата Сообщение от ninja2 Посмотреть сообщение
Ты просто не привык, через месяц два будешь так называть, привыкнешь, будет тебе казаться что просто i j называть глупо.
Мде... просто нет слов

Yandex
Объявления
12.10.2013, 23:58     Сказали некрасиво пишу
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru