2013-09-28 51 views
0

所以基本上我打开一个大文件(我认为它很大--19MB),并试图梳理出任何不是4个字母的.com,并在今天的日期从注册表中删除。所以我在下面试图做的是调用这个文件,并且为它的每一行找到带有日期的字符串,字符数小于或等于35(在剥离字符之前)。我在做什么错? strpos,strlen和preg_match

然后我希望它能检查它是一个.com没有缺失并且不包含连字符。

我错过了什么?

<?php 

date_default_timezone_set('UTC'); 

$extension = '.com'; 
$lines = file('PoolDeletingDomainsList.txt'); 
//$lines = file('testdomains.txt'); 
$date = date('n/j/Y'); 

echo "<b>4 Letter premiums for ". $date .":</b><br />"; 

foreach($lines as $line) 

if ((false !== strpos($line,$date)) && (35 <= strlen($line))) { 

    $line = preg_replace('/12:00:00 AM,AUC\b/','<br />', $line); 
    $line = preg_replace('/,9\/28\/2013/', '', $line); 

    if ((false !== strpos($line, $extension)) && (0 === preg_match('#\d#',$line)) && (0 === preg_match('/-/', $line))){ 
    echo $line; 
} 
} 
?> 
+1

我可以注意到的第一件事是35 <= strlen意味着“至少35个字符长的行”,而不是“35个字符或更少” –

+0

我是个白痴。 Drop文件今天不包含:@ – CodingNoob

+0

所有正在使用上述+ Paul的修复程序。 – CodingNoob

回答

1

尝试(35 >= strlen($line)而不是(35 <= strlen($line)如果你想行35个字符以下。就我个人而言,我更喜欢以其他方式进行比较,例如strlen($line) <= 35,我认为它更具可读性,可以避免像刚刚制作的那样的错误:)

+0

是的,我切换它们,原来如上所述,我想这没什么区别,但容易忘记翻转符号。 。 – CodingNoob