передача QString функции собственному классу Опции

[danya]

Там совсем в другом ошибка:
1. base::base file; замени на base file;
2. В base.cpp
baza(file,QSettings::IniFormat); замени на QSettings baza(file,QSettings::IniFormat);

2 так раньше и было)
а вот первое действительно тупанул спасибо!

[kwisp]

danya,
у тебя же есть член класса baza зачем локальную переменную в функции open_base тоже называешь baza при том она того же типа что и член класса!
Это тебя путает. Так делать не рекомендуется. назови по другому.

А если ты хочешь инициализировать член класса с именем baza то тебе надо написать не
baza(file,QSettings::IniFormat);
и не
QSettings baza(file,QSettings::IniFormat);
а
baza = QSettings(file,QSettings::IniFormat);
к моменту вызова open_base baza уже существует созданная конструктором по умолчанию.

[danya]

danya,
у тебя же есть член класса baza зачем локальную переменную в функции open_base тоже называешь baza при том она того же типа что и член класса!
Это тебя путает. Так делать не рекомендуется. назови по другому.

А если ты хочешь инициализировать член класса с именем baza то тебе надо написать не
baza(file,QSettings::IniFormat);
и не
QSettings baza(file,QSettings::IniFormat);
а
baza = QSettings(file,QSettings::IniFormat);
к моменту вызова open_base baza уже существует созданная конструктором по умолчанию.

сделал)
появилась следующая ошибка

 error: 'QSettings& QSettings::operator=(const QSettings&)' is private

в

baza = QSettings(file,QSettings::IniFormat);

[Litkevich Yuriy]

Всё верно, QSettings не имеет отрытого оператора присваивания и конструктора копирования.
Этот класс предназначен для создания локальной переменной, так всюду в примерах и применяется.

[kwisp]

danya,
я просто исходники твои скачал.
про то что оператор копирования закрыт это я проглядел в документации.
получается тебе не нужны члены класса QSettings baza и еще какой то не помню уже т.к. исходники удалил

[danya]

danya,
я просто исходники твои скачал.
про то что оператор копирования закрыт это я проглядел в документации.
получается тебе не нужны члены класса QSettings baza и еще какой то не помню уже т.к. исходники удалил

да я уже по другому сделал сделал общим переменную file вообщем сейчас всё работает спасибо всем

[Obey-Kun]

Не следует передавать QString через копию. Лучше — через константную ссылку: int base::open_base(const QString &file). Такой метод будет вызываться быстрее.

[Алексей1153]

Obey-Kun, иногда бывает так, что нет объекта, ссылку на который можно передать Тогда лучше иметь перегрузку с параметром по значению

[Obey-Kun]

Obey-Kun, иногда бывает так, что нет объекта, ссылку на который можно передать Тогда лучше иметь перегрузку с параметром по значению

Так перегрузку или параметр по умолчанию?

Перегрузка:

myMethod(const QString &string);
myMethod();

Параметр по умолчанию:

myMethod(QString string = "");

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

Сообщение отредактировал Obey-Kun - 1.12.2010, 10:55

[Litkevich Yuriy]

Такой метод будет вызываться быстрее.

в общем случае да, а в с лучае с QString - нет. Т.к. QString копирует данные, только при изменении оригинала/копии.
Этот класс специально оптимизирован