Codeigniter issue with loop and Condition - php

Question

I am loading a view like this
if($type == 'view'){
// do something
}else if($type == 'insert'){
// Here i am making a form
?>
<select>
<?php
foreach($applications as $row)
{
?>
<option value = "<?php echo $row->id;?>">
<?php echo $row->name;?>
</option>
<?php } ?>
</select>
<?
}else{
//do some thing else
}
Now this is the error i am encountering:
Parse error: syntax error, unexpected T_ELSE in url on line xx
The weird thing is that if i comment out the loop if works fine. What is the problem and how can it be resolved?

Answer 1

you need to use codeigniter loops in view way like this:
<?php foreach ($applications as $row):?>
<option value = "<?php echo $row->id; ?>">
<?php echo $row->name; ?>
</option>
<?php endforeach;?>

Answer 2

It works fine for me... try again
<?php
if($type == 'view'){
// do something
}else if($type == 'insert'){
// Here i am making a form
?>
<select>
<?
foreach($applications as $row){
?><option value = "<?php echo $row->id;?>"><?php echo $row->name;?></option>
<?php
}
?>
</select>
<?
}else{
//do some thing else
}

Answer 3

Using switch, you have a more elegant way of code writing and of course faster, try:
switch($type){
case 'view':
//do something
break;
case 'insert':
echo '<select>';
foreach($applications as $row){
echo '<option value="'.$row->id.'">'.$row->name.'</option>';
}
echo '</select>';
break;
default:
//do something else
break;
}

Answer 4

Maybe your want like this code:
<?php
if ($type == 'view')
{
// do something
}
elseif ($type == 'insert')
{
// Here i am making a form
?>
<select>
<?php
foreach($applications as $row)
{
?>
<option value = "<?php echo $row->id;?>"><?php echo $row->name;?></option>
<?php
}
?>
</select>
<?php
}
else
{
//do some thing else
}
?>