#12. Kevin on 09 November 2008

@ Nate: Thank you for noticing!

#11. Nate on 06 November 2008

It looks like someone forgot to declare "val" with var.

I think the first line of code should read:

[code]
var key = '', tmp_arr1 = {}, tmp_arr2 = {}, val;
[/code]

Otherwise, "val" is a global variable, I believe.

Is there any benefit to using var in a for loop? Would it be better to also declare "fkey" with "strict"?

Also, I don't know if this is important, but there is no space between "for" and "if" and the opening parentheses in __array_search().

#10. Kevin on 27 August 2008

@ Nosredna: That's an idea that calls for a function rewrite. Interested? ;)

#9. Nosredna on 05 August 2008

Stray thought. The nested loops make me wonder how slow this is for large arrays.

Perhaps you could clone the array, sort the clone, then walk the two arrays deleting the dupes as you go. Sort is probably O(n log n) and maybe it's especially fast when no user sort evaluation routine is passed in.

#8. Kevin on 25 July 2008

@ sankai: Thank you so much! But this array_unique does not support associative arrays (javascript objects). So unlike PHP's implementation, arrays that have non-numeric keys do not work.

Sad to say that my implementation didn't support it either, but I'm looking to replace array_unique with one that does support:

{firstname: 'Kevin', surname: 'van Zonneveld'}

.. style arrays

#7. sankai on 25 July 2008

the code I posted yesterday had big bug!I fixed it as the following:

function array_unique(array,numeric){
    // http://blog.doublekai.org/
    // +   original by: sankai (http://blog.doublekai.org/)
    // *     example 1: array_unique([1,2,3,'1','2','3',1,2,3],true);
    // *     returns 1: [1,2,3]
    // *     example 2: array_unique([1,2,3,'1','2','3',1,2,3]);
    // *     returns 2: [1,2,3,'1','2','3']
    // *     example 3: array_unique([1,'a','a','1','3',3,'b','c','b',1]);
    // *     returns 3: [1,'a','1','3',3,'b','c']
    // *     example 3: array_unique([1,'a','a','1','3',3,'b','c','b',1],true);
    // *     returns 3: [1,'a',3,'b','c']    
    // *     NOTE :require function in_array()    
    var tem_arr = new Array();
    for(i=0;i<array.length;i++){
        if(numeric === true && typeof(array[i]) == 'string' && !isNaN(array[i])){ 
            array[i]=parseInt(array[i],10);
        }
        if(!in_array(array[i],tem_arr)){
                tem_arr[i]=array[i];
        }
    }
    return tem_arr.join(' ').replace(/\s{2,}/g,' ').split(' ');
}

The version need require function in_array

function array_unique(array,numeric){
    // http://blog.doublekai.org/
    // +   original by: sankai (http://blog.doublekai.org/)
    // *     example 1: array_unique([1,2,3,'1','2','3',1,2,3],true);
    // *     returns 1: [1,2,3]
    // *     example 2: array_unique([1,2,3,'1','2','3',1,2,3]);
    // *     returns 2: [1,2,3,'1','2','3']
    // *     example 3: array_unique([1,'a','a','1','3',3,'b','c','b',1]);
    // *     returns 3: [1,'a','1','3',3,'b','c']
    // *     example 3: array_unique([1,'a','a','1','3',3,'b','c','b',1],true);
    // *     returns 3: [1,'a',3,'b','c']    
    // *     DON'T NEED require function in_array()    
    var tem_arr = new Array();
    for(i=0;i<array.length;i++){
        if(numeric === true && typeof(array[i]) == 'string' && !isNaN(array[i])){ 
            array[i]=parseInt(array[i],10);
        }    
        if(tem_arr.length == 0){
            tem_arr[i] = array[i];
        } else {
            var exist = false;
            for(var j=0; j<tem_arr.length; j++){
                if(tem_arr[j] === array[i]){
                    exist = true;
                }
            }
            if(!exist){
                if(array[i] != null){
                    tem_arr[i] = array[i];
                }    
            }
        }
    }
    return tem_arr.join(' ').replace(/\s{2,}/g,' ').split(' ');
}

The version don't need require funciton in_array()

#6. sankai on 24 July 2008

Hi,I try to write a code for array_unique() as the following:

function array_unique(array){
    var tem_arr = new Array();
    for(i=0;i<array.length;i++){
        if(!in_array(array[i],tem_arr)){
            tem_arr[i]=array[i];
        }
    }
    return tem_arr.join(',').split(',');
}

NOTE:It's require function in_array()

#5. sankai on 24 July 2008

Running:

array_unique(['Kevin','Kevin','van','Kevin']);

Return:

['Kevin','van','Kevin']

It's the same problem as @goshki ?

#4. Kevin on 16 June 2008

@ goshki: You're right, it seems that our current version doesn't support associative arrays (javascript objects). Since PHP doesn't distinct those, we need to work on a version that does support objects. It's on my todo list. Thank your for noticing!

#3. goshki on 13 June 2008

Well, there seems to be some kind of a problem with this function. Running:

array_unique(['a','b','c','a','b','c','a','b','c']);

returns:

['a','a','b','c']

#2. Kevin on 02 May 2008

@ duncan: You were not being stupid, we were. Thank you for noticing! The fixed version will be visible shortly.

#1. duncan on 30 April 2008

maybe i'm being stupid, but PHP array_unique returns an array, not a boolean.
"Takes an input array and returns a new array without duplicate values."