Здравствуйте мастера =)

Я начинающий кодер на cpp, и хочу оценить свой стиль написания.. возможно мне нужно его поменять. Думаю ни для кого не секрет что удача в кодинге напрямую зависит от стиля.

Собственно, вот здесь находится кусок моего кода, хотелось бы услышать, что вы об этом думаете :D

1stapp.cpp

Здравствуйте, Аноним, Вы писали:

А>Здравствуйте мастера =)

А>Я начинающий кодер на cpp, и хочу оценить свой стиль написания.. возможно мне нужно его поменять. Думаю ни для кого не секрет что удача в кодинге напрямую зависит от стиля.

А>Собственно, вот здесь находится кусок моего кода, хотелось бы услышать, что вы об этом думаете :D

А>1stapp.cpp

Я бы добавил чуточку коммента в код для разъяснения хода Выших мыслей

Здравствуйте, Аноним, Вы писали:

А>Здравствуйте мастера =)

А>Я начинающий кодер на cpp, и хочу оценить свой стиль написания.. возможно мне нужно его поменять. Думаю ни для кого не секрет что удача в кодинге напрямую зависит от стиля.

А>Собственно, вот здесь находится кусок моего кода, хотелось бы услышать, что вы об этом думаете :D

А>1stapp.cpp\

неужели ты понимаешь, что там делается после

if(*(WORD*)(vmem)==0x5A4D) {

Здравствуйте, Аноним, Вы писали:

А>Здравствуйте мастера =)

А>Я начинающий кодер на cpp, и хочу оценить свой стиль написания.. возможно мне нужно его поменять. Думаю ни для кого не секрет что удача в кодинге напрямую зависит от стиля.

А>Собственно, вот здесь находится кусок моего кода, хотелось бы услышать, что вы об этом думаете :D

А>1stapp.cpp

Санне, юзай константы, не все понимают наш маразм

Здравствуйте, Аноним, Вы писали:

А>Собственно, вот здесь находится кусок моего кода, хотелось бы услышать, что вы об этом думаете :D

Ален И. Голуб "Веревка достаточной длины, чтобы выстрелить себе в ногу"
Даст ответы на многие вопросы.

Здравствуйте, Аноним, Вы писали:

А>Собственно, вот здесь находится кусок моего кода, хотелось бы услышать, что вы об этом думаете :D

А исходный текст до применения обфускатора можно посмотреть?

Здравствуйте, igna, Вы писали:

I>Здравствуйте, Аноним, Вы писали:

I>А исходный текст до применения обфускатора можно посмотреть?

звеняйте, но обфускатором не пользовался, да и зачем использовать его для проэкта на cpp, это ведь не .net, который легко декомпилируется.

Здравствуйте, Аноним, Вы писали:

А>звеняйте, но обфускатором не пользовался, да и зачем использовать его для проэкта на cpp, это ведь не .net, который легко декомпилируется.

Я думаю для вас хотели донести простую мысль,
что программа в первую очередь пишется не для компьютреа,
а для человека, а ваш код совершенно не предназначен для прочтения.

Да вроде нормально, читается не плохо.Вроде все в одном стиле, ткст не разряженный, но дело вкуса.С математикой конечно сложно разобраться, но это в целом не проблема.Коментарии конечно нужны, но только такие что описываю картину в целом, а не то что делает конкретный оператор. Ворде таких:
// Находим корни таким-то методом

// Для опроса каждого устройства создаем отдельный поток

// устанавливаем параметры COM-порта в соответствие со
// спецификацией протокола обмена с APC's smart protocol
// http://eu1.networkupstools.org/protocols/apcsmart.html

// просто fuck!

Здравствуйте, Crackjack, Вы писали:

C>Да вроде нормально, читается не плохо.Вроде все в одном стиле, ткст не разряженный, но дело вкуса.С математикой конечно сложно разобраться, но это в целом не проблема.Коментарии конечно нужны, но только такие что описываю картину в целом, а не то что делает конкретный оператор. Ворде таких:
C>// Находим корни таким-то методом

C>// Для опроса каждого устройства создаем отдельный поток

C>// устанавливаем параметры COM-порта в соответствие со
C>// спецификацией протокола обмена с APC's smart protocol
C>// http://eu1.networkupstools.org/protocols/apcsmart.html

C>// просто fuck!

Самые распространенные забыли:
// Durty HACK
// TODO
(забавно, у меня такие комментарии Kate особо подствечивает).

Без таких комментариев можно всё сломать, решив что это (то или иное) ошибка и её срочно надо исправить.

Здравствуйте, Аноним, Вы писали:

А>1stapp.cpp

