这段 php 代码如何优化？

This topic created in 4089 days ago, the information mentioned may be changed or developed.

初学php不久，一道题目，个人的做法是吧这些标题分别存到不同的数组，然后针对不同的数组进行循环遍历分别+=. 不过别人说这样还不如原来的，所以想请教下正确的方法是什么

$xcount=0;
     $pos = strpos($newtitle, '賣腎');
 if($pos) $xcount+=3;
     $pos = strpos($newtitle, '卖肾');
 if($pos) $xcount+=3;
     $pos = strpos($newtitle, '收肾');
 if($pos) $xcount+=3;
     $pos = strpos($newtitle, '２４小时');
 if($pos) $xcount+=3;//２４小时
     $pos = strpos($newtitle, '咨询热线');
 if($pos) $xcount+=3;//２４小时
     $pos = strpos($newtitle, '找小姐');
 if($pos) $xcount+=6;//２４小时
     $pos = strpos($newtitle, '服务信息');
 if($pos) $xcount+=6;//２４小时
     $pos = strpos($newtitle, '学生妹');
 if($pos) $xcount+=6;//２４小时
     $pos = strpos($newtitle, '找小姐');
 if($pos) $xcount+=6;//２４小时

$pos = strpos($newtitle, '男女公关');
if($pos) { echo 'Fail, No Uid.';exit;}
     $pos = strpos($newtitle, '女公关');
if($pos) { echo 'Fail, No Uid.';exit;}
     $pos = strpos($newtitle, '男公关');
if($pos) { echo 'Fail, No Uid.';exit;}
     $pos = strpos($newtitle, '公关');
if($pos) { echo 'Fail, No Uid.';exit;}
     $pos = strpos($newtitle, '招聘');

strpos

newtitle

POS

24 replies • 2015-03-03 12:14:56 +08:00

nigelvon

Mar 2, 2015

循环遍历为啥不如原来的，同样的代码重复N遍很爽么。

lijinma

Mar 2, 2015

原题目是啥？

EXDestroyer

Mar 2, 2015

@nigelvon 对方的说法是，逻辑上没有原来的好，看上去不好维护
我是把+=+,+=3，以及fail的三部分分开存到三个数组，对他们分别进行遍历

EXDestroyer

Mar 2, 2015

@lijinma http://d1kt.cn/course/pluginfile.php?file=/5031/mod_assign/intro/ssw.php
修改这段代码，使程序结构更漂亮更好维护一点

invite

Mar 2, 2015

不是应该： {key:value} 么？

raincious

Mar 2, 2015

能将原题贴上来么？

这代码写的不好。不知道你什么意图，如果是搜索字符串是否存在，那么if($pos)这样的用法就是错的。

branchzero

Mar 2, 2015

猜测应该是违禁词屏蔽？
<?php
$words = ['賣腎','关键词太多了自己后面接着加'];
foreach($words as $word){
if(strpos($newtitle, $word)) exit('Fail, No Uid.');
}
?>
搞定

Moker

Mar 2, 2015

话说那些关键词没人注意到么

jswh

Mar 2, 2015

如果我写的话估计是这样子的

$config = [
'xCountPlus3' => [],
'xCountPlus6' => [],
'NoUidAndExit' => [],
];
$xCount = 0;
foreach($config as $action => $keywords) {
foreach($keywords as $word) {
if(strpos($newTitle, $word) !== false) {
$action($xCount);
}
}
}
function xCountPlus3(&$xCount) {
$xCount += 3;
}
function xCountPlus6(&$xCount) {

}
...

b821025551b

Mar 2, 2015

7L正解

hwiiago

Mar 2, 2015

$sensitiveWords = array('賣腎' => 3, '卖肾' => 3, '收肾' => 3, '找小姐' => 6);
$xcount = 0;
foreach($sensitiveWords as $key => $value)
{
if (strpos($newtitle, $key) !== false)
{
$xcount += $value;
}
}

zhujinliang

Mar 2, 2015

我的话会用正则搞，让正则处理器自己去优化

jk2r

Mar 2, 2015

和敏感词过滤类似呀。

你需要一个“多模式匹配”的PHP实现（AC状态机／Double-array Trie树），查出有的词；
然后k-v查分数吧

qqjt

Mar 2, 2015

只有我好奇这个是用在哪里的么，又是找小姐又是学生妹……

EXDestroyer

Mar 2, 2015 via Android

@raincious 原题在4楼了

msg7086

Mar 2, 2015 via iPhone

if pos 什么鬼

br00k

Mar 2, 2015

是我应该会这样吧，感觉这种还是用数组方便，就是title也是数组for嵌套，也比单纯if好，不然数组内容多了，会增加太多无用代码。

$title = "xxx";
$str = ["賣腎", "卖肾", "收肾", "２４小时", "咨询热线", "找小姐", "服务信息"];
foreach ($str as $value) {
if (false !== strpos($title, $value)) {
echo $value;
exit;
}
}

alex321

Mar 2, 2015

$str = array('賣腎', '卖肾', '收肾', '２４小时', '咨询热线', '找小姐', '服务信息','男女公关','男公关','女公关','公关','招聘');
preg_match('('.implode($str, '|').')', $newtitle, $matches);
return strlen(implode($matches, ''));