Говнокод #25924 — C++ — Говнокод.ру

C++ / Говнокод #25924

+4
1. 01
2. 02
3. 03
4. 04
5. 05
6. 06
7. 07
8. 08
9. 09
10. 10
11. 11
12. 12
```
// File.cpp
QString File::size() const
{
    return QString::number(QFileInfo(m_path).size());
}

// ... somewhere in the code...

File* message = ...
...
if (message->size() == "0")
        return;
```
Commit b1aef142 "Refactoring"
Р - Рефакторинг

Запостил: salamon_style, 10 Октября 2019

Tweet
Комментарии (43) RSS
- Steve_Brown 10.10.2019 13:13 # 0
  
  Ну, разве что если это какой-нибудь промежуточный класс для отображения в интерфейсе, сделанном на Quick/QML. На случай, если нужно будет добавить значения типа "<unknown>".
  Но сравнение все равно подозрительное.
  
  P.S. quick ui quick!
  Ответить
  - salamon_style 10.10.2019 13:16 # +1
    
    Нет, это файл модели, в которой хранятся все данные. UI тут пока и не пахнет :) Отображать размер на UI тоже не планируется
    Ответить
  - ropuJIJIa 10.10.2019 14:05 # 0
    
    Кстати, а для класса QString сравнение перегружено или будут сравниваться указатели?
    Ответить
    - Steve_Brown 10.10.2019 18:08 # 0
      
      Конечно, перегружено, и сравнивает содержимое строк.
      А вот если оператор сравнения в структуре не перегрузить, то
      ...
      черт.
      Забыл. То ли будет пополевое сравнение, то ли сравнение будет не определено. Скорее, второе. Но точно не сравнение адресов.
      Ответить
      
      gost 10.10.2019 18:10 # +102
      
      Поругается на отсутствие перегруженного оператора, ЕМНИП.
      Ответить
      
      salamon_style 10.10.2019 18:46 # 0
      
      Будет тоже самое, что и с классом - ошибка компиляции. Все отличие class от struct в том, что модификаторы доступа дефолтные разные: private у class и public у struct, больше ничего
      Ответить
      
      Steve_Brown 10.10.2019 18:49 # +1
      
      Это понятно, просто я забыл, генерируется ли оператор== по умолчанию. Ну и мало ли в новом стандарте как-нибудь по-другому.
      Ответить
      
      guest8 10.10.2019 18:53 # −999
      
      показать все, что скрытоvanished
      Ответить
      
      salamon_style 10.10.2019 18:57 # +1
      
      Ага, они тут оператор <=> (three-way comparison) введут в C++20 (это не шутка!), который можно будет сгенерировать (и одновременно все оставшиеся 6 операторов сравнения: >, <, >=, <=, ==, !=)
      
      https://en.cppreference.com/w/cpp/language/operator_comparison#Three-way_comparison
      https://en.cppreference.com/w/cpp/language/default_comparisons
      
      А я еще когда-то угорал с JS и его === :(
      Ответить
      
      guest8 10.10.2019 19:00 # −999
      
      показать все, что скрытоvanished
      Ответить
      
      salamon_style 10.10.2019 19:05 # 0
      
      Тип того, но тут не -1, 0, 1 будет возвращать, а один из 5 ordering типов (по второй ссылке, что скинул выше, есть примеры). Но я пока боюсь это трогать :)
      Ответить
      
      ropuJIJIa 11.10.2019 13:15 # 0
      
      std::strong_ordering::equal
      std::strong_equality::equal
      std::partial_ordering::equivalent
      std::weak_ordering::equivalent
      std::weak_equality::equivalent
      
      И это я перечислил не все значения... Как всё сложно...
      Ответить
      
      MAPTbIwKA 11.10.2019 22:05 # +102
      
      икарус, пинг
      https://govnokod.ru/25937
      Ответить
      
      Steve_Brown 11.10.2019 09:57 # 0
      
      auto operator<=>(const Point&) const = default;
      
      Вот так и думал, что в новом стандарте что-то такое введут.
      Ответить
      
      MAPTbIwKA 11.10.2019 22:05 # +102
      
      https://govnokod.ru/25937
      Ответить
      
      MAKAKA 10.10.2019 18:48 # 0
      
      Ты перепутал с C#:там структура это value type и будет попо-левое сравнение
      
      В C++ struct это синоним класса просто там всё public.
      Ответить
