Russian Qt Forum

Всем привет! Проблема следующая:
На форме есть объект txtLog (QTextBrowser), в который должны добавляться логи программы. Логи добавляются почти всеми исходными файлами в проекте.

Объект txtLog содержится в классе UniStitch.
Почитав про сигналы и слоты, я решил сделать так: в UniStitch добавить слот, добавляющий форматированный лог в txtLog, а в остальные исходники добавить в место отправки лог-информации emit logsignal(data)

Но тут получается, что надо будет в каждом исходнике создавать экземпляр объекта UniStitch и делать для каждого сигнала connect? Или я что-то не понимаю? Пытался сделать всё без сигналов и слотов - при помощи статичной функции, но что-то возникли проблемы (писал до этого на C#), наверняка я что-то делаю не так :) Вообщем, подскажите решение возникшей проблемы, если не затруднит! Спасибо!

С уважением,
Андрей

Почитай (http://doc.qt.nokia.com/4.7-snapshot/debug.html) и не велосипедь. :)

Конечно почитаю, только прошу ответ на мой вопрос, а не RTFM, а остальное я почерпну из документации ;)

А зачем вам в каждом исходнике создавать экземпляр? Для соединения достаточно указателя на уже существующий объект: вы создаёте UniStitch, а кому надо -- пускай просят указатель и соединяются, или как-то так. Возможно, имеет смысл воспользоваться статическими функциями, их ведь можно из любого места использовать.

Вот кусок кода класса UniStitch:

C++ (Qt)
static class UniStitch : public QMainWindow
{
	Q_OBJECT
 
public:
	explicit UniStitch(QWidget *parent = 0);
	~UniStitch();
 
	static void foo(QString logLine);

Статичная функция foo() будет выполнять запись лога в txtLog, только проблема в том, что txtLog - не статичный, и его методы из статичной функции foo() НЕ доступны ... что посоветуете (новичку :) )

Я не знаю... я всегда так делаю:

C++ (Qt)
class UniStitch : public QMainWindow
{
  Q_OBJECT
public:
  explicit UniStitch(QWidget *parent = 0);
public slots:
  void foo(QString logLine) { txtLog->append(logLine); }
private:
  QTextBrowser *txtLog;

C++ (Qt)
class Kitty : public QObject
{
  Q_OBJECT
public:
  explicit Kitty(QObject *parent = 0);
signals:
  void mew(QString msg);
};

и какой-нибудь

C++ (Qt)
int main(/*int argc, char *argv[]*/)
{
  // ...
  UniStitch us;
  Kitty k;
  QObject::connect( &k, SIGNAL(mew(QString)),
                    &us, SLOT(foo(QString)) );
}

А как использовать класс Kitty из других классов? Придется в каждом объявлять свой экземпляр Kitty? Или создать в одном месте и передавать указатель на него, скажем в конструкторе?

Э.. а я думал, что Kitty - это якобы один из многих классов, которому нужно выводить сообщения в лог :)

C++ (Qt)
class Log
{
public:
    static void ToFile(QString msg);                            
 
    static void ToFileName(QString fileName, QString msg);      
};
 

Далее используем его вызывая

C++ (Qt)
Log::ToFile(QString msg);
 

Такой вариант устраивает?

Мне нужно обращаться к нестатичному объекту txtLog. Те статичные функции, что вы написали, не смогут получить доступа к методам нестатичного txtLog :(

А почему обязательно статические функции?

Чтобы не создавать каждый раз новый экземпляр класса UniStitch в других классах, которые будут использовать функцию лога из UniStitch

p.s. Я приму любой интересный вариант решения моей проблемы, не обязательно с использованием статических функций

Если я правильно понимаю вашу идею. То ваш класс, UniStitch изначально создается, т.к. выводит на форму объекты и до окончания работы программы он никуда не пропадает. Тогда в чем проблема? Надо что-то залогить - вызывается public метод класса UniStich и ему в качестве аргумента передается сообщение.

Проблема в том, что запись в лог нужно делать из кучи других классов, и в каждом придётся держать указатель на UniStitch. У меня идея сделать UniStitch::foo(QString) слотом, а другим классам написать сигнал bar(QString), который бы отправлялся по надобности записи в лог - тогда остаётся соединить все классы с экземпляром UniStitch.
Я совсем запутался :-\

Чтобы не путаться: есть форма, на ней текстовое поле (ну пускай QTextBrowser). В проекте много исходных файлов (и конечно же других классов) - каждый что-то должен написать в это текстовое поле.

surendil, такое ощущение, что это ты пишешь :) Очень тяжело читать код с непонятными названиями классов и методов, типа

C++ (Qt)
class UniStitch;
static void foo(QString logline);
 

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

C++ (Qt)
static class UniStitch : public QMainWindow
{
	Q_OBJECT
 
public:
	explicit UniStitch(QWidget *parent = 0);
	~UniStitch();
 
	void appendLog(QString logLine);
 
private:
	txtLog QTextBrowser;
}
 

Когда нужно логить, просто вызываешь метод данного класса appendLog и заносишь инфу в виджет.
И да, вопрос - почему класс статический?

Нарисовал идею в коде :) Спасибо Даниилу, теперь я умею прикреплять файлы :D

