Jump to content
  • Advertisement
Sign in to follow this  
Butter

OpenGL Texture Binding with Material Class

This topic is 4679 days old which is more than the 365 day threshold we allow for new replies. Please post a new topic.

If you intended to correct an error in the post then please contact us.

Recommended Posts

Im having a problem binding textures that I am loading from an .obj Material library. My texture loading code works if I load each texture individually, then all textures work great. But if a load the textures while parsing the Material Library file only the last texture loaded can be rendered. I have written a smaller program to try to get to the root of the problem. Basically, an instance of my Material class will load a texture and store the texture ID so that it can be bound when rendering. Any suggestions? Can anyone tell what I am doing wrong? Thanks for any help. Main.cpp:
#include <windows.h>		// Header File For Windows
#include <stdio.h>			// Header File For Standard Input/Output
#include <gl\gl.h>			// Header File For The OpenGL32 Library
#include <gl\glu.h>			// Header File For The GLu32 Library
#include <gl\glaux.h>		// Header File For The Glaux Library

#include <ios>
#include <iostream>
#include <string>
#include <tchar.h>
#include <windowsx.h>
#include <fstream>

#include <vector>

#include "Material.h"

using namespace std;

HDC			hDC=NULL;		// Private GDI Device Context
HGLRC		hRC=NULL;		// Permanent Rendering Context
HWND		hWnd=NULL;		// Holds Our Window Handle
HINSTANCE	hInstance;		// Holds The Instance Of The Application

bool	active=TRUE;		// Window Active Flag Set To TRUE By Default

// Log file
ofstream logFile("C:\\Documents and Settings\\Dave\\My Documents\\Visual Studio Projects\\TextureManager\\Debug\\log.txt", ios::out);

// Materials Container
std::vector<Material> materials;

LRESULT	CALLBACK WndProc(HWND, UINT, WPARAM, LPARAM);	// Declaration For WndProc

// Number of textures loaded
int numTexture = 0;

void checkGlErrors() {
	int glError = 1;
	while (glError != 0) {
		glError = glGetError();
		if (glError != 0) {
			logFile << "[ERROR] glGetError = " << glError << endl;
		}
	}
}

// Call to load a Bitmap
int loadBitmap(const char *filename) 
{
	logFile << "	loadBitmap called: file name = " << filename << endl;
    FILE *file;				// File pointer
    unsigned char *texture; // The pointer to the memory in which we will load the texture
     
    // windows.h gives us these types to work with the Bitmap files
    BITMAPFILEHEADER fileheader; 
    BITMAPINFOHEADER infoheader;
    RGBTRIPLE rgb;

	// Increment the texture count
    numTexture++; 
	logFile << "		numTexture = " << numTexture << endl;

	// Open the file for reading
	if ( (file = fopen(filename, "rb")) == NULL) {
		logFile << "		COULDN'T OPEN FILE" << endl;
		return (-1); 
	} else {
		logFile << "		The file has been opened for reading." << endl;
	}
    
	// Read the fileheader
    fread(&fileheader, sizeof(fileheader), 1, file); 
    
	// Jump the fileheader
    fseek(file, sizeof(fileheader), SEEK_SET); 
	// and read the infoheader
    fread(&infoheader, sizeof(infoheader), 1, file); 

    // Allocate the memory for our image (width * height * color deep)
    texture = (byte *) malloc(infoheader.biWidth * infoheader.biHeight * 4);
    // And fill it with zeros
    memset(texture, 0, infoheader.biWidth * infoheader.biHeight * 4);
 
    // Read in the image
	int j = 0;
    for (int i = 0; i < infoheader.biWidth*infoheader.biHeight; ++i) {            
        // We load an RGB value from the file
        fread(&rgb, sizeof(rgb), 1, file); 

        // And store it
        texture[j+0] = rgb.rgbtRed;		// Red component
        texture[j+1] = rgb.rgbtGreen;	// Green component
        texture[j+2] = rgb.rgbtBlue;	// Blue component
        texture[j+3] = 255;				// Alpha value
        j += 4;							// Go to the next position
    }

	// Close the file stream
    fclose(file); 
     
	// Bind the ID texture
    glBindTexture(GL_TEXTURE_2D, numTexture); 

	// Define the 2d texture
    glTexImage2D(GL_TEXTURE_2D, 0, 4, infoheader.biWidth, infoheader.biHeight, 0, GL_RGBA, GL_UNSIGNED_BYTE, texture);

    // Sets the texture parameters
    glTexParameterf(GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, GL_REPEAT); 
    glTexParameterf(GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, GL_REPEAT);
    glTexParameterf(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_LINEAR); 
    glTexParameterf(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_LINEAR_MIPMAP_NEAREST);

	// Use only the texture map
    glTexEnvf(GL_TEXTURE_ENV, GL_TEXTURE_ENV_MODE, GL_REPLACE); 

    // And create 2d mipmaps for the minifying function
    gluBuild2DMipmaps(GL_TEXTURE_2D, 4, infoheader.biWidth, infoheader.biHeight, GL_RGBA, GL_UNSIGNED_BYTE, texture);

	// Free the memory used to load the texture
    free(texture); 

	checkGlErrors();

	// Returns texture ID for the new texture
	int isTexture = glIsTexture(numTexture);
	logFile << "		is a texture ? = " << isTexture << endl;
    return (numTexture); 
}

