Merging files with php

Question

So, I have two files In the first file, there is text:hash

In the other file there is hash:pass

I wrote some code to match the hash and then print the text (So when ran, I'd get an output of text:pass) It works fine and it finds all the information, however sometimes it will skip certain ones and just not find anything. (If I go and do it by hand the data is there) So I'm not sure why it will find most of them but not all. Anyway I'm hoping someone can help, the code is below:

<?php
$emailhash = file('emailhash.txt');
$hashpass = file('hashpass.txt');
$list = '';

foreach($emailhash as $data) {
    $data = str_replace("\r\n",'', $data);
    $array_emailhash = explode(":", $data);
    $email = $array_emailhash[0];
    $hash = $array_emailhash[1];

    foreach($hashpass as $data2) {
        $data2 = str_replace("\r\n",'', $data2);
        $array_hashpass = explode(":", $data2);
        $hash2 = $array_hashpass[0];
        $pass = $array_hashpass[1];

        if($hash2 == $hash)
            $list .= $email.':'.$pass."\r\n";

    }
}

file_put_contents('emailpass.txt', $list);

nested foreach, your condition IF will only run after the first foreach finish itself, after the second foreach finish itself, then your conditions will work. clear? — devasia2112, Apr 01 '14 at 17:28
also I found a question very close to yours, see - http://stackoverflow.com/questions/9299635/php-foreach-how-to-assign-to-two-variables perhaps duplicate question — devasia2112, Apr 01 '14 at 18:13

score 0 · Answer 1 · answered Apr 01 '14 at 18:09

You need to iterate your iterator, for example using multipleIterator, since you reads the entire file into an array, do like this.

$iterator = new MultipleIterator();
$iterator->attachIterator( new ArrayIterator( $emailhash ));
$iterator->attachIterator( new ArrayIterator( $hashpass ));

foreach( $iterator as $value ) {
    list($keys1, $keys2) = $iterator->key();
    list($value1, $value2) = $value;
}

Take a look at this -> http://br1.php.net/MultipleIterator

score 0 · Answer 2 · answered Apr 01 '14 at 18:15

This algorithm is very inefficient. try to do this like that

$emailhash = file('emailhash.txt'); $hashpass = file('hashpass.txt'); $hashToEmail = array(); $list = '';

foreach($emailhash as $data) {
    $data = str_replace("\r\n",'', $data);
    $array_emailhash = explode(":", $data);
    $email = $array_emailhash[0];
    $hash = $array_emailhash[1];

    $hashToEmail[$hash] = $email;
}

foreach($hashpass as $data2) {
    $data2 = str_replace("\r\n",'', $data2);
    $array_hashpass = explode(":", $data2);
    $hash2 = $array_hashpass[0];
    $pass = $array_hashpass[1];
    if (array_key_exists($hash2, $hashToEmail)) {
        $list .= $hashToEmail[$hash2].':'.$pass."\r\n";
    } else {
        //what to do if you don't have that password hash
    }
}

Your design has Big O complexity of n^2 [for 1000 rows it will take 1 000 000 iterations] the one without nested foreach has a complexity of 2n. [it will take 2000 iterations]. Try to find a solution for that :)

Anyway in both places there is a issue, because you can have a hash collisions (for two different emails you can have the same hash, then for all emails you will print out same (probably wrong password).

so then check the $hash and $hash2 variables, maybe you're parsing them incorrectly — PolishDeveloper, Apr 01 '14 at 18:52

Merging files with php

2 Answers2