Говнокод #13776 — Java — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

Java / Говнокод #13776

+77

float a = Float.parseFloat(new_size + "");

Приведение дабла ко флоту.

Запостил:

Vladiator, 11 Сентября 2013

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

bormand 11.09.2013 18:52 # +3
float a = Float.parseFloat(Double.toString(new_size));
Ответить
- Vladiator 11.09.2013 18:56 # +1
  
  Лучше же String.valueOf(new_size) — тип может поменяться.
  Ответить
  - bormand 11.09.2013 19:05 # +7
    Небольшой этюд на тему valueOf:
    public String toString() { return String.valueOf(this); }
    Ответить
    - 3.14159265 11.09.2013 20:29 # +1
      
      lol
      Ответить
    - kegdan 12.09.2013 05:40 # 0
      
      Это называется "эзотерическое программирование"
      Ответить
    - anonimb84a2f6fd141 12.09.2013 17:21 # −1
      
      Боты, скажите мне, тчоделает String.valueOf?
      Ответить
      - bormand 12.09.2013 19:10 # 0
        
        Возвращает строковое представление чего угодно. Там перегрузки на все примитивы + одна на Object. Перегрузка на Object выглядит так:
        public static String valueOf(Object obj) { return (obj == null) ? "null" : obj.toString(); }
        Ответить
        
        anonimb84a2f6fd141 12.09.2013 23:35 # +1
        
        Как я подозревал, бесконечная рекурсия?
        Ответить
        
        bormand 12.09.2013 23:58 # 0
        
        Она самая.
        Ответить
- Dummy00001 12.09.2013 02:24 # 0
  
  простите за дилетантский вопрос: а почему нельзя floatValue() из java.lang.Number воспользоватся?
  Ответить
  - bormand 12.09.2013 05:13 # 0
    
    Можно, но зачем? :) Я же даже пометил свой код зеленым, чтобы его в серьез не воспринимали...
    
    А аноним ниже написал правильный ответ: как и в сишке тупо (float)new_size.
    Ответить
- guest 12.09.2013 23:48 # −7
  
  показать все, что скрытоОтсосу член парню из Владикавказа. Прут наглые хуястые самцы.
  
  [email protected] (Спросить Тараса)
  Ответить
- guest 12.09.2013 23:59 # −9
  
  показать все, что скрытоОтсосу член парню из Владикавказа. Прут наглые хуястые самцы.
  
  [email protected] (Спросить Тараса)
  Ответить
- guest 13.09.2013 00:00 # −9
  
  показать все, что скрытоОтсосу член парню из Владикавказа. Прут наглые хуястые самцы.
  
  [email protected] (Спросить Тараса)
  Ответить
  - guest 13.09.2013 00:00 # −9
    
    показать все, что скрытоОтсосу член парню из Владикавказа. Прут наглые хуястые самцы.
    
    [email protected] (Спросить Тараса)
    Ответить
anonimb84a2f6fd141 12.09.2013 04:19 # +1

(float)new_size

//Кэп
Ответить
- kegdan 12.09.2013 05:44 # 0
  
  а жаба так умеет? Шарп разу говорит "Parse или GTFO"
  Ответить
  - bormand 12.09.2013 05:58 # 0
    
    Пруф: http://ideone.com/awmCau.
    Ответить
    - guest 12.09.2013 23:48 # −8
      
      показать все, что скрытоОтсосу член парню из Владикавказа. Прут наглые хуястые самцы.
      
      [email protected] (Спросить Тараса)
      Ответить
  - Vladiator 12.09.2013 13:47 # 0
    
    Кстати, приславший мне этот код раньше на Шарпе программировал. Неужели в этом причина такого приведения?
    Ответить
    - 3.14159265 12.09.2013 14:34 # +2
      
      Сишкокасты - неочевидное для многих (не только начинающих, но и пересевших) уродство. Более того, джва раза надо тип указывать.
      SomeVeryLongShit<String,Object> a=(SomeVeryLongShit<String,Object>) b;
      Потому часто юзают понятный для чтения хелпер-метод, который укорачивает код. Для примитивов такого к сожалению нельзя сделать ибо нет женериков.
      
      public static <T> T cast(Object o){ return (T)o; }
      Ответить
      - Vladiator 12.09.2013 14:49 # 0
        
        К счастью, примитивы не сильно длинные.
        Ответить
        
        3.14159265 12.09.2013 15:58 # 0
        
        Для некоторых - длинные. Вон пониже юзают var.
        Но я бы предпочел конечно
        float f=cast(doubleVal);
        Во-первых, уродские скобки каста часто надо оборачивать еще одними скобками, во-вторых, человек впервые увидевший каст - как он поймет что это? Слово cast хоть можно нагуглить.
        Ответить
    - kegdan 12.09.2013 15:12 # 0
      
      В шарпе
      
      var a = float.Parse(newSize);
      Ответить
      - bormand 12.09.2013 15:14 # 0
        
        Эм, как-то не интуитивно...
        
        З.Ы. newSize в данном треде это дабл. Этот парс на даблах тоже пашет, не только на строках?
        Ответить
        
        kegdan 12.09.2013 15:16 # 0
        
        я думал - строка
        
        нет, только для строк.
        для даблов
        
        var a = (float) newSize
        Ответить
        
        3.14159265 12.09.2013 15:57 # 0
        
        отож.
        PS var detected.
        Ответить
        
        guest 12.09.2013 23:48 # −8
        
        показать все, что скрытоОтсосу член парню из Владикавказа. Прут наглые хуястые самцы.
        
        [email protected] (Спросить Тараса)
        Ответить
- guest 12.09.2013 23:59 # −8
  
  показать все, что скрытоОтсосу член парню из Владикавказа. Прут наглые хуястые самцы.
  
  [email protected] (Спросить Тараса)
  Ответить
  - guest 12.09.2013 23:59 # −8
    
    показать все, что скрытоОтсосу член парню из Владикавказа. Прут наглые хуястые самцы.
    
    [email protected] (Спросить Тараса)
    Ответить

Говнокод: по колено в коде.

Java / Говнокод #13776

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

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