// Load an Alias Wavefront .obj Material Library
bool loadMaterialFile(const char filename[]) {

	std::ifstream ifs(filename);

	if (ifs) {
		Material *pmat = 0;

		while (!ifs.eof()) {

			std::string s;
			ifs >> s;

			if (s == "newmtl") {
				Material mat;
				ifs >> mat.name;
				materials.push_back(mat);
				pmat = &(materials[materials.size()-1]);

			} else if (s == "map_Kd") {

				char buffer[512];
				char* ptr = buffer;
				ifs.getline(buffer,512);
				while (*ptr == ' ' || *ptr == '\t') {
					++ptr;
				}
				
				pmat->diffuseTextureName = ptr;
				pmat->loadDiffuseTexture();
			}
		}
		ifs.close();
		return true;
	}
	logFile << "File not Found!!!" << endl;

	return false;
}

// Resize the GL Scene
GLvoid ReSizeGLScene(GLsizei width, GLsizei height)		
{
	// Prevent A Divide By Zero By
	if (height==0)										
	{
		height=1;
	}

	// Reset The Current Viewport
	glViewport(0,0,width,height);						

	// Select and Reset The Projection Matrix
	glMatrixMode(GL_PROJECTION);						
	glLoadIdentity();									

	// Calculate The Aspect Ratio Of The Window
	gluPerspective(75.0f,(GLfloat)width/(GLfloat)height,0.1f,100.0f);

	// Select and Reset The Modelview Matrix
	glMatrixMode(GL_MODELVIEW);
	glLoadIdentity();
}

// All Setup For OpenGL Goes Here
int InitGL(GLvoid)										
{
	glEnable(GL_TEXTURE_2D);							// Enable Texture Mapping ( NEW )
	glShadeModel(GL_SMOOTH);							// Enable Smooth Shading
	glClearColor(0.0f, 0.0f, 0.0f, 0.5f);				// Black Background
	glClearDepth(1.0f);									// Depth Buffer Setup
	glEnable(GL_DEPTH_TEST);							// Enables Depth Testing
	glDepthFunc(GL_LEQUAL);								// The Type Of Depth Testing To Do
	glHint(GL_PERSPECTIVE_CORRECTION_HINT, GL_NICEST);	// Really Nice Perspective Calculations

	// load material file
	if (!loadMaterialFile("squares2.mtl")) {
		logFile << "Materials NOT loaded." << endl;
	} else {
    	logFile << "Materials loaded - materials.size() = " << materials.size() << endl;
	}

	// Print out the loaded materials props of interest
	for (int i = 0; i < materials.size(); ++i) {
		logFile << "Material " << i << ":" << endl;
		logFile << "    name = " << materials.name << endl;
		logFile << "    diffuseTextureName = " << materials.diffuseTextureName << endl;
		logFile << "    diffuseTextureId = " << materials.diffuseTextureId << endl;
		int isTexture = glIsTexture(materials.diffuseTextureId);
		logFile << "    isTexture? = " << isTexture << endl;
	}

	// Check for errors
	checkGlErrors();

	// Initialization completed OK
	return TRUE;	
}

// Draw the scene
// Render 4 quads - each with a different Material
int DrawGLScene(GLvoid)	
{
	checkGlErrors();

	// Clear The Screen And The Depth Buffer and Reset The View
	glClear(GL_COLOR_BUFFER_BIT | GL_DEPTH_BUFFER_BIT);
	glLoadIdentity();

	// move the camera a little
	glTranslatef(0.0f,0.0f,-5.0f);

	int isTexture = glIsTexture(materials[0].diffuseTextureId);
	logFile << "  glIsTexture? materials[0] = " << isTexture << endl;
	materials[0].apply();
	glBegin(GL_QUADS);
		glTexCoord2f(0.0f, 0.0f); glVertex3f(-1.0f, -1.0f,  0.0f);
		glTexCoord2f(1.0f, 0.0f); glVertex3f( 0.0f, -1.0f,  0.0f);
		glTexCoord2f(1.0f, 1.0f); glVertex3f( 0.0f,  0.0f,  0.0f);
		glTexCoord2f(0.0f, 1.0f); glVertex3f(-1.0f,  0.0f,  0.0f);
	glEnd();

	isTexture = glIsTexture(materials[1].diffuseTextureId);
	logFile << "  glIsTexture? materials[1] = " << isTexture << endl;
	materials[1].apply();
	glBegin(GL_QUADS);
		glTexCoord2f(0.0f, 0.0f); glVertex3f(0.0f, 0.0f,  1.0f);
		glTexCoord2f(1.0f, 0.0f); glVertex3f( 1.0f, 0.0f,  1.0f);
		glTexCoord2f(1.0f, 1.0f); glVertex3f( 1.0f,  1.0f,  1.0f);
		glTexCoord2f(0.0f, 1.0f); glVertex3f(0.0f,  1.0f,  1.0f);		
	glEnd();

	return TRUE;
}

// Properly Kill The Window
GLvoid KillGLWindow(GLvoid)								
{
	// Do We Have A Rendering Context?
	if (hRC)											
	{
		// Are We Able To Release The DC And RC Contexts?
		if (!wglMakeCurrent(NULL,NULL))					
		{
			MessageBox(NULL,"Release Of DC And RC Failed.","SHUTDOWN ERROR",MB_OK | MB_ICONINFORMATION);
		}

		// Are We Able To Delete The RC?
		if (!wglDeleteContext(hRC))						
		{
			MessageBox(NULL,"Release Rendering Context Failed.","SHUTDOWN ERROR",MB_OK | MB_ICONINFORMATION);
		}
		hRC = NULL;
	}
	// Are We Able To Release The DC
	if (hDC && !ReleaseDC(hWnd,hDC))					
	{
		MessageBox(NULL,"Release Device Context Failed.","SHUTDOWN ERROR",MB_OK | MB_ICONINFORMATION);
		hDC = NULL;
	}
	// Are We Able To Destroy The Window?
	if (hWnd && !DestroyWindow(hWnd))					
	{
		MessageBox(NULL,"Could Not Release hWnd.","SHUTDOWN ERROR",MB_OK | MB_ICONINFORMATION);
		hWnd = NULL;
	}
	// Are We Able To Unregister Class
	if (!UnregisterClass("OpenGL",hInstance))			
	{
		MessageBox(NULL,"Could Not Unregister Class.","SHUTDOWN ERROR",MB_OK | MB_ICONINFORMATION);
		hInstance = NULL;
	}
}

