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 

Ło ja Cię kręcę ale po kolei...

[PHP] pobierz, plaintext 
global $db; // hmm nie słyszeli o przekazywaniu argumentów...
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$db->query("SELECT tresc FROM konfiguracja WHERE nazwa='blokada'"); // wydajność, wydajność i jeszcze raz wydajność 
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
if ($res['tresc'] == 'tak') {
echo '';                    // To echo jest tutaj po to by?
$this->checked = 'checked';
} else { 
echo '';                   // To echo jest tutaj po to by?
$this->checked = '';
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
 
switch ($_GET['do']) { // rozumiem że to jest dodane na zaś?
default:
$this->show();
break;
}
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
for ($i = 0; $i < count($r); $i++) { // a już myślałem że wiedzą co to switch(...)...
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)
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$db->query("UPDATE konfiguracja SET tresc='tak' WHERE nazwa='blokada'"); // brak konsekwencji - raz "$this->name" raz "konfiguracja" ...
[PHP] pobierz, plaintext 

Skąd Ty to wziąłeś? Nie mów, że zapłaciłeś tym profesjonalistom?

To jest kod głównego programisty w firmie w której pracuję...
Nie powiem jaka to firma, ale powiem, że musze stąd uciec...

Znaczy powiem tak, żeby nie było - luk bezpieczeństwa jakichś mega tu nie ma - ale jest nieład, niedbalstwo i co najgorsze - choć może tu nie najistotniejsze - brak pomyślunku jeśli chodzi o wydajność...

Może się już mu nie chce...

Co do wydajności to to:

[PHP] pobierz, plaintext 
$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']);
[PHP] pobierz, plaintext 

Można dać w jednym zapytaniu... itd... itp...

No i global $db :/ to mi nie da zasnąć...

Szybko zgadłeś, ale edycja i rozwijanie takiego kodu to koszmar...

Edit. Zauważ, że przy każdym zapisie wszystkie pola są kasowane i dodawane na nowo.

Czepiacie się, działać działa!

Zależy co to za obiekt $db i jak wygląda metoda query. Jeśli argumenty tej metody nie są escape'owane to mamy SQL injection.

[PHP] pobierz, plaintext 
$db->query("INSERT INTO " . $this->name . " (id_fraza,id_jezyk,tresc) VALUES (%d,%d,'%s')", 2, $_SESSION['id_jezyk'], $_POST['opis']);
[PHP] pobierz, plaintext 

Zgadza się, ale pocieszenie jest takie, że niezabezpieczone zapytanie są tylko w panelu administracyjnym.

@redeemer - Masz całkowitą rację

po prostu biorąc ten kod:

[PHP] pobierz, plaintext 
$db->query("INSERT INTO " . $this->name . " (id_fraza,id_jezyk,tresc) VALUES (%d,%d,'%s')", 2, $_SESSION['id_jezyk'], $_POST['opis']);
[PHP] pobierz, plaintext 

Założyłem że kolejne argumenty są automatycznie zabezpieczane ale w sumie jak teraz popatrzeć to chyba są walone wprost ... a to jest lipne strasznie...



heh Może to i jakieś pocieszenie ale i tak marne zabezpieczanie powinno być w krwi - zawsze - wszędzie. Nie można myśleć na zasadzie - to jest panel administracyjny a do niego dostęp mam tylko ja, a ja nie mam zamiaru robić SQL-inj. to mogę nie zabezpieczyć - złe nawyki

Jak napisał Sephirus to że jest to admin, nie ma znaczenia. Można też założyć, że ten obiekt oraz metoda query jest wykorzystywana również w frontendzie.

Nic dodać, nic ująć-kod z optymalnością mało wspólnego.Choć programuje w ASP.NET to nie chwaląc się ,pewnie zrobiłbym to lepiej.Brak pomysłu albo lenistwo -to nie pisał profesjonalista-albo mu się nie chciało i pisał to od ręki wyłączając przy tym część swoich szarych komórek.

[PHP] pobierz, plaintext 
switch ($_GET['do']) {
default:
$this->show();
break;
}
[PHP] pobierz, plaintext 

Tego to normalnie nie rozumiem-czy to ma być instrukcja wyboru.Jakiś hack and trick?Nie wystarczyło dać if-a?

[PHP] pobierz, plaintext 
    if ($res['tresc'] == 'tak') {
    echo ''; // To echo jest tutaj po to by?questionmark.gif
    $this->checked = 'checked';
    } else {
    echo ''; // To echo jest tutaj po to by?questionmark.gif
    $this->checked = '';
    }
[PHP] pobierz, plaintext 

Gdzie tutaj logika?Jeśli warunek spełniony wyświetl nic jeśli nie, także wyświetl nic

Poza tym o ile to możliwe można poniższe liniki zrobić na poziomie bazy danych:

[PHP] pobierz, plaintext 
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'");
}
[PHP] pobierz, plaintext 

