[Tenderfoot]

I'm creating a small PHP script to calculate taxes in my country and have been getting this error: Fatal error: Call to undefined function reiknaLifeyrisFramlagVinnuveitanda() in C:\apache\htdocs\Aefingaverkefni01\skattautreikningar.php on line 134

The function in question is the following one:

function reiknaLifeyrisFramlagVinnuveitanda($laun)
	{
		$this->lifeyrissFramlagVinnuveitanda = $laun * 0.06; //Finnur 6% framlag vinnuveitanda til lífeyrissjóðs
		echo "Þar sem launin eru $laun ætti framlag vinnuveitanda til lífeyrissjóðs að vera: $this->lifeyrissFramlagVinnuveitanda <br />";
	}

This function is written or perhaps I should say defined well above the function I call it from.

I call it on line 18 in the following snippet:

function reiknaLaun($laun) //Á að reikna út tekjuskatt og lífeyrissjóð
	{
		if ($laun <= 230000)
		{
			$tekjuskattur = 0.3734; //37,43% er tekið af launum undir 230 þúsund
		}
		
		else if ($laun > 230000 && $laun <= 704367)
		{
			$tekjuskattur = 0.4024; //40,24% er tekið af launum á milli 230001 og 704367
		}
		
		else
		{
			$tekjuskattur = 0.4624; //46,24% er tekið af launum yfir 704367
		}
		
		reiknaLifeyrisFramlagVinnuveitanda($laun); //Reikna út lífeyrisframlag vinnuveitanda og set það í private breytu
		reiknaLifeyrisFramlagLaunthega($laun); //Reikna út lífeyrisframlag launþega og set það í private breytu
		$laun += $this->lifeyrissFramlagVinnuveitanda; // (Síðan þarf að leggja 6% ofan á launin (framlag vinnuveitanda))
		
		//reikna tekjuskatt af allri upphæðinni
		$laun -= reiknaTekjuSkattMedTillitiTilPersonuafslatts($tekjuskattur, $laun);
		
		//og svo að draga lífeyrissjóðinn aftur frá því sem eftir stendur (10% samanlagt).
		
		reiknaHeildarLifEyrisFramlag(); 
		$laun -= $this->heildarLifeyrisFramlag; 
		
		return $laun; 
	}

And perhaps it's worth mentioning how I call the class in this file from another file:

require_once 'skattautreikningar.php'; 

$skattur1 = new Skattur(); 

$skattur1->reiknaLaun("200000");

Anyone have any idea what I'm doing wrong? Everything's written correctly as far as I can tell.

[StefanOnRails]

Well, in PHP in order to call a method (function) you use $this->function_name(); (if the function is called inside the class)

[JackOfAllTrades]

Damn, I thought *I* used long names

[Tenderfoot]

StefanOnRails, on 14 September 2012 - 10:43 AM, said:

Well, in PHP in order to call a method (function) you use $this->function_name(); (if the function is called inside the class)

Thanks, problem solved.

JackOfAllTrades, on 14 September 2012 - 11:00 AM, said:

Damn, I thought *I* used long names

Haha, yeah.. I'm aware of my tendency to use fairly long names. It's just, I try to make the name speak for the function. And sometimes no two words that can do that come to mind. Or three. Or four.

[JackOfAllTrades]

Better to have descriptive names (within reason). I was chalking it up to the language, actually

[Atli]

Icelandic is a lot of things, but succinct it is not

Out of curiosity, do your teachers want you to write it like that? I always wrote my code in English and never got any complaints.

[JackOfAllTrades]

I thought it might be your native language, Atli. Knew it was some variation of a Scandianavian language.

[Tenderfoot]

Atli, on 14 September 2012 - 03:19 PM, said:

Icelandic is a lot of things, but succinct it is not

Out of curiosity, do your teachers want you to write it like that? I always wrote my code in English and never got any complaints.

They don't make it a requirement. However, the instructions for the assignment are in Icelandic, and the functions he asked for were written in Icelandic as well, so he basically gave us the names for those. So I write it like this for the teacher. The majority of the teachers write their code in Icelandic, and translate a fair bit of the terms used in programming. I wonder if you went to the same school as I - most likely did if you studied PHP. But I'm given to understand that the subjects have changed a fair bit so who knows.

Always wondered if you were Icelandic by the way - seen your name around in the erm.. reputation-hiscore-box more than a few times.

This post has been edited by Tenderfoot: 15 September 2012 - 10:53 AM