// Create an OpenGL Window 
BOOL CreateGLWindow(char* title, int width, int height, int bits, bool fullscreenflag)
{

	logFile << "CreateGLWindow called." << endl;

	GLuint		PixelFormat;			// Holds The Results After Searching For A Match
	WNDCLASS	wc;						// Windows Class Structure
	DWORD		dwExstyle;				// Window Extended style
	DWORD		dwstyle;				// Window style
	RECT		WindowRect;				// Grabs Rectangle Upper Left / Lower Right Values
	WindowRect.left=(long)0;			// Set Left Value To 0
	WindowRect.right=(long)width;		// Set Right Value To Requested Width
	WindowRect.top=(long)0;				// Set Top Value To 0
	WindowRect.bottom=(long)height;		// Set Bottom Value To Requested Height

	hInstance			= GetModuleHandle(NULL);				// Grab An Instance For Our Window
	wc.style			= CS_HREDRAW | CS_VREDRAW | CS_OWNDC;	// Redraw On Size, And Own DC For Window.
	wc.lpfnWndProc		= (WNDPROC) WndProc;					// WndProc Handles Messages
	wc.cbClsExtra		= 0;									// No Extra Window Data
	wc.cbWndExtra		= 0;									// No Extra Window Data
	wc.hInstance		= hInstance;							// Set The Instance
	wc.hIcon			= LoadIcon(NULL, IDI_WINLOGO);			// Load The Default Icon
	wc.hCursor			= LoadCursor(NULL, IDC_ARROW);			// Load The Arrow Pointer
	wc.hbrBackground	= NULL;									// No Background Required For GL
	wc.lpszMenuName		= NULL;									// We Don't Want A Menu
	wc.lpszClassName	= "OpenGL";								// Set The Class Name

	if (!RegisterClass(&wc))									// Attempt To Register The Window Class
	{
		MessageBox(NULL,"Failed To Register The Window Class.","ERROR",MB_OK|MB_ICONEXCLAMATION);
		return FALSE;											// Return FALSE
	}

	dwExstyle=WS_EX_APPWINDOW | WS_EX_WINDOWEDGE;			// Window Extended style
	dwstyle=WS_OVERLAPPEDWINDOW;							// Windows style

	AdjustWindowRectEx(&WindowRect, dwstyle, FALSE, dwExstyle);		// Adjust Window To True Requested Size

	// Create The Window
	if (!(hWnd=CreateWindowEx(	dwExstyle,							// Extended style For The Window
								"OpenGL",							// Class Name
								title,								// Window Title
								dwstyle |							// Defined Window style
								WS_CLIPSIBLINGS |					// Required Window style
								WS_CLIPCHILDREN,					// Required Window style
								0, 0,								// Window Position
								WindowRect.right-WindowRect.left,	// Calculate Window Width
								WindowRect.bottom-WindowRect.top,	// Calculate Window Height
								NULL,								// No Parent Window
								NULL,								// No Menu
								hInstance,							// Instance
								NULL)))								// Dont Pass Anything To WM_CREATE
	{
		KillGLWindow();								// Reset The Display
		MessageBox(NULL,"Window Creation Error.","ERROR",MB_OK|MB_ICONEXCLAMATION);
		return FALSE;								// Return FALSE
	}

	static	PIXELFORMATDESCRIPTOR pfd=				// pfd Tells Windows How We Want Things To Be
	{
		sizeof(PIXELFORMATDESCRIPTOR),				// Size Of This Pixel Format Descriptor
		1,											// Version Number
		PFD_DRAW_TO_WINDOW |						// Format Must Support Window
		PFD_SUPPORT_OPENGL |						// Format Must Support OpenGL
		PFD_DOUBLEBUFFER,							// Must Support Double Buffering
		PFD_TYPE_RGBA,								// Request An RGBA Format
		bits,										// Select Our Color Depth
		0, 0, 0, 0, 0, 0,							// Color Bits Ignored
		0,											// No Alpha Buffer
		0,											// Shift Bit Ignored
		0,											// No Accumulation Buffer
		0, 0, 0, 0,									// Accumulation Bits Ignored
		16,											// 16Bit Z-Buffer (Depth Buffer)  
		0,											// No Stencil Buffer
		0,											// No Auxiliary Buffer
		PFD_MAIN_PLANE,								// Main Drawing Layer
		0,											// Reserved
		0, 0, 0										// Layer Masks Ignored
	};
	
	if (!(hDC=GetDC(hWnd)))							// Did We Get A Device Context?
	{
		KillGLWindow();								// Reset The Display
		MessageBox(NULL,"Can't Create A GL Device Context.","ERROR",MB_OK|MB_ICONEXCLAMATION);
		return FALSE;								// Return FALSE
	}

	if (!(PixelFormat=ChoosePixelFormat(hDC,&pfd)))	// Did Windows Find A Matching Pixel Format?
	{
		KillGLWindow();								// Reset The Display
		MessageBox(NULL,"Can't Find A Suitable PixelFormat.","ERROR",MB_OK|MB_ICONEXCLAMATION);
		return FALSE;								// Return FALSE
	}

	if(!SetPixelFormat(hDC,PixelFormat,&pfd))		// Are We Able To Set The Pixel Format?
	{
		KillGLWindow();								// Reset The Display
		MessageBox(NULL,"Can't Set The PixelFormat.","ERROR",MB_OK|MB_ICONEXCLAMATION);
		return FALSE;								// Return FALSE
	}

	if (!(hRC=wglCreateContext(hDC)))				// Are We Able To Get A Rendering Context?
	{
		KillGLWindow();								// Reset The Display
		MessageBox(NULL,"Can't Create A GL Rendering Context.","ERROR",MB_OK|MB_ICONEXCLAMATION);
		return FALSE;								// Return FALSE
	}

	if(!wglMakeCurrent(hDC,hRC))					// Try To Activate The Rendering Context
	{
		KillGLWindow();								// Reset The Display
		MessageBox(NULL,"Can't Activate The GL Rendering Context.","ERROR",MB_OK|MB_ICONEXCLAMATION);
		return FALSE;								// Return FALSE
	}

	ShowWindow(hWnd,SW_SHOW);						// Show The Window
	SetForegroundWindow(hWnd);						// Slightly Higher Priority
	SetFocus(hWnd);									// Sets Keyboard Focus To The Window
	ReSizeGLScene(width, height);					// Set Up Our Perspective GL Screen

	if (!InitGL())									// Initialize Our Newly Created GL Window
	{
		KillGLWindow();								// Reset The Display
		MessageBox(NULL,"Initialization Failed.","ERROR",MB_OK|MB_ICONEXCLAMATION);
		return FALSE;								// Return FALSE
	}

	return TRUE;									// Success
}