1)
Этот кусок кода, наверняка доступ к полям какой-то структуры, по адресу vmem+d2.
Лучьше преобразрвать указатель к указателю на эту структуру и использвать для доступа имена полей, а не вычисленные смещения.

*(BYTE*)(vmem+d2) = 0xB9;
*(DWORD*)(vmem+d2+1) = *(DWORD*)(PEstart+0x108);
*(BYTE*)(vmem+d2+5) = 0xB8;
*(DWORD*)(vmem+d2+6) = *(DWORD*)(PEstart+0x34)+*(DWORD*)(PEstart+0x104);
*(DWORD*)(vmem+d2+10) = 0x40113080;
*(WORD*)(vmem+d2+14) = 0xFAE2;
*(DWORD*)(vmem+d2+16) = 0x6AEC8B55;
*(BYTE*)(vmem+d2+20) = pv;
*(BYTE*)(vmem+d2+21) = 0x0E9;
*(DWORD*)(vmem+d2+22) = *(DWORD*)(PEstart+40)-(d0+0xA+11);

То-же относится и к указателю PEstart.

2) При сипользовании констант типа 0x746F7270, лучше дать им осмысленные имена.

3) Не верю, что процедуру нельзя развить хотябы на 2-3 отдельных.

4) Мне вообще не понятно, что этот код делает.

Превед, Санниасинчег!

А>Здравствуйте мастера =)

А>Я начинающий кодер на cpp, и хочу оценить свой стиль написания.. возможно мне нужно его поменять. Думаю ни для кого не секрет что удача в кодинге напрямую зависит от стиля.

А>Собственно, вот здесь находится кусок моего кода, хотелось бы услышать, что вы об этом думаете :D

А>1stapp.cpp

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

NG

Здравствуйте, Аноним, Вы писали:

Может моё мнение субъективно , но здесь уже было сказано , что код пишеться для человека , а не для машины.

А ты сам попробуй 3 вещи :

1. Просмотри свой код через месяц — другой . Сможешь ли еще понять , что код делает ?

2. Попробуй внести в него изменения или попроси друга . Насколько это будет легко тебе ( а особенно ему ) это сделать ?

3. Почитай "Рефакторинг " Фаулера.

Если после всего этого , ты решишь , что ничего менять не надо — то значить стиль у тебя хороший.

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

" Аноним " <0@users.rsdn.ru> wrote in message news:2119453@news.rsdn.ru...
> Здравствуйте мастера =)
>
> Я начинающий кодер на cpp, и хочу оценить свой стиль написания.. возможно мне нужно его поменять. Думаю ни для кого не секрет что удача в кодинге напрямую зависит от стиля.
>
> Собственно, вот здесь находится кусок моего кода, хотелось бы услышать, что вы об этом думаете :D
>
> 1stapp.cpp

Я думаю, тебе интересно будет прочесть книжку Саттера и Александреску "Стандарты программирования на C++. 101 правило и рекомендация". Здесь под словом стандарт подразумевается не стандарт языка C++, а стандарты стиля кодирования.

Здравствуйте, Аноним, Вы писали:

А>Я начинающий кодер на cpp, и хочу оценить свой стиль написания.. возможно мне нужно его поменять.

После просмотра кода мне показалось, что Вам еще рано задумываться о вопросах стиля. Вообще же (когда со временем этот вопрос станет актуальным) есть довольно очевидный способ выработки собственного стиля: попробуйте описать Ваш стиль кодирования. В такой форме, чтобы, руководствуясь этим описанием, ему могли следовать другие люди. При этом будьте готовы приводить аргументы в пользу выбранного Вами варианта из нескольких возможных. Короче говоря, попытайтесь объяснить себе, почему Вы делаете именно так, а не иначе. В процессе и сложится стиль.

P.S. В реальной жизни кодера, написавшего такое, я заставил бы переделывать всё, начиная с использования табуляций.

После закрывающей фигурной скобки '}' точка с запятой ';' не ставится. Никогда!

Здравствуйте, asdfghjkl, Вы писали:

A>После закрывающей фигурной скобки '}' точка с запятой ';' не ставится. Никогда!

class A {};

A>>После закрывающей фигурной скобки '}' точка с запятой ';' не ставится. Никогда!

CX>

CX>class A {};
CX>

В данном случае она стоит после списка переменных этого класса. Здесь список пустой. Но он есть!

Здравствуйте, asdfghjkl, Вы писали:

A>В данном случае она стоит после списка переменных этого класса. Здесь список пустой. Но он есть!

После закрывающей фигурной скобки '}' точка с запятой ';' не ставится. Никогда!

Просто надо быть точнее в формулировках.