C++ vraag

[SeySayux]

Hallo,

ik heb een paar problemen met het programmeren in C++. In java kan je vb doen:

Code: [Selecteer]

Object o;
//...
o = new Object();Nu wil ik in C++ doen:

Code: [Selecteer]

QWidget * q;
//...
q = new QWidget;maar dan krijg ik de volgende foutmelding:

Code: [Selecteer]

src/globals.h:7: error: expected constructor, destructor, or type conversion before '=' tokenIk gebruik de volgende versies van software:

gcc: i686-apple-darwin9-gcc-4.0.1
make: GNU Make 3.81, i386-apple-darwin9.0
qmake: QMake version 2.01a
qdevelop: version 0.25

Als het nodig is, kan ik de code van het gehele programma uploaden.

Alvast bedankt,

- SeySayux

[profoX]

Ik zie niet echt iets fout daar, maar inderdaad lijkt het me beter als je het in zijn context plaatst, dus het hele programma.

[SeySayux]

Je kan de code hier downloaden:

http://www.megafileupload.com/en/file/68483/syrpmbuild-zip.html

De bestanden die van belang zijn, zijn globals.h en globals.cpp, in Globals.h declareer ik Configuration * CONFIG; en in globals.cpp declareer ik CONFIG = new Configuration;

- SeySayux

[profoX]

Als je globals.h include heb je misschien wel die CONFIG variabele, maar daar is dan niets aan toegewezen. Het is niet zo dat die .cpp file zomaar wordt uitgevoerd. Om een globale configuration class te hebben zou je eventueel een Singleton kunnen gebruiken (of een static class indien je geen slots e.d. nodig hebt, maar die wil je wel hebben)... of je herziet de structuur van je programma wat en maakt ergens in het begin (bv. in je main of core class) een Configuration object aan, die je dan eventueel doorgeeft aan de constructor van objecten zoals QMetaBar..

Andere opmerkingen:

* wist je dat dit appje meer dan 500 MB ram in beslag neemt tijdens het compileren door die reuzegrote opsomming bij die QStringList? Ik zou die dingen eerder uitlezen vanuit een tekstbestand ofzo
* als je iets inpakt zou ik dat in een aparte map doen, want nu deed ik 'unzip file.zip' en kwamen er allemaal losse bestanden in de map waar ik het uitpakte, dat was een beetje vervelend want dan moest ik ze allemaal afzonderlijk verwijderen
* als je objecten op de heap aanmaakt, zorg er dan ook voor dat die objecten allemaal gedeleted worden, bij Qt gebeurt dit inderdaad grotendeels automatisch, maar alleen als je de objecten een parent geeft

edit: ik ga ervan uit dat jij zelf wel zult/kunt googlen naar de zaken die ik heb opgenoemd, maar als je wat extra uitleg wil vraag je het maar hé, hoewel ik zelf natuurlijk ook geen expert C++/Qt ben.. het IRC kanaal #qt op irc.freenode.net en www.qtcentre.org zijn wel handige hulpmiddelen om hulp te vragen vind ik

[SeySayux]

Hallo,

Ik heb het volgende al geprobeerd:

globals.h:

Code: [Selecteer]

#ifndef __GLOBALS_H__
#define __GLOBALS_H__
#include "configuration.h"

class Global {
public:
static Configuration * CONFIG;
};
#endifglobals.cpp:

Code: [Selecteer]

#include "globals.h"

Global::CONFIG = new Configuration;Maar dan krijg ik dezelfde fout.

Over jou opmerkingen:
* Ik heb hier 2 GB ram, ik merk dat dus nog eens niet . Ik heb proberen uit te zoeken hoe ik iets van een tekstbestand kan lezen, maar er is geen goede manier om dat cross-platform te doen. Op Mac OS X zou ik bv. het tekstbestand willen halen uit de map "Resources", terwijl in Linux ik het zou willen halen uit "$dataroot/syrpmbuild" (dataroot is meestal /usr/share, maar dit kan normaal gezien veranderd worden)
* Sorry, meestal doe ik dit wel, maar ik had het snel ingepakt deze keer. Rechtsklikken -> inpakken als...
* Worden die objecten niet automatisch vewijderd als je het programma afsluit?