LRESULT CALLBACK WndProc(	HWND	hWnd,			// Handle For This Window
							UINT	uMsg,			// Message For This Window
							WPARAM	wParam,			// Additional Message Information
							LPARAM	lParam)			// Additional Message Information
{
	switch (uMsg)									// Check For Windows Messages
	{
		case WM_ACTIVATE:							// Watch For Window Activate Message
		{
			if (!HIWORD(wParam))					// Check Minimization State
			{
				active=TRUE;						// Program Is Active
			}
			else
			{
				active=FALSE;						// Program Is No Longer Active
			}

			return 0;								// Return To The Message Loop
		}

		case WM_SYSCOMMAND:							// Intercept System Commands
		{
			switch (wParam)							// Check System Calls
			{
				case SC_SCREENSAVE:					// Screensaver Trying To Start?
				case SC_MONITORPOWER:				// Monitor Trying To Enter Powersave?
				return 0;							// Prevent From Happening
			}
			break;									// Exit
		}

		case WM_CLOSE:								// Did We Receive A Close Message?
		{
			PostQuitMessage(0);						// Send A Quit Message
			return 0;								// Jump Back
		}

		case WM_SIZE:								// Resize The OpenGL Window
		{
			ReSizeGLScene(LOWORD(lParam),HIWORD(lParam));  // LoWord=Width, HiWord=Height
			return 0;								// Jump Back
		}
	}

	// Pass All Unhandled Messages To DefWindowProc
	return DefWindowProc(hWnd,uMsg,wParam,lParam);
}

int WINAPI WinMain(	HINSTANCE	hInstance,			// Instance
					HINSTANCE	hPrevInstance,		// Previous Instance
					LPSTR		lpCmdLine,			// Command Line Parameters
					int			nCmdShow)			// Window Show State
{
	MSG		msg;									// Windows Message Structure
	BOOL	done=FALSE;								// Bool Variable To Exit Loop

	// Create Our OpenGL Window
	if (!CreateGLWindow("Materials", 640, 480, 16, false))
	{
		// Quit If Window Was Not Created
		return 0;									
	}

	// main loop
	while (!done)									
	{
		// Is There A Message Waiting?
		if (PeekMessage(&msg, NULL, 0, 0, PM_REMOVE))	
		{
			// Have We Received A Quit Message?
			if (msg.message==WM_QUIT)				
			{
				// If So done=TRUE
				done=TRUE;							
			}
			else
			{
				// Translate and Dispatch the message
				TranslateMessage(&msg);
				DispatchMessage(&msg);
			}
		}
		else
		{
			// Draw The Scene.
			if ((active && !DrawGLScene()))
			{
				// Quit signaled
				done = TRUE;
			}
			else
			{
				// Swap Buffers (Double Buffering)
				SwapBuffers(hDC);					
			}
		}
	}

	// Shutdown
	KillGLWindow();									// Kill The Window
	return (msg.wParam);							// Exit The Program
}


End Main.cpp Material.h:
#ifndef MATERIAL_H
#define MATERIAL_H

#include <stdio.h>
#include <fstream>
#include <ios>
#include <iostream>
#include <string>

//#include "Engine.h"
#include <windows.h>		// Header File For Windows
#include <stdio.h>			// Header File For Standard Input/Output
#include <gl\gl.h>			// Header File For The OpenGL32 Library
#include <gl\glu.h>			// Header File For The GLu32 Library
#include <gl\glaux.h>		// Header File For The Glaux Library

using namespace std;


class Material
{
public:
	Material();
	Material(const char *name);
	Material(const Material &mat);

	virtual ~Material();

	void loadDiffuseTexture();

	/** apply the material */
	void apply() const;

	std::string name;		//< material name

	int illum;				//< don't know :| Seems to always be 4
	float transparency[3];	//< transparency
	float intensity;		//< intensity
	float specularPower;	//< specular power
	float bumpDepth;		//< bump map depth. Only used if bump is relevent.

	mutable float ambientColor[4];	//< ambient
	mutable float diffuseColor[4];	//< diffuse
	mutable float specularColor[4];	//< specular

