validate url array?

Question

I'm writting a small script where the user adds urls to a textarea, I found this litttle function to validate single links:

function isValidURL($url){
 return preg_match('|^http(s)?://[a-z0-9-]+(.[a-z0-9-]+)*(:[0-9]+)?(/.*)?$|i', $url);
}

but I need it to validate an array of links

I use explode to seperate each like this:

$urls = explode("\n", $_POST['urls']);

I understand that I need to loop the array of urls using a foreach but how can I get the foreach $var inside my elseif

Example:

function isValidURL($url){
 return preg_match('|^http(s)?://[a-z0-9-]+(.[a-z0-9-]+)*(:[0-9]+)?(/.*)?$|i', $valid);
}

$urls = array('http://iamvalid.com','iamnotvalid.com','http://iamvalidaswell.com');

foreach ($urls as $check) {
 $validate = isValidURL($check);
}

if($check != $validate) {
//  If fail
 $result = 'fail';
} else {
//  If pass
 $result = 'pass';
}  // if

echo $result;

(used array instead of expode for demo purposes)

Update: Deepaks answer works great but conflicts with my mysql injection prevention function

Conflicting function:

function sanitizeSQL($input) {
 if(get_magic_quotes_gpc() == true){
   $input = stripslashes($input);
  }
   return mysql_real_escape_string(htmlspecialchars($input));
}

What's the question? Explode, iterate, possibly trim each value before validating. What doesn't work? — Fabrício Matté
– Fabrício Matté, Commented Sep 6, 2012 at 4:32

Deepak · Accepted Answer · 2012-09-06 05:52:28Z

1

function isValidURL($url){
   return preg_match('|^http(s)?://[a-z0-9-]+(.[a-z0-9-]+)*(:[0-9]+)?(/.*)?$|i', $url);
}

function sanitizeSQL($input) {
   if(get_magic_quotes_gpc() == true){
      $input = stripslashes($input);
   }
   return mysql_real_escape_string(htmlspecialchars($input));
}

$urls = explode("\n", $_POST['urls']);

$errorcount = 0;
foreach($urls as $url) {
   // next line fixes conflict by cleaning here rather then before
   $url = sanitizeSQL($url);
   $result = isValidURL(trim($url));
   // Do something with ur result
   if(!$result) $errorcount++;

}
if($errorcount>0){
  //Failed
} else {
  //Passed
}

Note

Try to avoid mysql_* functions use mysqli_* or PDO.

edited Sep 6, 2012 at 5:52

answered Sep 6, 2012 at 4:40

Deepak

6,82718 gold badges71 silver badges121 bronze badges

Sign up to request clarification or add additional context in comments.

6 Comments

Anna Riekic Over a year ago

This works but it is conflicting with another function (see first post)

Deepak Over a year ago

What unexpected results are you getting with tat conflicting function ?

Anna Riekic Over a year ago

Your code works without my function in place but not with, my function "cleans" the textarea input before exploding EG: $links = sanitizeSQL($_POST['urls']); $urls = explode("\n", $links);

Deepak Over a year ago

Ok sanitize the urls individually after exploding. because if you sanitize before exploding you might wanna use \, as the exploding delimiter i guess!! can you print the value of $links after sanitizing ?

Anna Riekic Over a year ago

Thank you Deeepak, I put $url = sanitizeSQL($url); inside the foreach loop and its working now.

|

Erik · Accepted Answer · 2012-09-06 04:52:15Z

1

Using your example code (array instead of explode) I suggest something like

function isValidURL($url){
  return preg_match('|^http(s)?://[a-z0-9-]+(.[a-z0-9-]+)*(:[0-9]+)?(/.*)?$|i', $valid);
}

$urls = array('http://iamvalid.com','iamnotvalid.com','http://iamvalidaswell.com');

$result = 'pass';
foreach ($urls as $check) {
  if(!isValidURL($check)) {
    $result = 'fail';
    break;
  }
}

echo $result;

edited Sep 6, 2012 at 4:52

answered Sep 6, 2012 at 4:32

Erik

2,2641 gold badge20 silver badges20 bronze badges

Comments

Romain Bruckert · Accepted Answer · 2017-07-18 08:32:07Z

0

Or maybe a little bit shorter... ?

$urls = array_map('trim', explode("\n", $string));
$validUrls = array_filter(filter_var_array($urls, FILTER_VALIDATE_URL));

answered Jul 18, 2017 at 8:32

Romain Bruckert

2,62032 silver badges50 bronze badges

Collectives™ on Stack Overflow

validate url array?

3 Answers 3

6 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

6 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related