- SeySayux

[profoX]

Ik heb het volgende al geprobeerd:

globals.h:

Code: [Selecteer]

#ifndef __GLOBALS_H__
#define __GLOBALS_H__
#include "configuration.h"

class Global {
public:
static Configuration * CONFIG;
};
#endifglobals.cpp:

Code: [Selecteer]

#include "globals.h"

Global::CONFIG = new Configuration;Maar dan krijg ik dezelfde fout.

Ik zou iets op bv. deze manier doen:

verandering aan configuration.h

Code: [Selecteer]

void setName(const QString & name);verandering aan configuration.cpp

Code: [Selecteer]

void Configuration::setName(const QString & name) {
this->setName(name);
}[ edit: ik zie nu pas dat deze functie setName() eigenlijk niet kan kloppen... je zal gewoon recursief iets blijven uitvoeren; ik had zelf ook een heap/stack foutje gemaakt in main.cpp hieronder, heb ik even aangepast ]

globals.h

Code: [Selecteer]

#ifndef __GLOBALS_H__
#define __GLOBALS_H__
//#include "configuration.h"

class Configuration;
class Global {
public:
Global();
~Global();
Configuration * config;
};
#endif // __GLOBALS_H__globals.cpp

Code: [Selecteer]

#include "globals.h"
#include "configuration.h"

Global::Global() {
config = new Configuration;
}

Global::~Global() {
delete config;
}verandering aan main.cpp

Code: [Selecteer]

