Die Nachteile merkst Du wenn die Fremdcode mit tausend functions debuggen darfst.
Das Login wird einmal im Script gebraucht - wieso dann eine function ?
Das ist nur Mehraufwand beim Tippen.
Sehr beliebt sind z.B. auch functions welche mysql-Abfragen beinhalten
und welche dann zig mal aufgerufen werden.
Ist ja so schön einfach:
Dann findet man z.B: getUserData($id); an allen möglichen und unmöglichen Stellen im Script und jedes mal wird eine Query abgesetzt.
Hier ein Beispiel:
PHP-Code:
function makeAdminList ()
{
$active = array("Nein", "Ja");
$result = mysqlGetObjects("admin");
while($data = @mysql_fetch_array($result))
{
if($data['picture']=="")
$picture = "dummy.jpg";
else
$picture = $data['picture'];
echo "
<form action = '".self."?action=admin' method = 'POST'>
<br>
<br>
<table width = '100%' cellspacing = '0' cellpadding = '0'>
<tr>
<td>
<img src='img/objekt-img/".$picture."' alt='Objekt-Ansicht' title='Objekt-Ansicht' width='180' height='134' />
</td>
<td valign='top'>".$data['title']."</td>
<td width = '*'> </td>
</tr>
<tr>
<td width='134' class='fliesstext-schwarz-detail' valign='top' style = 'background-color:#CACACA'>Objekt-Nummer:</td>
<td width='350' valign='top' class='fliesstext-schwarz-detail' style = 'background-color:#CACACA'>".$data['isNr']."</td>
<td width = '*'> </td>
</tr>
".objectCheck($data['id'], $data, "yes")."
<tr>
<td width='134' class='fliesstext-schwarz-detail' valign='top' style = 'background-color:#CACACA'>Ort / Ortsbereich:</td>
<td width='350' valign='top' class='fliesstext-schwarz-detail' style = 'background-color:#CACACA'>".$data['location']."</td>
<td width = '*'> </td>
</tr>
<tr>
<td width='134' class='fliesstext-schwarz-detail' valign='top'>Aktiviert:</td>
<td width='350' valign='top' class='fliesstext-schwarz-detail'>".$active[$data['status']]."</td>
<td width = '*'> </td>
</tr>
<tr>
<td width='134' class='fliesstext-schwarz-detail' valign='top'>Top Objekt:</td>
<td width='350' valign='top' class='fliesstext-schwarz-detail'>".$active[$data['ref']]."</td>
<td width = '*'></td>
</tr>
<tr>
<td width='134' class='fliesstext-schwarz-detail' valign='top' style = 'background-color:#CACACA'>Exposé-Nummer:</td>
<td width='350' valign='top' class='fliesstext-schwarz-detail' style = 'background-color:#CACACA'>".$data['isNr']."</td>
<td width = '*'> </td>
</tr>
<tr>
<td>
<br>
<input type = 'hidden' name = 'id' value = '".$data['id']."'>
<input type = 'submit' name = 'adminaction' value = 'Objekt editieren'>
<br>
</td>
<td colspan = '2'> </td>
</tr>
</table>
</form>
";
}
}
function makeAdminRefList ()
{
$result = mysqlGetRefObjects();
while($data = @mysql_fetch_array($result))
{
echo "
<form action = '".self."?action=admin' method = 'POST'>
<br>
<br>
<table width = '100%' cellspacing = '0' cellpadding = '0'>
<tr>
<td>
<img src='img/objekt-img/".$data['path']."' width='180' height='134' />
</td>
</tr>
<tr>
<td>
<br>
<input type = 'hidden' name = 'delRefObject' value = '".$data['id']."'>
<input type = 'submit' name = 'adminaction' value = 'Referenz Objekt löschen'>
<br>
</td>
</tr>
</table>
</form>
";
}
}
function makeAdminContactList ()
{
$result = mysqlGetContactPersons();
if(mysql_num_rows($result)>0)
{
echo "
<br>
<br>
<table width = '100%' cellspacing = '0' cellpadding = '2'>
<tr bgcolor=\"#888888\" height=\"25\">
<td align = 'center'>
<b>Name</b>
</td>
<td align = 'center'>
<b>E-Mail</b>
</td>
<td align = 'center'>
<b>Telefon</b>
</td>
<td align = 'center'>
<b>Mobil</b>
</td>
<td align = 'center' width=\"250\">
<b>Position</b>
</td>
<td align = 'center' >
<b>Bearbeiten</b>
</td>
<td align = 'center'>
<b>Löschen</b>
</td>
</tr>
";
$counter=0;
while($data = @mysql_fetch_array($result))
{
$counter++;
if ($counter%2==0)
$col=" bgcolor=\"#ffffff\"";
else $col="";
echo "
<tr style=\"font-size:11px;\" ".$col." height=\"30\">
<td>
".$data['name']."
</td>
<td>
".$data['email']."
</td>
<td>
".$data['tel']."
</td>
<td>
".$data['mobil']."
</td>
<td>
".$data['position']."
</td>
<td align=\"center\">
";
if($data['id'] != "1")
{
echo "
<form action = '".self."?action=admin' method = 'POST'>
<input type = 'submit' name = 'editContact' value = 'Bearbeiten'>
<input type = 'hidden' name = 'adminaction' value = 'Ansprechpartner bearbeiten'>
<input type = 'hidden' name = 'contactID' value = '".$data['id']."'>
<input type = 'hidden' name = 'edit' value = '1'>
</form>
";
}
else
echo "";
echo "
</td>
<td align=\"center\">
";
if($data['id'] != "1")
{
echo "
<form action = '".self."?action=admin' method = 'POST'>
<input type = 'submit' name = 'delContact' value = 'Löschen'>
<input type = 'hidden' name = 'adminaction' value = 'Ansprechpartner'>
<input type = 'hidden' name = 'contactID' value = '".$data['id']."'>
<input type = 'hidden' name = 'edit' value = '0'>
</form>
";
}
else
echo "<i>Keine Aktion möglich</i>";
echo "
</td>
</tr>
";
}
echo "
</table>
";
}
else
echo "<br><br><b>Keine Ansprechpartner verfügbar</b>";
}
function makeCatList ($type, $cat = false)
{
$cats = mysqlGetCats ($type);
$return = "<select name = 'cat'>";
while($data = @mysql_fetch_array($cats))
{
$index=$data['id'];
if ($cat == $data['id']){
$selected[$index]=" selected ";
} else $selected[$index]="";
$return .= "<option value = '".$data['id']."' ".$selected[$index].">".$data['name']."</option>";
}
$return .= "</select>";
return $return;
}
function makeRefCount(){
$res = mysqlGetSettings ($type);
$data = @mysql_fetch_array($res);
return $data['References'];
}
function makeContactList ($id)
{
$select = mysqlGetContactPerson($id);
$result = mysqlGetContactPersons();
$return = "<select name = 'contactPerson'>";
while($data = @mysql_fetch_array($result))
{
if($data['id'] == $select)
$return .= "<option value = '".$data['id']."' selected>".$data['name']."</option>";
else
$return .= "<option value = '".$data['id']."'>".$data['name']."</option>";
}
$return .= "</select>";
return $return;
}
function makeDetailListContact ($id, $oid = false)
{
$data = mysqlGetContactData($id);
if(mysqlGetContactDataCheck($oid))
{
echo "
<tr>
<td align='left' valign='top'>
<table width='210'>
<tr>
<td width='10' height='150'></td>
<td width='200'></td>
</tr>
<tr>
<td></td>
<td class='fliesstext-schwarz-10'>Ansprechpartner für dieses Objekt:<br /><br />
".$data['name']."<br />
".$data['position']."<br />
".$data['email']."<br />
".$data['tel']."<br />
".$data['mobil']."
</td>
</tr>
</table>
</td>
</tr>
<tr>
<td align='center'></td>
</tr>
";
}
else
{
echo "
<tr>
<td align='left'>
</td>
</tr>
";
}
}
function makeList ($cat)
{
$result = mysqlGetObjects($cat);
if(@mysql_num_rows($result)==0)
noObjects();
while($data = @mysql_fetch_array($result))
{
$bigImg = @mysql_num_rows(@mysql_query("SELECT * FROM object_has_image WHERE oid = '".$data['id']."'"));
$contact = mysqlGetContactPerson($data['id']);
$name = mysqlGetContactPersonName($contact);
if($bigImg>1)
$bigImgTxt = "Mehrere Großansichten verfügbar!";
else if($bigImg==1)
$bigImgTxt = "Eine Großansicht verfügbar!";
else
$bigImgTxt = "";
if(mysqlCheckPdf($data['id']))
$expose = "<a href = 'pdf/".mysqlGetPdfFile(mysqlGetPdfInfo($data['id']))."' title = 'Exposé als PDF einsehen' target = '_blank' class='text-nav-link'>Exposé als PDF einsehen</a>";
else
$expose = "Kein PDF-Exposé vorhanden";
if($data['picture'] == "")
$picture = "dummy.jpg";
else
$picture = $data['picture'];
echo "
<tr>
<td valign='top'><table width='655' border='0' cellspacing='0' cellpadding='0'>
<tr>
<td colspan='3' height='20'></td>
</tr>
<tr>
<td width='200' height='155' valign='top'><table width='200' border='0' cellspacing='0' cellpadding='0'>
<tr>
<td height='155' class='objekt-foto' valign='top'><a href = '".self."?action=".$_GET['action']."&details=".$data['id']."'><img src='img/objekt-img/".$picture."' alt='Objekt-Ansicht' title='Objekt-Ansicht' width='180' height='134' border = '0' /></a><br /><br /><form action = '".self."?action=contact' method = 'POST'><input type = 'hidden' name = 'isnr' value = '".$data['isNr']."'><input type = 'hidden' name = 'contactPerson' value = '".$contact."'><input type = 'submit' value = '> ".$name." > E-Mail' style = 'background-color:#DFDFDF;color:#e2001a;border:0;'></form></td>
</tr>
<tr>
<td height='34' class='fliesstext-grau-9' align='center' valign='top'>".$bigImgTxt."</td>
</tr>
</table></td>
<td width='15'> </td>
<td valign='top'><table width='440' border='0' cellspacing='0' cellpadding='0'>
<tr>
<td colspan='3'><a href = '".self."?action=".$_GET['action']."&details=".$data['id']."' class='fliesstext-schwarz-bold'>".$data['title']."</a></td>
</tr>
<tr>
<td colspan='3' height='10'></td>
</tr>
<tr>
<td width='134' class='fliesstext-schwarz-detail' valign='top'>Objekt-Nummer:</td>
<td width='20'> </td>
<td width='286' valign='top' class='fliesstext-schwarz-detail'>".$data['isNr']."</td>
</tr>
".objectCheck($data['id'], $data, "no")."
<tr>
<td colspan='3' height='15'></td>
</tr>
<tr>
<td colspan='3' align='right'>
<table width='320' border='0' cellspacing='0' cellpadding='0'>
<tr>
<td width='120' align='right'><img src='img/button_pfeil_s.gif' width='11' height='11' /><a href = '".self."?action=".$_GET['action']."&details=".$data['id']."' class='text-nav-link'>Mehr Details</a></td>
<td width='200' align='right'><img src='img/button_pfeil_s.gif' width='11' height='11' class='text-nav-link' />".$expose."</td>
</tr>
</table>
</td>
</tr>
</table></td>
</tr>
<tr>
<td height='1' colspan='3' bgcolor='#d1d1d1'></td>
</tr>
</table></td>
</tr>
";
}
}
function makeDetailList ($cat)
{
$result = mysqlGetObjects($cat);
while($data = @mysql_fetch_array($result))
{
echo "
<tr>
<td></td>
<td height='211' align='left' class='fliesstext-grau-9'>
";
$bigImg = @mysql_num_rows(@mysql_query("SELECT * FROM object_has_image WHERE oid = '".$data['id']."'"));
$pdf = mysqlCheckPdf($data['id']);
if(trim($data['details'])!="")
echo "Detail-Beschreibung: <span class='fliesstext-gruen-bold'>ja</span><br />";
else
echo "Detail-Beschreibung: <span class='fliesstext-rot-bold'>nein</span><br />";
if($bigImg>1)
echo "Großansichten: <span class='fliesstext-gruen-bold'>ja</span><br />";
else if($bigImg==1)
echo "Großansicht: <span class='fliesstext-gruen-bold'>ja</span><br />";
else if ($bigImg==0)
echo "Großansicht(en): <span class='fliesstext-rot-bold'>nein</span><br />";
if(trim($data['drawing'])!="")
echo "Skizze / Grundriss: <span class='fliesstext-gruen-bold'>ja</span><br />";
else
echo "Skizze / Grundriss: <span class='fliesstext-rot-bold'>nein</span><br />";
if($pdf>0)
echo "Exposé als PDF-Download: <span class='fliesstext-gruen-bold'>ja</span>";
else
echo "Exposé als PDF-Download: <span class='fliesstext-rot-bold'>nein</span>";
echo "
</tr>
";
}
}
function makeFilterList ($cat)
{
$result = mysqlSearchObjects($cat, $_POST['roomA'], $_POST['roomB'], $_POST['sizeA'], $_POST['sizeB'], $_POST['priceA'], $_POST['priceB']);
if(@mysql_num_rows($result)==0)
noObjects();
while($data = @mysql_fetch_array($result))
{
$bigImg = @mysql_num_rows(@mysql_query("SELECT * FROM object_has_image WHERE oid = '".$data['id']."'"));
if($bigImg>1)
$bigImgTxt = "Mehrere Großansichten verfügbar!";
else if($bigImg==1)
$bigImgTxt = "Eine Großansicht verfügbar!";
else
$bigImgTxt = "";
if(mysqlCheckPdf($data['id']))
$expose = "<a href = 'pdf/".mysqlGetPdfFile(mysqlGetPdfInfo($data['id']))."' title = 'Exposé als PDF einsehen' class='text-nav-link'>Exposé als PDF einsehen</a>";
else
$expose = "Kein PDF-Exposé vorhanden";
if($data['picture'] == "")
$picture = "dummy.jpg";
else
$picture = $data['picture'];
if($_GET['action'] == "industry")
{
$sizeText = "Hallenfläche";
if($data['price']>0 && $data['size']>0)
{
$pricePerSqmText = "Preis je m²:<br />";
$pricePerSqm = @round($data['price'] / $data['size'],2)." €<br />";
}
else
{
$pricePerSqmText = "";
$pricePerSqm = "";
}
}
else
{
$sizeText = "Wohnfläche";
$pricePerSqmText = "";
$pricePerSqm = "";
}
echo "
<tr>
<td valign='top'><table width='655' border='0' cellspacing='0' cellpadding='0'>
<tr>
<td colspan='3' height='20'></td>
</tr>
<tr>
<td width='200' height='155' valign='top'><table width='200' border='0' cellspacing='0' cellpadding='0'>
<tr>
<td height='155' class='objekt-foto' valign='top'><img src='img/objekt-img/".$picture."' alt='Objekt-Ansicht' title='Objekt-Ansicht' width='180' height='134' /></td>
</tr>
<tr>
<td height='34' class='fliesstext-grau-9' align='center' valign='top'>".$bigImgTxt."</td>
</tr>
</table></td>
<td width='15'> </td>
<td valign='top'><table width='440' border='0' cellspacing='0' cellpadding='0'>
<tr>
<td colspan='3' class='fliesstext-schwarz-bold'>".$data['title']."</td>
</tr>
<tr>
<td colspan='3' height='10'></td>
</tr>
<tr>
<td width='134' class='fliesstext-schwarz-detail' valign='top'>Objekt-Nummer:</td>
<td width='20'> </td>
<td width='286' valign='top' class='fliesstext-schwarz-detail'>".$data['isNr']."</td>
</tr>
<tr>
<td width='134' class='fliesstext-schwarz-detail' valign='top'>Anzahl der Zimmer:<br />
".$sizeText.":<br />
Preis:<br />
".$pricePerSqmText."
Baujahr:<br />
Ort / Ortsbereich:</td>
<td width='20'> </td>
<td width='286' valign='top' class='fliesstext-schwarz-detail'>".$data['rooms']."<br />
ca. ".$data['size']." m²<br />
".number_format($data['price'], 2, ",", ".")." EUR<br />
".$pricePerSqm."
".$data['year']."<br />
".$data['location']."
</td>
</tr>
<tr>
<td colspan='3' height='15'></td>
</tr>
<tr>
<td colspan='3' align='right'><img src='img/button_pfeil_s.gif' width='11' height='11' class='text-nav-link' />".$expose."</td>
</tr>
</table></td>
</tr>
<tr>
<td height='1' colspan='3' bgcolor='#d1d1d1'></td>
</tr>
</table></td>
</tr>
";
}
}
function makeFilterDetailList ($cat)
{
$result = mysqlSearchObjects($cat, $_POST['roomA'], $_POST['roomB'], $_POST['sizeA'], $_POST['sizeB'], $_POST['priceA'], $_POST['priceB']);
while($data = @mysql_fetch_array($result))
{
echo "
<tr>
<td height='211' align='left' class='fliesstext-grau-9'>
";
$bigImg = @mysql_num_rows(@mysql_query("SELECT * FROM object_has_image WHERE oid = '".$data['id']."'"));
$pdf = mysqlCheckPdf($data['id']);
if(trim($data['details'])!="")
echo "Detail-Beschreibung: <span class='fliesstext-gruen-bold'>ja</span><br />";
else
echo "Detail-Beschreibung: <span class='fliesstext-rot-bold'>nein</span><br />";
if($bigImg>1)
echo "Großansichten: <span class='fliesstext-gruen-bold'>ja</span><br />";
else if($bigImg==1)
echo "Großansicht: <span class='fliesstext-gruen-bold'>ja</span><br />";
else if ($bigImg==0)
echo "Großansicht(en): <span class='fliesstext-rot-bold'>nein</span><br />";
if(trim($data['drawing'])!="")
echo "Skizze / Grundriss: <span class='fliesstext-gruen-bold'>ja</span><br />";
else
echo "Skizze / Grundriss: <span class='fliesstext-rot-bold'>nein</span><br />";
if($pdf>0)
echo "Exposé als PDF-Download: <span class='fliesstext-gruen-bold'>ja</span>";
else
echo "Exposé als PDF-Download: <span class='fliesstext-rot-bold'>nein</span>";
echo "
</tr>
";
}
}
function makeRefPics()
{
$result = mysqlGetRef();
$text='';
while($data = @mysql_fetch_array($result))
{
$text.="<div style=\"background:#fff;border:5px solid white;margin:15px; 0 15px 0;\"><img src=\"img/objekt-img/".$data['path']."\"></div>";
}
return $text;
}
function makeObjPics()
{
$result = mysqlGetObjPictures();
$data = @mysql_fetch_array($result);
$text="<div style=\"background:#fff;border:5px solid white;margin:5px; 0 15px 0;\"><a href=\"?action=".$data['type']."&details=".$data['id']."\"><img border=\"0\" src=\"img/objekt-img/".$data['picture']."\"></a></div>";
return $text;
}
function makeInvestPics()
{
$result = mysqlGetInvestPictures();
$data = @mysql_fetch_array($result);
$text="<div style=\"background:#fff;border:5px solid white;margin:5px; 0 15px 0;\"><a href=\"?action=investments&details=".$data['id']."\"><img border=\"0\" src=\"img/objekt-img/".$data['picture']."\"></a></div>";
return $text;
}
function makeTopPics()
{
$result = mysqlGetTopPictures();
$data = @mysql_fetch_array($result);
$text="<div style=\"background:#fff;border:5px solid white;margin:5px; 0 15px 0;\"><a href=\"?action=".$data['type']."&details=".$data['id']."\"><img border=\"0\" src=\"img/objekt-img/".$data['picture']."\"></a></div>";
return $text;
}
function makeISNrSearchList ($search)
{
$result = mysqlSearchISNr($search);
while($data = @mysql_fetch_array($result))
{
$bigImg = @mysql_num_rows(@mysql_query("SELECT * FROM object_has_image WHERE oid = '".$data['id']."'"));
if($bigImg>1)
$bigImgTxt = "Mehrere Großansichten verfügbar!";
else if($bigImg==1)
$bigImgTxt = "Eine Großansicht verfügbar!";
else
$bigImgTxt = "";
if(mysqlCheckPdf($data['id']))
$expose = "<a href = 'pdf/".mysqlGetPdfFile(mysqlGetPdfInfo($data['id']))."' title = 'Exposé als PDF einsehen' class='text-nav-link'>Exposé als PDF einsehen</a>";
else
$expose = "Kein PDF-Exposé vorhanden";
echo "
<tr>
<td valign='top'><table width='655' border='0' cellspacing='0' cellpadding='0'>
<tr>
<td colspan='3' height='20'></td>
</tr>
<tr>
<td width='200' height='155' valign='top'><table width='200' border='0' cellspacing='0' cellpadding='0'>
<tr>
<td height='155' class='objekt-foto' valign='top'><img src='img/objekt-img/".$data['picture']."' alt='Objekt-Ansicht' title='Objekt-Ansicht' width='180' height='134' /></td>
</tr>
<tr>
<td height='34' class='fliesstext-grau-9' align='center' valign='top'>".$bigImgTxt."</td>
</tr>
</table></td>
<td width='15'> </td>
<td valign='top'><table width='440' border='0' cellspacing='0' cellpadding='0'>
<tr>
<td colspan='3' class='fliesstext-schwarz'>".$data['title']."</td>
</tr>
<tr>
<td colspan='3' height='10'></td>
</tr>
<tr>
usw. ... mit so einer gequirlten ... muß ich mich dann rumschlagen.
Bei mir sieht es so aus:
eine function hab ich z.B. beim umwandeln eines englischen Datumsformats in deutsche.
Diese Funktion ruf ich in manchen Projekten 30 mal auf.