Witacie.

Fajna firma do której apliuję życzy sobię próbki kodu (jak z resztą spora część firm), jednak pierwszy raz na taką trafiam.
W aktualnej pracy zajmuję się rozbudową i wdrażaniem systemu CRM B2B i raz, że próbki stąd nie działałyby same w sobie, a dwa - nie bardzo mam praco wynosić cokolwiek z firmy a tymbardziej pokazywać konkurencji.

To co robie jako freelancer to zwykle przeróbki jakichś istniejących strukturalnych partyzantek które też się nie nadajć.

Moje pytanie: co w takiej sytuacji robić?

Pisać coś specialnie pod pokazanie?
Co to miałoby być?
Czego takie firmy oczekują?
Czy powinno tego być dużo? Jakaś działająca aplikacja?

Pozdrawiam.

Próbka kodu nie służy ocenie aplikacji, nie trzeba nic działającego wysyłać. Ma ona pokazać, że umiesz pisać prawidłowo sformatowany kod, dobrze nazywasz zmienne, używasz odpowiednich metod etc.

100-200 linijek wystarczy w zupełności, wybierz coś, o czym wiesz, że jest fajnie napisane i prezentuje Twoje umiejętności.

Większość osób na siłę coś klepie komentuje i dba o każdy detal a w codziennej pracy leci ciurkiem bez wcięć i innych ozdób. Według mnie takie wysyłanie próbki kodu jest nic nie warte

Prawdę mówiąc nie wyobrażam sobie pisania kodu ciurkiem bez wcięc - sam bym sobie utrudiał pracę ale ogólnie zgadzam się co do wypowiedzi.

Mam zamiar zaprezentować wspólną część kodu każdego modułu, z akcjami usuń dodaj edytuj itp. Klasę po któej dziedziczy każdy moduł i dzięki temu od razu posiada wszystke podstawowe akcje + jeden moduł. Co o tym myślicie?

Cały moduł jak pokażesz będzie najlepiej, wtedy pracodawca będzie mógł ocenić czy potrafisz podzielić MVC na M, V i C a to jakie akcje to już jest mało ważne. Najlepiej wyjmij cos z istniejącego projektu, a nie pisz od zera bo na siłę będziesz wymyślał zbędne kwiatki A najlepiej pokaż tu to Ci powiemy czy się się nadajesz

http://forum.php.pl/index.php?showtopic=168262

Dobrze działa na pracodawców, jak się dłubie coś swojego i trzyma to na githubie (oczywiście jako publiczne ). Z doświadczenia wiem, że nawet jak ten kod nie jest jakiejś super jakości (jak mój ), to robi pozytywne wrażenie (szczególnie jak jest to jakieś całościowe rozwiązanie).
Oczywiście, czym jest to mniejszy projekt, tym musi być lepiej napisany i tym większa szansa, że ktoś się w niego szczegółowo wgryzie.
Projekt(y) na githubie, czy ogólnie jakaś własna dłubanina, czy też wkład w jakiś projekt OS jest pozytywnie widziane bo pokazuje, że jesteś faktycznie programistą, a nie siedzisz przed telewizorem po odwaleniu swojej zmiany w tesco czy innej korporacji a nagle wpadłeś na pomysł zostania programistą
Także, że nie wstydzisz się pokazać swojego kodu. No i jest to w jakimś stopniu gwarancja, że to faktycznie twój autorski kod a nie copy-paste cudzego.
Oczywiście akcja typu sforkowanie cudzego projektu i zrobienie kilku kosmetycznych poprawek raczej nie zagwarantuje powodzenia

Poza wszystkim jest to dość wygodne - wrzucasz linka i niech sobie patrzą, nie musisz się zastanawiać ile wysłać, co wyciąć, co zostawić itp.

