28-04-2005 01:12									#91
	Get a bigger avatar today!

I'm still looking and I'm going to go back to my Dummies to get the terminology down right but what I'm seeing is that the code should be:

code:

if(irec->GetCultureOnlyIndex(s) == g_player->GetCivilisation()->GetCityStyle()) {

code:

if(irec->GetCultureOnlyIndex(s) == GetCivilisation()->GetCityStyle()) {

	28-04-2005 16:35									#92
	Get a bigger avatar today!

quote: Originally posted by E but you said that was wrong before so I'm going to keep digging because I don't think it should be: code: if(irec->GetCultureOnlyIndex(s) == GetCivilisation()->GetCityStyle()) {

And why do you think this is wrong?

-Martin

	29-04-2005 04:55									#93
	Support Apolyton, buy Civilization: The Boardgame

because I thought it was too easy an answer....

I guess this worked but only ifthe function is in player.cpp so it already knows it applies to the player. My lack of C++ vocabulary confused me. I'll make the changes and post the files to this thread.

	29-04-2005 09:23									#94
	Increase the size of your Attachments

Martin,

I made the changes. One thing I want to double check though:

code:

Terrailutil.cpp

		if(rec->IsRestrictedToGood == 0) {
			for(i = 0; i < rec->GetNumCantBuildOn(); i++) {
				if(rec->GetCantBuildOnIndex(i) == cell->GetTerrain()) {
					return false;
				}
			}
		} 
		else {
			sint32 good;
			if (g_theWorld->GetGood(pos, good)) {
				for(i = 0; i < rec->GetNumIsRestrictedToGood(); i++) {
					if(rec->GetIsRestrictedToGood(i) == good) {
						return true; 
					}  
				}	 
				return false;
			}
		}
	}

Last edited by E on 02-05-2005 at 02:24

	30-04-2005 01:25									#95
	Lose 30 kilos (of popups)

quote: Originally posted by E code: Terrailutil.cpp if(rec->IsRestrictedToGood == 0) { for(i = 0; i < rec->GetNumCantBuildOn(); i++) { if(rec->GetCantBuildOnIndex(i) == cell->GetTerrain()) { return false; } } } else { sint32 good; if (g_theWorld->GetGood(pos, good)) { for(i = 0; i < rec->GetNumIsRestrictedToGood(); i++) { if(rec->GetIsRestrictedToGood(i) == good) { return true; } } return false; } } } When we went through this code awhile ago so it should be good, but I don't understand why the first line is: if(rec->IsRestrictedToGood and not CantBuildOn like the code was originally. Will the game still check the CantBuildOn in this case?

Yes we went through a while ago. But there are two possibilities that happend.

First I overlooked that it doesn't work.
Second possibility you forgot to fix something I told you to fix.

The indention was that it is a stupid idea to restrict a tile improvement to a certain good and to forbid the construction on the underlying terrain.

So the design is if there is no good restriction the original code is used and the CanBuildOn stuff is used. If there is at least one restriction to a certain good your new code is used.

But the code does not to this what I just described. It even does not compile, because you didn't do the right test whether there are any restrictions to certain goods.

So again how do you get figure out that the number of your good restrictions is zero?

Well I leave this queston to you.

-Martin

	30-04-2005 09:45									#96
	Inflate your Upload Space

Well I know I have to GetNum so I think the code should be:

code:

		if(rec->GetNumIsRestrictedToGood () == 0) {
			for(i = 0; i < rec->GetNumCantBuildOn(); i++) {
				if(rec->GetCantBuildOnIndex(i) == cell->GetTerrain()) {
					return false;
				}
			}
		} 
		else {
			if(irec->GetNumIsRestrictedToGood() > 0) {
			sint32 good;
				if (g_theWorld->GetGood(pos, good)) {
					for(i = 0; i < rec->GetNumIsRestrictedToGood(); i++) {
						if(rec->GetIsRestrictedToGood(i) == good) {
							return true; 
						}  
					}	 
				return false;
				}
			}
		}

code:

		if(rec->GetNumIsRestrictedToGood () == 0) {
			for(i = 0; i < rec->GetNumCantBuildOn(); i++) {
				if(rec->GetCantBuildOnIndex(i) == cell->GetTerrain()) {
					return false;
				}
			}
		} 
		else {
			sint32 good;
			if (g_theWorld->GetGood(pos, good)) {
				for(i = 0; i < rec->GetNumIsRestrictedToGood(); i++) {
					if(rec->GetIsRestrictedToGood(i) == good) {
						return true; 
					}  
				}	 
				return false;
			}
		}

Last edited by E on 02-05-2005 at 02:24

	30-04-2005 22:42									#97
	Help yourself to an AD-FREE life

quote: Originally posted by E Well I know I have to GetNum so I think the code should be:

Everything else would be a surprise.

quote: Originally posted by E BUT I added that extra line for the check but if thats wrong then the code should be:

Why do you want the extra check. At first it doesn't compile, because you forgot a closing brace, and arrays can't have a size that is smaller then 0. If you really want to check this you can ask whether you have less or zero elements in your IsRestrictedToGood array and if this holds you execute the original code otherwise your new code.

quote: Originally posted by E Where there any other code problems with the CultureOnly, etc?

Well I told you that g_player is a list of players and not player properties. You didn't look on the descriptions of the functions in terrainutil.cpp.

Then something like this TerrainImprovementRecord *rec is not a variable for improvement flag it is a terrain improvement record in the terrain improvement database.

In comparision to this:

code:

// Parameters :  sint32 pl              : Variable for player

This is wrong:

code:

//              pl: The player's attributes if they can build wonder  

It is rather the index of the player that is checked for whether the terrain improvement in question can be build.

Then this:

code:

// - Added  a check in CanPlayerBuild for the CultureOnly flag to see if a Unit  
//   is limited to certain CityStyles (E 2005/03/12)

Well that is nice that the unit is limited to a certain city style. Of which style by the way the style of the owner city or the owner civilisation and why are you talking about units in terrainutil.cpp

code:

// - Added a check in CanPlayerBuild for the GovernmentOnly flag to see if a Unit  
//   is limited to certain CityStyles (E 2005/03/12)

What do you mean limitation to certain city styles or to certain governments?

code:

// - Added  a check in CanPlayerBuildAt for the IsRestrictToGood flag so an   
//   improvement can only be built on a good tile (-E 2005/03/12)

Should be a tile with a certain good on it. And of course these functions all start with the prefix terrainutil_.

Then let's take a look on the terrainutil_CanPlayerBuild function itsself. In this function there is no irec defined. Therefore this doesn't compile.

And lease use the GetGovernmentType function instead of m_government_type if you don't want to modify its value.

In the Player::CanBuildUnit method you find this irec thing again. And the description of the function tells you that it is called BOOL CityData::CanBuildUnit, of course this is wrong.

And the best is that the top of the file doesn't say anything about your changes you have done in this file.

OK let's come to CityData.cpp:

code:

// - CityStyleOnly code added to CanBuildUnit, CanBuildBuilding, CanBuildWOnder 
//   by E April 20th 2005
// - Governmenttype and CultureOnly added to CanBuildBuilding and canBuildWonder
//   by E April 20th 2005

Nice that you have added these stuff but for what?

Again in CityData::CanBuildUnit is no irec defined. The only method that has an irec is CityData::CanBuildBuilding.

And CityData::CanBuildWonder has no irec as well, but there is no database record defined at all, so you have do more then renaming it in comparision to the other methods. You have to retrieve a wonder record first and please don't call it irec like city improvement record.

Well that should be all for now.

-Martin

	01-05-2005 10:43									#98
	Support Apolyton buy from Amazon

Thanks for going easy on me. I updated all the stuff and attached here ( i did my wonder rec from what I saw in another file).

Everything else seemed straight forward.

Also I did install the C++for dummies CD and using its cpp program. It has a compiler too so as a warning I may be compiling soon.

{removed}

Last edited by E on 02-05-2005 at 02:25

	01-05-2005 22:06									#99
	Browse Apolyton AD-FREE

quote: Originally posted by E Thanks for going easy on me. I updated all the stuff and attached here ( i did my wonder rec from what I saw in another file).

So far we aren't finished.

code:

// - Added a check in terrainutil_CanPlayerBuild for the GovernmentOnly flag 
//   to see if a tile improvement is limited to certain player 
//   governments - added by E March 12th 2005

Actual this should be something along whether the player has the right government or his government is matching to the record you find in tileimp.txt. And check the description of the terrainuti.cpp again there are two added with two following spaces.

code:

// Added GovernmentType flag from Units to use for Improvements
	if(rec->GetNumGovernmentType() > 0) {
		sint32 i;
		bool found = false;
		for(i = 0; i < rec->GetNumGovernmentType(); i++) {
			if(rec->GetGovernmentTypeIndex(i) == g_player[pl]->m_government_type->GetGovernmentType()) {
				found = true;
				break;
			}
		}
		if(!found)
			return false;
	}

m_government_type is a member of Player like GetGovernmentType(), so this doesn't compile. Actual you was supposed to replace all instances of m_government_type by GetGovernmentType() you have added in all the three files. This has the advantange that you cannot modify this variable by acciendent. And I can make all these members in the long run private.

The rest of terrainutil.cpp seems to be ok. So let's go to Player.cpp:

code:

// - CultureOnly code added to CanBuildUnit; checks the CultureOnly flag to a
//   player's citystyle by E April 20th 2005

Nice that it checks it but what for?

code:

// Remark(s)  : CultureOnly added by E; checks if player an unit have the
//              same CityStyle in order to build		

Is this English? And by the way remove the last four trailing tabs.

OK let's go to CityData.cpp

code:

//            : Added PrerequisiteBuilding check to see if a cit has a building
//              needed to build a unit.

A city without y.

code:

// Added by E - checks if a city has a building required to build the unit
	if(rec->GetNumPrerequisiteBuilding() > 0) {
	    sint32 o;
    	for(o = 0; o < rec->GetNumPrerequisiteBuilding(); o++) {
			sint32 b = rec->GetPrerequisiteBuildingIndex(o);
			if(!(GetEffectiveBuildings() & ((uint64)1 << (uint64)b)))
				return FALSE;
		}
	}

And some additional code fortunatly for you it compiles and it is correct, but please stop now adding anything else, before the rest is correct. Therefore I had again to check the *.cdb files, fortunately there where correct as well.

Well, the only thing you can do is to check the white space, there are two tabs replaced by spaces in.

And there again is a m_government_type instead of a GetGovernmentType()

code:

// Returns    : Whether the city can build the building specified by type.

This line is wrong at least if your method is called CityData::CanBuildWonder

code:

//            : GovernmentType flag for wonderss limits wonders to govt type.

wonders with an s too much. And the rest of the CityData::CanBuildWonder looks ok, except that there are again some spaces where tabs should be. And I rather tought of a more generic name than wrec, so that you can paste the piece of code more easily into other locations. But well there is no real need to change it now.

quote: Originally posted by E Also I did install the C++for dummies CD and using its cpp program. It has a compiler too so as a warning I may be compiling soon.

Maybe you should finish this project first and then commit it to the respiratory. So that you have a good code base. I hope you have a Microsoft compiler, as noone as far as I know has compiled it on anything else.

-Martin

	02-05-2005 02:23									#100
	Support Apolyton, pre-order Civilization IV

Martin,

Thanks for checking the stuff. Hopefully the end is insight. I didn't add anything new and I think once this code is done and I commit it to the svn I'm going to take a coding break. I think you need it I'll get back into my graphics I need to work on and go through my C++ book more and do the exercises. I notice I can understand the code fairly well (like with PreReqBuilding) but still not good enough to write so I'm going to do some self study and maybe take a class before I do any more coding after this.

Attachment: 2005.05.01.e-files.zip
This has been downloaded 2 time(s).

	05-05-2005 01:15									#101
	Support Apolyton, buy Call to Power 2

Martin,
are their any problems or should I commit them to ther server now?

	05-05-2005 03:25									#102
	Support Apolyton, buy Galactic Civilizations: Deluxe Edition

quote: Originally posted by E Martin, are their any problems or should I commit them to ther server now?

No, you shouldn't. The code seems to be ok, but there are problems with your comments.

CityData.cpp:

code:

// - Governmenttype added to CanBuildWonder and CanBuildBuilding; Checks if
//   the pler's government matches the flag for the wonder or building - by E 
//   April 20th 2005 

What is a pler?

And you didn't take care about the white space usage when you copied the PrerequisiteBuilding check. There are two tabs replaced by spaces. The same holds for the line when you retrieve the pointer on the wonder record.

Player.cpp seems to be ok.

terrainutil.cpp:

code:

// - Added  a check in CanPlayerBuild for the CultureOnly flag to see if a   
//   Tile improvement is limited to certain CityStyles (E 2005/03/12)
// - Added a check in terrainutil_CanPlayerBuild to check a to see if player's 
//   government is matching to the record you find in tileimp.txt - (E 2005/03/12)
// - Added  a check in terrainutil_CanPlayerBuildAt for the IsRestrictToGood 
//   flag so a tile improvement can only be built on a tile with a 
//   certain good on it - (E 2005/03/12)

First there are three "Added"s after the first and the third "Added" there are 2 spaces after each even if there should be just one space.

And now two the middle statement. I don't think this is good English:

code:

// - Added a check in terrainutil_CanPlayerBuild to check a to see if player's 
//   government is matching to the record you find in tileimp.txt - (E 2005/03/12)

There is an indefinete article at the wrong place. And I think it is better to say that an additional check was added to figure out if the given unit is restricted to a certain types of government and whether the player has the according government. Your task by the way is now to sum it up so that it is still correct. And of course correct another thing terrainutil.cpp has nothing to do with units. It has to do with terrain and what you can build on it.

And this statement is also wrong:

code:

// Name       :  terrainutil_CanPlayerBuildAt
//
// Description:  Checks terrain improvement properties to see if it can build 
//               on a tile only if it has a good

That's to focussed on your particular addition. The terrainutil_CanPlayerBuildAt is more general.

Well that should be now for today. And of course all the fixes has to be done.

-Martin

	05-05-2005 06:39									#103
	Support Apolyton, buy Alpha Centauri

inching toward the finish line....I added your corrections, the only other thing I did was change wrec to rec like you suggested before (not replace I did each one by hand just to make sure I didn't miss) and in Unit.cdb I removed the futureuse flag NeedsBuildingToBuild since PreRequisiteBuilding was already added and you saw that it works. Other than that I made your corrections.

Attachment: 2005.05.04.e-files.zip
This has been downloaded 2 time(s).

	07-05-2005 02:32									#104
	Support Apolyton buy from Amazon

Martin,

Any other problems?

	07-05-2005 03:02									#105
	Support Apolyton, buy Galactic Civilizations: Deluxe Edition

quote: Originally posted by E Martin, Any other problems?

No real problems, except that this line in CityData::CanBuildWonder still contains spaces even if it should be tabs, well actual a single tab:

code:

    const WonderRecord* rec = wonderutil_Get(type);

Once you have done this you can commit it to the respository. Of course first you have to Update your files, so that all the stuff gets merged properly. But I think there shouldn't be any problems.

By the way MergeTortoise is a powerfull tool that can spot every space at the wrong place easily. And it is very helpfull to concentrate on your modifications.

I would say the next project is to fix the CanSettleOn feature at least the part that can be done in UnitData.cpp.

And maybe before you should get your compiler working. By the what compiler did you got your hands on?

-Martin

	07-05-2005 03:24									#106
	Support Apolyton, buy Civilization III: Complete

Thanks Martin,
Will do the stuff tonight (after I get the most recent update etc)

My compiler came with my C++ for dumies I think its Visual Basic 6.0 or something to that affect.

I'll also refresh the CanSettleOn issue but HasResource keeps bugging me that it has something to do with checking or assigning a resource(good) to a city and that it could be used to do a needsCityGood function...

code:

IN CITYDATA.CPP


BOOL CityData::HasResource(sint32 resource) const
{
	return m_collectingResources[resource] > m_sellingResources[resource];
}

BOOL CityData::IsLocalResource(sint32 resource) const
{
	return m_collectingResources[resource] > 0;
}



bool CityData::GetResourceTradeRoute(sint32 resource, TradeRoute & route) const
{
	sint32 i;
	for(i = 0; i < m_tradeSourceList.Num(); i++) {
		ROUTE_TYPE type;
		sint32 rr;
		m_tradeSourceList[i].GetSourceResource(type, rr);
		if(type != ROUTE_TYPE_RESOURCE) continue;
		if(rr != resource) continue;

		
		route = m_tradeSourceList[i];
		return true;
	}

	
	return false;
}


bool CityData::IsSellingResourceTo(sint32 resource, Unit & destination) const
{
	sint32 i;
	for(i = 0; i < m_tradeSourceList.Num(); i++) {
		ROUTE_TYPE type;
		sint32 rr;
		m_tradeSourceList[i].GetSourceResource(type, rr);
		if(type != ROUTE_TYPE_RESOURCE) continue;
		if(rr != resource) continue;

		
		destination.m_id = m_tradeSourceList[i].GetDestination().m_id;
		return true;
	}

	
	destination.m_id = 0;
	return false;
}

code:

GOVERNOR.CPP



void Governor::ManageGoodsTradeRoutes()
{
	Assert(g_player[m_playerId] != NULL);
	Player *player_ptr = g_player[m_playerId];

	sint32 cur_round = player_ptr->GetCurRound();

	Unit city;
	sint16 i,g,d;;
	UnitDynamicArray *city_list = player_ptr->GetAllCitiesList();
	double unused_freight = player_ptr->GetUnusedFreight();
	double total_freight = player_ptr->GetTotalFreight();
	GoodsRoute new_route;
	GoodsRouteList new_routes;

	
	m_neededFreight = 0.0;

	
	for (i = 0; i < city_list->Num(); i++) {
		city = city_list->Access(i);

		
		for(g = 0; g < g_theResourceDB->NumRecords(); g++) {
			if(city.CD()->IsLocalResource(g)) {
				
				sint32 op;
				Unit maxCity;
				sint32 maxPrice = 0;
				sint32 bestPrice = 0;
				double maxCost = 0.0;
				double maxNeededFreight = 0.0;
				sint32 sellingPrice = -1;
				TradeRoute curDestRoute;

				
				if(	city.CD()->HasResource(g) == FALSE &&	
					city.CD()->GetResourceTradeRoute(g, curDestRoute))
				{
					sellingPrice = 
						tradeutil_GetTradeValue(m_playerId, curDestRoute->GetDestination(), g);
				}
				else 
				{
					curDestRoute.m_id = 0;
					sellingPrice = -1;
				}

				
				for(op = 1; op < k_MAX_PLAYERS; op++) {

Last edited by E on 07-05-2005 at 03:29

	14-05-2005 21:10									#107
	Support Apolyton, buy GURPS/ Alpha Centauri

Well the first four methods seem to be accurate for your project, but your next task is to get the game to compile on your computer.

Your compiler can't be Visual Basic 6.0 that's a compiler for another language, so it must be Visual C++ 6.0 including Microsoft Visual Studio and of course the introductory version.

But for testing your code it is enough.

To answer your other question what I had to correct. Actual the answer is very easy if you use the Show Log feature of TortoiseSVN and mark there on the list Revision 346 and right click on the CityData.cpp and use from there the Show Differences option.

In CityData.cpp I corrected wrong cases. You have to figure out on your own what I have done.

And you have to make sure that in the settings of TortoiseMerge the option Ignore All Whitespaces is used.

In Player.cpp I didn't fix anything at your code, I just removed .NET warnings in other parts of the file.

In terrainutil.cpp I fixed something at your code, again you have to figure this out on your own. And I added an #include "Civilisation.h".

And in the *.cdb files I fixed some syntax errors, again you have to find them on your own but with TortoiseMerge this should be easy.

All these fixes have in common is that they are very hard to catch especially this #include "Civilisation.h" as long as you don't try to compile your changes, therefore you need a compiler.

-Martin