Моя ошибка, давно её исправил :) Сейчас UniStitch без static

Не все так просто - для того, чтобы вызвать метод, надо сначала получить к нему доступ:
1. Если метод (appendLog) нестатичный, то придется создавать экземпляры класса, содержащего этот метод, для КАЖДОГО иного класса, использующего этот метод
2. Если метод (appendLog) статичный, то в нем НЕ можем обратиться к нестатичному объекту (виджету)

Стоп. Ты класс свой изначально вызываешь же? Он у тебя никуда не пропадает? Т.е. ты в любой точке программы можешь просто вызвать этот метод для УЖЕ созданного объекта класса?!
Или я чего-то не понимаю?

Попробуйте так:

А для каждого класса, который будет отправлять сигнал, нужно сделать специальный конструктор например такой:


И для всех екземпляров этого класса в качестве parent_unistitch ставить указатель на тот QTextBrowser в который нужно отправить лог. В этом случае не надо парится об создании сигналов/слотов, если в всех класах будете использовать конструктор такого рода, то сигналы/слоты будут настраиватся автоматически.

Сделай UniStitch  синглтоном и не парься со статичными методами, если ты хочешь что бы как я понял UniStitch был один на весь проект

Тогда придется передавать указатель на этот класс в каждый другой класс, чтобы использовать метод appendLog() ... вот товарищ Рома предложил интересный (похожий, но чуточку удобнее) вариант ... сейчас обдумаю ...

Да, на весь проект один объект QTextBrowser класса UniStitch
Что значит "синглтон" - просто я новичек не только в Qt, но и в C++ :)

Это ? http://habrahabr.ru/blogs/cpp/116577/

вот http://ru.wikipedia.org/wiki/%D0%9E%D0%B4%D0%B8%D0%BD%D0%BE%D1%87%D0%BA%D0%B0_(%D1%88%D0%B0%D0%B1%D0%BB%D0%BE%D0%BD_%D0%BF%D1%80%D0%BE%D0%B5%D0%BA%D1%82%D0%B8%D1%80%D0%BE%D0%B2%D0%B0%D0%BD%D0%B8%D1%8F)

там и пример есть, грубо говоря, это когда создается один экземпляр класса на все приложение.

О, здорово ^^

Обычно делают так

C++ (Qt)
// из любого места
UniStitch::Instance()->Write2Log("abc");
 

Instance - static. Write2Log - нет

Никуда, никаких указателей передавать не нужно. Просто будет вызываться сразу два метода:

C++ (Qt)
//LogData - return QString;
mainWindow->appendLog(client->LogData());
 

А как я получу доступ к mainWindow из другого класса?

Логика программы прояснилась. Мой вариант идет, только если в main'e танцевать с методами.

C++ (Qt)
// unistitch.h
#include <QMainWindow>
#include <logwidget.h>
class UniStitch : public QMainWindow
{
  Q_OBJECT
public:
  explicit UniStitch(QWidget *parent = 0);
private:
  QTextBrowser *txtLog;
};

C++ (Qt)
// unistitch.cpp
UniStitch::UniStitch(QWidget */*parent*/)
  : txtLog(NULL)
{
  txtLog = LogWidget::Instance();
  setCentralWidget(txtLog);
}

C++ (Qt)
// logwidget.h
#include <QTextBrowser>
class LogWidget : public QTextBrowser
{
  LogWidget(){}
public:
  static LogWidget *Instance()
  {
    if( m_instance == NULL )
      m_instance = new LogWidget;
    return m_instance;
  }
private:
  static LogWidget *m_instance;
};

C++ (Qt)
// logwidget.cpp
#include "logwidget.h"
LogWidget* LogWidget::m_instance = NULL;

В каждый .h файл класса, из которого нужно отправлять сообщения, я включаю logwidget.h. Использую так:

C++ (Qt)
LogWidget::Instance()->append("Hello, World!");

Это работает. А как правильно? Куда девается объект, который создаётся в Instance()?
Я пробовал понять, но не получилось, поэтому просто из Википедии переписал :D

Всё гениальное - просто? А ведь правда :)

