Re-Jigging Arrays

[tomwerner]

I have looked under the array section of the php manual but to no succses ...

I have an array which stores some info:

$event["0"]["title"] = "Event 1";
$event["1"]["title"] = "Event 2";
$event["2"]["title"] = "Event 3";
$event["3"]["title"] = "Event 4";
$event["4"]["title"] = "Event 5";

I am offering a remove function from this array which uses

unset($event["$this->x"] );

However how do I move the indexes up once a 'row'(is that what it is called?) has been removed. Thanks in advance,

Tom

[ahundiak]

http://www.php.net/manual/en/function.array-values.php

[weekender]

instead of actually stating the index in the array, why not just add the data to the next index

$event[] = array("title"=>"Event1", "date"=>"11/03/04");

then, you will have all the events

$event[0]
$event[1]
$event[2]

remembering that arrays start at 0

when you delete and array unset($event[1]), array 2 will now be array 1

[ahundiak]

Weekender,
The problem is that unset does not work quite as expected. Try:

<?php
$a = array('zero','one','two');
print_r($a);
unset($a[1]);
print_r($a);
echo 'Item 1 ' . $a[1] . "\n";
$a = array_values($a);
print_r($a);
?>

Output is:
Array
(
[0] => zero
[1] => one
[2] => two
)
Array
(
[0] => zero
[2] => two
)
Item 1
Array
(
[0] => zero
[1] => two
)
If you are using foreach then it does not matter. But if you unset $a[1] then $a[1] simply does not exist anymore. The indexes do not automatically shift themselves up.

[tomwerner]

Thanks for that, I think array_values is the way to go as user comment on that page of the manual fitted exactly what I needed. On another not any idea what is wrong with these two functions, they are part of a class but I don't think it effects them in anyway. clearUp() just doesn't work, clearUpEvent() does something weird to the array because when I use print_r($this->event) it prints Array() wheras before it gave the contents of multi-dimensional array. Here is the code, if you feel you need the rest of the class I can post it.

  function cleanUp() {
		// Do the clean up loop
  	for ($this->x = 0; $this->x <= $this->count; $this->x = $this->x + 1) {
		  // Check for Non-Valid Line
			// $test = substr_count($this->eventsdata["{$this->x}"], "%^");
			if ($this->eventsdata["{$this->x}"] == "") {
			// Delete that bad row
			echo $this->eventsdata["{$this->x}"];
			unset($this->eventsdata["{$this->x}"]);
			}
		}
  }

function cleanUpEvent() {
	// Do the clean up loop
for ($this->x = 0; $this->x <= $this->count; $this->x = $this->x + 1) {
	  // Check for Non-Valid Line
		if ($this->event["{$this->x}"]["title"] == "" || " ") {
		// Delete that bad row
		unset($this->event["{$this->x}"]);
		}
	}
  }

any help would be appreciated.

[ahundiak]

There is no need to use quotes when accessing an array via a variable. So:
$this->event["{$this->x}"]
should be
$this->event[$this->x]

And don't use instance variables when you don't have to.
$count = count($this->event);
for($i = 0; $i < $count; $i++)
{
echo $this->event[$i];
}

And this won't work as desired:
if ($this->event["{$this->x}"]["title"] == "" || " ") {
Try
$title = $this->event[$i]['title'];
if (($title == "") || ($title == ' ')) {

Take things one step at a time and test as you develop.

[tomwerner]

I tried both but the same behaviour is still exhibited. I normally do do a few lines then test it but it went out of the window because I haven't done OOP before, I thought because quite a few functions depended on eachother it would be better to do the lot then test it, a lesson for next time I suppose. Attached is:

1) add_event.php - Administers events, not finished
2) addEvent.class.php - The class
3) events.data.php - The datafile

I am dispising every line I write to make this script use a flatfile instead of a database (due to client's budget), I suppose I have got complacent with having MySQL!