	std::string ambientTextureName;		//< ambient texture map name
	std::string diffuseTextureName;		//< diffuse texture map name
	std::string specularTextureName;	//< specular texture map name
	std::string bumpTextureName;		//< bump texture map name

	// OpenGL texture IDs
	unsigned int ambientTextureId;		//< ambient texture object ID
	unsigned int diffuseTextureId;		//< diffuse texture object ID
	unsigned int specularTextureId;		//< specular texture object ID
	unsigned int bumpTextureId;			//< bump map texture object ID

	
private:
};

#endif


End Material.h Material.cpp:
#include "Material.h"

#include "TextureManager.h"
#include "Globals.h"

//#include "lesson6.h"

/**
 * Material constructor
 */
Material::Material() : name(), illum(4), intensity(1), specularPower(10), bumpDepth(1),
		ambientTextureName(), diffuseTextureName(), specularTextureName(), bumpTextureName() {

	ambientColor[0] = ambientColor[1] = ambientColor[2] = 
	diffuseColor[0] = diffuseColor[1] = diffuseColor[2] = 
	specularColor[0] = specularColor[1] = specularColor[2] = 0;
	ambientColor[3] = diffuseColor[3] = specularColor[3] = 1;
	transparency[0] = transparency[1] = transparency[2] = 1;
	ambientTextureId = diffuseTextureId = specularTextureId = bumpTextureId = 0;
}

/**
 * Material Copy Constructor
 */
Material::Material(const Material& mat) {
	ambientColor[0] = mat.ambientColor[0]; 
	ambientColor[1] = mat.ambientColor[1]; 
	ambientColor[2] = mat.ambientColor[2]; 
	ambientColor[3] = mat.ambientColor[3];

	diffuseColor[0] = mat.diffuseColor[0]; 
	diffuseColor[1] = mat.diffuseColor[1]; 
	diffuseColor[2] = mat.diffuseColor[2]; 
	diffuseColor[3] = mat.diffuseColor[3];

	specularColor[0] = mat.specularColor[0];
	specularColor[1] = mat.specularColor[1];
	specularColor[2] = mat.specularColor[2];
	specularColor[3] = mat.specularColor[3];

	transparency[0] = mat.transparency[0];
	transparency[1] = mat.transparency[1];
	transparency[2] = mat.transparency[2];

	intensity = mat.intensity;
	specularPower = mat.specularPower;
	name = mat.name;
	ambientTextureName = mat.ambientTextureName;
	diffuseTextureName = mat.diffuseTextureName;
	specularTextureName = mat.specularTextureName;
	bumpTextureName = mat.bumpTextureName;
	illum = mat.illum;
	bumpDepth = mat.bumpDepth;

	ambientTextureId = mat.ambientTextureId;
	diffuseTextureId = mat.diffuseTextureId;
	specularTextureId = mat.specularTextureId;
	bumpTextureId = mat.bumpTextureId;
}

/** 
 * The material class destructor.
 */
Material::~Material() {
	if (ambientTextureId) glDeleteTextures(1, &ambientTextureId);
	if (diffuseTextureId) glDeleteTextures(1, &diffuseTextureId);
	if (specularTextureId) glDeleteTextures(1, &specularTextureId);
	if (bumpTextureId) glDeleteTextures(1, &bumpTextureId);
}

/**
 *
 */
void Material::apply() const {

	// HACK !?!??
	glColor3f(1.0f, 1.0f, 1.0f);

	float average_transp = (transparency[0]+transparency[1]+transparency[2])/3.0f;
	ambientColor[3] = diffuseColor[3] = specularColor[3] = average_transp;
	glMaterialfv(GL_FRONT_AND_BACK, GL_AMBIENT, ambientColor);
	glMaterialfv(GL_FRONT_AND_BACK, GL_DIFFUSE, diffuseColor);
	glMaterialfv(GL_FRONT_AND_BACK, GL_SPECULAR, specularColor);
	glMaterialf(GL_FRONT_AND_BACK, GL_SHININESS, specularPower);

	if (diffuseTextureId) {
		// maya always sets the diffuse color to black when a texture is applied.
		float color[4] = {0.8f, 0.8f, 0.8f, average_transp};
		glMaterialfv(GL_FRONT_AND_BACK, GL_DIFFUSE, color);

		// apply texture
		glBindTexture(GL_TEXTURE_2D, diffuseTextureId);

		// Enable texturing
		glEnable(GL_TEXTURE_2D);
	} else {
		glDisable(GL_TEXTURE_2D);
	}
}

/**
 * Loads a bitmap and converts it to a texture.
 * @return The new texture's ID.
 */
void Material::loadDiffuseTexture() {
	diffuseTextureId = loadBitmap(diffuseTextureName.c_str());
}


End Material.cpp Material Library file: newmtl grass illum 4 Kd 0.00 0.00 0.00 Ka 0.00 0.00 0.00 Tf 1.00 1.00 1.00 map_Kd grass.bmp Ni 1.00 Ks 0.33 0.33 0.33 Ns 0.06 newmtl Brick_dark_grout illum 4 Kd 0.00 0.00 0.00 Ka 0.00 0.00 0.00 Tf 1.00 1.00 1.00 map_Kd Brick_dark_grout.bmp Ni 1.00 Ks 0.33 0.33 0.33 Ns 0.06 end Material Library file [edit: added source tags -SiCrane]

Share this post


Link to post
Share on other sites
Advertisement
You need to call glGenTextures. You can't just assume gl will allow you to use sequential texture IDs. If you replace your numTexture stuff with glGenTextures it should work OK, I think.

Share this post


Link to post
Share on other sites
Here is the log file:

