DGL • Thema anzeigen - C++: Problem

C++: Problem

Moderator: DGL-Team

Seite 1 von 2

[ 21 Beiträge ]

Gehe zu Seite 1, 2 Nächste

Vorheriges Thema | Nächstes Thema

Autor

Nachricht

Hondo

Betreff des Beitrags: C++: Problem

Verfasst: Mi Mär 22, 2006 17:47

DGL Member

Registriert: Mi Mär 22, 2006 17:39
Beiträge: 7

Mahlzeit!

Ich programmiere C++ und Perl an der Arbeit, und will privat jetzt mal nen bisschen mit Grafikprogrammierung rumbasteln. Hoffe ihr helft mir, obwohl mein Code nicht Delphi ist.

Zum Einstieg hab ich mal was gebastelt, was mir den Bildschirm in Quadrate einteilt. Villeicht mach ich mal sowas wie Schiffe versenken draus oder so, aber ich scheitere hier schon am Anfang. Folgendes Problem:

Ich habe eine 2D-Welt erstellt, mit

glOrtho(0, this->screen->w, 0, this->screen->h, -1.0f, 1.0f);

screen, ist dabei ein SDL_Surface*. Also so wie in dem SDL-Tut von wiki.delphigl.com.
Die Einteilung des Bildschirms in 32x32px Quadrate funzt auch. Jetzt wollt ich mal ne Textur auf die Einzelnen Quadrate kleben, allerding bleibt jetzt mein Fenster weiß. Zum Laden des Bildes benutze ich den Targa-Loader von der NeHe Seite, Lektion 33. So sieht mein Code aus (die wichtigen Stellen):

die header datei mit den Weltklassen

Code:

 
#ifndef _WORLD_H_
#define _WORLD_H_
#include <stdlib.h>
 
#include "constants.h"
 
 
class Area
{
 
private:
    int ID;
    Pos pos;
    int size;
    int type;
    Texture tex;
 
public:
    Area();
    ~Area();
    void Setup(Pos position, int Type); 
    
    void setID(int id){this->ID = id;}
    int getID(){return this->ID;}
    void setSize(int Size){this->size = Size;}
 
    void Draw();
};
 
class World
{
private:
    Area **matrix;
    int fieldsize;
    int fieldsX;
    int fieldsY;
public:
    World(int FieldSize, int solX, int solY);
    ~World();
 
    Area **Matrix(){return this->matrix;}
 
    void Draw();
};
 
#endif
 

und die Implementationen dazu:

Code:

 
#include "World.h"
 
 
World::World(int Fieldsize, int solX, int solY)
{
    int idgiver = 0;
    this->fieldsize = Fieldsize;
    this->fieldsX = solX / Fieldsize;
    this->fieldsY = solY / Fieldsize;   
    Pos tmpos;
 
    this->matrix = (Area**) malloc(this->fieldsX*sizeof(Area*));
    tmpos.X = 0;
    tmpos.Y = 0;
    for(int i = 0; i < this->fieldsX; i++)
    {
        tmpos.Y = 0;
        this->matrix[i] = (Area*) malloc(this->fieldsY*sizeof(Area));
        for(int j = 0; j < this->fieldsY; j++)
        {
            this->matrix[i][j].setID(idgiver);
            idgiver++;
            this->matrix[i][j].setSize(Fieldsize);
            this->matrix[i][j].Setup(tmpos, GRASLAND);
            tmpos.Y += this->fieldsize;
        }
        tmpos.X += this->fieldsize;
        
    }
}
 
World::~World()
{
    free(matrix);
}
 
/*void World::CreateLevelData()
{
    for(int i = 0; i < this->fieldsX; i++)
    {
        for(int j = 0; j < this->fieldsY; j++)
        {
            this->matrix[i][j].Setup();
        }
        
    }
}*/
 
void World::Draw()
{
    for(int i = 0; i < this->fieldsX; i++)
    {
        for(int j = 0; j < this->fieldsY; j++)
        {
            this->matrix[i][j].Draw();
        }
    }
}
 
 
Area::Area()
{
    
}
 
Area::~Area()
{
 
}
 