Przyjrzałem się klasie do tych akcji o których pisałem i to raczej nie przejdzie...
(2 posty niżej wrzuciłem w wygodniejszy sposó
Pisanie czegoś na nowo też raczej mija się z celem...

Wrzucaj na http://pastebin.com/ lub podobne


Dobre formatowanie, trzymaj się PSR-* + dokumentacja.

Dzięki !*!
http://pastebin.com/B63RbzkG

Ogólnie nie wygląda to źle - trochę mnie raziły statyczne odwołania do Message czy currentUser ale to jeszcze przejdzie.

Nie wnikam w sam kod ale jeśli chodzi o prezentację to masz niekompletne opisy metod:

[PHP] pobierz, plaintext 
    /**
     * Metoda służy do zapisania większej ilość rekodrów
     */
    public function saveFromArray($array, $mapper = array(),$defaults = array()) {
        // ...
    }
[PHP] pobierz, plaintext 

Dla takiego laika jak ja - nieznającego się w tej klasie zupełnie nie wiem co podstawic pod $mapper co pod $defaults itd... Dodatkowo nie wiem czego się spodziewać na wyjściu - co zwróci funkcja

[PHP] pobierz, plaintext 
    /**
     * Metoda służy do zapisania większej ilość rekodrów
     * @param array $array Opis ...
     * @param array $mapper Opis ...
     * @param array $defaults Opis ...
     * @return typ...
     */
    public function saveFromArray($array, $mapper = array(),$defaults = array()) {
        // ...
    }
[PHP] pobierz, plaintext 

Wiem wiem - czepiam się - ale kod wygląda ładnie i przydałby mu się taki opis.

Opisy były robione pode mnie. Tak właśnie się obawiałem, że nie minie mnie rozwinięcie ich opisów.
Nie wiem dlaczego rażące są odwołania do metod statycznych, mnie wydało się to lepsze niż globalsy czy instancje...
Może zrobić tak jak z instancją połączenia do bazy danych? Na stałe w obiekt i siedzi.

Pomijając sam kod to tego "haos", "zamierzam" w głównym opisie, mogą Ci nie wybaczyć...

Oprócz tego, że wszystko wrzucone do jednego worka to nie jest źle. Na pewno ponad przeciętność ;-)

Moim zdaniem należy zacząć od pojemnika na próbki:


Każdą próbkę należy wyciąć starannie i opisać dokładnie kiedy, skąd i w jakich okolicznościach została pobrana.

Pamiętajmy też o tak zwanej "próbce kontrolnej"

Wszystkie próbki dobrze obłożyć suchym lodem (by zachowały świeżość) przy pomocy takiego np. pojemnika:


Mając tak przygotowane próbki śmiało zgłoś się do HRu!

Policz ile w tym fragmencie kodu użyłeś $_SESSION['order'][$this->name]['order'], nie uważasz że można by to skrócić i zapisać to do zmiennej, chyba że na siłe robisz wymaganą liczbę lini