Ustawić default(domyślną wartość) dla kolumny treść na 'nie' i jeśli warunek if ($_POST['blokada'] == 'tak') zostałby spełniony zrobić tylko update , else nie musiałoby w ogóle istnieć.
Przykład:

[PHP] pobierz, plaintext 
if ($_POST['blokada'] == 'tak') {
$db->query("UPDATE konfiguracja SET tresc='tak' WHERE nazwa='blokada'");
}
[PHP] pobierz, plaintext 

Else nie potrzebne gdyż domyślna wartość kolumny tresc jest ustawiona na nie i nie trzeba jej zmieniać.

@Niktoś - ja to wiem i szczerze załamuje mnie to.
To co przedstawiłem to tylko kawałeczek. Ja muszę przy tym pracować.
Tutaj ludzie uważają, że to jest programowanie obiektowe, mówię nie - "przecież są klasy"...
Pyta mnie - "split deprecated" - mowie przestarzała, użyj preg_math - "przecież preg_math służy do validacji"... (po aktualizacji PHP)

@Adi32 czepiasz się każdy może mieć gorszy dzień buahahaha

To jakaś nowa matematyczna funkcja ?

Mam nadzieję, że nie starasz się zbyt mocno nad rozwijaniem czegoś takiego .

Kiedy tu trafiłem pomyślałem, że użyje doświadczenia jakie mam aby to "rozwinąć" ale nie, spotkało się to z oporem gdyż tek kto ma ciepłą posadkę i nikogo nad sobą nie potrzebuje sie uczyć.

Wkleiłem tylko niewielki kawałek kodu nic nie prezentujący żeby nikt się nie przyczepił ale spróbujcie sobie wyobrazić jak wygląda kod sklepu...
Proponowałem Zenda - zapomnij ... "jak chcesz to sam zrób coś prostszego", kto coś łapie wie, że mija się to z celem...

W takim razie szukaj innej firmy, bo niczego się nie nauczysz.

@Adi32
Jesteś romantykiem. Rzeczywistość jest dużo gorsza niż to co podałeś

Szukam... Mam nawet zaproszenia na kilka rozmów.



A co dokładniej przez to rozumiesz? Ja rozumiem, że mogą być gorsze kody, ale chyba nie w "profesjonalnej firmie"...

Jak to jest Jego obiektówka, to wklej jakiś kod strukturalny, chciałbym wiedzieć jak wygląda.

Żeby ukazać jego moc musiałbym wkleić całe 500 linij pliku top.
Boje się w ogóle tam zaglądać...

Dobrze że dodałeś cudzysłów bo skoro tak wygląda programowanie w tej firmie to ciężko nazwać ją profesjonalną.
Rzuć linki do arcydzieł tego pana to czempioni od sqlInjection i XSS popracują nad tym

A tak już całkiem na poważnie to uciekaj z takiej firmy, Lepiej pracować za mniejsze pieniądze niż uwsteczniać się pisząc takie rzeczy. Co z tego że łykniej doświaczenia na papier jak twoje umiejętności (chociaż tu nabywasz umiejętności pracy z cudzym kodem ) się nie rozwiną.

Też tak sądzę.Taki proffesjonalista nawet nie powinien szkolić osoby po gimnazjum lub szkole średniej.