void Area::Setup(Pos position, int Type)
{
 
    this->pos = position;
    this->type = Type;
    switch(this->type)
    {
    case GRASLAND:
        if(LoadTGA(&this->tex, "gras1.tga"))
        {
            glGenTextures(1, &this->tex.texID);
            glBindTexture(GL_TEXTURE_2D, this->tex.texID);
            glTexImage2D(GL_TEXTURE_2D, 0, this->tex.bpp / 8, this->tex.width, this->tex.height, 0, this->tex.type, GL_UNSIGNED_BYTE, this->tex.imageData);
            glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_LINEAR);
            glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_LINEAR);
 
            if (this->tex.imageData)                        // If Texture Image Exists ( CHANGE )
            {
                free(this->tex.imageData);                  // Free The Texture Image Memory ( CHANGE )
            }
        }
        else
        {
            exit(1);
        }
 
        break;
    default:
        break;
    }
}
 
void Area::Draw()
{
    glColor3ub(255, 255, 255);
 
    glEnable(GL_TEXTURE_2D);
    glBindTexture(GL_TEXTURE_2D, this->tex.texID);
    glBegin(GL_QUADS);
        glTexCoord2i(0,1);/*    glColor3f(1,0,0);*/ glVertex2i(this->pos.X, this->pos.Y+this->size);
        glTexCoord2i(0,0);/*    glColor3f(0,1,0);*/ glVertex2i(this->pos.X, this->pos.Y);
        glTexCoord2i(1,0);/*    glColor3f(0,0,1);*/ glVertex2i(this->pos.X+this->size, this->pos.Y);
        glTexCoord2i(1,1);/*    glColor3f(1,1,0);*/ glVertex2i(this->pos.X+this->size, this->pos.Y+this->size); 
    glEnd();
}
 

In der Hauptschleife rufe ich dann diese Funktion auf:

Code:

 
void Core::InGame()
{
    glClearColor(0.0f, 0.0f, 0.0f, 1.0f);
    glClear(GL_COLOR_BUFFER_BIT);
 
    this->world->Draw();
 
    this->keys = SDL_GetKeyState(NULL);
    if(keys[SDLK_ESCAPE])
    {
        this->done = true;
    }  
}
 

wenn ich in der Area::Draw Funktion glTexCoord2i auskommentiere und stattdessen die Kommentare um glColor3f entferne, erscheinen mir die Quadrate in den Richtigen Farben und auch richtigerweise gegen den Uhrzeigersinn gezeichnet. Die tga Datei liegt auch an der richtigen Stelle, die scheint er auch korrekt zu finden und zu laden. Sieht jemand einen Fehler? Könnte Fehler irgendwo anders im code liegen, in Teilen die ich nicht mitgepostet habe (viel mehr isses nich)?

Bin dankbar für jede Hilfe...

grüße...

Nach oben

Magellan

Betreff des Beitrags:

Verfasst: Mi Mär 22, 2006 18:18

DGL Member

Registriert: Di Nov 26, 2002 22:12
Beiträge: 259
Wohnort: Dresden

Einen Fehler kann ich nicht finden. Allerdings ist es eine ganz schlechte Idee die Textur für jedes Feld neu zu laden und in den Grafikkartenspeicher zu schieben. Es reicht doch völlig die Textur einmal zu laden (oder prüft das die LoadTGA-Funktion?).

Besitzt deine Textur vielleicht nicht die Breite/Höhe von 2^n? Probier' es mal mit einer Textur der entsprechenden Größe (z.B. 128*128).

_________________
Nichts auf der Welt ist so gerecht verteilt wie der Verstand. Denn jederman ist überzeugt, dass er genug davon habe.
Rene Descartes, frz. Mathematiker u. Philosoph, 1596-1650

Nach oben

Shaijan

Betreff des Beitrags:

Verfasst: Mi Mär 22, 2006 18:19

DGL Member

Registriert: Di Sep 06, 2005 18:34
Beiträge: 362
Wohnort: Hamburg

Hi ...

