Pošljite nam vašo najboljšo kodo

Iz dolgčasa gledam zaposlitvene oglase in mnogi pišejo, da naj se priloži naša najboljša/ponosna stvaritev.

Sam pri tem nimam pojma kaj bi priložil. Mogoče kakšno bash skripto, kjer sem v eni vrstici rešil problem, za katerega bi rabil v drugih programskih jezikih par 10 vrstic. Še najraje pa bi poslal posnetke zaslona uporabniškega vmesnika, katerega sem naredil (ko stranke ne priloži dizajna in postanem deklica za vse).

Kaj bi vi poslali na tak oglas?

Link na GitHub, link na kakšne prispevke odprtokodnim projektom.

Za ostalo sam nisem prepričan, če ima smisla.

Če delovno mesto vključuje načrtovanje UI/UX, potem je posnetek GUI dobra stvar, sicer pa ne vidim njegove vrednosti.

Tako je. Človek lahko pokaže kvečjemu tiste projekte, ki jih naredi sam (izven podjetja) ali pa so odprtokodni.

Kaj bi vi poslali na tak oglas?

Nekaj, kar se ti zdi zanimivo. Ce bi sprasevali za programiranje, bi dal spodnjo skripto. Nekje je treba začet :)

#!/usr/bin/python3.4

import time;

index = 0;
sign = "+";
color_1 = "\033[9";
color_2 = "m";
width = 8;
edge = " | ";
direction = "right";

while True:
    _line = "";
    _line = str(_line) + str(edge);

    if int(index) > 0:
        _counter = int(index);
        while int(_counter) > 0:
            _line = str(_line) + " ";
            _counter = int(_counter) - 1;
    _line = str(_line) + str(color_1) + str(index) + str(color_2) + str(sign) + "\033[0m";
    if int(index) < int(width):
        _counter = int(index);
        while int(_counter) < int(width):
            _line = str(_line) + " ";
            _counter = int(_counter) + 1;

    if str(direction) == "right":
        if int(index) < int(width):
            index = int(index) + 1;
        else:
            direction = "left";
            index = int(index) - 1;
    else:
        if int(index) > 0:
            index = int(index) - 1;
        else:
            direction = "right";
            index = int(index) + 1;

    _line = str(_line) + str(edge);

    print(_line);
    time.sleep(0.2);

Hotfix 1.1 :)

try:
    time.sleep(0.2);
except KeyboardInterrupt:
    print("\nCtrl + C");
    break;

SimplyMiha je 31. maj 2017 ob 10:16 izjavil:

Tako je. Človek lahko pokaže kvečjemu tiste projekte, ki jih naredi sam (izven podjetja) ali pa so odprtokodni.

In zakaj ne bi smel pokazat na katerih projektih si delal prej, če je bilo to v okviru druge firme? Če je na projektu delalo več ljudi, potem ne, če pa si bil sam zadolžen za npr. kompletni grafični vmesnik, potem pa ne vidim razloga. Sploh če gre sicer za izdelek, ki se trži na prostem trgu in ni bil spisan specifično za interno uporabo nekega podjetja.

HotBurek je 2. jun 2017 ob 12:21 izjavil:

Kaj bi vi poslali na tak oglas?

Nekaj, kar se ti zdi zanimivo. Ce bi sprasevali za programiranje, bi dal spodnjo skripto. Nekje je treba začet :)

Strinjam se, da je nekje potrebno začeti.

Predlogi za izboljšave:

1) Python in podpičja? Vrstic ni potrebno zaključevati s podpičjem in zato naj se to znakovno smetje odstrani.

2) Manjka dokumentacija čemu aplikacija služi na vrhu.

3) Manjka razdelitev kode na logične sklope - funkcije.

4) Dodaj "standarden vzorec" za funkcijo main, kar bo olajšalo/omogočilo pisanje unit testov.

5) Spremenljivke na nivoju module niso zaželjene in so nevarne, če se jih uporabi, ne da bi za to obstajal dober razlog.

6) Popravi nekonsistentno poimenovanje spremenljivk. Zakaj enkrat index, spet naslednjič pa _count? To mi smrdi na nepoznavanje pravil za definiranje imen. Spremenljivki _line in _counter sta namreč tudi globalni spremenljivka in nista omejena na obseg while oz. if stavkov. Sploh pa je uporaba prefiksa "_" v tem primeru neskladna z njegovo običajno uporabo v Python-u - za označevanje imen, ki jih avtor modula oz. razreda šteje za zasebne.

7) Popravi nekonsistentno uporabo konstant. Enkrat imaš definirano "\033[9" kot color_1, potem pa nekje drugje uporabiš "\033[0m" neposredno v stavku. Oboje bi moralo biti globalna konstanta.

8) Popravi imena, da bodo upoštevala običajen Python stil (PEP 8). Npr. IME_KONSTANTE.

9) Odstrani neuporabno kodo (solato). Npr.:

_line = "";
_line = str(_line) + str(edge);

Zadoščalo bi že:

_line = edge

10) Odpravi neumnosti tipa:

if int(index) > 0:

Zadošča že:

if index > 0:

11) Izogibaj se uporabi znakovnih nizov, še zlasti, če jih želiš uporabljati kot enum. Namesto:

direction = "right";

# ...

if foo:
  direction = "left"
else:
  direction = "right"

Uporabiš Enum:

from enum import Enum

class Direction(Enum):
  RIGHT = 1
  LEFT = 2

direction = Direction.RIGHT

# ...

if foo:
  direction = Direction.LEFT
else:
  direction = Direction.RIGHT

12) Uporabljaj operator "a += b" namesto "a = a + b".

Zagotovo sem še kaj izpustil, ampak je bolje, da najprej odpraviš te težave, potem pa se še enkrat pogleda popravljen izdelek.