Jeden Wielki OffTopick, jeśli masz pisać na temat, nie wchodź! Opcje

[Damonsson]

[*] znicz wiary i nadziei dla osoby, która dotknie kodu po Tobie (IMG:style_emoticons/default/biggrin.gif)

[Pyton_000]

Like It!

@Damonsson Takie potworki widuję dość często. Albo się od nich oddalam albo robię reformat całego kodu w Stormie z nadzieją że On mi otworzy oczy (IMG:style_emoticons/default/smile.gif)

[buliq]

zastanawiam się co było myślą przewodnia podczas tworzenia takiej ładnej linijki (IMG:style_emoticons/default/tongue.gif)

[tzm]

inlineowcy (IMG:style_emoticons/default/biggrin.gif) wlos sie jezy na glowie

[starach]

Tak z ciekawości. Jakbyście to wy zapisali w takim razie?

[aniolekx]

Tak z ciekawości. Jakbyście to wy zapisali w takim razie?

może obejrzyj to wideo (głownie mam na myśli ten kawałek pod koniec, kiedy to autor robi refaktoryzacje kodu) i opowiedz wtedy na pytanie która wersja kodu jest czytelniejsza.

[scanner]

Zacznijmy od tego, że fajnie by było, gdybyś dał jakieś dane wejściowe i wynik spodziewany - jakiegoś unittesta by człowiek mógł zrobić - a tak to trzeba zgadywać, co autor miał na myśli grzesząc bardziej niż przy użyciu GoTo…
Nigdy nie zagnieżdża się ternary, nigdy.

[solificati]

Tak z ciekawości. Jakbyście to wy zapisali w takim razie?

Napisz testy jednostkowe to może ktoś się pokusi o rozwiązanie, bo analizowanie tego kodu jest średnio przyjemne. Ten trzeci argument replace to jest "wszystko do ostatniego (z maksymalnie dwóch) slasha"?

Coś w stylu:

Kod

(defn magic-function [ class ]
  (let [ magic-part (join "\\" (pop (split class #"\\"))) ]
    (lower-case (replace magic-part #"\\|/" "."))))

[Pyton_000]

Domyślam się że autor sam nie będzie wiedział po tygodniu.

[starach]

Trochę mi się wierzyć nie chce że dwie ( banalne ) instrukcje warunkowe zapisane liniowo i zagnieżdżone sprawiają takim starym wygom jak wy problem. Wybaczcie, ale mam wrażenie że szukacie dziury w całym.

Ten fragment odpowiada za tworzenie identyfikatora z nazwy klasy \ImTheVendor\MySweetProject\Library\UberClass = imthevendor.mysweetproject i napisałem go dla celów identyfikowania DIC w moim frameworku dla celów jego keszowania.

@aniolekx: Zaraz obejrzę ten screencast.

[solificati]

Ten fragment odpowiada za tworzenie identyfikatora z nazwy klasy \ImTheVendor\MySweetProject\Library\UberClass = imthevendor.mysweetproject i napisałem go dla celów identyfikowania DIC w moim frameworku dla celów jego keszowania.

Czyli

Kod

(defn magic-function [ path ]
  (lower-case (join "." (take 2 (split path #"\\|/")))))

Clojure to jednak ładny język.

Apropo, niesystematycznie sprawdzasz separatory ścieżek - w replece oba, ale w strpos już nie.

Ten post edytował solificati 27.02.2015, 17:57:31

[starach]

(...)
Apropo, niesystematycznie sprawdzasz separatory ścieżek - w replece oba, ale w strpos już nie.

Masz rację, ale to są naleciałości z poprzedniej wersji. Metodę ::createId($class) zmieniałem o drugiej w nocy. (IMG:style_emoticons/default/tongue.gif) W finalnej wersji jest tylko backslash jako separator sprawdzany.
Tak ona wygląda w całej okazałości.

[PHP] pobierz, plaintext 
/**
 * Create id from class name
 * Takes two first parts of a class and turns them into id
 * 
 * example:
 * <pre>
 *     ::createId('\Vendor\MyProject\SomeClassName');
 *     output: vendor.myproject
 * </pre>
 * 
 * @param string $class
 * @return string
 */
static public function createId($class) {
    $class = trim($class, '\\');
    return strtolower(str_replace('\\', '.', (($s1 = strpos($class, '\\')) ? substr($class, 0, (($s2 = strpos($class, '\\', $s1+1)) ? $s2 : $s1)) : $class)));
}
[PHP] pobierz, plaintext 

Ten post edytował starach 27.02.2015, 18:07:00

[solificati]

Czyli w php możnaby zrobić coś w stylu:

[PHP] pobierz, plaintext 
  return strtolower(implode(".",array_slice(explode("\\", $class),0,2)))
[PHP] pobierz, plaintext 

No fakt, php miało zawsze nieprzemyślaną kolejność argumentów.

[starach]

W poprzedniej wersji było na implode i explode, ale te funkcje są wolniejsze od strpos i substr.

edit>
Chociaż przyznam że dla tego akurat przykładu nie sprawdzałem. W związku z użyciem str_replace rozwiązanie na implode i explode może być szybsze.

Ten post edytował starach 27.02.2015, 18:29:44

[Pyton_000]

A co ty tego potworka wykonujesz 1kkk razy (IMG:style_emoticons/default/questionmark.gif)

[starach]

A co ty tego potworka wykonujesz 1kkk razy (IMG:style_emoticons/default/questionmark.gif)

Szacuję, że w ogólnym rozrachunku "ten potworek" (IMG:style_emoticons/default/tongue.gif) może być wywoływany nawet trzydzieści kilka razy w jednym wywołaniu. Coś za coś. Nie jest to na pewno jeszcze tak wydaje jak być powinno, ale definitywnie zadecyduje o tym profilowanie.

Za jakiś bliżej nieokreślony czas może wypuszczę całe stado swoich potworków i wtedy ocenisz czy kod jest faktycznie taki tragiczny.

<joke>
,a na razie MILCZ NIEWIERNY, bo spadnie na ciebie gniew script-kiddie i rzeki spłyną pastą termoprzewodzącą z twych lędźwi. (IMG:style_emoticons/default/tongue.gif)

[tzm]

poda mi ktos wady i zalety czystej instalacji php, mysql, apache, phpmyadmin vs xampp/mamp/lamp na linuxie? pierwszy raz sie skusilem o manualne instalowanie srodowiska i komponentow do pracy

[Posio]

Na linuxie to bez róznicy moim zdaniem, na windowsie troche czasu zaoszczędza. To po prostu prekonfigurowana paczka, nic wiecej.

[!*!]

Czyli w php możnaby zrobić coś w stylu:

[PHP] pobierz, plaintext 
  return strtolower(implode(".",array_slice(explode("\\", $class),0,2)))
[PHP] pobierz, plaintext 

No fakt, php miało zawsze nieprzemyślaną kolejność argumentów.

lub

[PHP] pobierz, plaintext 
preg_match('#([\w]+)\\\([\w]+)#', $class, $match);
return preg_replace('#\\\#','.', strtolower($match[0]));
[PHP] pobierz, plaintext 

[tzm]

Na linuxie to bez róznicy moim zdaniem, na windowsie troche czasu zaoszczędza. To po prostu prekonfigurowana paczka, nic wiecej.

ok, a ogarnij mi to... mam w pracy skrypty ponizej php 5.3 i takie co na 5.5+ dzialaja. nie do konca kompatybilne byly te wersje i na macu sie przelaczam w mampie miedzy wersjami. jak taki patent zrobic majac recznie instalowane php? bo rozumiem ze musze miec dwie wersje php zainstalowane i co dalej?