Function to call another function about multiplewindow3dscene HOT 2 OPEN

The windowsUpdated function in your code seems redundant as it's merely calling another function, updateNumberOfCubes(), without performing any additional logic. As it stands, removing windowsUpdated and directly calling updateNumberOfCubes() wherever windowsUpdated() is called would likely simplify the code without losing any functionality.

Sometimes, developers create intermediate functions like windowsUpdated for potential future expansion or to maintain a consistent structure across the codebase. However, if it doesn't serve any clear purpose beyond calling another function and there are no plans for extending its functionality in the future, removing it could indeed streamline the code. Always remember to ensure that removing such functions won't impact other parts of the codebase before doing so.

	function setupWindowManager ()
	{
		windowManager = new WindowManager();
		windowManager.setWinShapeChangeCallback(updateWindowShape);
		windowManager.setWinChangeCallback(windowsUpdated);

		// here you can add your custom metadata to each windows instance
		let metaData = {foo: "bar"};

		// this will init the windowmanager and add this window to the centralised pool of windows
		windowManager.init(metaData);

		// call update windows initially (it will later be called by the win change callback)
		windowsUpdated();
	}

	function windowsUpdated ()
	{
		updateNumberOfCubes();
	}

from multiplewindow3dscene.

The windowsUpdated function in your code seems redundant as it's merely calling another function, updateNumberOfCubes(), without performing any additional logic. As it stands, removing windowsUpdated and directly calling updateNumberOfCubes() wherever windowsUpdated() is called would likely simplify the code without losing any functionality.

Sometimes, developers create intermediate functions like windowsUpdated for potential future expansion or to maintain a consistent structure across the codebase. However, if it doesn't serve any clear purpose beyond calling another function and there are no plans for extending its functionality in the future, removing it could indeed streamline the code. Always remember to ensure that removing such functions won't impact other parts of the codebase before doing so.

function setupWindowManager ()
{
	windowManager = new WindowManager();
	windowManager.setWinShapeChangeCallback(updateWindowShape);
	windowManager.setWinChangeCallback(windowsUpdated);

	// here you can add your custom metadata to each windows instance
	let metaData = {foo: "bar"};

	// this will init the windowmanager and add this window to the centralised pool of windows
	windowManager.init(metaData);

	// call update windows initially (it will later be called by the win change callback)
	windowsUpdated();
}

function windowsUpdated ()
{
	updateNumberOfCubes();
}

from multiplewindow3dscene.