VD
да, как там твой движок
Сегодня собес был какой движок?)
Size: a a a
2021 April 05
m
надо в @Qt_chat залезть, спросить , как там qtdoc раскочегарить
Я чёт пробовал давно, но чёт забил и взял doxygen
R
Так, а нафиг мне смак по сути ...
R
эти файлы же будут в другие проекты включаться
A
# Доебы
1. https://github.com/Eixini/QNixieNumber/blob/main/qnixienumber.cpp#L9-L17 - одинаковые блоки;
2. https://github.com/Eixini/QNixieNumber/blob/main/qnixienumber.cpp#L35 - нейминг плохой, лучше setSegmentsCount тогда (?);
3. Что за number? (там же, рядом);
4. https://github.com/Eixini/QNixieNumber/blob/main/qnixienumber.cpp#L42 - опять дублирование кода;
5.
1. https://github.com/Eixini/QNixieNumber/blob/main/qnixienumber.cpp#L9-L17 - одинаковые блоки;
2. https://github.com/Eixini/QNixieNumber/blob/main/qnixienumber.cpp#L35 - нейминг плохой, лучше setSegmentsCount тогда (?);
3. Что за number? (там же, рядом);
4. https://github.com/Eixini/QNixieNumber/blob/main/qnixienumber.cpp#L42 - опять дублирование кода;
5.
A
Так, а нафиг мне смак по сути ...
вообще-то надо сделать правильный find_package(Eixini_QNixieNumber)
A
потому что нахуй мне бандлить это, что концептуально планируется быть библиотекой
R
вообще-то надо сделать правильный find_package(Eixini_QNixieNumber)
R
а смак не нужен ведь, да?
A
нужен
A
только надо создавать не executable, а library
R
тупо *.h , *.cpp и папка ресурсов, а еще и сам файл *.qrc
A
а для тестов уже надо и оставить executable
A
погляди тут, как я смак сделал
A
со строки 33 интересное
A
только надо DESTINATION другое сделать, ибо у тебя архитектурно-зависимая библиотека
A
# Доебы
1. https://github.com/Eixini/QNixieNumber/blob/main/qnixienumber.cpp#L9-L17 - одинаковые блоки;
2. https://github.com/Eixini/QNixieNumber/blob/main/qnixienumber.cpp#L35 - нейминг плохой, лучше setSegmentsCount тогда (?);
3. Что за number? (там же, рядом);
4. https://github.com/Eixini/QNixieNumber/blob/main/qnixienumber.cpp#L42 - опять дублирование кода;
5.
1. https://github.com/Eixini/QNixieNumber/blob/main/qnixienumber.cpp#L9-L17 - одинаковые блоки;
2. https://github.com/Eixini/QNixieNumber/blob/main/qnixienumber.cpp#L35 - нейминг плохой, лучше setSegmentsCount тогда (?);
3. Что за number? (там же, рядом);
4. https://github.com/Eixini/QNixieNumber/blob/main/qnixienumber.cpp#L42 - опять дублирование кода;
5.
хедер - вообще помойка, нахуя мне твои массивы сдались и куча депсов от них)
A
их надо вынести в реализацию, и сделать статическими переменными хотя бы
A