5

我正在用 PHP 编写一个基本的莫尔斯电码转换器，它可以接受一个字符串并将其转换为莫尔斯电码。它使用关联数组、foreach 循环和 for 循环。它可以工作，但由于某种原因，它会在每个转换后的字符后输出等效于“0”的莫尔斯电码。我不知道 0 是从哪里来的。如果我从关联数组中删除 0，则没有问题，但我也希望能够转换数字。如果有人能够给我一些反馈，那将不胜感激。

这是代码：

<?php
$string = "dog";
$string_lower = strtolower($string);
$assoc_array = array(
    "a"=>".-",
    "b"=>"-...", 
    "c"=>"-.-.", 
    "d"=>"-..", 
    "e"=>".", 
    "f"=>"..-.", 
    "g"=>"--.", 
    "h"=>"....", 
    "i"=>"..", 
    "j"=>".---", 
    "k"=>"-.-", 
    "l"=>".-..", 
    "m"=>"--", 
    "n"=>"-.", 
    "o"=>"---", 
    "p"=>".--.", 
    "q"=>"--.-", 
    "r"=>".-.", 
    "s"=>"...", 
    "t"=>"-", 
    "u"=>"..-", 
    "v"=>"...-", 
    "w"=>".--", 
    "x"=>"-..-", 
    "y"=>"-.--", 
    "z"=>"--..", 
    "0"=>"-----",
    "1"=>".----", 
    "2"=>"..---", 
    "3"=>"...--", 
    "4"=>"....-", 
    "5"=>".....", 
    "6"=>"-....", 
    "7"=>"--...", 
    "8"=>"---..", 
    "9"=>"----.",
    "."=>".-.-.-",
    ","=>"--..--",
    "?"=>"..--..",
    "/"=>"-..-.",
    " "=>" ");
    for($i=0;$i<strlen($string_lower);$i++){
        foreach($assoc_array as $letter => $code){
            if($letter == $string_lower[$i]){
                echo "$code<br/>";
            }
        }
    }
?>
4

4 回答 4

5

主要问题是你做的“超过”了必要。当您可以使用字符串从中获取所需数据时，无需循环您的$assoc_array喜欢。

这也使用更少的资源，而不是循环，a-z并且0-9您只循环所需的字母/数字/空格的确切数量。

/*Rest of your code above*/
for($i=0;$i<strlen($string_lower);$i++){
    echo (isset($assoc_array[$string_lower[$i]])) ? $assoc_array[$string_lower[$i]] . '<br />' : 'ERROR';       
}

由于您的数组包含所有内容，a-z因此0-9您可以轻松调用所需的字母，而不必担心丢失数据。

编辑： 添加了一张isset()支票，几乎不需要它，因为它$assoc_array涵盖了每个需要的字母/数字，但比抱歉更安全。（感谢@Farkie 提醒我）

于 2016-03-22T09:16:29.310 回答
2

最简单的修复方法很简单，就是在回显之后添加一个“中断”：

foreach($assoc_array as $letter => $code){
                if($letter == $string_lower[$i]){
                        echo "$code<br/>";
                        break;
                }
        }

真正的问题是 0 被评估为假，这意味着当它循环它时，它将是一个真值（假 == 假）。

===您可以通过相同的 ( ) 匹配更好地解决它：

foreach($assoc_array as $letter => $code){
                if($letter === $string_lower[$i]){
                        echo "$code<br/>";
                        break;
                }
        }
于 2016-03-22T09:12:11.910 回答
1

也可以使用一点php函数式编程，比如array_reduce()函数http://php.net/manual/en/function.array-reduce.php

避免所有那些丑陋的 for 循环并大大简化您的代码：

$convert = function($carry, $item) {
    $table = array(
        "a" => ".-",
        "b" => "-...",
        "c" => "-.-.",
        "d" => "-..",
        "g" => "--.",
        "o" => "---");
    // Get the correspondent value for the given letter
    $morse = $table[$item];
    // Return the string with appended morse character
    return $carry . $morse;
};

// Split 'dog' into an array, then apply a reduce to convert it to morse
array_reduce(str_split('dog'), $convert);
// ➜  ~ php morse.php                                                     
// -..-----.
于 2016-03-22T09:35:08.180 回答
1

我知道自从发布问题和给出答案以来已经有一段时间了。我想我会添加我为将来发现这个问题的人编写的这个函数。

代码

/**
 * Convert string to morse
 *
 * @param string $string
 * @return string
 */
function str_to_morse(string $string) {
    // Make the string lowercase and create an array of the characters
    $stringParts = str_split(strtolower($string));

    // Define the dictionary
    $morseDictionary = [
        'a' => '.-',
        'b' => '-...',
        'c' => '-.-.',
        'd' => '-..',
        'e' => '.',
        'f' => '..-.',
        'g' => '--.',
        'h' => '....',
        'i' => '..',
        'j' => '.---',
        'k' => '-.-',
        'l' => '.-..',
        'm' => '--',
        'n' => '-.',
        'o' => '---',
        'p' => '.--.',
        'q' => '--.-',
        'r' => '.-.',
        's' => '...',
        't' => '-',
        'u' => '..-',
        'v' => '...-',
        'w' => '.--',
        'x' => '-..-',
        'y' => '-.--',
        'z' => '--..',
        '0' => '-----',
        '1' => '.----',
        '2' => '..---',
        '3' => '...--',
        '4' => '....-',
        '5' => '.....',
        '6' => '-....',
        '7' => '--...',
        '8' => '---..',
        '9' => '----.',
        '.' => '.-.-.-',
        ',' => '--..--',
        '?' => '..--..',
        '/' => '-..-.',
        ' ' => ' ',
    ];

    $morse = '';
    foreach ($stringParts as $stringPart) {
        if (array_key_exists($stringPart, $morseDictionary)) {
            $morse .= $morseDictionary[$stringPart] . '<br />';
        }
    }

    return $morse;
}
于 2018-12-13T11:08:54.707 回答