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

В общем, что вы думаете об этом проекте? Акцентируя внимание на его восприятии. И учтите, что я пытаюсь прокатить не за зеленого новичка, поэтому отнеситесь критично и за каждый промах валите меня гнилым помидором. Этот код должен стать идеальным. Ответственность за это лежит на вас (драматизирую).

Итак, налетай народ, валим нуба!
Для начала ознакомьтесь с описанием: диаграммы (я старался), структура, поэзия, недостатки (очень много недостатков): http://www.scribd.com/doc/138457565/Flying
Ну и виновник торжества, сам код: https://github.com/CompleteBrains/Flying

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

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

CB>Я переместил тему потому что в форуме cpp глухо а мне нужно мясо. Поскольку мне этот код демонстрировать в придачу к резюме, раздел форума выбрал правильный, надеюсь

Шоб былО мясо, нужно такое в священные войны постить )))

Здравствуйте, Alexéy Sudachén, Вы писали:
AS>Шоб былО мясо, нужно такое в священные войны постить )))
по странному недоразумению "О работе" досихпор находится не в разделе СВ. Хотя группа состоит из холиварских тем чуть менее чем полностью.

Здравствуйте, CompleteBrains, Вы писали:
CB> Поскольку мне этот код демонстрировать в придачу к резюме, раздел форума выбрал правильный, надеюсь
Забей, "идеальный код" — понятие очень субьективное. Как бы тут его не вылизывали, интервьювер всеравно найдет кчему докапаться.

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

CB>В этом проекте я старался писать код, на который было бы приятно смотреть. Приятность кода состоит из множества параметров к тому же здесь должна присутствовать некая неописуемая магия. Приятность начинается на уровне отдельных модулей и простирается вплоть до локальных переменных. Хотелось, чтобы код можно было всунуть в рамочку и повесить в музее.

CB>В общем, что вы думаете об этом проекте? Акцентируя внимание на его восприятии. И учтите, что я пытаюсь прокатить не за зеленого новичка, поэтому отнеситесь критично и за каждый промах валите меня гнилым помидором. Этот код должен стать идеальным. Ответственность за это лежит на вас (драматизирую).

CB>Итак, налетай народ, валим нуба!
CB>Для начала ознакомьтесь с описанием: диаграммы (я старался), структура, поэзия, недостатки (очень много недостатков): http://www.scribd.com/doc/138457565/Flying
CB>Ну и виновник торжества, сам код: https://github.com/CompleteBrains/Flying

Так... Пустые методы — зачем растаскивать скобки на несколько строк? Ты ещё комментарий напиши, что это — пустой метод.

using namespace std — то есть, то нет. Забавно вот это:

using namespace std;
// ...
std::map ...

Табуляции в форматировании — зло.

Зачем нужна struct Pointer? Ты собираешься регулярно менять реализации указателей? И почему она повторена в нескольких местах?

А тесты где?

Магические числа 0.15f, 10.0f, 20.0f.

try/catch, проверка кодов возврата, я так понимаю, не царское дело?

В хидерах что-то мрачное, намекающее на необходимость отлучения автора от клавиатуры (мои комментарии в: /* */):

// Library functions export
#define exportdll __declspec(dllexport)
/* Это лучше оформить так, чтобы запись символа экспорта отличалась от системной,
и не создавала путаницы, если будет несколько библиотек. Например:

#define FLYING_DLL_EXPORT
*/

// I hate CapsLock
#define null NULL
/* Да мне наплевать, что ты там hate,
вынесу эту строчку при первом удобном случае.
Не надо косить под компилятор. */

/* За такие вещи полагается месяц читать Страуструпа вслух по пять часов в день. */
// Logic symbols
#define equals      ==
#define not          !
#define notequal  !=
#define and          &&
#define or          ||

/* Убрать!!! Содержательного смысла в них нет. */
// Literals
#define Space    " "
#define Tab        "    "
#define Endl    "\n"

