<?php if ($arFields[$key]['type'] == 'text' )
{echo ' ';}
else
{echo '' . $_POST[$key] . '';}
?>
стоило бы написать так:
<?php if ($arFields[$key]['type'] == 'text' ) : ?>
<input size="40" type="<?=$arFields[$key]['type']?> name="<?=$key . ' value ="' . $_POST[$key] ?>">
<?php else: ?>
<textarea rows="10" cols="35" name="<?=$key?>" ><?=$_POST[$key]?></textarea>
<?php endif; ?>
Это касаемо читабельности. Плюс комменты не мешало бы делать в коде.
foreach тоже более читабельный в виде:
<?php foreach($fff as $key=>$value): ?>
тут html
<?php endforeach; ?>
Плюс куча лишних переносов и пустых строк, где они не нужны, и нет там где просятся.
ВОТ ЭТО ВАЩЕ ЗЛО:
<?php
}
?>
ПС: почитайте
www.php-fig.org/psr/psr-1
