[menukadevinda]

Hi all,

I just implemented count function in php that return number of elements in a array.

 function mycount($arr)
{
       $i=0;
       while(!empty($arr[$i])){
           
           $coun=$i;
           $i++;
           
       }
        return $coun+1;
}

now we can use this methode to get number of elements in array. If you think there is any problem or issue let me know.

thanks in advance..

[Atli]

I see two issues with your function:

• You are duplicating the counter variable. You already have $i, which represents the count. Why do you copy it over to $coun? You may as well just remove $coun altogether and return $i.
  
  
• Using empty() to determine if an element is set skews your results, if your intent is really to return the number of set elements in an array. It will exclude elements set to values like the number 0, FALSE, empty arrays, empty strings, and of course NULL. An element set to any of these (except perhaps NULL, one might argue) should be considered a part of the count. - You'd do better to use isset()

But this function is ultimately unnecessary, as I am sure you already know, seeing as PHP already has a count() function built in. Unless of course your goal is to do some sort of manual filtering of what is counted.

[Dormilich]

Atli, on 07 July 2012 - 09:15 AM, said:

Unless of course your goal is to do some sort of manual filtering of what is counted.

and in that case you would filter the array and count() the resulting array.

[JackOfAllTrades]

Quote

Posts: 435
Joined: 14-April 11

/facepalm

[menukadevinda]

thanks atli,

I could figure out weak points of my programme. yes u r correct it is usless to overide count function but I intended to learn something from my weak points.

 function mycount($arr)
{
       $i=0;
       while(!empty($arr[$i])){
           
           $coun=$i;
           $i++;
           
       }
        return $i;
}

thanks you guys and I am too thinking of JackOfAllTrades post.

hi ,

 function mycount($arr)
{
       $i=0;
       while(!empty($arr[$i])){
           
        
           $i++;
           
       }
        return $i;
}

corrected one should like this be and I notice my editing previlage also remove you guys

[macosxnerd101]

Yes, you upvoted me for removing your editing privileges here. You removed your code b/c it was a school assignment, which violates the rules here.

[menukadevinda]

ok. I am sorry about that and can I get my rights to edit back?

[CTphpnwb]

If you're trying to learn something, start with proper indenting:

function mycount($arr)
{
	$i=0;
	while(!empty($arr[$i])){           
		$i++;
	}
	return $i;
}

Also note that this function will not always return the actual size of the array, but merely the first empty index. For example, this returns 2:

<?php
$test = array();

$test[0] = 100;
$test[1] = 200;
$test[3] = 400;
$test[4] = 500;

echo mycount($test);

function mycount($arr)
{
	$i=0;
	while(!empty($arr[$i])){           
		$i++;
	}
	return $i;
}
?>

[menukadevinda]

Hi CTphpnwb,

it is amazing can you explain how it out puts 2? it should be 4.

and as Atli mention to use isset() methode instead of empty() , out put 0.

[CTphpnwb]

Does $test[2] contain a value? What will happen when $i is 2?

[menukadevinda]

ahhh,though I knew what you was talking,I didnt notice $test[2] was empty.

thanks..

[bluman]

A much easier way to do this is using the foreach structure. I'm not saying it's optimal. Also there is the count function already implemented in the standard library so this is kind of pointless.

function mycount($array)
{
	$count = 0;
	foreach ($array as $record)
		$count++;
	return $count;
}

[Dormilich]

that is not even doing any filtering, i.e. it is the same as count()

[menukadevinda]

HI,

this will out put the correct result.
finally, this works fine and I could grasp clearly empty() and isset() method in php.

<?php
        //function to count number of elemenst in a array
        function mee($arr){
       $i=0;
       while(isset($arr[$i])){
           
        
           $i++;
           
       }
        return $i;
       }
       
       
$test = array(100,200,300,400);

$test[0] = 100;
$test[1] = 200;
$test[3] = 400;
$test[4] = 500;

echo mee($test);
    
   
?> 

[CTphpnwb]

It may work, but you've failed to grasp the concept of proper indenting. It's important, so learn it.