Some help with a loop calculation

[ExpendableDecoy]

Hi. 🙂

I'm currently trying to create a loop in a function but I'm struggerling quite badly on a how to get this code to work.

This is the dev code I currently have:

function modifier_calc($skill_level)
{
	if($skill_level > '11')
	{
		// Must have at least 12 points in the skill/characteristic before positive modifier is applied
		// Gains +1 modifier for every 2 levels, ie. 12-13 (+1), 14-15 (+2), 16-17 (+3), etc, etc
		for($p = 12; $p < $skill_level; $p++)
		{
			$modifier = ;
		}
	}
	else
	{
		// Less than 12 skill/characteristic points means no modifier to user
		$modifier = '0';
	}

return $modifier;
}

What I'm trying to do is if the variable ($skill_level) is at least '12' then I need it to go into the loop and work out how which modifier it is related too.

		for($p = 12; $p < $skill_level; $p++)
		{
			$modifier = ;
		}

For example:

$skill_level = 12 or 13 then $modifier = 1,
$skill_level = 14 or 15 then $modifier = 2,
$skill_level = 16 or 17 then $modifier = 3,
etc, etc.

I cannot for the life of me work out how to achieve this.

Any help would be greatly apprieciated. 🙂

[laserlight]

Here's a hint - consider:
$modifier = (int)(($skill_level - 12) / 2) + 1;

[JPnyc]

In most scripting languages when you put ' ' around an integer, you designate it as string. Is that the case for PHP as well?

[sapoxgn]

Originally posted by JPNYC
In most scripting languages when you put ' ' around an integer, you designate it as string. Is that the case for PHP as well?

I think so

[JPnyc]

In that case he should remove the '' from '11'. That might be the entire problem, although obviously I'm not experienced with PHP or I wouldn't have had to ask. But it looks like he's comparing an integer to a string. Come to think of it that should throw an error though.

[ScubaKing22]

Originally posted by JPNYC
Come to think of it that should throw an error though.

Nope. Perhaps a disadvantage of a language such as PHP 😉

[JPnyc]

Aha! Well then if he ever comes back, let's suggest he remove the ''. Thanks.

[ExpendableDecoy]

My error when copying some code from another function and just forgot to remove the ''.

Thanks for the info laserlight. I'll give it a go.