crossplatform.ru

Здравствуйте, гость ( Вход | Регистрация )

> Правила именования идентификаторов в С++
igorenja
  опции профиля:
сообщение 2.2.2010, 13:16
Сообщение #1


Студент
*

Группа: Участник
Сообщений: 22
Регистрация: 18.1.2010
Пользователь №: 1384

Спасибо сказали: 1 раз(а)




Репутация:   0  


Цитата(lucifer_vrn @ 2.2.2010, 16:07) *
QLabel *help; у тебя определен в хедере.
а ты еще в сипипишнике еще один определяешь, в итоге у тебя выходит, что тот что в хедере не иницилизирован и обращение в слоте происходит к рандомному участку памяти.


Спасибо большое тупанул по чёрному (плюсы тока изучаю). Пару комментариев добавлю
Цитата(lucifer_vrn @ 2.2.2010, 16:07) *
1. Зачем у тебя часть переменных в private часть в protected. Если ты конечно от этого класса не собираешь потом наследоваться....

Это определение базового класса, его свойства будут наследовать производные классы.
Цитата(lucifer_vrn @ 2.2.2010, 16:07) *
2. But->setGeometry(QRect(QPoint(300, 300), But->sizeHint())); - мега жесть. Кури про QLayout, в частности QVBoxLayout и QHBoxLayout.

Спасибо за совет. Лэйауты я курил.... и много... Дело в том что сие твоерение будет работать на Embedded Linux в очень специфичной целезке, Там нужно точно позиционировать объекты на экране. Менеджеры компоновки там юзать не желательно....

Ну так а всё же, как скомпилировать так проект чтоб исполняемый файл работал хоть где?
Перейти в начало страницы
 
Быстрая цитата+Цитировать сообщение
 
Начать новую тему
Ответов
trdm
  опции профиля:
сообщение 3.2.2010, 1:09
Сообщение #2


Дмитрий Трошин
****

Группа: Участник
Сообщений: 575
Регистрация: 12.1.2008
Пользователь №: 68

Спасибо сказали: 21 раз(а)




Репутация:   6  


Цитата(igorenja @ 2.2.2010, 13:16) *
Спасибо большое тупанул по чёрному (плюсы тока изучаю).

а классы с приставкой "T" это привет из дельфы?
Что эта приставка означает?
ПС. Терпеть не могу когда так классы зовут...
Перейти в начало страницы
 
Быстрая цитата+Цитировать сообщение
Litkevich Yuriy
  опции профиля:
сообщение 4.2.2010, 17:55
Сообщение #3


разработчик РЭА
*******

Группа: Сомодератор
Сообщений: 9669
Регистрация: 9.1.2008
Из: Тюмень
Пользователь №: 64

Спасибо сказали: 807 раз(а)




Репутация:   94  


Цитата(AD @ 4.2.2010, 17:34) *
В 8-10 символов название функций удается поместить крайне редко.
В данный момент я склоняюсь к тому, что не нужно жёстко определять длину имени. Хотя краткое удобнее. Я пишу код в программерском блокноте, автодополнения нет. Поэтому мне важно чтобы названия были легко вспоминаемые. И интуитивные.
Немного свыкнувшись со словами используемыми в Qt, можно ожидать определённое название. Например, по твоим названиям что я ожидаю:
takeLoader - берёт какой-то загрузчик, вынимая его из чего-то, т.е. его потом нужно будет удалять вручную или куда-то вставить, чтоды удалился сам
indexDirectory - непонятно, что это такое, какой индекс каталога
isDirListEmpty - "список каталогов пуст?", по моему избыточное название
addChartNameToList - "добавить имя диаграммы в список", по моему избыточное название

Если не знать название класса то сложно сказать точное значение. А по поводу двух последних похоже что в имени функции имеется дубль имени класса
Перейти в начало страницы
 
Быстрая цитата+Цитировать сообщение
AD
  опции профиля:
сообщение 4.2.2010, 19:45
Сообщение #4


Профессионал
*****

Группа: Участник
Сообщений: 2003
Регистрация: 4.2.2008
Из: S-Petersburg
Пользователь №: 84

Спасибо сказали: 70 раз(а)




Репутация:   17  


Цитата(Litkevich Yuriy @ 4.2.2010, 17:55) *
Например, по твоим названиям что я ожидаю:
takeLoader - берёт какой-то загрузчик, вынимая его из чего-то, т.е. его потом нужно будет удалять вручную или куда-то вставить, чтоды удалился сам
indexDirectory - непонятно, что это такое, какой индекс каталога
isDirListEmpty - "список каталогов пуст?", по моему избыточное название
addChartNameToList - "добавить имя диаграммы в список", по моему избыточное название

Если не знать название класса то сложно сказать точное значение. А по поводу двух последних похоже что в имени функции имеется дубль имени класса

Да я понимаю, что некоторые названия не сократить. :) Дело не в этом. Первые два - это просто показ того, что не все имена функций у меня кривые. :) Вторая функция именно то и делает, что написано. Определяет индекс каталога, который был выбран. А вот изменение последних двух имен я бы сделал. Но не знаю как лучше! Дело в том, что в классе, где есть функция isDirListEmpty, isEmpty уже есть и относится к другому списку (в классе их два).

Да.... Название класса - BaseLoader.

P.S. Юра, спасибо за критический разбор и отклик. Буду очень благодарен за дальнейшее обсуждение. Поправить кривые название - большая польза!

Сообщение отредактировал AD - 4.2.2010, 19:48
Перейти в начало страницы
 
Быстрая цитата+Цитировать сообщение

Сообщений в этой теме


Ответить в данную темуНачать новую тему
Теги
Нет тегов для показа


13 чел. читают эту тему (гостей: 13, скрытых пользователей: 0)
Пользователей: 0




RSS Текстовая версия Сейчас: 4.12.2024, 17:06