1. Kann es sein, dass deine Texturen evtl nicht power of two größen haben? Normal müssen alle OpenGL-Texturen seitenlänge haben die power of two sind...
2. Evtl ist dein Code zum Laden der Textur falsch. Hast du mal überprüft of LoadTGA(...) wirklich true zurück gibt?

Gruß
Shai

_________________
Der Mensch hat neben dem Trieb der Fortpflanzung und dem zu essen und zu trinken zwei Leidenschaften: Krach zu machen und nicht zuzuhören. (Kurt Tucholsky)
Schwabbeldiwapp, hier kommt die Grütze. (Der Quästor)

Nach oben

Hondo

Betreff des Beitrags:

Verfasst: Mi Mär 22, 2006 18:37

DGL Member

Registriert: Mi Mär 22, 2006 17:39
Beiträge: 7

Mahlzeit!

Geht ja fix hier.

Also die tga sind im Format 32x32px. Sollten auch genau in die Quadrate passen, die sind auch 32x32. Die Funktion LoadTGA überprüft zuerst, ob es sich um ne komprimierte oder ne unkomprimierte Datei handelt und ruft entsprechend LoadCompressedTGA(...) oder LoadUnCompressedTGA(...) auf. Da wird mehrfach überprüft, ob das Öffnen und das Lesen der Datei erfolgreich waren. Falls was schief geht, wird false zurückgegeben. Das Fang ich ab mit

if(LoadTGA(&this->tex, "gras1.tga"))
{
glGenTextures(1, &this->tex.texID);
glBindTexture(GL_TEXTURE_2D, this->tex.texID);
glTexImage2D...
....

}
else
{
exit(1);
}

Ansonsten geh ich davon aus das der Loader funktioniert, er wurde mir mehrfach empfohlen, und die Leute von NeHe (denk ich mal) haben das schon ordentlich funktionierend Programmiert. Wegen mangelden eigenen Kenntnissen in Sachen Dateien von Hand lesen (bin da Perl-verwöhnt ;) kann ich aber auch nicht selber überprüfen, ob das korrekt programmiert ist.

Stimmt, das mit dem mehrfachen Laden ist natürlich blödsinn, das werd ich gleich mal in ne extra Funktion auslagern.

danke schonmal, sieht sonst noch jemand was?

Nach oben

Shaijan

Betreff des Beitrags:

Verfasst: Mi Mär 22, 2006 18:58

DGL Member

Registriert: Di Sep 06, 2005 18:34
Beiträge: 362
Wohnort: Hamburg

Hi ...

bin den Code jetzt nochmal bissl genauer durchgegangen und kann in dem schnipsel jetzt auch keinen Fehler entdecken... versuch mal mitm Debugger durchzusteppen und rauszufinden ob 1. die Texturladefunktion richtig funktioniert und ob deine "Areas" richtig initialisiert werden ...

und überprüf mal ob tex.texID beim glBindTexture einen Wert hat ...

Gruß
Shai

Nach oben

Betreff des Beitrags:

Verfasst: Mi Mär 22, 2006 19:38

DGL Member

Registriert: Sa Jan 22, 2005 21:10
Beiträge: 225

Zitat:

glTexImage2D(GL_TEXTURE_2D, 0, this->tex.bpp / 8, this->tex.width, this->tex.height, 0, this->tex.type, GL_UNSIGNED_BYTE, this->tex.imageData);

Ich glaube der dritte Parameter is falsch: this->tex.bpp / 8
Der heißt internalFormat und gibt an, wie das intern gemanaged werden soll. Probier mal GL_RGB8 oder GL_RGBA8, je nachdem was this->tex.type ist.

Edit: Im Redbook steht, dass die so ausgelegt sind, dass GL_LUMINANCE, GL_LUMINANCE_ALPHA, GL_RGB und GL_RGBA den Zahlen 1, 2, 3 und 4 entsprechen. In sofern müsste es mit this->tex.bpp / 8 doch gehen. Kannst es ja aber trotzdem mal mit GL_RGB / GL_RGB8 bzw. GL_RGBA / GL_RGBA8 versuchen.

_________________
[18:30] tomok: so wie ich das sehe : alles. was nich was anderes ist als nen Essay ist nen Essay

hi, i'm a signature viruz, plz set me as your signature and help me spread