Takich ,rzeczy to on Ciebie się nie powinien w ogóle pytać.Jeśli chodzi o polecenie split() to jest ona depracet dlatego,że w swojej metodzie używa ereg'ów ,należało by użyć polecenia preg_split lub prościej str_split.

Można powiedzieć, że trafiłem tu z ulicy... Bez żadnego udokumentowanego doświadczenia ani pracy związanej programowaniem, także chciałem chociaż 2 lata posiedzieć żeby mieć "te 2 lata" w CV ale i tego nie będę miał. Płaca to mniej niz 2k także też nieciekawe tylko problem z pracą w Lublinie...

Jeszcze nie raz zobaczysz taki kod, a nawet gorszy i jeszcze mający być w swoich założeniach narzędziem do zarabiania kasy, z konkretnymi planami biznesowymi.
Świat jest daleki od doskonałości.

Zgadzam się z tym. Często gęsto bywa tak że ludzie strają się pisać piękny kodzić i tak jest ale przychodzi dzień kiedy musza coś na szybko wpisać i wtedy takie babole powstają upchnięte w każde wolne miejsce w pliku. Takie rzeczy można zobaczyć w wielu projektach które na pierwszy rzut oka wydaja się byc profesjonalne i takie są do pewnego pliku / lini.

Taki kod można wybaczyć freelancerowi, taka osoba napisze coś dostanie za to pieniądze i nic go już to nie obchodzi, pewnie i tak nigdy już tego nigdy nie zobaczy. Ale jeśli jest to osoba która wie że będzie na tym pracować to sama strzela sobie w kolano. Rozwój/pielęgnacja takiego potworka to męczarnia.

Dla jaj możesz przy jakimś zadaniu zapytac tego swojego profesjonalistę jakiego wzorca projektowego użyłby aby uzyskać dobrą uniwersalność tego co będziesz robił

Chciałbym zobaczyć jego minę

Swoją drogą z czymś takim spotyka się chyba każdy... Ja w firmie gdzie pracuje też mam parę "starych" aplikacji i ostatnio w jednej z nich napotkałem $HTTP_POST_VARS['...']... pamiętam to sprzed X lat ale dla kogoś kto niedawno zaczął prace z PHP wywołało by to co najmniej ciekawość Nie mówiąc już o tym, że spora część aplikacji nie jest upgrade'owana a łatana jedynie... jeśli coś działa mniej-więcej jak należy a ma 10 lat to firmie nie chcę się tracić zasobów na upgrade... Wtedy dopiero jest kosmos jak trzeba pracować w starym, 10-letnim kodzie strukturalnym pisanym przez takich właśnie "profesjonalistów"

Yymmmnn taaak... Spotykam się z tym często...
W moim przypadku hehe jest wiele gorzej, trzeba zrobić nową stronę - sklep - stawia się ją na tym skrypcie na którym stawiało się ostatni sklep. I w ten sposób nie wiadomo ile osób nawpisywało swoje kwiatuszki do tego "wzoru". Dodatkowo często pojawia się kompletnie nie uważana przeze mnie kwestia "przekazania" projektu innej osobie bo ktoś idzie na tydzień na urlop. W zespole zgranym, który pisze obiektowo jeszcze mogłoby to przejść ale kiedy każdy ma własny zamysł co do metody rozwiązania konkretnego problemu to takie "upłynnienie" pracy nad projektem powoduje nawet kilkukrotne wydłużenie pracy nad stroną gdy zajmuje się nią na zmianę 4 osoby...

@Adi32
Jeśli pracujesz tam, gdzie myślę, że pracujesz, to powiem tak- nie ładnie jest tak psioczyć na kolegów.
Najpierw poznaj realia pracy pod presją czasu i sens powiedzenia: "czas to pieniądz".

Każdy kod może być piękny, ale jeśli odłożymy go na półkę i po roku okaże się, że trzeba dodać/zmienić w nim funkcjonalność, dopisać cokolwiek, to co- będziesz analizował wszystko od początku i restrukturyzował logikę aplikacji, czy po prostu dopiszesz co masz dopisać i zamkniesz? Jeśli wybierzesz drugą opcję, a z biegiem czasu dojdą kolejne poprawki, to oczywiście- zacznie się robić śmietnik, a naprawa tego śmietnika może być równoznaczna z przepisywaniem wszystkiego od początku.

