Který kód je lepší?

[dustin]

podstata je, ze jde o konzolovou aplikaci, ktera kdyz ma prepinac -p, tak se pouzije nejakej connection string. kdyz se nezada prepinac, tak je pouzitej nejakej default connection string.

Zde jsi popsal pravidla pro nastaveni connection stringu. Zatímco tvůj kód:

Ja mam default connectionstring jeste pred podminkou.

Takže tvůj algoritmus je jiný. Při čtení kódu si musím pamatovat, že na začátku už je nastavený default, takže princip je opačný, než jsi to sám vyjádřil vlastními slovy.

Osobně bych to dal do funkce třeba determineConnectionString() a při zjištění connection stringu bych jej rovnou vracel returnem. Ale někdo má rád v metodě jenom jeden return, proč ne.

jine prepinace tam nebudou.

Tipnul bych si, že časem budou. Je to tak snad vždycky.

Jenom pár drobností mimo dotaz:

* Váš systém na správu zdrojáků nepodporuje blame, kde bys rovnou viděl, kdo změnil příslušný kód?

* Pokud je connstr1 klíč k defaultnímu a connstr2 klíč k nějakému speciálnímu connection stringu, proč se klíče nejmenují rovnou "defaultconnstr" a třeba "customconnstr"? Čísla 1 a 2 nemají žádný význam, narozdíl od konkrétního názvu. Zdá se to jako blbost, ale správné významové pojmenování je úplně zásadní a ušetří spoustu chyb.

[Reklama]

[lopata]

Zprasit? Proc myslis? Mel jsme pouzit knihovnu? Ne, neni to zapotrebi. Nemel jsem jeste udelat to vic OOP jak je zvykem se na rootu bavit o OOP?

Podívej, to tvoje řešení je úplně blbě, to neokecáš. Když zadáš nějaký parametr navíc, tak se -p tiše ignoruje a použije se default. Proto ti to teamleader přepsal. Kdybys použil knihovnu, je menší pravděpodobnost, že to dokážeš takhle doprasit. Trochu sebereflexe by neškodilo.

[Tuxik]

Tak nad tím neuvažuj vůbec a dej tam

Kód: [Vybrat]

string connStr;
if (args.length == 1 && args[0] == "-p")
    {
        connStr = ConfigurationManager.ConnectionStrings["connstr2"].ConnectionString;
    } else {
        connStr = ConfigurationManager.ConnectionStrings["connstr1"].ConnectionString;
    }Máš to kratší a ještě víc jednoúčelový.

[Tuxik]

Zprasit? Proc myslis? Mel jsme pouzit knihovnu? Ne, neni to zapotrebi. Nemel jsem jeste udelat to vic OOP jak je zvykem se na rootu bavit o OOP?

Ne, není to o knihovně, je to o tom, že ignoruješ chyby zadání parametrů a to se ti jednoho krásnýho dne vymstí. Není to o téhle jedné aplikaci, které se použije 5x a z toho 3x při jejím testování, je to obecný problém, na který dojedeš. Opravil ti to správně, i když to není žádná krása, aspoň to funguje.

[bohous]

no jo mate pravdu. jenomze osobne resit knihovnu mi tady neprijde vhodny. aplikace je urcena jenom jednoucelove, export dat z db do souboru, coz se dalo udelat i pomoci sql a bcp utilitky. ne to nemuzu, mi bylo receno, protoze xp_cmdshell neni povolen. tak jsem spravil tuhle appku.

[Reklama]

[Ivan Nový]

Novej, ani jeden. Protoze ta aplikace se mozna pouzije ani ne 5x.

Pak ale nemá smysl zabývat se optimalitou nebo krásou kódu a použití knihovny je na místě, jak bude výsledek velký je v tomto případě jedno, protože se použije jen 5x a naprogramované to bude rychleji. Přitom se ještě tu knihovnu naučíte, což v budoucnosti jistě zvýší vaši produktivitu.

To, že nemáte standardy na psaní CLI je chyba, ale ne vaše.

Jinak podobné skripty se aplikací pomocí copy&paste šíří jako mor.

[bohous]

kdyz najdu knihovnu, vyzkousim ji, tak rychleji je to takle jak to mam naspany.

[Ivan Nový]

Tak nad tím neuvažuj vůbec a dej tam

Kód: [Vybrat]

string connStr;
if (args.length == 1 && args[0] == "-p")
    {
        connStr = ConfigurationManager.ConnectionStrings["connstr2"].ConnectionString;
    } else {
        connStr = ConfigurationManager.ConnectionStrings["connstr1"].ConnectionString;
    }Máš to kratší a ještě víc jednoúčelový.

A úplně neprůstřelné a připravené k rozšíření:

Kód: [Vybrat]

   for(var i in args) {
        if (args[i] == '-p')
           connStr = ConfigurationManager.ConnectionStrings["connstr2"].ConnectionString;
        else
           connStr = ConfigurationManager.ConnectionStrings["connstr1"].ConnectionString;
   }


[lopata]

kdyz najdu knihovnu, vyzkousim ji, tak rychleji je to takle jak to mam naspany.

Tak si do toho započítej čas teamleadera, který to musel zbytečně studovat a fixovat. Ve výsledku to bez knihovny není rychlejší.

[Tuxik]

A úplně neprůstřelné a připravené k rozšíření:

Kód: [Vybrat]

   for(var i in args) {
        if (args[i] == '-p')
           connStr = ConfigurationManager.ConnectionStrings["connstr2"].ConnectionString;
        else
           connStr = ConfigurationManager.ConnectionStrings["connstr1"].ConnectionString;
   }


a chybu tam máme oba, do toho ifu patří args[0].equals("-p")

[Kryštof]

kdyz najdu knihovnu, vyzkousim ji, tak rychleji je to takle jak to mam naspany.

Proč se vlastně tady ptáš, když žádná odpověď Ti není dost dobrá. Každé začátky jsou těžké a trocha pokory by chlapče neškodila.

[bohous]

zacatky? rofl. ja uz delam nekolik let vyvojare a za ne-maly penize. 

[Kryštof]

zacatky? rofl. ja uz delam nekolik let vyvojare a za ne-maly penize. 

Pak se Ti hluboce omlouvám, z toho kódu to nebylo poznat.

[bohous]

to byl jenom test, kde se ukazalo, jaki kokoti existuji na rootu. se divim, ze jeste nikdo nerekl, nech to udelam v lispu a ze to neni OOP

[Kiwi]

to byl jenom test, kde se ukazalo, jaki kokoti existuji na rootu. se divim, ze jeste nikdo nerekl, nech to udelam v lispu a ze to neni OOP

Tak sem nechoď a bude tu o kokota méně.