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

[danya]

попробовал сделать следующим образом:
в основной форме:

QString str = QFileDialog::getOpenFileName(this, "Select File", "", "*.ini");
    base::base file;
    flat=file.open_base(str);

в классе есть функция:

int base::open_base(QString file)
{
    QSettings baza(file,QSettings::IniFormat);
    baza.beginGroup("Config");
    flat=baza.value("FLATS","").toInt();
    return flat;
}

в результате получаю ошибку:

error: no match for call to '(QSettings) (QString&, QSettings::Format)'

подскажите что я делаю не так или хотябы что в какую сторону копять?

[panter_dsd]

Все, вроде, правильно. Версия Кьюта какая?

[danya]

Все, вроде, правильно. Версия Кьюта какая?

4.7.0

[panter_dsd]

Интересно. Код сюда приложить сможешь?

[kwisp]

error: no match for call to '(QSettings) (QString&, QSettings::Format)'

по-моему такая ошибка если из текущей области видимости(в данном случае файла) не видно обЪявления функции(в данном случае конструктора QSettings) проще говоря предполагаю что не хватает # include <QtCore/QSettings> в файле в котором ошибка.

[panter_dsd]

Тогда писало бы что

error: 'QSettings' was not declared in this scope

[danya]

Интересно. Код сюда приложить сможешь?

да конечно
только не пугайтесь на кол-во закомментированного кода
я ведь только учусь Link

[panter_dsd]

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

[kwisp]

2. В base.cpp
baza(file,QSettings::IniFormat); замени на QSettings baza(file,QSettings::IniFormat);

в приведенном в посте коде так и есть. или я туплю?

[panter_dsd]

Ты прав. Но в исходниках другое. ТС походу запутался немного.

[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 копирует данные, только при изменении оригинала/копии.
Этот класс специально оптимизирован

[Алексей1153]

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

про перегрузку

myMethod(const QString &string); //тут можно передать ссылку на QString
myMethod(const QString string); // а тут можно просто строку символов. Объект QString создастся и будет равносильно передаче по значению

но зато гибко

QString копирует данные,

мфсишный CString себя так же ведёт Я однажды это для себя открыл вдруг - передал в функцию CString по значению, потом в функции редактировал внутренний буфер вручную. Хоп! Смотрю, внешний объект тоже поменялся )) Костыль - сначала явно скопировать в другой объект.

[Obey-Kun]

Да, забыл об implicit sharing.

Но в любом случае, для передачи объектов, sizeof которых больше sizeof указателя (а это ограничивается почти всеми стандартными типами и элементами enum'ов), в качестве аргументов, во всех методах в Qt применяются именно константные ссылки, куда ни глянь.

Возможно, потому, что тут всё равно приходится совершать какие-никакие действия по копированию объекта.

[BRE]

Что-то я не пойму о чем речь?
И первый и второй метод спокойно можно вызывать так:

myMethod( "text" );

В обоих случаях будет создан временный объект QString.

[Obey-Kun]

Если ты о моём примере, то нет, при передаче по ссылке никакой копии создаваться не будет.
При передаче по копии, будет создана копия, но содержимое QString (буковки) не будут копироваться (из-за Impicit Sharing).

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

[Алексей1153]

BRE, да, но когда передаёшь QString, не происходить аллокации для строки, а когда передаётся строку символов, происходит создание объекта и аллокация памяти под копию буфера символов. Вот об этом речь

[Obey-Kun]

По моему, чем делать заведомо менее эффективный метод для иллюзии удобства, лучше делать так:

myFunc(const QString &string) {
    qDebug() << string;
}

QString s("tatata");
myFunc(s);
myFun(QString("nanana"));

[Алексей1153]

Obey-Kun, визуальное загромождение кода тоже мало способствует удобству Мы же знаем, что QString так работает (да и сами разработчики так его применяют - это о чём-то говорит).
А в общем случае, конечно, по ссылке или по указателю большие объекты передавать следует. За исключением случаев, когда внутренний объект будет меняться - тогда проще сразу на стеке его и создать

[BRE]

Если методу нужен в качестве параметра QString, то по любому нужно создавать объект.
А передача константной ссылки всегда "легче" чем передача по значению, хотя с классами используемыми implicit sharing это и не так явно.

[Obey-Kun]

Obey-Kun, визуальное загромождение кода тоже мало способствует удобству Мы же знаем, что QString так работает (да и сами разработчики так его применяют - это о чём-то говорит).
А в общем случае, конечно, по ссылке или по указателю большие объекты передавать следует. За исключением случаев, когда внутренний объект будет меняться - тогда проще сразу на стеке его и создать

Поищите, например, setName в документации. Да у них там везде QString передаётся по константной ссылке. Абсолютно везде.
При создании копии QString, несмотря на общее использование данных, кое-какие лишние действия всё равно будут проводиться.

А передача константной ссылки всегда "легче" чем передача по значению, хотя с классами используемыми implicit sharing это и не так явно.

Вот именно. Но разница всё равно есть. И разработчики Qt, видимо, посчитали её значительной, иначе в собственных методах использовали бы передачу QString по значению.

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

[Алексей1153]

Obey-Kun, непонятно, о чём такой долгий спор Всем и так известно, что ссылка (или указатель) передаётся быстрее и меньше стека есть. Думаешь, Антарктиду открыл ?
Только в каждом случае , когда пишешь свою функцию, нужно продумывать интерфейс, так как тупо всегда ссылку или указатель нельзя передавать. Бывает так, что надо именно объект, чтобы он сразу скопировался