Nach oben

Hondo

Betreff des Beitrags:

Verfasst: Mi Mär 22, 2006 19:56

DGL Member

Registriert: Mi Mär 22, 2006 17:39
Beiträge: 7

Mahlzeit!

Die tex.texID ist 3452816845 (immer die größenordnung). Das sollte in ordnung gehen oder?
tex ist ja eine Objekt der Klasse Textur, die Member width, height, und bpp enthalten nach dem Laden der tga korrekte werte, also 32,32,24. (vielleicht dumme Frage: sollt bpp nicht 32 sein bei Targa?). tex->imageData enthält auch nen ganzen Haufen Daten. Ist es denn richtig diese ImageData gleich wieder mit free freizugeben? Aber auch wenn ich das in den Destruktor der Areas auslagere, bleibt das Fenster weiß.
Also da scheint auch alles in Ordnung zusein.
glBindTexture wie gesagt, bekommt da oben solche IDs.

Die feldmatrix sieht auch gut aus, die Werte werden korrekt initialisiert, ich kann mir die IDs der areas auch in die Konsole ausgeben lassen. Das sieht bis dahin alles sehr gut aus. Wie gesagt, wenn ich statt Texturen Farben auf die Quadrate packe, sieht alles so aus, wie es soll.

Die Texturen sind mit GIMP erstellt, ich hab das jetzt auch in verschiedenen Versionen abgespeichert (komprimiert, unkomprimiert, anfang unten links, ...), das hilft auch nicht.

@AL:

Ich habs probiert, der Compiler akzeptiert GL_RGB8 , aber es bleibt beim weißen Schirm. Gibt es da noch andere Optionen? obwohl das RGB -> 8bit jeweils stimmt ja (ich nehme man an, das sich GL_RGB8 so aufschlüsseln lässt)

jetzt hab ich mal nen Alphakanal in GIMP zur tga zugefügt und GL_RGBA8 eingesetzt -> keine veränderung (aber jetzt ist mir zumindest klar, warum es eben nur 24 in bpp waren, jetzt ist tex->bpp korrekterweiße 32)

*dank und auf weitere Ideen hoff*

Nach oben

Lyr

Betreff des Beitrags:

Verfasst: Mi Mär 22, 2006 20:12

DGL Member

Registriert: So Sep 26, 2004 05:57
Beiträge: 190
Wohnort: Linz

Nur mal als kleine Anmerkung am Rande ohne den Thread jetzt gelesen zu haben:
3452816845 = 0xCDCDCDCD = das wie (beispielsweise) Visual C den Speicher initialisiert

Nach oben

Shaijan

Betreff des Beitrags:

Verfasst: Mi Mär 22, 2006 20:31

DGL Member

Registriert: Di Sep 06, 2005 18:34
Beiträge: 362
Wohnort: Hamburg

Hi ...

hab mir den nehe cide mal angesehen ... hast du den exakt so übernommen?
Mir ist nämlich aufgefallen, dass da ne große Sicherheitslücke drin steckt.
Und zwar wird der Rückgabewert von LoadUncompressedTGA und LoadCpmpressedTGA nicht überprüft:

Code:

 
if(memcmp(uTGAcompare, &tgaheader, sizeof(tgaheader)) == 0)
{
    LoadUncompressedTGA(texture, filename, fTGA);
}
else if(memcmp(cTGAcompare, &tgaheader, sizeof(tgaheader)) == 0)
{
    LoadCompressedTGA(texture, filename, fTGA);
}
else
{
    MessageBox(NULL, "TGA file be type 2 or type 10 ", "Invalid Image", MB_OK); // Display an error
    fclose(fTGA);
    return false;   
}
return true;    
 

D.h. egal ob Uncompressed oder Compressed false oder true zurückgeben gibt LoadTGA true zurück...

Da solltest du lieber vor die beiden funktionen ein return schreiben, damit deren Wert zurückgegeben wird.
Hab mir den Code der beiden nicht genau angesehen und hab auch keine Ahnung von TGAs, weiß also nicht ob die nach den Prüfungen in LoadTGA noch false zurückgeben können, aber evtl liegt es daran ...

