How hard would it be to move it to php8

[Nolan81]

https://github.com/pheroh/TSP-GA-PHP

need this lib for my lab , is there a way to automatically upgrade it to modern php8 version?

[requinix]

Have you tried running it? Does it work? Are there any errors?

[Nolan81]

running is not enough. Code will be watched by teacher, code should not be that old looking

[requinix]

Your teacher is going to grade you on the quality of someone else's library? What?

It doesn't look old. It's not like PHP 8 introduced some incredibly obvious and immensely helpful features that everybody had to stop what they were doing and migrate to using. Like, unless I were to look over it with a fine-toothed comb, I don't think I could tell you whether it was written for PHP 5 or PHP 8.

But no, there aren't really any tools to do what you want automatically. Not that I know of. But the project isn't that large - it should only take, like maybe, 15 minutes if you wanted to "spruce it up" or something.

[Barand]

One thing to watch for is the stricter variable typing in version 8.

Whereas in earlier versions this would work (the empty string being treated as "0") ...

$a = 5;
$b = '';
echo $a + $b;

in version 8 it throws an error

Quote

PHP Fatal error: Uncaught TypeError: Unsupported operand types: int + string in .....

 

[kicken]

12 hours ago, requinix said:

But no, there aren't really any tools to do what you want automatically. Not that I know of.

I've never used it personally but I've heard good things about rector.

[Nolan81]

rector is what chatgpt proposed too.

17 hours ago, requinix said:

Like, unless I were to look over it with a fine-toothed comb, I don't think I could tell you whether it was written for PHP 5 or PHP 8

missing types of the arguments, properties is obvious sign. or code like : = array()

[requinix]

1 hour ago, Nolan81 said:

missing types of the arguments, properties is obvious sign. or code like : = array()

Casual code doesn't always use types. And there are people who find "array" to be easier to scan and read than brackets.

[gizmola]

I looked at the code, and it's fine.  As others said, I'm not sure why it matters, if you are planning to use this as the basis for a project of your own, unless you are planning to try and pass the code off as something you wrote yourself.

[Nolan81]

I will need to use any of such swarm libraries as a basis for my own tasks, they do not perform them, they concentrate on TSP.

Edited December 12, 2023 by Nolan81