Duplicita kódu Bash

[tojsemja]

ahoj,
jsem prudič, mám rád čistej kód, bez duplicit, návrhový vzory, takovej kód, co se čte jako poezie viz http://knihy.cpress.cz/cisty-kod.html

a lámu si hlavu, jak z tohodle kódu udělat jednu funkcičku. jednou potřebuju do logu zapsat datum, jednou vlastní nějakej řetězec, a předpokládám že nějaký takový podobný zapisování přijde...

Kód [Vybrat] Rozbalit


function printDatum {
if [ -f $LOG ]; then
date >> $LOG
fi
}

function printString {
if [ -f $LOG ]; then
echo $1 >> $LOG
fi
}


[Reklama]

[Kolemjdoucí]

mám rád čistej kód

Kód [Vybrat] Rozbalit



Tak to BASH není nic pro Vás.

[code]
function printDatum
{
    if [ -f $LOG ]; then
        if [ -z $1 ]; then
            date >> $LOG
        else
            echo $1 >> $LOG
fi
    fi
}


[tojsemja]

v budoucnu budu do logu potrebovat vkladat info o sitovnym nastaveni, info o disku... a predstavuju si ze to vlepim sem. a z pohledu co pisou v ty knize tak si nejsem jistej jestli je zcela vhodne.. pokud $1 je prazdne, tak zapsat, jinak zapsat datum. takovej kod se neda nijak hezky rozsirit..

[linux_noob]

Kód [Vybrat] Rozbalit


function printString {
if [ -f $LOG ]; then
echo $1 >> $LOG
fi
}
function printDatum {
printString `date`
}


jinak ocenuju jednotnou konvenci pojmenovani funkci (Datum vs. String).

[aaaa]

Tak ty funkci dej tech parametru vic, jeden z nich vystupni format, ne?
Tva otazka mi pripomina:
A: Na co jezdi spalovaci motor?
B: Na benzin
A: Ale ja chci mit kontejnerovou lod a letadlo a ty potrebujou mazut a na avgas, tva odpoved je prinejmensim nedostatecna

[Reklama]

[tojsemja]

Tak ty funkci dej tech parametru vic, jeden z nich vystupni format, ne?
Tva otazka mi pripomina:
A: Na co jezdi spalovaci motor?
B: Na benzin
A: Ale ja chci mit kontejnerovou lod a letadlo a ty potrebujou mazut a na avgas, tva odpoved je prinejmensim nedostatecna

v ty knize (fakt doporucuji precist) se dlouze pojednava ze cim vic ma funkce parametru, tim vetsi zlo to je... jinak nad tim resenim jsem taky premyslel

[Kolemjdoucí]

Až budete vědět co všechno to má dělat, tak nám dejte vědět.

v ty knize (fakt doporucuji precist) se dlouze pojednava ze cim vic ma funkce parametru, tim vetsi zlo to je...

Tady tohle se používá zhruba tak 20 let a zlo tam kromě autora knihy za tu dobu nikdo nenašel:

BOOL StretchBlt(HDC hdcDest, int nXOriginDest, int nYOriginDest, int nWidthDest, int nHeightDest, HDC hdcSrc, int nXOriginSrc, int nYOriginSrc, int nWidthSrc, int nHeightSrc, DWORD dwRop);

[tojsemja]

Až budete vědět co všechno to má dělat, tak nám dejte vědět.

v ty knize (fakt doporucuji precist) se dlouze pojednava ze cim vic ma funkce parametru, tim vetsi zlo to je...

ja nevim, ale ja nikdy nebudu vedet s cim zakaznik prijde a jak se zmeni pozadavky zitra? proto uz dnes chci mit kod tip-top?

[Kolemjdoucí]

ja nikdy nebudu vedet s cim zakaznik prijde a jak se zmeni pozadavky zitra?

S čím přijde v budoucnosti se bude řešit v budoucnosti.

[karel]

pletes si pojmy s dojmy, cisty, funkcni a hlavne jednoduchy kod proc ne ale u kompilovaneho jazyka do interpretovanych duplicity sem tam patrej ma to docela vliv na vykon

[JardaP .]

Pouzijte jednu funkci pro oba pripady (function printString). Jako parametr jednou predejte date ve zpetnych apostrofech, jednou retezec dle vaseho vkusu.

[OMG]

...
ja nevim, ale ja nikdy nebudu vedet s cim zakaznik prijde a jak se zmeni pozadavky zitra? proto uz dnes chci mit kod tip-top?

vyborny napad, takze u evidence dochazky treba rovnou naprogramuj ucetnictvi, kamerovy system a ridici system pro atomovou elektrarnu... co kdyby si to to pekarstvi nahodou zitra rozmyslelo a prislo s tim za tebou...
budto mas jasne requirements, nebo delas dort jak pejsek s kocickou...

[Tomáš Marný]

Mně dala nejvíc kniha Leo Brodie: Thinking Forth.

[Worker]

...
ja nevim, ale ja nikdy nebudu vedet s cim zakaznik prijde a jak se zmeni pozadavky zitra? proto uz dnes chci mit kod tip-top?

vyborny napad, takze u evidence dochazky treba rovnou naprogramuj ucetnictvi, kamerovy system a ridici system pro atomovou elektrarnu... co kdyby si to to pekarstvi nahodou zitra rozmyslelo a prislo s tim za tebou...
budto mas jasne requirements, nebo delas dort jak pejsek s kocickou...

To uz vypada ako Firefox, ci Emacs.

[nobody]

printString(){
    if [ -f LOG ]; then
        echo "${1:-$(date)}" >>$LOG
    fi
}