Ansonsten kommt mir eine solch große texID schon irgendwie komisch vor ...
Initialisier die texIDs mal auf 0 und schau, ob sich der Wert ändert ...

Gruß
Shai

Nach oben

Hondo

Betreff des Beitrags:

Verfasst: Mi Mär 22, 2006 21:08

DGL Member

Registriert: Mi Mär 22, 2006 17:39
Beiträge: 7

Mahlzeit!

Die ID bleibt tatsächlich 0, wenn ich sie mit 0 initialisiere... da liegt wohl der Hund begraben...

Wenn ich bei dem tga Loader die return hinzufüge, entstehen trotzdem keine Compiler- oder Laufzeitfehler.

Woher kann das kommen das die ID 0 ist? Die anderen Werte werden ja anscheinend richtig initialisiert....

Ich hab grade mal die Datei TGALoader.cpp nach dem Begriff texID durchsucht. Der kommt dort drinnen nicht vor... Haben die vielleicht vergessen, der Textur nach dem laden ne ID zu verpassen? Wie macht man das, b.z.w. an welcher Stelle bei OpenGL geschieht das?

grüße...

Nach oben

Hondo

Betreff des Beitrags:

Verfasst: Mi Mär 22, 2006 21:17

DGL Member

Registriert: Mi Mär 22, 2006 17:39
Beiträge: 7

achso, ich seh grad, dass das bei glGenTextures geschieht. na jedenfalls bleibt die ID 0...

Nach oben

Shaijan

Betreff des Beitrags:

Verfasst: Mi Mär 22, 2006 21:48

DGL Member

Registriert: Di Sep 06, 2005 18:34
Beiträge: 362
Wohnort: Hamburg

Hi ...

So langsam bin ich überfragt

Das einzige was mir noch einfällt ist die möglichkeit, dass er garnicht bis zum glGenTextures kommt (aus irgend einem grund, nicht weil LoadTGA false zurück gibt)
Überprüf mal ob LoadTGA überhaupt aufgerufen wird ...

Gruß
Shai

Nach oben

Lyr

Betreff des Beitrags:

Verfasst: Mi Mär 22, 2006 22:29

DGL Member

Registriert: So Sep 26, 2004 05:57
Beiträge: 190
Wohnort: Linz

Habs jetzt noch mal etwas genauer überflogen. Wäre es möglich, dass du zum Zeitpunkt wo du Area::Setup aufrufst (was ja im World::World erfolgt) OpenGL noch nicht initialisiert hast?

Nach oben

Shaijan

Betreff des Beitrags:

Verfasst: Mi Mär 22, 2006 22:35

DGL Member

Registriert: Di Sep 06, 2005 18:34
Beiträge: 362
Wohnort: Hamburg

Ich gehe mal davon aus, dass du opengl statisch gelinkt hast, also wird es initialisiert sein ... und wenn nicht würds 100% n laufzeitfehler geben ...

Nach oben

Lyr

Betreff des Beitrags:

Verfasst: Do Mär 23, 2006 02:36

DGL Member

Registriert: So Sep 26, 2004 05:57
Beiträge: 190
Wohnort: Linz

Sorry, aber statisches linken hat wenig mit Initialisierung zu tun. Wenn kein OpenGL-Kontext vorhanden ist (wo Bittiefe und andere Parameter welche nicht durchs linken bekannt sein können erforderlich sind) dann werden auch keine Texturen erstellt und es werden in diesem Fall auch keine Laufzeitfehler auftreten.

Nach oben

Seite 1 von 2

[ 21 Beiträge ]

Gehe zu Seite 1, 2 Nächste

Foren-Übersicht » Programmierung » OpenGL

Wer ist online?

Mitglieder in diesem Forum: 0 Mitglieder und 7 Gäste

Du darfst keine neuen Themen in diesem Forum erstellen.
Du darfst keine Antworten zu Themen in diesem Forum erstellen.
Du darfst deine Beiträge in diesem Forum nicht ändern.
Du darfst deine Beiträge in diesem Forum nicht löschen.
Du darfst keine Dateianhänge in diesem Forum erstellen.