[Create array in class and call function]

Is the purpose of using the foreach (...): and endforeach; to make the code easier to read without having to find the end curly bracket? or is that the correct syntax?

[Create array in class and call function]

<select name="startMonth1" id="startMonth1" class="form-control">
<?php foreach($months as $rows){ ?>
	<option value="<?php echo $rows ?>"><?php echo $rows ?></option>
	<?php } ?>
</select>

Nevermind! I see my error now, thanks for the nudge in the right direction

[Create array in class and call function]

Thanks but when I try this the dropdown list with a 'Warning: illegal string offset' error.

function listMonths() {
return ['january'=>'January', 'february'=>'February', 'march'=>'March', 'april'=>'April', 'may'=>'May', 'june'=>'June', 'july'=>'July', 'august'=>'August', 'september'=>'September', 'october'=>'October', 'november'=>'November', 'december'=>'December'];
}
						
$months = listMonths();

<select name="startMonth1" id="startMonth1" class="form-control">
<?php foreach ($months as $rows) { ?>
	<option value="<?php echo $rows['rows']; ?>" <?php if($startmonth1 == $rows['rows']) echo 'selected'; ?>><?php echo $rows['rows']; ?></option>
	<?php } ?>
</select>

when I print_r($months) I get:

 

Array ( [january] => January [february] => February [march] => March [april] => April [may] => May [june] => June [july] => July [august] => August [september] => September [october] => October [november] => November [december] => December )

[Create array in class and call function]

Hi,

On my site I have dropdown selection boxes which use months, years and others. So to save myself repeating I wrote a quick function which selects all rows in a mysql database so I can call into any dropdown box

Class Fetch {
function getMonths($db){
		try {
			$sql = 'SELECT monthName FROM months';
			$stmt = $db->prepare($sql);
			$stmt->execute();
			return $stmt->fetchAll(PDO::FETCH_ASSOC);
			
		} catch(PDOException $e) {
			echo '<p class=bg-danger center">'.$e->getMessage().'</p>';
		}
	}
}

Then I just use this on any page I need to populate dropdowns (using a foreach)

$months = $fetch->getMonths($db);

This works but I thought it might be better if I created an array which would be easier to edit should it be needed, and so I can get rid of the various tables in my database. 

 

I have tried several combinations thinking I just have the syntax wrong but now am wondering if it's even possible or sensible. Is there a simple way to do this or shall I just stick with what's working at the moment?!

 

This is my latest failure, it bring back empty dropdowns

var $montharray = array('January', 'February', 'March');
	
	function monthsArray(){
		print($montharray);
	}

$months = $fetch->monthsArray();