/* Это всё тоже лишнее. */
// Visual Studio Macrocosm
#define FunctionName __FUNCTION__
#define Line __LINE__
#define CurrentTime __TIME__

Вывод: автор нуб и опозорился.

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

CB>В этом проекте я старался писать код, на который было бы приятно смотреть. Приятность кода состоит из множества параметров к тому же здесь должна присутствовать некая неописуемая магия. Приятность начинается на уровне отдельных модулей и простирается вплоть до локальных переменных. Хотелось, чтобы код можно было всунуть в рамочку и повесить в музее.

Да, с подписью: "Первое впечатление бывает обманчивым!"

Ещё убери пустые комментарии "Description:", зачем они? Или заполни их содержательным текстом.

CB>В общем, что вы думаете об этом проекте? Акцентируя внимание на его восприятии. И учтите, что я пытаюсь прокатить не за зеленого новичка, поэтому отнеситесь критично и за каждый промах валите меня гнилым помидором. Этот код должен стать идеальным. Ответственность за это лежит на вас (драматизирую).

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

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

ГВ>В хидерах что-то мрачное, намекающее на необходимость отлучения автора от клавиатуры (мои комментарии в: /* */):

Не хватает еще

#define begin {
#define end }

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

CB>В этом проекте я старался писать код, на который было бы приятно смотреть. Приятность кода состоит из множества параметров к тому же здесь должна присутствовать некая неописуемая магия. Приятность начинается на уровне отдельных модулей и простирается вплоть до локальных переменных. Хотелось, чтобы код можно было всунуть в рамочку и повесить в музее.


А смысл ? Код должен решать задачу, допускать расширение в разумных пределах и его можно было бы саппортить. Зачем его в рамочку ставить ?

Начал читать описание , так и не понял что программа должна делать ?

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

ГВ>>В хидерах что-то мрачное, намекающее на необходимость отлучения автора от клавиатуры (мои комментарии в: /* */):

П>Не хватает еще
П>

П>#define begin {
П>#define end }
П>

Тогда можно было бы смело принять автора за тролля.

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

CB>Ну и виновник торжества, сам код: https://github.com/CompleteBrains/Flying

код полное говно.

С++ там и рядом не лежал, ни С++11, ни вообще С++
синглтоны глаза режут
Syntax.h это вообще жесть, я думал так только индусы могут

Порнография в Render.h
Во первых

        // Frame
        void BeginScene();
        void FinishScene();

Если что то begin то оно должно end а не finish.

А во вторых голая баба в комментах только время сборки увеличивает.

https://github.com/CompleteBrains/Flying/blob/master/Render/Sources/Render.h

а еще полно тупых комментов, типа:

// Retrieve keyboard state 
if (not GetKeyboardState (keyBuffer))    return;

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

ГВ>Табуляции в форматировании — зло.

А за пробелы повбывав бы.

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

ГВ>>Табуляции в форматировании — зло.
IT>А за пробелы повбывав бы.

За какие именно?

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

ГВ>>>Табуляции в форматировании — зло.
IT>>А за пробелы повбывав бы.

ГВ>За какие именно?

За ведущие.

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

CB>В общем, что вы думаете об этом проекте? Акцентируя внимание на его восприятии. И учтите, что я пытаюсь прокатить не за зеленого новичка, поэтому отнеситесь критично и за каждый промах валите меня гнилым помидором. Этот код должен стать идеальным. Ответственность за это лежит на вас (драматизирую).

нубас, смари, кароч, первое чо те надо уяснить:
дефайны на стандартные вещи — удел полных *даков
statefull синглтоны — удел полных *даков
геттеры/сеттеры для полей, изменение которых не нарушает контракт интерфеса — удел полных *даков
использование в хидерах типов, которые подключаются неявно, как следствие важна очерёдность подключения хидеров в cpp — удел полных *даков

а я ведь только 2 минуты поглядел, больше не хочется, т.к. налицо ООП головного мозга и возраст автора < 20 лет.

Походил по папкам, почитал исходники. Этот код smells.

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

CB>Итак, налетай народ, валим нуба!

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