Need help with class?

[AquariaXI]

Hi, I'm new here! So I'm trying to create a class & I need to grab the title of my program through `get_appTitle ( )`  whose data is changed by `__InitializeData ( )`

How can I do that?

<?php
class MainframeData {
	public $appTitle;
	public $appName = "App Name";
	public $version = "1.0";
	public $verPrefix = "";

	function __construct ( ...$params ) {
		$this -> param = $params;
	}

	function get_appTitle ( ) {
		return $this -> appTitle;
	}

	function __InitializeData ( ) {

		if ( $this -> verPrefix !== "" && empty ( $this -> verPrefix ) === false ) {
			$this -> appVer = ( $this -> version . " { " . $this -> verPrefix . " } " );
			$this -> appTitle = $appName . " - " . "Version" . " " . $appVer;
		}

		if ( $this -> verPrefix !== "" || empty ( $this -> verPrefix ) === true ) {
			$this -> appVer = $this -> version;
			$this -> appTitle = $this -> appName . " - " . " { " . "Version" . " " . $this -> appVer . " } ";
		}
	}
}

$mfrdata = new MainframeData ( );
$mfrdata -> __InitializeData ( );

?>

Any help is GREATLY appreciated!

 

 

 

Edited May 19, 2021 by AquariaXI

[requinix]

Have you tried calling that get_appTitle() method?

[AquariaXI]

Yes & it returns nothing at all.

[requinix]

What's the code where you call it? Is it part of the code you posted before or is it written somewhere else?

[AquariaXI]

It is written on index.php which is separate but here it is : 

<?php $mfrdata = new MainframeData ( ); ?>
<h1><?php echo "AppTitle :: " . $mfrdata -> get_appTitle ( ); ?></h1><br>
<hr><br>
	

Edited May 19, 2021 by AquariaXI

[requinix]

You already know that the appTitle is set by InitializeData. If you don't call InitializeData then there will be no appTitle.

All that InitializeData code should exist in the constructor anyways. Move it and you won't have to call any special methods before you can use appTitle.

[AquariaXI]

@requinix: I don't understand...

[requinix]

Did you write the code for the MainframeData class? If not, where did it come from?

[AquariaXI]

yes i wrote the code for the mainframedata class. I'm just not understanding what & / or how you're trying to tell me what to do.

[requinix]

All that code in InitializeData should be inside the constructor instead. There is no need for InitializeData to exist. It forces you to call that method every time you create a new instance of MainframeData, and if you forget then you get problems like the one that started this thread.
If you need to set up appVer and appTitle when the object is created then you should be literally setting up appVer and appTitle when the object is created.

[AquariaXI]

Ah thanks, @requinix! That fixed it!