- neTyx_npoTKHyTbIu 10.10.2019 13:18 # 0
  
  а поясните для питухов, как скомпилируется возвращение нумбера, когда в сигнатуре стринг?
  Ответить
  - salamon_style 10.10.2019 13:23 # 0
    
    [static] QString QString::number(int n, int base = 10)
    Просто переводит число в строчку
    Ответить
    - guest8 10.10.2019 13:35 # −999
      
      показать все, что скрытоvanished
      Ответить
    - MAPTbIwKA 11.10.2019 22:05 # +102
      
      https://govnokod.ru/25937
      Ответить
  - Steve_Brown 10.10.2019 13:26 # +1
    
    QString File::size() const { return 123; }
    Это не скомпилируется, поскольку в QString нет неявного конструктора/преобразования из int.
    А QString::number(int) возвращает QString.
    Ответить
    - guest8 10.10.2019 13:37 # −999
      
      показать все, что скрытоvanished
      Ответить
      
      Steve_Brown 10.10.2019 13:43 # +2
      
      Преобразование типов
      
      class Foo { public: operator int() { return 42; } }; int main() { Foo f = 1; int i = (int)f; // 42 int j = f; // 42 (неявное преобразование) }
      Ответить
      
      guest8 10.10.2019 15:08 # −999
      
      показать все, что скрытоvanished
      Ответить
      
      salamon_style 10.10.2019 13:46 # +2
      
      class Foo { public: operator int() const { return m_value; } private: int m_value { 42 }; }; int main() { Foo obj; int res = 1 + obj; // res == 43 return 1; }
      Ответить
      
      guest8 10.10.2019 15:09 # −999
      
      показать все, что скрытоvanished
      Ответить
      
      salamon_style 10.10.2019 17:35 # 0
      
      42
      Ответить
      
      Steve_Brown 10.10.2019 17:53 # 0
      
      Наверное, мы одинаковые люди.
      Ответить
- Web_Monkey 10.10.2019 16:52 # 0
  
  Блядь, зачем размер фала возвращать строкой?
  Ответить
  - guest8 10.10.2019 17:06 # −999
    
    показать все, что скрытоvanished
    Ответить
  - guest8 10.10.2019 17:25 # −999
    
    показать все, что скрытоvanished
    Ответить
    - guest8 10.10.2019 17:37 # −999
      
      показать все, что скрытоvanished
      Ответить
      
      ropuJIJIa 10.10.2019 17:47 # 0
      
      Именно поэтому я за «PHP».
      Ответить
  - Elvenfighter 10.10.2019 18:36 # +2
    
    Чтоб единицы измерения легче было дописать
    Ответить
    - guest8 10.10.2019 18:57 # −999
      
      показать все, что скрытоvanished
      Ответить
      
      KingKong 12.10.2019 01:05 # 0
      
      parseInt-ом лучше, куик
      Ответить
    - MAPTbIwKA 11.10.2019 22:05 # +102
      
      https://govnokod.ru/25937
      Ответить
  - MAPTbIwKA 11.10.2019 22:05 # +102
    
    https://govnokod.ru/25937
    Ответить
    - Web_Monkey 12.10.2019 00:59 # 0
      
      Неа, мне до тудава 20 часов на мамолете с 2-мя пересадками за 17 тыр. И это только в одну сторону.
      
      Да и я малость ебнуты и не способне к общению IRL.
      Ответить
      
      guest8 12.10.2019 01:27 # −999
      
      показать все, что скрытоvanished
      Ответить
      
      BATHbIu_nemyx 12.10.2019 02:25 # −102
      
      показать все, что скрытоЗа мили лети
      Ответить
      
      XYPO3BO3 17.12.2019 05:54 # 0
      
      Зачем ты малость ёбнуты и не способне к общению IRL?
      Ответить
- cmepmop 11.10.2019 23:48 # 0
  
  давай
  Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только в понедельник, среду, четверг или воскресение
Семь раз отмерь — один отрежь, guest!

А не использовать ли нам bbcode?

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

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

C++ / Говнокод #25924

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

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