Dodatkowo nie zapomnij, że ludzie nie pracują nad jednym kodem non-stop. To też ma znaczenie, gdy nagle masz do czegoś wrócić, w między czasie coś tam jeszcze gdzieś poprawić. Często w takich sytuacjach stawia się na prostotę, a nie kompleksowość.

I jeśli ktoś Ci mówi: "to zrób lepszy", to po prostu zrób lepszy. Na pewno nikt Ci nie będzie miał tego za złe, a przyczynisz się do rozwoju w dobrym kierunku. Pytanie tylko, czy czasowo będziesz w stanie się z tym wyrobić. I czy ogarniesz ogrom aplikacji, która bądź co bądź nie skupia się w kilku pliczkach.

Nie oczerniaj kolegów sypiąc jakimiś kodami sprzed dziesięciu lat, które były poprawiane przez n+1 osób. Ktoś czegoś nie zrozumiał na szybko, to po prostu dopisał po swojemu nie chcąc psuć tego, co już działa.

A ja jestem tego zdania -jeśli coś robisz ,rób to najlepiej lub staraj się robić.Nie widać starania w kodzie , który autor przedstawił. Nie wygląda on na profesjonalny kod .Jeśli robi tak nagminnie to też bym był wkurzony, gdyż poprawiać po kimś to już dodatkowe obciążenie.

A skąd wiesz, że się nie starali? : )
Pewnie, że tak- ja też jestem takiego zdania i zawsze staram się dążyć do perfekcji.
Szkoda tylko, że nie zawsze warunki zewnętrzne mi na to pozwalają...

I dlatego PHP ma taka opinie jaka ma.

Psioczyć na kolegów a pokazać poziom środowiska swojej spracy to (może niewielka) ale różnica. Znajomi z pracy to świetni ludzie i doskonale znają moje zdanie na temat ich kodów które tu przedstawiłem. Dlatego postanowiłem pokazać "kawałek".


Chyba nie przyjrzałeś się zbyt dokładnie temu kodowi. Uważasz, że był pisany na szybko? Zrobić to w sposób "normalny" byłoby 2 razy szybciej.
Na tym kodzie pracuje się cały czas a mi znudziło się już przerabianie tego z każdym projektem który przejmuję.


Bywa, ale robić to zawsze i wszędzie? Nigdy nie używać wcięć? Tak sie po prostu nie robi...



Rozumiem, że chciałeś pokazać całą sprawę z drugiej strony ale nie masz racji. Są rzeczy które można robić jak sie pisze dla kolegi w domu ale nie gdy pracuje się w "poważnej" firmie. To przyczynia się do kilkukrotnego wydłużenia całych projektów. Skryty napisane w tym roku wyglądają tak samo a rozwinięcie czegokolwiek to koszmar. "Zostałem oszukany", obiecano mi tu rozwój a jest pływanie w kisielu i pogoń za zyskiem. Na nikogo nie psioczę poza złą organizacją firmy.

Adi32, wiesz każdy może znaleźć jakiegoś "potworka" z realizacji komercyjnej roku 2005 i narzekać na brak profesjonalizmu . Nie zrozum mnie źle, nie twierdzę że się mylisz i pracujesz z niedocenianymi geniuszami. Wiem jednak jak potrafi wyglądać realizacja produktów stricte komercyjnych. Istnieje wiele czynników, które (niestety) mogą mieć wpływ na jakość... czas, nagła zmiana "widzimisię" klienta, zmęczenie projektem, który się niemiłosiernie ciągnie i mamy ochotę go zwyczajnie zamknąć itd..

Wiem, że nie powinno nas to usprawiedliwiać, niemniej jednak zdarza się, że coś zostanie dorobione przy pomocy programistycznego odpowiednika taśmy klejącej . Pytanie brzmi czy był to szczyt możliwości programisty pracującego na etacie w komfortowych warunkach, czy kod, który powstał na wczoraj i przewrócił cały projekt do góry nogami. Być może ostatecznie firma stwierdziła, że kod jaki będzie taki będzie, ale firma musi zarabiać.