void initialize(QWidget * win)
{
Global * global = new Global;

win->setMinimumSize(640,480);
QVBoxLayout *layout = new QVBoxLayout;

QTabWidget * tabbar = new QTabWidget;
layout->addWidget(tabbar);

MetaTab * meta = new MetaTab(0, &global);
tabbar->addTab(meta, QObject::tr("Meta"));

QWidget * prep = new QWidget;
tabbar->addTab(prep, QObject::tr("Prep Section"));
// etc...verandering aan metatab.cpp

Code: [Selecteer]

//valueChanged
connect(name, SIGNAL(textChanged(const QString &)), global->config, SLOT(setName(const QString &)));Maar eigenlijk zit de code wat messy in elkaar. Ik zou eens een goed boek over C++ aanschaffen. Eentje die je ook 'design patterns' (ontwerppatronen) aanleert. Ook helpt het wel om de voorbeeldcode van bv. Qt 4 demos/examples te bekijken. Die zijn altijd heel netjes geprogrammeerd.

Voor de GUI zou je btw Designer kunnen gebruiken in plaats van het handmatig te hoeven doen.
Zodat je dit soort gekke dingen niet doet:

Code: [Selecteer]

QScrollArea * scroller = new QScrollArea(this);
scroller->setWidget(this);Bij mij gebruikt dit gewoon 100% CPU zonder iets nuttigs te doen.

* Ik heb hier 2 GB ram, ik merk dat dus nog eens niet . Ik heb proberen uit te zoeken hoe ik iets van een tekstbestand kan lezen, maar er is geen goede manier om dat cross-platform te doen. Op Mac OS X zou ik bv. het tekstbestand willen halen uit de map "Resources", terwijl in Linux ik het zou willen halen uit "$dataroot/syrpmbuild" (dataroot is meestal /usr/share, maar dit kan normaal gezien veranderd worden)

Als je op verschillende OS'en verschillende mappen wil gebruiken zou je platform defines kunnen gebruiken: Q_WS_X11 voor Linux, Q_WS_WIN voor Windows, Q_WS_MAC voor Mac (en Q_WS_QWS voor Qtopia Core) bv.:

#ifdef Q_WS_X11
resPath = "/usr/share/syrpmbuild/"
#else
resPath = "Resources/"
#endif

Een andere mogelijkheid is volgens mij om de static function QDir::setSearchPaths te gebruiken om in te stellen in welke mappen gezocht moet worden: http://doc.trolltech.com/4.4/qdir.html#setSearchPaths

* Worden die objecten niet automatisch vewijderd als je het programma afsluit?

Bij alle moderne besturingssystemen wel, maar dat is geen reden om niet netjes te programmeren In het midden van een programma zou zoiets gemakkelijk kunnen uitmonden in de befaamde 'memleaks' of geheugenlekken. Eigenlijk geldt: voor elke "new" heb je een "delete" nodig op de gepaste plaats, maar bij Qt objecten volstaat het meestal als je bij het aanmaken van het object een parent meegeeft, wanneer die parent dan wordt gedeleted, worden ook automatisch alle 'child' objecten van die parent gedeleted.

[SeySayux]

wat jij zegt dat ik moet veranderen aan configuration.h/cpp heb ik toch al? Is er ook een manier waarmee ik gemakkelijker/beter de gegevens die op alle tabbladen aangeduid zijn door de gebruiker zijn ingevoerd bijeen te krijgen zodat deze verwerkt kunnen worden? En ik heb toch ongeveer dezelfde stijl gebruikt als die ze gebruikten in de Qt-tutorial? Ik zal de voorbeeld-code bekijken.

En mensen die van mening verschillen of het beter is om

Code: [Selecteer]

void test (int i)
{
dummy();
}of

Code: [Selecteer]

void test (int i) {
dummy();
}te gebruiken, begrijp ik niet hoor...

- SeySayux

[profoX]

wat jij zegt dat ik moet veranderen aan configuration.h/cpp heb ik toch al?

Ik heb er 'const' bijgezet zodat de types in de connection overeenkomen:
textChanged(const QString &) is het signaal dat van QLineEdit komt.
Ik denk dat je daaroorspronkelijk gewoon QString & had staan.... en dan zou je deze run-time error krijgen:
Object::connect: No such signal QLineEdit::textChanged(QString&)

Is er ook een manier waarmee ik gemakkelijker/beter de gegevens die op alle tabbladen aangeduid zijn door de gebruiker zijn ingevoerd bijeen te krijgen zodat deze verwerkt kunnen worden? En ik heb toch ongeveer dezelfde stijl gebruikt als die ze gebruikten in de Qt-tutorial? Ik zal de voorbeeld-code bekijken.

Als ik het goed begrijp wil je die gegevens die daar zijn ingevuld verwerken bij het klikken op één van die knoppen?
Dan zou je dat rechtstreeks kunnen verwerken vanuit de GUI velden zelf.

En mensen die van mening verschillen of het beter is om

Code: [Selecteer]

void test (int i)
{
dummy();
}of

Code: [Selecteer]

void test (int i) {
dummy();
}te gebruiken, begrijp ik niet hoor...

Wie zegt zoiets? Dat is allebei hetzelfde.

[SeySayux]

Hoe moet ik dan vanuit een andere klasse aan de pointers naar deze widgets komen? En door ze door te geven aan de constructor...

Code: [Selecteer]

Builder::Builder(QLabel * name, QLabel * url QLabel * version /* reken hier nog een 500-tal andere wigdets bij */ , QCheckBox actuallyBuild) {
}Ik denk niet dat dat gaat lukken .

En hoe moet ik dan eenvoudig gegevens inladen? Ook door de QWidget::setSomething(QString * q const) calls?

Bedankt dat je me wilt helpen,

- SeySayux

[profoX]

Ik wist niet dat je vanuit een andere klasse wou werken...
Je kan natuurlijk dat config object wel gebruiken als je wil.
Alleen moet zoiets dan wel anders:

Code: [Selecteer]

void Configuration::setName(const QString & name) {
    this->setName(name);
}je wil wellicht zoiets:

Code: [Selecteer]

void Configuration::setName(const QString & name) {
    _name = name;
}waarbij _name een public variabele is (of: _name is een private variabele en een public functie "const QString & getName() {return _name;};"