[PHP] pobierz, plaintext 
if (!$_SESSION['order'][$this->name]['order'] or $_SESSION['order'][$this->module]['how'] or $_REQUEST['order']) {
 
            if (in_array('position', $this->columnList) or in_array($_REQUEST['order'], $this->columnList)) {
 
                if ($_REQUEST['order'] and in_array($_REQUEST['order'], $this->columnList)) {
                    $_SESSION['order'][$this->name]['order'] = $_REQUEST['order'];
                } else if (!$_SESSION['order'][$this->name]['order']) {
                    $_SESSION['order'][$this->name]['order'] = 'position';
                }
 
                if ($_REQUEST['how'] AND $_SESSION['order'][$this->name]['order']) {
                    $_SESSION['order'][$this->name]['how'] = $_REQUEST['how'];
                } else if (!$_SESSION['order'][$this->name]['how']) {
                    $_SESSION['order'][$this->name]['how'] = 'ASC';
                }
 
                if ($_SESSION['order'][$this->name]['how']) {
                    $order = ' ORDER BY ' . $_SESSION['order'][$this->name]['order'] . ' ' . $_SESSION['order'][$this->name]['how'];
                }
 
            } else {
                unset($order);
            }
        } else {
            if ($_SESSION['order'][$this->name]['how'])
                $order = ' ORDER BY ' . $_SESSION['order'][$this->name]['order'] . ' ' . $_SESSION['order'][$this->name]['how'];
        }
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
    /**
     * Wstępna metoda służąca do zapisywania i aktualizowania elementów w bazie
     */
    public function save() {
 
        global $config;
[PHP] pobierz, plaintext 

global ? Serio

[PHP] pobierz, plaintext 
    public function getView() {
 
        $this->process();
 
        if (class_exists($_REQUEST['action'].'View')) {
            $viewClassName = $_REQUEST['action'].'View';
            $viewFocus = new $viewClassName($this);
 
            $viewFocus->display();
        } else {
            echo 'Nie odnaleziono widoku'; //throw new Exception('Nie odnaleziono widoku');
        }
 
    }
[PHP] pobierz, plaintext 

Według mnie kiszka panie.
- Pomieszanie z poplątaniem baza danych z widokiem
- Komentarze (sensowne) a raczej ich brak
- Nawiasy
- W metodzie delete nie sprawdzasz parametru (nigdzie nie sprawdzasz)

@skowron-line - Wszystko masz rację i dzięki za komentarz. To dlatego, że to jest w trakcie rozwoju zaczęte kiedyś tam... Odkąd zaczęło działać rozwijane tylko w razie potrzeby.

- $global - raz może być... A na poważnie... ...na szybko...
- komentarze - oczywiście do zrobienia
- co nawiasy?
- Parametr w delete - sprawdzam, w klasach które dziedziczą
- wszystko w jednej klasie - w tym przypadku czytelność jest, było więcej ale zacząłem rozdzielać sukcesywnie. Lepiej to robić powoli i z planem bo się można pogubić.
- $_SESSION['order'][$this->name]['order'] - a to chyba nawet nie działa tak jak powinno

Zrobiłem niedawno aktualizację, klasa sama rozpoznaje relacje i posiada akcje dodawania usuwania i edycji powiązań relacyjnych 1:n i n:n.
Wystarczy w pliku z definicjami modułu dać tablicę np:

[PHP] pobierz, plaintext 
    public $relationships = array (
        'products_categories' => array (
            'relationship_type' => 'many-to-many',
            'left_table' => 'product',
            'left_column' => 'id',
            'right_table' => 'category',
            'right_column' => 'id',
            'connection_table' => 'products_categories',
            'left_key' => 'product_id',
            'right_key' => 'category_id',
        ),
    );
[PHP] pobierz, plaintext 

I tak się zastanawiam czy by tego nie sprezentować... ale musiałbym poukładać w którymś module żeby było coś widać :/

Edit: najważniejsze - faktycznie tam są elementy które są żywcem z widoku ale tylko tymczasowo. Musze zrobić ład z widokami ale może faktycznie "na razie" to wywale...


Starałem się jakoś usprawiedliwić fakt, że sporo różnych rzeczy znajduje się w jednym worku. Czemu mogą nie wybaczyć?

Ponieważ "haos" pisze się "chaos", a "zamirzam" > "zamierzam"...

fuck... Zamierzam to literówka a z tym chaosem byłem pewien że pisze się przez samo h :/ Nie wiem skąd mi się to ubzdurało...

Adi32: Jaką wartość według Ciebie będzie miała teraz ta "próbka kodu"? Przecież ten fragment ma sobą reprezentować Twoje umiejętności, nawyki, sposób tworzenia kodu, a nie efekt wspólnej pracy forum. Jeżeli chciałbyś dowiedzieć się czy Twój kod zawiera jakieś błędy (projektowe, stylistyczne itp. itd.) zapytaj, na pewno kilka osób wypowie się na ten temat, ale nie podsyłaj tego później swojemu pracodawcy, bo jest to przecież zwyczajne okłamywanie go.

PS. W języku polskim brakuje czasami słówek, ale na "fuck" mamy ładny odpowiednik - kurwa, na pewno słyszałeś.

Przywykłem do nie rzucania kurwami na forach aby nikogo nie obrazić i nie łapać ostrzeżeń. fuck wydało mi się łagodniejszym odpowiednikiem, który odpowiednio odda to co aktualnie odczułem - wtopa, o rzesz kurwa albo krócej - fuck :/.

skowron-line zaproponował wrzucenie kodu. Chciałem jedynie poznać zdanie forumowiczów, czy się nadaje (wstępnie chciałem dowiedzieć się ogólnie czego oczekują pracodawcy).

Zaprezentowanie tego kodu moim zdaniem nie będzie oszustwem. Nie wiem dlaczego tak uważasz - ten kod wciąż jest mój, o jego poplątaniu (częściowe wtrącenie zarządzania widokiem) wiedziałem. Dowiedziałem się, że ogólnie się nadaje. Komentarze w każdym razie miałem dopisać. Mógłbyś rozwinąć/uzasadnić swoją wypowiedź?

Dodam jeszcze, że nie jest to efekt wspólnej pracy forum, nie pytam przecież jak napisać kod tylko czy to sie nadaje, czy to lepiej wywalić czy tamto dodać...

Jak pisząc po polsku, używasz obcych wyrazów, wychodzisz na kretyna.


Tu chodzi o sam fakt. Namaluj kotka, My Ci powiemy żebyś domalował wąsy, a kotek z wąsami zejdzie za milion $, tylko dlatego że je ma, ale to nie Ty wpadłeś na pomysł ich domalowania.

Nie chcesz rzucać kurwami, a jedną rzuciłeś tylko w innym, powszechnie znanym języku. Chociaż faktycznie "fuck" ma słabszy wydźwięk - przynajmniej dla mnie. Przyczepiłem się, bo nasz język ma naprawdę bogaty asortyment mniej lub bardziej dosadnych wulgaryzmów. Jednak porzućmy już ten nieistotny temat, na przyszłość jedynie: cholera.

Właśnie to miałem na myśli pisząc o wspólnej pracy forum. Taka próbka ma zaprezentować przeciętny kod jaki wychodzi spod Twoich palców. Ani jakiś śmieciowy, odpalany raz na pół roku, ani jakiś dopracowany i dopieszczony specjalnie pod prezentację. Innymi słowy, taki jaki będziesz na co dzień klepać w pracy, gdzie nikt nie powie Ci czy to lepiej wywalić, czy tamto dodać. Może określenie "oszustwo" było nieco za mocne, ale na pewno jest to pewnego rodzaju wprowadzenie w błąd swojego potencjalnego pracodawcę.

Zaś co do samej próbki. Najlepiej gdy zamiast jakiejś pojedynczej klasy możesz podlinkować jakiś mały projekt, moduł projektu czy najlepiej jakąś bibliotekę. Tam widać czy potrafisz organizować swój projekt, czy utrzymujesz jako taki porządek. Daje to pełniejszy obraz, szczególnie jeżeli chodzi o OOP.

Byłem przygotowany na tego typu posty z Twojej strony
Jednak przyznaje - mogłem użyć polskiego odpowiednika.

To by się zgadzało, jednak ja prezentuje swoje produkty. Nie pytam co warto byłoby "domalować".
Może faktycznie temat posunął się za daleko. Z całą pewnością nie chciałbym "oszukać" nikogo a tymbardziej przyszłego pracodawcy. Zresztą wcześniej czy później obróciłoby się to przeciwko mnie.


Gdybym się w ten sposób zastanowił, na pewno użyłbym polskiego odpowiednika. Przeważało w tedy odczucie wstydu spowodowane walnięciem błędu ortograficznego w czymś dość ważnym.


Zawsze wolę się do wszystkiego jak najdokładniej przygotować. Też pomyślałem, żeby nie przedobrzyć bo wyjdzie sztucznie.
Jednak nie o to mi chodziło. Jak już pisałem, chciałem poznać ogólne oczekiwania pracodawcy, a jak już wrzuciłem kod to dodatkowo doszło pytanie: 'czy się nadaje'.


I tak też postanowiłem. Udokumentuje ładnie kod, oddziele widok od imitacji modelu, dodam kilka rzeczy które zrobiłem wczoraj i zaprezentuję całość.

Ciszej, bo jeszcze ktoś pomyśli że jestem wredny i przewidywalny

Myślę, że już wszyscy wiedzą...

Szlag...