Wiele firm w ofertach pracy umieszcza informację o ogromnych "możliwościach rozwoju", niestety tych możliwości to ani widu ani słychu. Wiecie coś na zasadzie "Młoda, dynamiczna i prężnie rozwijająca się firma poszukuje...". Ostatnio niemalże każda firma poszukująca pracownika jest nie tylko młoda i dynamiczna, ale i prowadzi niebywałą ekspansję pracowniczą .

Innymi słowy należy doceniać firmy, które faktycznie pozwalają nam się rozwijać - gdyż, wbrew pozorom, odnoszę wrażenie iż pracodawcom wcale nie przychodzi to tak naturalnie .

Oni z ofertami pracy robią dokładnie to samo co z kodem: ctrl+c/ctrl+v i oczywiście obowiązkowo "jesteśmy liderami w branży".

Przykre jest to co piszecie. Miałem kiedyś nauczyciela, który dość ostro powtarzał mi "pisz kod zawsze najwyższej jakości jak potrafisz".
Wziąłem to do siebie bardzo poważnie i zawsze się do tego stosuje.



Właśnie dlatego. Z doświadczenie mimo, że niewielkiego wiem, że w większości sytuacji kod który został napisany wyższej jakości niż niższej jest bardziej podatny na zmiany z powodu "widzimisię" klienta.
Dla mnie pojęcie programowania zamieniło się w swego rodzaju sztukę, przyjemność sprawia tylko w tedy gdy zrobione jest najlepiej jak się da, z pasją.
Bolesne jest wkładanie siebie w kod który zaraz zostanie spiepszony a jeszcze gorsze jest to, że mówicie, że wszędzie tak jest.
Mam nadziję, że sie mylicie.

Wypalisz się w ten sposób.

Adi32, nikt nie napisał, że tak jest wszędzie. Po prostu zdarza się, że nasza rzeczywistość pisania błyskotliwego kodu w domu i na półkę, zderza się z rzeczywistością, w której Twój kod musi na siebie zarabiać. Ba, w grę wchodzi jeszcze niezbyt sterylne środowisko, w którym cudze błędy (PM stwierdzi, że poprawki naniesiesz w 3 dni, gdzie realny czas to 5) mają wpływ na to co robisz. Tak jak pisałam, jeżeli ktoś daje nam się rozwijać i jeszcze za to płaci - powinniśmy się cieszyć.

Wypalę? A ja myślałem, że taka praca to spełnienie marzeń. Robić coś najlepiej jak się potrafi dążąc do "doskonałości" (wiem, że tu nie ma takiego pojęcia).



Ja uważam, że to jest cofanie się. Wszyscy wiemy, że OOP to przyszłość a ja nie mam z tym praktycznie styczności.
Miałem projekt, który był niemożliwy do wykonania w takim terminie jaki podano klientowi. I tu się z Tobą zgodzę, wstyd mi za kod który tam jest ale nie miałem wyjścia. Jak mam tam coś rozwinąć to ciarki mnie przechodzą.

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.

Ponieważ pasja programowania i klepania kodu mija z czasem, szczególnie gdy dostajesz kod po kimś, lub musisz napisać coś na wczoraj. Pasje możesz mieć, gdy tworzysz coś swojego, całkowicie od podstaw i nikt Ci niczego nie narzuca, wtedy ok, ale nie oszukujmy się, ile jest takich zleceń? W większości trafia się klient który nie doceni Twojego talentu eleganckiej typografi, 20 godzin spędzenia na obmyślaniu czy logo dać o 20px niżej , oraz pięknej dokumentacji kodu, formatowania, już nie wspominając o bebechach słitaśnego PDO.

Zdarza mi się napisać kod na "odwal się" (ten którego nie widać), ale idę o zakład że nie ma tu osoby na forum która mogłaby powiedzieć inaczej, pracującej więcej niż 2 miesiące w wakacje


Nie oszukuj się i nigdy nie mów nigdy...


Niekoniecznie, jak to co robisz jest jednorazowe.

