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

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

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

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

#include <map>
#include <stdio.h>
#include <vector>
using namespace std;
 
int main() {
   map<string, int> karta;
   for (auto pituh : vector<string> {"foo", "bar", "foo"}) {
    	karta[pituh]++;
   }
   printf("foo: %d\nbar: %d\n", karta["foo"], karta["bar"]);
   return 0;
}

<?php $karta = []; foreach (["foo", "bar", "foo"] as $pituh) { if (!isset($karta[$pituh])) { $karta[$pituh] = 0; } $karta[$pituh]++; } echo "foo: {$karta['foo']}\nbar: {$karta['bar']}\n";

Запостил:

guestinxo, 24 Сентября 2018

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

Steve_Brown 24.09.2018 11:15 # 0

И в чем проблема? Как инициализируется элемент при создании? Так int() - это 0.
Ответить
- guest8 24.09.2018 11:31 # −999
  
  показать все, что скрытоvanished
  Ответить
Elvenfighter 24.09.2018 11:33 # +3
Гиест, нафейхоа тебе вектор из стрингов? std::initializer_list наше всьо!
```
std::map<std::string, int> karta;
for (const auto pituh : {"foo", "bar", "foo"})
  karta[pituh]++;
```
https://wandbox.org/permlink/Tb7a3Odz3tqa0rgV
Ответить
roman-kashitsyn 24.09.2018 12:04 # +1

> <stdio.h>

Фи, настоящие крестолюбы используют <cstdio>.
Ответить
- Elvenfighter 24.09.2018 12:34 # 0
  
  Тут вообще <іострём> надо было.
  Ответить
KaKou-To_xyu 24.09.2018 13:36 # +2
Питония!
```
from collections import defaultdict

karta = defaultdict(lambda: 0)

for pituh in ("foo", "bar", "foo"):
    karta[pituh] += 1

print("foo: %s\nbar: %s" % (karta["foo"], karta["bar"]))
```
Ответить
- gost 24.09.2018 16:40 # 0
  Питониевый O(n^2):
  
  goroda = ("foo", "bar", "foo") karta = dict([(x, goroda.count(x)) for x in set(goroda)]) print("foo: %s\nbar: %s" % (karta["foo"], karta["bar"]))
  Ответить
- guest8 24.09.2018 18:25 # −999
  
  показать все, что скрытоvanished
  Ответить
  - gost 24.09.2018 22:47 # 0
    И в итоге в олимпиаде побеждает Питон:
    
    from collections import Counter print(Counter(('foo', 'bar', 'foo')))
    
    https://ideone.com/LQsoSw
    Ответить
    - guest8 24.09.2018 23:16 # −999
      
      показать все, что скрытоvanished
      Ответить
3oJIoTou_xyu 24.09.2018 13:39 # 0

На луа переведите.
Ответить
- KaKou-To_xyu 24.09.2018 14:25 # +2
  Держи, Хуй:
  karta = setmetatable({}, {__index = function(x) return 0; end}) for k, i in pairs({"foo", "bar", "foo"}) do karta[i] = karta[i] + 1 end print(("foo: %s\nbar: %s"):format(karta["foo"], karta["bar"]))
  Ответить
  - 3oJIoTou_xyu 24.09.2018 14:38 # 0
    
    Спасибо, Хуй.
    Ответить
    - KaKou-To_xyu 24.09.2018 14:55 # 0
      
      Пожалуйста, Хуй!
      
      ЗЫ. Хуй, расскажи молодому и неопытному Хую, можно ли этот цикл переписать как-нибудь по-другому?
      Ответить
      - TOPT 24.09.2018 16:09 # 0
        
        Можно более забавнее, например метаметод паирс вроде бы в lua 5.3 или 5.2 завезли, или через call как то такой метаметод, но не думаю короче.
        Посплю, попробую уделить этому время, Хуй.
        Ответить
gost 24.09.2018 14:18 # 0

Слишком процедурно и несовременно. Надо так: https://wandbox.org/permlink/Kr2SqCQCNfsthfZw
Ответить

roman-kashitsyn 24.09.2018 15:14 # +1

import qualified Data.Map as M
import Text.Printf

mapM_ (uncurry $ printf "%s → %d\n") $ M.toList $ M.fromListWith (+) $ zip ["foo", "baz", "foo"] [1,1..]

-- baz → 1
-- foo → 2

Ответить

OBEH 24.09.2018 21:46 # 0

Э-эээммм...
Ответить
- guest8 24.09.2018 22:20 # −999
  
  показать все, что скрытоvanished
  Ответить
  - guest8 24.09.2018 22:21 # −999
    
    показать все, что скрытоvanished
    Ответить
    - gost 24.09.2018 22:39 # 0
      
      Так в посте же есть перевод.
      Ответить
      - OBEH 24.09.2018 23:02 # 0
        
        Всё, вопрос снят (по привычке всегда пропускаю описания к кодам).
        Ответить

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

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

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

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