Как улучшить

@Lactusspooch · Регистрация: 02.10.2017

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

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

https://bitbucket.org/LactusLu... ?at=master

@Kastaneda · 02.10.2017, 20:21

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

Что б вообще подумали что я сеньором работал

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

@FateOri · 02.10.2017, 20:26

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

Что б прям взяли. Что б вообще подумали что я сеньором работал.

не подумают)))
идите на джуна а там видно будет на что вы способны))

@DobroAlex · 02.10.2017, 20:41

Lactusspooch, все равно вскроют вопросом типа

C++
1
2
i = 5;
i = i++ + ++i; // чему равно i?

@DU3 · 02.10.2017, 20:47

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

@Lactusspooch · 02.10.2017, 20:52 **[ТС]**

Я кстати не знаю что с отступами.... В IDE с ними все прекрасно. А вот на битбакете уже съехало. Мб это из-за того что в 2010 экспресс пишу

@DU3 · 02.10.2017, 20:56

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

@COKPOWEHEU · 02.10.2017, 22:15

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

C
1
2
i = 5;
i = i++ + ++i; // чему равно i?

---------
Но это неточно

Подпись как никогда в тему

@nonedark2008 · 02.10.2017, 22:56

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

Мб это из-за того что в 2010 экспресс пишу

Со времен 2010 года много чего изменилось в мире C++, советую начать наверстывать упущенное.

@hoggy · 03.10.2017, 00:20

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

Я кстати не знаю что с отступами.... В IDE с ними все прекрасно

вот так и вскрываются джуны)

@DU3 · 03.10.2017, 00:41

C++
1
char *c = new char[TOTAL_CLASTERS*CLASTER_SIZE];

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

@GbaLog- · 03.10.2017, 06:29

Не по теме:

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

чему равно i?

это даже джуниор должен знать, имхо.

MrGluck · 03.10.2017, 12:44

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

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

Я наверное сейчас разобью вам все представления о жизни, но на ваш код скорее всего даже никто и не посмотрит. Всё решится устной частью.
А по поводу кода - отформатируйте его, дайте понятные названия и уберите для себя куда-нибудь в архив. Приятно будет потом достать и ~~ужаснуться~~ поностальгировать. А понятные имена и форматирование помогут в будущем хотя бы разобраться что хотели написать в тот конкретный момент.

Добавлено через 7 минут
Глянул код - слишком много признаков того, что писал его новичок. Причём по всем параметрам (архитектура, код стайл, именование переменных, комментарии, даже ярко выраженное желание написать "в стиле профи").
Но в этом нет ничего плохого (кроме того, что код лучше не показывать). Все мы когда-то писали код на таком уровне. Вам может помочь лишь Опыт. Со временем, если продолжите учиться, вы сами всё поймёте. А пока могу посоветовать лишь книгу Макконнелла, Совершенный код.
И про код лучше просто не заикаться, слишком подставляете себя и даёте возможность тыкнуть в конкретные места.

@Raali · 03.10.2017, 17:37

player.h

C++
1
struct Player:Entity

c непривычки без пробелов сначала даже не понял что это )

Добавлено через 9 минут
#include лучше конкретные, только те, что используются. Иначе потом проект по 2 часа собираться будет, когда разрастется

C++
1
2
3
4
5
#include "Entity.h"
#include ...
 
struct Player : Entity
{...

using namespace std в хедере тоже лучше никогда не писать

#pragma once не все компиляторы воспримут, если проект кроссплатформенный - лучше избегать

члены структуры разного стиля, то с большой то с маленькой буквы

почему структура, а не класс, лучше чтобы все было по умолчанию приватно, для таких комплексных классов как Player

на realize.cpp закончил смотреть данный проект)

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .	SDL3 для Web (WebAssembly): Работа со звуком через SDL3_mixer 8Observer8 08.02.2026 Содержание блога Пошагово создадим проект для загрузки звукового файла и воспроизведения звука с помощью библиотеки SDL3_mixer. Звук будет воспроизводиться по клику мышки по холсту на Desktop и по. . .	SDL3 для Web (WebAssembly): Основы отладки веб-приложений на SDL3 по USB и Wi-Fi, запущенных в браузере мобильных устройств 8Observer8 07.02.2026 Содержание блога Браузер Chrome имеет средства для отладки мобильных веб-приложений по USB. В этой пошаговой инструкции ограничимся работой с консолью. Вывод в консоль - это часть процесса. . .
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .

@Lactusspooch 0 / 0 / 0 Регистрация: 02.10.2017 Сообщений: 3

	Как улучшить 02.10.2017, 19:50. Показов 774. Ответов 13 Метки нет (Все метки) Я самоучка, сам мало что знаю по поводу красивого кода. А на работу хотелось бы т к денег нет. Посоветуйте как улучшить данный код что б прям ваще. Что б прям взяли. Что б вообще подумали что я сеньором работал. Т.к. собираюсь этот код как портфолио сдать https://bitbucket.org/LactusLu... ?at=master 0

@DU3 284 / 232 / 114 Регистрация: 07.09.2016 Сообщений: 584
	02.10.2017, 20:47
	хотябы с отступами бардак уберите. почерк джуниора из этого кода по простому не убрать. да и не зачем. как уже было сказано на собеседовании вскроется на раз два. 0

@Lactusspooch 0 / 0 / 0 Регистрация: 02.10.2017 Сообщений: 3
	02.10.2017, 20:52 [ТС]
	Я кстати не знаю что с отступами.... В IDE с ними все прекрасно. А вот на битбакете уже съехало. Мб это из-за того что в 2010 экспресс пишу 0

@DU3 284 / 232 / 114 Регистрация: 07.09.2016 Сообщений: 584
	02.10.2017, 20:56
	это потому что в иде таб остается табом. по хорошему надо в настройках поставить, чтобы там заменялся на спейсы. количество сами определяйте как удобно. в таком случае в любом вьювере код будет ровный, а не только в вашей ide. если бы вы были сеньером, вы бы знали про это 0