Спасибо Пантер (http://www.prog.org.ru/index.php?action=profile;u=4067), surendil (http://www.prog.org.ru/index.php?action=profile;u=11621), Даниил (http://www.prog.org.ru/index.php?action=profile;u=12212), Рома (http://www.prog.org.ru/index.php?action=profile;u=12221), ecspertiza (http://www.prog.org.ru/index.php?action=profile;u=7026) и конечно, Igors (http://www.prog.org.ru/index.php?action=profile;u=7236) за помощь - проблема решена!

Сделал проще, по совету Igors:

C++ (Qt)
static class UniStitch : public QMainWindow
{
	Q_OBJECT
 
public:
	explicit UniStitch(QWidget *parent = 0);
	~UniStitch();
 
	void Write2Log(QString logLine);
 
	static UniStitch *stitch;
 

Подключил unistitch.h ко всем *.cpp и использую так:

C++ (Qt)
UniStitch::stitch->Write2Log("Требуется больше изображений!");

Бррр. Это громоздко. Гораздо проще заюзать qDebug и написать свою функцию сохранения лога.

Через qInstallMsgHandler ?

Ага. Вот тебе пример:

C++ (Qt)
void logMessageOutput (QtMsgType type, const char *msg)
{
	static QFile logFile;
	static QTextStream stream;
	stream.setCodec ("UTF-8");
 
	static QMap <QtMsgType, QString> messages;
 
	if (messages.isEmpty()) {
		messages [QtDebugMsg] = "Debug";
		messages [QtWarningMsg] = "Warning";
		messages [QtCriticalMsg] = "Critical";
		messages [QtFatalMsg] = "Fatal";
	}
 
	if (!logFile.isOpen ()) {
		logFile.setFileName (QCoreApplication::applicationDirPath ()
							 + "/"
							 + QCoreApplication::applicationName ()
							 + ".log");
		logFile.open (QIODevice::WriteOnly | QIODevice::Append);
		stream.setDevice (&logFile);
	}
 
	if (logFile.isOpen ()) {
		stream << QDateTime::currentDateTime ().toString ("yyyy-MM-dd_hh:mm:ss")
			   << " " << messages [type] << ": " << msg << endl;
	}
}
 

Окей, учту и в следующем билде использую ваш вариант ... а он потоко-безопасен?

Мой вариант использовать не нужно - это всего лишь пример. У меня в 3-х проектах разные варианты. На счет потокобезопасности - не знаю. По идее, да - потокобезопасен, если проинициализировать в main. Вот, кстати, еще вариант:

C++ (Qt)
const size_t maxLogFileSize = 1 * 1024 * 1024; //1 MB
 
class LogFile
{
public:
	explicit LogFile (QtMsgType type) : type_ (type) {
 
		messages_ [QtDebugMsg] = "Debug";
		messages_ [QtWarningMsg] = "Warning";
		messages_ [QtCriticalMsg] = "Critical";
		messages_ [QtFatalMsg] = "Fatal";
 
		fileNames_ [QtDebugMsg] = appName + "_debug.log";
		fileNames_ [QtWarningMsg] = appName + "_warning.log";
		fileNames_ [QtCriticalMsg] = appName + "_critical.log";
		fileNames_ [QtFatalMsg] = appName + "_fatal.log";
 
		file_.setFileName (QCoreApplication::applicationDirPath ()
						   + "/"
						   + fileNames_ [type_]);
		QIODevice::OpenMode openMode = QIODevice::WriteOnly;
 
		if (file_.exists () && file_.size () < maxLogFileSize) {
			openMode |= QIODevice::Append;
		}
 
		file_.open (openMode);
 
		stream_.setDevice (&file_);
		stream_.setCodec ("UTF-8");
	}
 
	~LogFile () {
		stream_.flush ();
		file_.flush ();
		file_.close ();
	}
 
	template <class T>
	void setMessage (const T &msg) {
		if (file_.isOpen ()) {
			stream_ << QDateTime::currentDateTime ().toString ("yyyy-MM-dd hh:mm:ss")
					<< " " << messages_ [type_] << ": " << msg << endl;
		}
	}
 
private:
	LogFile (const LogFile &);
	LogFile &operator= (const LogFile &);
 
private:
	QtMsgType type_;
	QMap <QtMsgType, QString> messages_;
	QMap <QtMsgType, QString> fileNames_;
 
	QFile file_;
	QTextStream stream_;
};
 
typedef QSharedPointer <LogFile> LogFilePtr;
 
void logMessageOutput (QtMsgType type, const char *msg)
{
	static QMap <QtMsgType, LogFilePtr> files;
 
	if (!files.contains (type)) {
		files [type] = LogFilePtr (new LogFile (type));
	}
 
	files [type]->setMessage (QString::fromUtf8 (msg));
}
 

Повесил qInstallMsgHandler, запускаю обработку в отдельном потоке ... должны обрабатываться 7 изображений, а получаю только 6, причем если делаю брейкпоинт перед функцией вызова обработки, то считает все 7 ... думаю, что проблемы из-за доступа потока обработки к GUI (к компоненту QTextBrowser - запись в него логов) :( Может быть такое?

p.s. при отключении qInstallMsgHandler, все работает отлично.