refactor(scheduler): replace init methods with constructor functions

[yxxhero]

What type of PR is this?
/kind bug

What this PR does / why we need it:

avoid literal copies lock value from

Which issue(s) this PR fixes:
This pull request simplifies the initialization of nodeManager and podManager by replacing the init methods with constructors. Additionally, it updates the Scheduler struct to use pointers for these managers and modifies the related tests accordingly.

Initialization improvements:

• pkg/scheduler/nodes.go: Replaced the init method with a newNodeManager constructor to initialize the nodeManager struct.
• pkg/scheduler/pods.go: Replaced the init method with a newPodManager constructor to initialize the podManager struct.

Scheduler struct updates:

• pkg/scheduler/scheduler.go: Updated the Scheduler struct to use pointers for nodeManager and podManager, and modified the NewScheduler function to use the new constructors. [1] [2]

Test updates:

• pkg/scheduler/nodes_test.go: Updated tests to use the newNodeManager constructor instead of the init method.
• pkg/scheduler/scheduler_test.go: Updated tests to use the newNodeManager and newPodManager constructors instead of the init methods. [1] [2]

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

[hami-robott]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: yxxhero
Once this PR has been reviewed and has the lgtm label, please assign archlitchi for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[yxxhero]

@archlitchi