smiley到图像优化的PHP代码

这是一个好代码还是可以优化

 $tempmsg = str_replace("[:)]","<img src='img/smiley0.png' title='Smile' height='100' width='100'>",$tempmsg);

$tempmsg=str_replace（“[：）]”，“”，$tempmsg）；

如何优化？通过一行代码很难对事情进行优化。我建议添加笑脸作为ALT文本，但是对于那些没有图像功能的人或者如果图像由于某种原因无法传输。但这不是“优化”。这是一个很好的设计

 $tempmsg = str_replace(blah blah blah .... alt=":)" ...);

---

如何优化？通过一行代码很难对事情进行优化。我建议添加笑脸作为ALT文本，但是对于那些没有图像功能的人或者如果图像由于某种原因无法传输。但这不是“优化”。这是一个很好的设计

 $tempmsg = str_replace(blah blah blah .... alt=":)" ...);

1/可以优化吗？ 这很好，您没有使用正则表达式（preg_replace），所以这里不用担心

2/这是一个好代码吗 对我来说，它不是真正可读的。如果你需要替换其他的笑脸怎么样？它是不可重复使用的。这是提高可读性、增加可重用性的第一步。。。它可能会慢一点，但在需要优化之前，我倾向于支持重用性而不是性能

function replace_smiley($text) {
  $replacements = array( 
     ":)" => "image1.jpg", 
     ":(" => "image2.jpg"
  );

  $out = $text;
  foreach ($replacements as $code => $image) {
    $html = '<img src="img/' . $image . ' alt="' . $code . '" height="100" width="100" />';
    $out = str_replace($code, $html, $out);
  }
  return $out;
}

函数替换\u smiley（$text）{
$replacements=数组（
“：）”=>“image1.jpg”，
“：（“=>”image2.jpg”
);
$out=$text；
foreach（$替换为$code=>$image）{
$html=''；
$out=str_replace（$code，$html，$out）；
}
退回$out；
}

当然，根据要求，总有改进的余地。

1/可以优化吗？ 这很好，您没有使用正则表达式（preg_replace），所以这里不用担心

2/这是一个好代码吗 对我来说，它不是真正可读的。如果你需要替换其他的笑脸怎么样？它是不可重复使用的。这是提高可读性、增加可重用性的第一步。。。它可能会慢一点，但在需要优化之前，我倾向于支持重用性而不是性能

function replace_smiley($text) {
  $replacements = array( 
     ":)" => "image1.jpg", 
     ":(" => "image2.jpg"
  );

  $out = $text;
  foreach ($replacements as $code => $image) {
    $html = '<img src="img/' . $image . ' alt="' . $code . '" height="100" width="100" />';
    $out = str_replace($code, $html, $out);
  }
  return $out;
}

函数替换\u smiley（$text）{
$replacements=数组（
“：）”=>“image1.jpg”，
“：（“=>”image2.jpg”
);
$out=$text；
foreach（$替换为$code=>$image）{
$html=''；
$out=str_replace（$code，$html，$out）；
}
退回$out；
}

---

当然，根据需求，总有改进的余地。

我将这样构建它：

function replace_smiley($text){
    $replacements = array(
        ':)' => 'image1.jpg',
        ':(' => 'image2.jpg'
    );

    $search = array();
    $replace = array();

    foreach($replacements as $key => $val){
        $search[] = $key;
        $replace[] = '<img src="images/' . $val . '" alt="' . $key . '" />';
    }

    return str_replace($search, $replace, $text);
}

函数替换\u smiley（$text）{
$replacements=数组(
“：）”=>“image1.jpg”，
“：（”=>“image2.jpg”
);
$search=array（）；
$replace=array（）；
foreach（$替换为$key=>$val）{
$search[]=$key；
$replace[]=''；
}
返回str_replace（$search，$replace，$text）；
}

---

它高效、易于维护，并且可能尽可能优化。

我将这样构建它：

function replace_smiley($text){
    $replacements = array(
        ':)' => 'image1.jpg',
        ':(' => 'image2.jpg'
    );

    $search = array();
    $replace = array();

    foreach($replacements as $key => $val){
        $search[] = $key;
        $replace[] = '<img src="images/' . $val . '" alt="' . $key . '" />';
    }

    return str_replace($search, $replace, $text);
}

函数替换\u smiley（$text）{
$replacements=数组(
“：）”=>“image1.jpg”，
“：（”=>“image2.jpg”
);
$search=array（）；
$replace=array（）；
foreach（$替换为$key=>$val）{
$search[]=$key；
$replace[]=''；
}
返回str_replace（$search，$replace，$text）；
}

---

它高效、易于维护，并且可能尽可能优化。

使用“优化”我想知道我的代码中是否有错误使用“优化”我想知道我的代码中是否有错误使用单引号而不是双引号：更快，在生成的HTML中获得双引号，如果使用XHTMLAnd使用单引号而不是双引号，哪一个更好或是必需的：更快，在生成的HTML中会得到双引号，如果使用XHTMLLike向函数添加

return$out

，哪一个更好或是必需的？；-）是的，这对于函数的可用性来说是一个很大的改进：）比如在函数中添加

return$out

？；-）是的，这在功能的可用性方面是一个很大的改进：）