Closed ExSport closed 7 years ago
ExSport
commented
7 years ago ExSport
commented
7 years ago @Lubo Porad jakou máš představu ohledně toho XML parsování, nasměruj mě a já to zkusím přepsat. Díky
lubo
commented
7 years ago @ExSport Do komentaru som ti dal link na clanok z dokumentacie ohladom parsovania XML.
ExSport
commented
7 years ago @Lubo Dik za odkaz ohledne XML parsingu. Zatim jsem prepsal zbytek tebou zmineny, XML cast jak bude zase cas...
ExSport
commented
7 years ago Opravoval jsem kod a vylepsoval detekci, neprepisoval jsem cely puvodni kod, takze ty dve stejne casti neni ma prace:) Chtel jsem provest rychlou opravu at se zbytecne lidem nevygeneruji nevalidni NFO soubory/odkazy, ktere pak jen tezko opravi (ve vetsim poctu), tak z meho pohledu slo o rychly fix a ne prepisovani celeho kodu, protoze neni schick. Pokud s tim nemas problem, tak to please mergni pro ostatni a ja snad behem tydne, dvou prepisu tu XML cast. Driv to urcite nestihnu, tak at lidi zbytecne necekaji, pripadne kdyby se toho nekdo zhostil driv... 👍
lubo
commented
7 years ago Dobre. Normalne by som toto nespravil, ale ak to ludom nefunguje a ty proste nemas momentalne viacej casu, tak musime siahnut po kompromisnom riesenie. Este ale aspon treba opravit tie problemy, co naslo Code Climate.
ExSport
commented
7 years ago CodeClimatu se nelíbí délka řádku o 1 znak a když jsem na tvůj popud odstranil ty závorky když to bylo na dva řádky, nikdy se mi to nezkompilovalo ať už plus byl na začátku či na konci, takže mě napadlo už jen lomítko na konci ale to prý už není doporučováno/deprecated v novějších verzích Pythonu, pokud si pamatuji dobře. Proto jsem to dal na jeden řádek protože jinde jsem se dočetl, že ikdyz Climate řve, tak do 120 či kolika znaků je to stále ok. Btw. ten kód funguje, jen pro některé tituly vytvoří link na něco jiné nebo vytvoří neexistující link, či vytvoří nfo file ikdyz neexistuje žádný validní link. U seriálů navíc používal zbytečně bbaron.sk web. Nyní by odkaz na thetvdb měl být přesnější, je totiž generovan z imdb id a pokud se nenajde, zkusí to přes název bez roku (thetvdb nepodporuje rok takže s ním většinou nic nenajde), následně se pokusí i s rokem. No a hledání ještě upřesňuje přes Lang=CS a když nic nenajde, tak zkusí ještě speciální Lang=all, který hledá napříč jazyky (jen pro některé tituly pod all nebyla český titul, ale pod Lang=CS to bylo...Proto první preferuje lang=cs). A ještě jsem opravil bug, že pokud se nenalezl thetvdb link, pak se vůbec negeneroval nfo file ani když existoval ČSFD či imdb link. Na to XML parsování ale časem ještě mrknu, aspoň se něco nové naučím. Díky
lubo
commented
7 years ago Pockaj, aky plus ? V tych if statementoch nie su ziadne plusy... Ale nevedel som, ze tie statementy su pridlhe, v tom pripade su zatvorky OK a mal si mi to povedat. V kazdom pripade, Code Climate tu mame na enforcement nasej konvencie, cize ak reve, nie je to OK.
Ak to XML parsovanie teraz nevyriesis, tak ja to stale mozem mergnut s tym, ze ked to nebude fungovat, tak budeme prstom ukazovat na teba.
ExSport
commented
7 years ago Ok sorry ty plusy bylo něco jiné co jsem vyřešil:) Tu mě to asi nebralo NOT na konci ani na začátku řádku a bez těch původních závorek to házelo chybu při kompilaci, jestli si pamatuji dobře. No sice jsem ten původní XML parsing nepsal já, ale klidně ukaž na mě...dokud to neprepisu tak se vymluvim na původního autora 😄 Díky.
lubo
commented
7 years ago Slusnost kaze, ze kto sa toho posledny chytal, ten je za to zodpovedny :) Pridaj tam tie zatvorky a bude to na teraz stacit.
ExSport
commented
7 years ago Ok vrátím je tam ale ne dřív než v pondělí asi. Z mobilu to nedám:)
ExSport
commented
7 years ago Dik za merge