通过函数将未知数量的对象添加到数组

Question

所以，首先。我是一名 C# 程序员，所以我的 PHP 代码可能会受到你用 C# 编写的方式的强烈影响。无论如何，我正在尝试解决一个问题，即我有一个带有两个轴 x 和 y 的二维网格。用户输入，程序会告诉你你的新位置。

例如，如果用户输入是“>^”，则位置将为 1,1。
我还希望程序做的是告诉您您重访的第一个位置。所以，如果你从 0,0 -> 0,1 -> 0,0 -> 1,0 -> 1,1 开始。然后您重新访问的第一个位置是 0,0。 我的问题是我试图将我的 int 发送到一个函数，然后将它添加到一个数组中，如果数组包含该对象，则搜索该数组。但似乎数组只添加了一个值（？）

declare(strict_types=1);
<?php

$NewInput = ">>^^<>>";
$arr = str_split($NewInput, 1);
$x = 0;
$y = 0;
Directions($x, $y, $arr);
function Directions (int $x, int $y, $arr)
{
    foreach ($arr as $item)
    {
        if ($item == '^')
        {
            $y += 1;
        }
        elseif ($item == 'v')
        {
            $y -= 1;
        }
        elseif ($item == '>')
        {
            $x += 1;
        }
        elseif ($item == '<')
        {
            $x -= 1;
        }
        Previous($x,$y);
    }
}
function Previous (string $x, string $y)
{
    $Location = $x . '.' . $y;
    echo "<br>";
    $NewArray = array();
    $NewArray[] = $Location;
    $ReVisited = "";
    if ($ReVisited == "")
    {
       if (array_search($Location, $NewArray))
        {
            $ReVisited = $Location;
            echo $ReVisited;
        }
    }
    echo $Location;
}
?>

我做错了什么？ 另外，如果您看到可以改进的代码的整体内容，请告诉我。 感谢任何提示或答案！

Answer 1

与其他答案一样：您的代码不起作用的关键原因是您每次调用函数 Previous() 时都在重新初始化 $newArray，因此数组为 null。

要解决这个问题，您可以按照@Andrew 的建议使用 global。

但是，您的代码似乎有点杂乱无章且过于复杂......似乎不需要两个单独的函数？整个过程实际上是一个功能。

如果你把它当作一个函数，那么你就不需要使用globals，你可以用更少的代码行来完成它！

$NewInput    = ">>^^<>>";
$arr         = str_split($NewInput, 1);
$x           = 0;
$y           = 0;

directions($x, $y, $arr);

function directions($x, $y, $arr){
    $used_points = []; // Array of points which have been passed through
    $repeats     = []; // Array of points which have been repeated
    foreach ($arr as $item) {
        switch ($item) {
            case '^':
                $y += 1;
                break;
            case 'v':
                $y -= 1;
                break;
            case '>':
                $x += 1;
                break;
            case '<':
                $x -= 1;
                break;
        }

        $current_point = "{$x}.{$y}";
        $repeat        = in_array($current_point, $used_points); // Checks if the point is repeated
        echo $current_point . (($repeat) ? " REPEAT\n" : "\n");  // Outputs current point and whether it's a repeat

        $used_points[] = $current_point; // Add current point to $used_points array
    }
}

/*  Output:
        1.0
        2.0
        2.1
        2.2
        1.2
        2.2 REPEAT
        3.2
*/

补充说明

开关

switch 语句实际上只是以更易于阅读的方式编写if...elseif 语句的另一种方式。从功能上讲，它执行相同的工作（即将给定值与另一个值进行比较）。

下面的三个代码示例显示了比较值的不同方式。第三个例子实际上就是switch 语句。

注意在switch 和while 语句中需要break 才能退出语句。如果您省略它，那么代码将继续在页面下方运行，超出您的预期。

## Example code with switch
switch ($test_case){
    case 1:
        // Do something if $test_case == 1...
        break;
    case 2:
        // Do something if $test_case == 2...
        break;
}

## Example code with if/elseif
if($test_case == 1){
    // Do something if $test_case == 1...
}
elseif($test_case == 2){
    // Do something if $test_case == 2...
}

## Example of if statements in a loop equivalent to switch
while(TRUE){
    if($test_case == 1){
        // Do something if $test_case == 1...
        break;
    }
    if($test_case == 2){
        // Do something if $test_case == 2...
        break;
    }
    break;
}

{ 花括号 } 和双引号

你可以直接在双引号内引用变量；这通常会使代码更容易阅读和更好看。示例：

echo $a . ", " . $b . ", " . $c . ", " $d;
echo "$a, $b, $c, $d";

但是，在某些情况下，执行上述操作可能会产生意想不到的结果。花括号有助于将其保持在最低限度，并且是一个很好的实践 - 它们清楚地指定了您要引用的变量。

$current_point = "$x.$y"; // is equivalent to....
$current_point = "{$x}.{$y}";

in_array($needle, $haystack)

in_array 只是一种检查数组中是否存在值的方法。如果找到该值，则该函数返回TRUE，否则返回FALSE。

线...

$repeat = in_array($current_point, $used_points);

...因此将$repeat 设置为TRUE 或FALSE。

三元运算符

三元运算符实际上是一个简写的if...else 语句。

## Example of if/else
$output = "";
if($value == TRUE){
    $output = 1;
}
else{
    $output = 2;
}

## Example with ternary operator
$output = ($value) ? 1 : 2;

在您的情况下，我们使用它来检查 $repeat 是否为 TRUE（即当前坐标存在于已访问的坐标数组中）。如果是TRUE，那么我们就输出 REPEAT\n或者\n，如果不是。

echo ($repeat) ? " REPEAT\n" : "\n";

整行……

echo $current_point . (($repeat) ? " REPEAT\n" : "\n");

...然后输出$current_point，后跟 REPEAT\n 或\n。

Answer 2

如果我正确理解您想要的结果，我不是 100%，但以下是我看到的问题： 在 Previous 函数中不断地重新定义 $NewArray，因此每次运行它时，数组都会变为空。您必须将该变量放在函数之外，并使用global 范围引用它。我还将它重命名为 $previousLocations。这是新功能：

$previousLocations = array();
function Previous (string $x, string $y)
{
    global $previousLocations;
    $Location = $x . '.' . $y;
    echo "<br>";
    if(in_array($Location, $previousLocations)) {
      // Already visited location
      echo "Revisiting: " . $Location;
    } else {
      $previousLocations[] = $Location;
      echo $Location;
    }
}

输出： 1.0 2.0 2.1 2.2 1.2 重访：2.2 3.2

如果我误解了什么，请纠正我。