CreateGLWindow called.
loadBitmap called: file name = grass.bmp
numTexture = 1
The file has been opened for reading.
is a texture ? = 1
loadBitmap called: file name = Brick_dark_grout.bmp
numTexture = 2
The file has been opened for reading.
is a texture ? = 1
Materials loaded - materials.size() = 2
Material 0:
name = grass
diffuseTextureName = grass.bmp
diffuseTextureId = 1
isTexture? = 0
Material 1:
name = Brick_dark_grout
diffuseTextureName = Brick_dark_grout.bmp
diffuseTextureId = 2
isTexture? = 1
glIsTexture? materials[0] = 0
glIsTexture? materials[1] = 1
glIsTexture? materials[0] = 0
glIsTexture? materials[1] = 1
. . .

Share this post


Link to post
Share on other sites
mrb.

I don't think that that is the problem. If I change the code that loads the textures (init) to the following, all the textures work:
int InitGL(GLvoid)
{
glEnable(GL_TEXTURE_2D); // Enable Texture Mapping ( NEW )
glShadeModel(GL_SMOOTH); // Enable Smooth Shading
glClearColor(0.0f, 0.0f, 0.0f, 0.5f); // Black Background
glClearDepth(1.0f); // Depth Buffer Setup
glEnable(GL_DEPTH_TEST); // Enables Depth Testing
glDepthFunc(GL_LEQUAL); // The Type Of Depth Testing To Do
glHint(GL_PERSPECTIVE_CORRECTION_HINT, GL_NICEST); // Really Nice Perspective Calculations

//// load material file
//if (!loadMaterialFile("squares2.mtl")) {
// logFile << "Materials NOT loaded." << endl;
//} else {
// logFile << "Materials loaded - materials.size() = " << materials.size() << endl;
//}

Material mat1;
Material mat2;
materials.push_back(mat1);
materials.push_back(mat2);
materials[0].diffuseTextureId = loadBitmap("Grass.bmp");
materials[1].diffuseTextureId = loadBitmap("Brick_dark_grout.bmp");


// Print out the loaded materials props of interest
for (int i = 0; i < materials.size(); ++i) {
logFile << "Material " << i << ":" << endl;
logFile << " name = " << materials.name << endl;
logFile << " diffuseTextureName = " << materials.diffuseTextureName << endl;
logFile << " diffuseTextureId = " << materials.diffuseTextureId << endl;
int isTexture = glIsTexture(materials.diffuseTextureId);
logFile << " isTexture? = " << isTexture << endl;
}

// Check for errors
checkGlErrors();

// Initialization completed OK
return TRUE;
}

The difference is that I am explicitly calling the loadBitmap method for each texture instead of loading the Material library. I don't know why the texture IDs would be valid when I call loadBitmap() directly and not when I call it from loadMaterialFile() ???

I tried to use glGenTextures() to give me valid texture IDs like so:

GLunit name[1];
glGenTexture(1, name);

But, with this, each call to loadBitmap() returns the same ID! The Texture doesn't really get bound to it for some reason.

Share this post


Link to post
Share on other sites
Well, I can't see exactly why it's happening, unless the vector is resizing itself after a single push_back, but I'm almost certain your problem is the destructor for Material. When you push_back an element into a vector the vector stores a copy, which means the following sequence of operations occurs:
Material::Material(); // your default constructed material object
Material::Material(Material const &); // the vector copy constructs your object into storage
Material::~Material(); // your material object leaves scope and is destructed

The last of those calls will result in you calling glDeleteTextures for any set textures ids. None of your texture ids should be set in your object, but if the vector has to further copy them then glDeleteTextures will be called and your texture object will be deleted. To solve this you'll probably need to store a reference count in your Material object or work with (smart) pointers to Materials or any other equivalent technique.

By the way, this code:
char buffer[512];
char* ptr = buffer;
ifs.getline(buffer,512);
while (*ptr == ' ' || *ptr == '\t') {
++ptr;
}
pmat->diffuseTextureName = ptr;

would be better implemented as:
std::getline(ifs, pmat->diffuseTextureName);
std::string::size_type whitespace = pmat->diffuseTextureName.find_first_not_of(" \t");
if (whitespace != std::string::npos)
{
pmat->diffuseTextureName = pmat->diffuseTextureName.substr(whitespace);
}

Enigma

Share this post


Link to post
Share on other sites
enigma,

You are right - I will have to re-write the Material class to handle this, or use a different container. Thanks for your help.

Share this post


