Здравствуйте, гость ( Вход | Регистрация )
|
AD |
5.8.2008, 16:26
Сообщение
#1
|
|
Профессионал ![]() ![]() ![]() ![]() ![]() Группа: Участник Сообщений: 2003 Регистрация: 4.2.2008 Из: S-Petersburg Пользователь №: 84 Спасибо сказали: 70 раз(а) Репутация: 17
|
Собственно предлагаю сюда задавать и выкладывать куски кода (с пояснениями, конечно, что этот код делает) для того, чтобы помогли его ну как упростить, сделать более читабельным. Возможно, кому-то это поможет.
Если разрешите, то могу начать. |
|
|
|
![]() |
|
ViGOur |
5.8.2008, 21:59
Сообщение
#2
|
![]() Мастер ![]() ![]() ![]() ![]() ![]() ![]() Группа: Модератор Сообщений: 3296 Регистрация: 9.10.2007 Из: Москва Пользователь №: 4 Спасибо сказали: 231 раз(а) Репутация: 40
|
Ну я дуаю ты сам мог бы догадаться, если бы был внимательней.
Не нужно два цикла, вместо entryList используй entryInfoList в и foreach получай QFileInfo с помощью которого ты можешь узнать файл это или директория (isFile или isDir). Ну а как потянешь за эту ниточку, остальное думаю наложится, если не будет понятно что-то спрашивай. p.s. как сделаешь код здесь выложи, чтобы покритиковали если будет что. |
|
|
|
AD Упрощение кода 5.8.2008, 16:26
void* ну вот у меня такая дилема. Нижеприведенный код по... 5.8.2008, 18:46
void* ViGOur, насчет entryInfoList я знал, просто мне ка... 5.8.2008, 22:21
Red Devil void ProjectView::AddDirs(const QDir ... 6.8.2008, 11:59
void* упростил однако из двух функций сделал четыре
хо... 6.8.2008, 12:49
ViGOur void createItems(const QDir &dir, QTreeWid... 6.8.2008, 14:22
void* ViGOur, спасибо! примерно такое у меня и верте... 6.8.2008, 17:26
Tonal 2 ViGOur
Я бы всё таки выделил из этого кода функц... 7.8.2008, 8:09
ViGOur Цитата(Tonal @ 7.8.2008, 9:09) Я бы всё т... 7.8.2008, 8:27
AD Выдалась возможность, может сможете и мне упростит... 7.8.2008, 9:10
Tonal Вынеси длинные проверки с next -> status... в о... 7.8.2008, 12:02
AD Цитата(Tonal @ 7.8.2008, 13:02) Вынеси дл... 7.8.2008, 12:55
Tonal Без тега кода всяко только уменьшилась!
По к... 8.8.2008, 15:21
Tonal Чё непонятно-то? 10.8.2008, 17:31
AD Цитата(Tonal @ 10.8.2008, 18:31) Чё непон... 10.8.2008, 22:06
Tonal ЦитатаДай - понимаю, курить - понимаю, дай курить ... 11.8.2008, 8:05
AD Цитата(Tonal @ 11.8.2008, 9:05) После про... 11.8.2008, 8:48
Tonal 1) После проверок где...
Код
if(...) isC... 11.8.2008, 11:07
AD Теперь это выглядит более внятно и легче править. ... 11.8.2008, 14:54
Red Devil Кодclass ScrTrack: public TImageClass
{
privat... 12.8.2008, 10:12
rich Может кому поможет:
"Рефакторинг. Улучшение с... 13.12.2008, 0:03
Admin rich, а ты темой не ошибся? Причем тут упрощение к... 13.12.2008, 0:52
alex977 Цитата(Admin @ 13.12.2008, 0:52) rich, а ... 13.12.2008, 11:28
rich Цитата(Admin @ 13.12.2008, 0:52) rich, а ... 14.12.2008, 18:41![]() ![]() ![]() |
|
Текстовая версия | Сейчас: 15.12.2025, 8:56 |