@Adi32 nie daj się przekonać takim zgryźliwym tetrykom jak !*!, który sam już się wypalił zawodowo i teraz wszystkim wmawia, że ich czeka to samo, jeśli nie będą klepać kodu na" odwal się"

A z tej pracy, jeśli nic się nie zmieni, uciekaj. Nawet nie chodzi o to, że niczego ciekawego tam nie zrobisz, ale po prostu jak zostaniesz, to się z czasem wypalisz i naprawdę nie będzie Ci się chciało. Nie mówiąc już o tym, że zdegenerujesz się do roli klepacza, stracisz jakąkolwiek przyjemność z programowania i nauczysz jedynie "tumiwisizmu".

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.

Niestety, smutna prawda


Ty świntuszku...

Nie można mieszać pisania na szybko , z pisaniem źle, to że do pokoju wpada PM i mówi, że na jutro jest potrzebna poprawka nie oznacz jeszcze, że mamy zapomnieć o podstawach optymalizacji. Tworzenie pustych switchów to przecież nie wynik pośpiechu tylko niechlujstwo lub niedouczenie.

Każdy programista ma lepsze i gorsze dni, ale gorszy dzień nie oznacza zapomnienia podstaw. Jasne wówczas pisze sie wolniej, testuje jeszcze wolniej czasem 1h myślisz nad rozwiązaniem na które wczoraj na pewno wpadłbyś po 10 min zastanowienia.

Na obronę autora przytoczonego skryptu można by powiedzieć, że jeżeli robimy coś "na odwal się" to PM powinien dostać info : "działa, ale wymaga optymalizacji / poprawek" to co on z tym zrobi to jego broszka być może nikt nie zgodził się na poprawianie tego ....

Zawsze mówiłem, że lepszy dobry kod strukturalny niż sieka pełna słów "this"

A co do kodu to hmmm, pracując w dużych korporacjach widywałem dużo gorsze Wszystko zależy od tego, kiedy był pisany. Jeśli w czasach zegarów na stronie to można strzelić piwko i się śmiać, sam nieraz jak się nawalę to nachodzi mnie nostalgia, otwieram wtedy swoje stare skrypty i buahaha, ehehehe, hyhyhyhy a kto nie znał global ten lame

Ale jeśli ktoś niedawno popełnił taki kod to hmmm, można strzelić piwko i zastanowić się, czy to programista CNC który dorwał się przypadkiem do PHP (a co ja, w PHP nie napiszę?) czy ktoś zatrzymał się na etapie zegarów na stronie i nie potrafi ruszyć dalej, w stronę imienin i frejmów.

Może to być też działanie celowe i zamierzone - w myśl hasła "jaka płaca taka praca" - przy obowiązującym w większości firm ujemnym budżecie na szkolenia (wyszło PHP 5, więc zrzucamy się na książkę) może to dać taki efekt, możliwe też że mamy tu działanie szatańsko cwane - tworząc kod zrozumiały tylko dla siebie i mocno zawiły zapewniamy sobie kolejne roboczogodziny i wyłączność na pracę nad projektem, niemożliwe staje się możliwe - jesteśmy NIEZASTĄPIENI! Potem byle hołysz bez paranteli nie rozkmini archaicznego kodzidła, potrzebny będzie pra-pra-pra-parent siegający korzeni pehapa, matuzalem skryptu co wykiełkował prosto z globala Młodzież wychowana na jakimś OOP nie da rady

Wow, Pilsener, postawiłbym Ci piwko za tą wypowiedź

//EDIT
@Adi32
Wracając do tematu, nie chciałem ukazać problemu z drugiej strony. Chciałem pokazać, że ludzie tam siedzący, to nie półgłówki, bo tak zabarwiałeś swoje wypowiedzi...

@Pilsener - Również postrawiłbym Ci piwko za tego posta
@prowseed - Miałem na myśli wyłącznie niedbalstwo co do pisania w PHP, nie zwracanie uwagi, że ktoś inny będzie się z tym..., zero przemyślenia jak zaprojektować nowy projekt, obrzydzenie do uczenia się noeych rzeczy w tym OOP. Jeżeli moje wypowiedzi wskazywały na coś innego to to jest sprostowanie.

Teraz programowanie funkcyjne jest trendy.