Тухлый Александр (0xd34df00d)

*programming *c++
Yet another one. За такое руки оторвут?

const int pos = std::max (0, Ui_.VariantBox_->findText (current));
Ui_.VariantBox_->setCurrentIndex (pos);

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

#1198512
from Azoth_primary, 6 months ago

← All messages

Replies (9)

Post a reply (xmpp)

@netneladno:
АРХИТЕКТУРА @ ПЛАГИНЫ
#1198512/1
from 02CC40EE, 6 months ago
@aix27249:
хотя на первый взгляд конструкция верная, я бы руки за такое оторвал, ибо без детального чтения документации по findText и setCurrentIndex нельзя точно сказать, корректная ли она или нет
#1198512/2
from AgiliaLinux, 6 months ago
@0xd34df00d:
@aix27249 Ну, любой кутешник с более-менее опытом привык к тому, что find* возвращает -1, если оно не найдено.
Да и вообще стандартная практика же.
#1198512/3
from web, 6 months ago, in reply to /2
@aix27249:
@0xd34df00d это то да, а вот что выдаст setCurrentIndex(-1) я, честно говоря, не помню
#1198512/4
from AgiliaLinux, 6 months ago, in reply to /3
@aix27249:
@aix27249 да и в документации что-то найти не могу. ЕМНИП, будет сегфолт.
#1198512/5
from AgiliaLinux, 6 months ago, in reply to /4
@0xd34df00d:
@aix27249 Ну да. Поэтому мы устанавливаем не меньшее нуля значение, за этим и std::max.
И да, по опыту — будет сегфолт.
#1198512/6
from web, 6 months ago, in reply to /4
@aix27249:
@0xd34df00d еще стоит учесть то, что если count() равен 0 (то бишь бокс пустой), то даже setCurrentIndex(0) может вызвать сегфолт. Этот момент я тоже не могу найти описанным в доках, но судя по тому что я уже на автомате ставлю такие проверки — так оно и есть :)
#1198512/7
from AgiliaLinux, 6 months ago, in reply to /6
@0xd34df00d:
@aix27249 Ну, тут надо отметить, что этот код выполняется в if (Ui_.VariantBox_->count () > 1)... )
#1198512/8
from web, 6 months ago, in reply to /7
@aix27249:
@0xd34df00d а, тогда ок :)
#1198512/9
from AgiliaLinux, 6 months ago, in reply to /8

Тухлый Александр (0xd34df00d)

Subscribe

Replies (9)

Messages

Tags all

Search

Statistics