これはもっときれいにすることができます...あなたの変数に説明的な名前を付け、いくつかのコメントを追加することを学ぶときは、良いアイデアです。
私はここでこれをやって、より明確になっていることを期待しています。
は、だから私は...
NOTEをより消化のセクションにこれを分解して、それをコメントしている。これと同じ機能を実行するためのより良い方法があります...
/**
* Determine if String2 matches the end of String 1
*
* Example: String1 Python
* String2 __thon
* would return true, otherwise false
*
* @param $string1
* @param $string2
* @return string
*/
function is_str2_at_end_of_str1($string1, $string2) {
$length_of_string1 = strlen($string1);
$length_of_string2 = strlen($string2);
// Get the position of where string2
// Only works if $length_string1 > $length_string2
$position_in_string1 = $length_of_string1 - $length_of_string2;
// Get the characters at the end of string1 that is the same size as string2.
// Go to php.net and look up substr()
$string1_end_section = substr($string1, $position_in_string1, $length_of_string2);
if ($string1_end_section == $string2) {
return "true"; // This is a string and not a boolean true/false
} else {
return "false"; // This is a string and not a boolean true/false
}
}
echo is_str2_at_end_of_str1("Python", "thon") . "\n";
echo is_str2_at_end_of_str1("JavaScript", "ript") . "\n";
それはちょうど戻っている '真サブストリングがメインの 'string'に存在するか否かに基づいて' false'を返します。あなたの関数名は自明ですが、混乱は何ですか? –