2

I can't decide on w/c technique to go w/... although I am tempted to go the Array way as it is much more straight forward and easier/cleaner to code...

lemme explain in a simple way..

For my messaging app, I have 2 tables

  1. Contacts - contains name, and phone number (currently 5,000+ records)
  2. Messages - messages and the phone number, but not the name of the member

I need to list 1000 Messages, w/ the corresponding phone number and name if possible...

eg.

  1. "Hi dude" - from 565-1111 (John)
  2. "Bring Cheese" - from 565-2222 (Bieber)
  3. "Hehehe " - from 565-2332
  4. "iron man is cool!" - from 565-7748 (Arnie)

etc...

I would normally join contacts to messages via the phone number... pretty standard stuff... (but in reality my queries are a bit more complex than a simple left join)

Now, i've been playing around w/ the idea of fetching all contacts and putting it into an array where my app/system could use it in a much easier , more straight forward and cleaner code... where:

  1. query SELECT * contacts

and store them into an array:

contacts['555-7748'] = "John";
contacts['555-1111'] = "Joe";
contacts['555-2233'] = "Borat";
contacts['555-4234'] = "Arnie";
etc

this way i can easily lookup the NAME of the Contact via accessing the array, many times in my page/script, just by simply doing:

$name = contacts[$phoneNumber];

I have actually implemented this and it's really cool .. no more join headaches and re-querying contacts table etc..

now the question is.. is this OK? Practical? or could i be hitting some memory cap or something here?

it's basically running JOIN query on EACH message my script outputs or store ALL THE CONTACTS/RECORDS into array (memory) and refer to that instead, making queries are more simple.. my only worry is, querying thousands of records into an array, may be a bit too much? or is this really OK/practical? specially since there are times when my script needs to fetch specific contacts more than once...

To add, I was also thinking of storing that array into a Session variable! lolz.. so my scripts can simply share this data.. lol

looking forward to some good inputs Thanks guys.

---- UPDATE as of 03-31-2013 (Sunday) ----

Ok... I'm thinking about a totally different approach now..

1) SINCE all this is really me needing to show the "CONTACT NAME" for each message for neat/presentation purposes (better to see the name of the contact on a message rather than phone number yeh?) ....

2) and since each message will need to show 2 contact names (receiver and sender) based on their phone numbers on the MESSAGES table.....

3) I now am thinking of ADDING 2 fields into the MESSAGES table instead... "sender_name" and "receiver_name" .. I have seen Bulletin board / Forum software use this technique a lot ..

4) when system receives a message , it checks for existing Contacts entry of the numbers associated w/ the message...and if it exists, get the respective names and include it into storage/insert

5) ..problem is if during the message was received, contact info isnt available, so no NAMES are saved.. and later CONTACT info is then created... the OLD messages wont show the contact names as intended...

6) 3 way solution... 1st, evertime a new contact is added/changed my php will make updates to all messages w/ the matching numbers... 2nd, from time to time perhaps once / month, run a "synchronize routine" to make sure things are in sync.. or, when a user Actually OPENS a message, mandatory routine will run to check if contact name of that particular message is still in sync w/ the contacts table information... and make updates as necessary..

hmmm... better approach you thing? this way, i just need to issue straight forward SELECT queries , simple, fast.. but am able to display the names..

then again, i still kinda find that QUICK array reference thing so cool though.. lolz

  • indecisive
Improve this question
8
  • 1
    Most people would prefer pulling the data in one query, no repetition of code. Therefore, Inner Joins. Commented Mar 30, 2013 at 15:09
  • 1
    Redis.io and possibly Memcache would work better if you're trying to make it set across all users. Commented Mar 30, 2013 at 15:18
  • @DarylGill - well Array is more straight forward than having to do Joins and re-issuing SQL query again further down the script... say for ex, i have 3 sections in my page where in each section i output messages specific to it, and for each message i need to lookup/issue join on contacts table... vs.. just issuing a very simple query on the messages table and find the contact via existing array (of about 5-10K records though) Commented Mar 30, 2013 at 15:28
  • @JaredFarrish - perhaps later down the road..for now i wana stay away from dependencies and complications.. tnx tho Commented Mar 30, 2013 at 15:28
  • 1
    @BrownChiLD It's down to preference at the end of the day. I prefer Joins when performing something such as this, but if you prefer arrays. There will be no difference.. Just preference. Commented Mar 30, 2013 at 15:39

2 Answers 2

Reset to default
1

Do you have access to the DB? If so, I'd suggest creating a view so that your queries from PHP are more simple. Maybe that will simplify things for you. I'd still suggest using queries instead of simply large arrays. That being said, if you know the load you are going to place on these and the array works fine there shouldn't be any reason to avoid it. Best practices are good, but there is something to be said for ease of coding which breeds better maintenance.

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

2 Comments

yes but the thing w/ queries is i have to re-issue queries to specifc contacts a few times in the script.. obviously displaying contacts w/ messages here and there on the page... so it's actually pin-point loading each contact to a message via individual and/or join queries or loading all the contacts into array and simply use array-referencing to get the needed contact.. that said what bothers me is the idea that i've loaded say, 10,000 contacts into a multi-dimentional array , for the purpose of using perhaps just 1000 contacts .. but it is really the easiest/cleanest way to go about this..
Sounds like you've made your decision then.
1

It sounds as if you've decided how to proceed, in which case I'd suggest looking into the most efficient method to manage your list. "Sharing" it across sessions I've not encountered (who do not share sessions for security reasons, or shouldn't). You've nixed redis.io and Memcached as too complicated.

One way you could do this, and minimize it's impact in efficiency, would be to setup a CRON or configure some workers to periodically run a script that outputs the contacts table to a PHP-parseable file.

For instance:

<?php

// Let's not run this if the app isn't accessing it.
if (!defined('APP_RUNNING_CONST')) {
    header('HTTP/1.1 401');
    exit;
}

$contacts = $db->query()... // Get the contacts list.
$lines = array();
$num_contacts = $contacts->getRows();

while ($contact = $contacts->nextRow()) {
    $contact->field->contactname = addslashes($contact->field->contactname);

    array_push($lines, "\$contacts['$contact->field->phone'] => '$contact->field->name';");
}

if (!empty($lines)) {
    $o_array = implode(PHP_EOL, $lines);
    $c_date = date('r');

    $o_array = "<?php

// CRON contact table cache to array
// Saved $c_date by __file__
// Number rows: $num_contacts

$contacts = array();
$o_array

";
}

// Write the PHP file over the last:
$contactsfilename = 'contacts.php' ...

Then all you need to do is 

include '/secure/lists/contacts.php';

in the pages you need it. You should be careful how you replace the file (copy, save and delete when finished) as well.

Some other improvements you could make might be to output files for each area code/prefix....

contacts.403.555.inc.php
contacts.403.585.inc.php
contacts.461.550.inc.php

And then create a function to autoload the appropriate file group only as you need them ("lazy loading") and use static variables to keep the references in the function. This also overcomes the global variable problem, as well as helps keep the functional code accessing the files in a central place. Of course you could use a class, maybe with an array interface implementation.

Anyhow, yet another option could be to use a ini file format.

Improve this answer

1 Comment

Hi Jared I actually thought about a similar way like this, using files, but this you've suggested is much better so thanks for the idea... my only concern is file access is much slower than a simple pull from db or session though, i would think (?).. but this is indeed a good way to share things securely. I will way things based on my needs and load..

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.