Link to post
Share on other sites
Sign in to follow this  

  • Advertisement
  • Advertisement
  • Popular Tags

  • Similar Content

    • By nOoNEE
      hello guys , i have some questions  what does glLinkProgram  and  glBindAttribLocation do?  i searched but there wasnt any good resource 
    • By owenjr
      Hi, I'm a Multimedia Engineering student. I am about to finish my dergree and I'm already thinking about what topic to cover in my final college project.
      I'm interested in the procedural animation with c++ and OpenGL of creatures, something like a spider for example. Can someone tell me what are the issues I should investigate to carry it out? I understand that it has some dependence on artificial intelligence but I do not know to what extent. Can someone help me to find information about it? Thank you very much.
       
      Examples: 
      - Procedural multi-legged walking animation
      - Procedural Locomotion of Multi-Legged Characters in Dynamic Environments
    • By Lewa
      So, i'm still on my quest to unterstanding the intricacies of HDR and implementing this into my engine. Currently i'm at the step to implementing tonemapping. I stumbled upon this blogposts:
      http://filmicworlds.com/blog/filmic-tonemapping-operators/
      http://frictionalgames.blogspot.com/2012/09/tech-feature-hdr-lightning.html
      and tried to implement some of those mentioned tonemapping methods into my postprocessing shader.
      The issue is that none of them creates the same results as shown in the blogpost which definitely has to do with the initial range in which the values are stored in the HDR buffer. For simplicity sake i store the values between 0 and 1 in the HDR buffer (ambient light is 0.3, directional light is 0.7)
      This is the tonemapping code:
      vec3 Uncharted2Tonemap(vec3 x) { float A = 0.15; float B = 0.50; float C = 0.10; float D = 0.20; float E = 0.02; float F = 0.30; return ((x*(A*x+C*B)+D*E)/(x*(A*x+B)+D*F))-E/F; } This is without the uncharted tonemapping:
      This is with the uncharted tonemapping:
      Which makes the image a lot darker.
      The shader code looks like this:
      void main() { vec3 color = texture2D(texture_diffuse, vTexcoord).rgb; color = Uncharted2Tonemap(color); //gamma correction (use only if not done in tonemapping code) color = gammaCorrection(color); outputF = vec4(color,1.0f); } Now, from my understanding is that tonemapping should bring the range down from HDR to 0-1.
      But the output of the tonemapping function heavily depends on the initial range of the values in the HDR buffer. (You can't expect to set the sun intensity the first time to 10 and the second time to 1000 and excpect the same result if you feed that into the tonemapper.) So i suppose that this also depends on the exposure which i have to implement?
      To check this i plotted the tonemapping curve:
      You can see that the curve goes only up to around to a value of 0.21 (while being fed a value of 1) and then basically flattens out. (which would explain why the image got darker.)
       
      My guestion is: In what range should the values in the HDR buffer be which then get tonemapped? Do i have to bring them down to a range of 0-1 by multiplying with the exposure?
      For example, if i increase the values of the light by 10 (directional light would be 7 and ambient light 3) then i would need to divide HDR values by 10 in order to get a value range of 0-1 which then could be fed into the tonemapping curve. Is that correct?
    • By nOoNEE
      i am reading this book : link
      in the OpenGL Rendering Pipeline section there is a picture like this: link
      but the question is this i dont really understand why it is necessary to turn pixel data in to fragment and then fragment into pixel could please give me a source or a clear Explanation that why it is necessary ? thank you so mu
       
       
    • By Inbar_xz
      I'm using the OPENGL with eclipse+JOGL.
      My goal is to create movement of the camera and the player.
      I create main class, which create some box in 3D and hold 
      an object of PlayerAxis.
      I create PlayerAxis class which hold the axis of the player.
      If we want to move the camera, then in the main class I call to 
      the func "cameraMove"(from PlayerAxis) and it update the player axis.
      That's work good.
      The problem start if I move the camera on 2 axis, 
      for example if I move with the camera right(that's on the y axis)
      and then down(on the x axis) -
      in some point the move front is not to the front anymore..
      In order to move to the front, I do
      player.playerMoving(0, 0, 1);
      And I learn that in order to keep the front move, 
      I need to convert (0, 0, 1) to the player axis, and then add this.
      I think I dont do the convert right.. 
      I will be glad for help!

      Here is part of my PlayerAxis class:
       
      //player coordinate float x[] = new float[3]; float y[] = new float[3]; float z[] = new float[3]; public PlayerAxis(float move_step, float angle_move) { x[0] = 1; y[1] = 1; z[2] = -1; step = move_step; angle = angle_move; setTransMatrix(); } public void cameraMoving(float angle_step, String axis) { float[] new_x = x; float[] new_y = y; float[] new_z = z; float alfa = angle_step * angle; switch(axis) { case "x": new_z = addVectors(multScalar(z, COS(alfa)), multScalar(y, SIN(alfa))); new_y = subVectors(multScalar(y, COS(alfa)), multScalar(z, SIN(alfa))); break; case "y": new_x = addVectors(multScalar(x, COS(alfa)), multScalar(z, SIN(alfa))); new_z = subVectors(multScalar(z, COS(alfa)), multScalar(x, SIN(alfa))); break; case "z": new_x = addVectors(multScalar(x, COS(alfa)), multScalar(y, SIN(alfa))); new_y = subVectors(multScalar(y, COS(alfa)), multScalar(x, SIN(alfa))); } x = new_x; y = new_y; z = new_z; normalization(); } public void playerMoving(float x_move, float y_move, float z_move) { float[] move = new float[3]; move[0] = x_move; move[1] = y_move; move[2] = z_move; setTransMatrix(); float[] trans_move = transVector(move); position[0] = position[0] + step*trans_move[0]; position[1] = position[1] + step*trans_move[1]; position[2] = position[2] + step*trans_move[2]; } public void setTransMatrix() { for (int i = 0; i < 3; i++) { coordiTrans[0][i] = x[i]; coordiTrans[1][i] = y[i]; coordiTrans[2][i] = z[i]; } } public float[] transVector(float[] v) { return multiplyMatrixInVector(coordiTrans, v); }  
      and in the main class i have this:
       
      public void keyPressed(KeyEvent e) { if (e.getKeyCode()== KeyEvent.VK_ESCAPE) { System.exit(0); //player move } else if (e.getKeyCode()== KeyEvent.VK_W) { //front //moveAmount[2] += -0.1f; player.playerMoving(0, 0, 1); } else if (e.getKeyCode()== KeyEvent.VK_S) { //back //moveAmount[2] += 0.1f; player.playerMoving(0, 0, -1); } else if (e.getKeyCode()== KeyEvent.VK_A) { //left //moveAmount[0] += -0.1f; player.playerMoving(-1, 0, 0); } else if (e.getKeyCode()== KeyEvent.VK_D) { //right //moveAmount[0] += 0.1f; player.playerMoving(1, 0, 0); } else if (e.getKeyCode()== KeyEvent.VK_E) { //moveAmount[0] += 0.1f; player.playerMoving(0, 1, 0); } else if (e.getKeyCode()== KeyEvent.VK_Q) { //moveAmount[0] += 0.1f; player.playerMoving(0, -1, 0); //camera move } else if (e.getKeyCode()== KeyEvent.VK_I) { //up player.cameraMoving(1, "x"); } else if (e.getKeyCode()== KeyEvent.VK_K) { //down player.cameraMoving(-1, "x"); } else if (e.getKeyCode()== KeyEvent.VK_L) { //right player.cameraMoving(-1, "y"); } else if (e.getKeyCode()== KeyEvent.VK_J) { //left player.cameraMoving(1, "y"); } else if (e.getKeyCode()== KeyEvent.VK_O) { //right round player.cameraMoving(-1, "z"); } else if (e.getKeyCode()== KeyEvent.VK_U) { //left round player.cameraMoving(1, "z"); } }  
      finallt found it.... i confused with the transformation matrix row and col. thanks anyway!
    • By Lewa
      So, i'm currently trying to implement an SSAO shader from THIS tutorial and i'm running into a few issues here.
      Now, this SSAO method requires view space positions and normals. I'm storing the normals in my deferred renderer in world-space so i had to do a conversion and reconstruct the position from the depth buffer.
      And something there goes horribly wrong (which has probably to do with worldspace to viewspace transformations).
      (here is the full shader source code if someone wants to take a look at it)
      Now, i suspect that the normals are the culprit.
      vec3 normal = ((uNormalViewMatrix*vec4(normalize(texture2D(sNormals, vTexcoord).rgb),1.0)).xyz); "sNormals" is a 2D texture which stores the normals in world space in a RGB FP16 buffer.
      Now i can't use the camera viewspace matrix to transform the normals into viewspace as the cameras position isn't set at (0,0,0), thus skewing the result.
      So what i did is to create a new viewmatrix specifically for this normal without the position at vec3(0,0,0);
      //"camera" is the camera which was used for rendering the normal buffer renderer.setUniform4m(ressources->shaderSSAO->getUniform("uNormalViewMatrix"), glmExt::createViewMatrix(glm::vec3(0,0,0),camera.getForward(),camera.getUp())//parameters are (position,forwardVector,upVector) ); Though i have the feeling this is the wrong approach. Is this right or is there a better/correct way of transforming a world space normal into viewspace?
    • By HawkDeath
      Hi,
      I'm trying mix two textures using own shader system, but I have a problem (I think) with uniforms.
      Code: https://github.com/HawkDeath/shader/tree/test
      To debug I use RenderDocs, but I did not receive good results. In the first attachment is my result, in the second attachment is what should be.
      PS. I base on this tutorial https://learnopengl.com/Getting-started/Textures.


    • By norman784
      I'm having issues loading textures, as I'm clueless on how to handle / load images maybe I missing something, but the past few days I just google a lot to try to find a solution. Well theres two issues I think, one I'm using Kotlin Native (EAP) and OpenGL wrapper / STB image, so I'm not quite sure wheres the issue, if someone with more experience could give me some hints on how to solve this issue?
      The code is here, if I'm not mistaken the workflow is pretty straight forward, stbi_load returns the pixels of the image (as char array or byte array) and you need to pass those pixels directly to glTexImage2D, so a I'm missing something here it seems.
      Regards
    • By Hashbrown
      I've noticed in most post processing tutorials several shaders are used one after another: one for bloom, another for contrast, and so on. For example: 
      postprocessing.quad.bind() // Effect 1 effect1.shader.bind(); postprocessing.texture.bind(); postprocessing.quad.draw(); postprocessing.texture.unbind(); effect1.shader.unbind(); // Effect 2 effect2.shader.bind(); // ...and so on postprocessing.quad.unbind() Is this good practice, how many shaders can I bind and unbind before I hit performance issues? I'm afraid I don't know what the good practices are in open/webGL regarding binding and unbinding resources. 
      I'm guessing binding many shaders at post processing is okay since the scene has already been updated and I'm just working on a quad and texture at that moment. Or is it more optimal to put shader code in chunks and bind less frequently? I'd love to use several shaders at post though. 
      Another example of what I'm doing at the moment:
      1) Loop through GameObjects, bind its phong shader (send color, shadow, spec, normal samplers), unbind all.
      2) At post: bind post processor quad, and loop/bind through different shader effects, and so on ...
      Thanks all! 
    • By phil67rpg
      void collision(int v) { collision_bug_one(0.0f, 10.0f); glutPostRedisplay(); glutTimerFunc(1000, collision, 0); } void coll_sprite() { if (board[0][0] == 1) { collision(0); flag[0][0] = 1; } } void erase_sprite() { if (flag[0][0] == 1) { glColor3f(0.0f, 0.0f, 0.0f); glBegin(GL_POLYGON); glVertex3f(0.0f, 10.0f, 0.0f); glVertex3f(0.0f, 9.0f, 0.0f); glVertex3f(1.0f, 9.0f, 0.0f); glVertex3f(1.0f, 10.0f, 0.0f); glEnd(); } } I am using glutTimerFunc to wait a small amount of time to display a collision sprite before I black out the sprite. unfortunately my code only blacks out the said sprite without drawing the collision sprite, I have done a great deal of research on the glutTimerFunc and  animation.
  • Advertisement
  • Popular Now

  • Forum Statistics

    • Total Topics
      631394
    • Total Posts
      2999755
×

Important Information

By using GameDev.net, you agree to our community Guidelines, Terms of Use, and Privacy Policy.

Participate in the game development conversation and more when you create an account on GameDev.net!

Sign me up!