1. Java / Говнокод #2101

    +77.2

    1. 1
    2. 2
    3. 3
    4. 4
    public static long hash(URL url) {
	FNV1 fnv; (fnv = new FNV1a32()).init(url.toExternalForm());
	return fnv.getHash();
}

    Какой читаемый код... :)

    Запостил: ShadowX, 05 Ноября 2009

    Комментарии (16) RSS

    • ava guest 05.11.2009 15:56 # +1

      Все читабельно, надо просто немного погуглить, чтобы понять что значит абревиатура FNV
      http://en.wikipedia.org/wiki/Fowler_Noll_Vo_hash
      Ответить

      • ava ShadowX 05.11.2009 16:07 # 0

        Вообще, код мой :) Но я 2-ю строку не назвал бы читабельной :)
        Не каждый сразу поймет последовательность действий.
        Ответить

        • ava g26g 05.11.2009 16:46 # 0

          а смысл был так писать ради одной сэкономленной строчки? :)
          Ответить

          • ava ShadowX 05.11.2009 16:53 # 0

            Вобще, если было бы возможно, написал бы в одну.
            Вся штука в том, что информационной нагрузки при чтении кода такие методы не несут, все в названии. В особенности это касается отлаженого кода :)
            Ответить

            • ava g26g 05.11.2009 18:45 # +1

              раз нагрузки при чтении нет, то к чему вопрос по поводу читабельности? :)
              значит, таки, нагрузка есть.
              Ответить

    • ava guest 06.11.2009 06:35 # 0

      public static long hash(URL url) {
      FNV1 fnv = new FNV1a32();
      fnv.init(url.toExternalForm());
      return fnv.getHash();
      }

      По-моему так намного лучше
      Ответить

      • ava ShadowX 06.11.2009 10:32 # 0

        По правилам оформления K&R да :) Но, на мой вкус, читается всеравно плохо :) В таком виде у меня был код до того как я наговнокодил :)
        Спасибо, за не равнодушие :)
        Ответить

      • ava guest 06.11.2009 18:03 # 0

        public static long hash(URL url)
        {
        FNV1 fnv = new FNV1a32();
        string someUrl = url.toExternalForm();
        fnv.init(someUrl);
        string hash = fnv.getHash()
        return hash ;
        }
        Ответить

        • ava ShadowX 06.11.2009 18:06 # 0

          Stop! Без фанатизма, это же не JavaScript, только там такие изыски дают пользу - скорость выполнения ;)
          Ответить

    • ava guest 19.11.2009 01:21 # 0

      А нельзя было просто
      URL.toString.getHash() // ???
      и что такое FNV - не понятно. Так что как говнокод не засчитывается.
      Ответить

      • ava guest 19.11.2009 01:23 # 0

        точнее
        return url.toString.getHash();
        или как в Java ME - там вообще хэши можно считать для любого класса.
        Ответить

        • ava ShadowX 19.11.2009 10:49 # 0

          Наверное хеш, особенный... Да и в коментах уже писали люди и пример уже был выше.
          Ответить

      • ava ShadowX 19.11.2009 10:47 # 0

        Глянь http://lmgtfy.com/?q=FNV1a32 - может поможет!
        Ответить

    • ava guest 23.11.2009 23:55 # 0 

      public static long hash(URL url) {
    return (new FNV1a32()).init(url.toExternalForm()).getHash();
}
      Ответить

      • ava ShadowX 24.11.2009 00:03 # 0

        Гениально, только функция init(String) возвращате void :-p
        Ответить

    Добавить комментарий