Skrypt konfiguracji ogólnej pisany przez profesionalistów... Opcje

[Adi32]

Sam nie jestem profesjonalistą ale ten kod to chyba kpina, pisany w jednej linij...
Może ja czegoś nie wiem, co Wy o tym myślicie?

[PHP] pobierz, plaintext 
<?public function __construct() {
global $db;
 
$this->name = 'konfiguracja';
$this->title = 'Konfiguracja serwisu';
 
$db->query("SELECT tresc FROM konfiguracja WHERE nazwa='blokada'");
$res = $db->getNextRow();
 
if ($res['tresc'] == 'tak') {
echo '';
$this->checked = 'checked';
} else {
echo '';
$this->checked = '';
}
 
switch ($_GET['do']) {
default:
$this->show();
break;
}
}
 
public function show() {
global $db;
 
$db->query("SELECT * FROM " . $this->name . " WHERE id_jezyk=%d", $_SESSION['id_jezyk']);
$r = $db->res();
 
for ($i = 0; $i < count($r); $i++) {
if ($r[$i]['id_fraza'] == 1)
$keyw = $r[$i]['tresc'];
if ($r[$i]['id_fraza'] == 2)
$opis = $r[$i]['tresc'];
if ($r[$i]['id_fraza'] == 3)
$tytul = $r[$i]['tresc'];
if ($r[$i]['id_fraza'] == 4)
$email = $r[$i]['tresc'];
if ($r[$i]['id_fraza'] == 5)
$szer = $r[$i]['tresc'];
if ($r[$i]['id_fraza'] == 6)
$wys = $r[$i]['tresc'];
if ($r[$i]['id_fraza'] == 7)
$s1 = $r[$i]['tresc'];
if ($r[$i]['id_fraza'] == 8)
$s2 = $r[$i]['tresc'];
if ($r[$i]['id_fraza'] == 9)
$konto = $r[$i]['tresc'];
 
if ($r[$i]['id_fraza'] == 10)
$licznik = $r[$i]['tresc'];
 
if ($r[$i]['id_fraza'] == 11)
$updateczas = $r[$i]['tresc'];
 
if ($r[$i]['id_fraza'] == 12)
$updateczestosc = $r[$i]['tresc'];
}
 
if (isset($_POST['save'])) { //zapisz do bazy
 
if ($r) {
$db->query("DELETE FROM " . $this->name . " WHERE id_jezyk=%d", $_SESSION['id_jezyk']);
}
 
$db->query("INSERT INTO " . $this->name . " (id_fraza,id_jezyk,tresc) VALUES (%d,%d,'%s')", 1, $_SESSION['id_jezyk'], $_POST['keyw']);
$db->query("INSERT INTO " . $this->name . " (id_fraza,id_jezyk,tresc) VALUES (%d,%d,'%s')", 2, $_SESSION['id_jezyk'], $_POST['opis']);
$db->query("INSERT INTO " . $this->name . " (id_fraza,id_jezyk,tresc) VALUES (%d,%d,'%s')", 3, $_SESSION['id_jezyk'], $_POST['tytul']);
$db->query("INSERT INTO " . $this->name . " (id_fraza,id_jezyk,tresc) VALUES (%d,%d,'%s')", 4, $_SESSION['id_jezyk'], $_POST['email']);
$db->query("INSERT INTO " . $this->name . " (id_fraza,id_jezyk,tresc) VALUES (%d,%d,'%s')", 5, $_SESSION['id_jezyk'], $_POST['szer']);
$db->query("INSERT INTO " . $this->name . " (id_fraza,id_jezyk,tresc) VALUES (%d,%d,'%s')", 6, $_SESSION['id_jezyk'], $_POST['wys']);
$db->query("INSERT INTO " . $this->name . " (id_fraza,id_jezyk,tresc) VALUES (%d,%d,'%s')", 7, $_SESSION['id_jezyk'], $_POST['szkontakt']);
$db->query("INSERT INTO " . $this->name . " (id_fraza,id_jezyk,tresc) VALUES (%d,%d,'%s')", 8, $_SESSION['id_jezyk'], $_POST['s2']);
$db->query("INSERT INTO " . $this->name . " (id_fraza,id_jezyk,tresc) VALUES (%d,%d,'%s')", 9, $_SESSION['id_jezyk'], $_POST['konto']);
$db->query("INSERT INTO " . $this->name . " (id_fraza,id_jezyk,tresc) VALUES (%d,%d,'%s')", 10, $_SESSION['id_jezyk'], $_POST['licznik']);
$db->query("INSERT INTO " . $this->name . " (id_fraza,id_jezyk,tresc) VALUES (%d,%d,'%s')", 11, $_SESSION['id_jezyk'], $_POST['updateczas']);
$db->query("INSERT INTO " . $this->name . " (id_fraza,id_jezyk,tresc) VALUES (%d,%d,'%s')", 12, $_SESSION['id_jezyk'], $_POST['updateczestosc']);
 
if ($_POST['blokada'] == 'tak') {
$db->query("UPDATE konfiguracja SET tresc='tak' WHERE nazwa='blokada'");
} else {
$db->query("UPDATE konfiguracja SET tresc='nie' WHERE nazwa='blokada'");
}
 
 
//pobierz meta
$db->query("SELECT * FROM konfiguracja WHERE id_jezyk=%d", $_SESSION['id_jezyk']);
$mres = $db->res();
if ($mres)
foreach ($mres as $d => $b)
$_SESSION['meta'][$b['id_fraza']] = $b['tresc'];
 
header("location:admin.php?module=" . $this->name . "&msg=Zmiany zapisane");
}
else { //pokaz
?>
<h1>Konfiguracja serwisu</h1>
<a href="admin.php?module=pomoc&do=<? echo $this->name; ?>">Pomoc</a>
<form action="" method="post">
<table cellpadding="0" cellspacing="0" style="width:90%">
<tr>
<td style="width:200px">Słowa kluczowe: </td>
<td><input type="text" class="pole" size="60" name="keyw" value="<? echo $keyw; ?>" /></td>
</tr>
<tr>
<td >Opis strony: </td>
<td><input type="text" class="pole" size="60" name="opis" value="<? echo $opis; ?>" /></td>
</tr>
<tr>
<td >Tytuł strony: </td>
<td><input type="text" class="pole" size="60" name="tytul" value="<? echo $tytul; ?>" /></td>
</tr>
<tr>
<td >E-mail kontaktowy: </td>
<td><input type="text" class="pole" size="60" name="email" value="<? echo $email; ?>" /></td>
</tr>
<tr>
<td >Licznik: </td>
<td><input type="text" class="pole" size="60" name="licznik" value="<? echo $licznik; ?>" /></td>
</tr>
<tr>
<td >Dane do wpłat: (bank, numer, dane odbiorcy) </td>
<td><textarea style="width: 379px; height: 69px;" name="konto"><? echo $konto; ?></textarea></td>
</tr>
<tr>
<td >Szybki kontakt: </td>
<td><textarea name="szkontakt" /><? echo $s1; ?></textarea></td>
</tr>                    
<tr>
<td >Update produktów - ostatni: </td>
<td><input type="text" class="pole" size="60" name="updateczas" value="<? echo $updateczas; ?>" /></td>
</tr>
<tr>
<td >Update produktów - częstość (min): </td>
<td><input type="text" class="pole" size="60" name="updateczestosc" value="<? echo $updateczestosc; ?>" /></td>
</tr>
 
</table>
<input type="submit" class="go" name="save" value="Zapisz" />
<input type="button" class="go" name="cancel" value="Anuluj" onclick="window.location='admin.php'" />
</form>
[PHP] pobierz, plaintext 

Ten post edytował Adi32 13.07.2012, 08:45:11

[Adi32]

Rzeczywistość jest zupełnie inna. W pracy, w zespole wypalenie przychodzi później. I nie zdziw się jak będziesz za jakiś czas pisać kod tak, jak przedstawiłeś wyżej. To w pewnym momencie jest nieuniknione, po prostu zadajesz sobie pytanie "po co mam się męczyć, nie chce mi się".
Jako jednoosobowa działalność, jest jeszcze gorzej, bo masz na głowie nie tylko kod, ale i wszytko inne.

Twoje sceptyczne podejście widać w wielu Twoich postach. Dlaczego tak pesymistycznie do tego podchodzisz? Czy Ty piszesz taki kod?
Mam nadzieje, że nie masz racji co do tego wypalenia ale jedno wiem na pewno - nigdy nie będę tak pisał. Nawet jakby mi się nie chciało to jest to utrudnianie sobie życia.

[prowseed]

Twoje sceptyczne podejście widać w wielu Twoich postach. Dlaczego tak pesymistycznie do tego podchodzisz? Czy Ty piszesz taki kod?
Mam nadzieje, że nie masz racji co do tego wypalenia ale jedno wiem na pewno - nigdy nie będę tak pisał. Nawet jakby mi się nie chciało to jest to utrudnianie sobie życia.

Hmm, nawiązując do tego, że napisałeś iż jest to Twoja pierwsza praca, wydaje mi się, że zderzyłeś się z czymś co Cię przeraża i popadasz w pewną skrajność. Niestety, w firmach jest tak, że za projekt płaci klient i uwierz mi, go nie obchodzi jak zaprojektujesz dodawanie newsów, go obchodzi by mógł tego newsa dodać i wstawić tam tabelkę i by zrobione to było jak najszybciej, bo ma dużo zamówień z Niemiec. Albo to zaakceptujesz i będziesz starał się przejść przez tą burzę znajdując z czasem złoty środek (jak u Kochanowskiego : ) albo się przestraszysz i odejdziesz, miej tylko na uwadze to, że w różnych firmach jest różnie. W moim przypadku to akurat nie jest pierwsza praca (zaczynałem w wieku 16 lat, ale zgodnie z umową nie mogę powiedzieć gdzie ) i patrząc przez pryzmat starej firmy tu gdzie teraz jestem jest o niebo lepiej. Ba, nawet o dwa nieba. Ja akurat pracuję w trochę innym dziale (na pewno z nas żartujecie ;] ) jako grafik i jest fajnie, jestem kreatywnym, moje pomysły wchodzą bezpośrednio w życie, to ja proponuję rozwiązania, ale czasem zdarzy się klient typu "logo większe i na środek" i wtedy pozostaje tylko rozłożyć ręce i ustawić mu to logo na środku... Bo w końcu to jego projekt i on za niego płaci.