Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > Песочница > Отвратительный код гл программиста


Автор: chaos 19.2.2008, 08:53
День добрый!!! столкнулся с такой проблемой:
встала задача перенести проект из под винь в линух. часть проекта котоую писал я перенес за пол дня тк я изначально все строил с использованеием boost, stl. все остальное писал гл программист. как он сообщил перед улетом в отпус начальнику отдела что его код кросс и проблем с переносом не будет. ответственность по переносу пала на меня.
кроссом там и не пахнет тк кругом используется win32(работа с потоками, синхронизация, работа с временем, BOOL, DWORD....)вообщем это не так уж сильно страшно - медлоенно упорно можно весь код перелапатить. 

плюс шокировали подобные функции(их гораздо больше):
Код

template <class _I>
void UIntToStr(_I i, std::string &dst) 
{
    char buf[200];
    char result_buf[200];
    BOOL neg = FALSE;

    unsigned buf_ptr = 0;

    do 
    {
        buf[buf_ptr++] = (i%10)+int('0');
        i /=10;
    } while (i);

    int ix = 0;
    while ( buf_ptr-- > 0)
    {
        result_buf[ix] = buf[buf_ptr];
        ix++;
    }
    result_buf[ix] = '\0';
    dst = result_buf;
}


Код

template <class _I>
void IntToStr(_I i, std::string &dst) 
{
    char buf[200];
    char result_buf[200];
    BOOL neg = FALSE;

    if ( i < 0 ) 
    {
        neg = TRUE;
        i = -i;
    }
    unsigned buf_ptr = 0;

    do 
    {
        buf[buf_ptr++] = (i%10)+int('0');
        i /=10;
    } while (i);

    if ( neg ) 
    {
        buf[buf_ptr++] = '-';
    }

    int ix = 0;
    while ( buf_ptr-- > 0)
    {
        result_buf[ix] = buf[buf_ptr];
        ix++;
    }
    result_buf[ix] = '\0';
    dst = result_buf;
}


Код

template < class MAP_TYPE >
void copy_map(MAP_TYPE &src_map, MAP_TYPE &dst_map)
{
    dst_map.clear();
    MAP_TYPE::iterator it = src_map.begin();
    while ( it != src_map.end() )
    {
        dst_map[it->first] = it->second;
        it++;
    }
}


Код

int BSTR2str(std::string &s, const BSTR &bstrVal)
{
    int len = wcslen(bstrVal);
    s.reserve(len+1);
    int ix=0;
    for( ; ix< len; ix++)
    {
        s[ix] = (char)bstrVal[ix]; // trunkate !
    }
    s[ix] = '\0';
    return len;
}


так же в проект есть как минимум 3 реализации сокетов(наследованием и агрегацией не пахнет) smile больше я не нашел 


Скажите пдалуйста: это я много о себе думаю или же действительно что-то не то у гл программиста?
если вы поддерживаете мою досаду, как решать подобные проблемы? жаловаться начальнику отдела на отвратительное программирование гл программиста или есть еще какие способы?
Как бы поступили вы на моем месте?

Заранее благодарен!


Автор: Lazin 19.2.2008, 09:26
Цитата(chaos @  19.2.2008,  08:53 Найти цитируемый пост)
Скажите пдалуйста: это я много о себе думаю или же действительно что-то не то у гл программиста?

у него ФГМ
код - полный ахтунг
Цитата(chaos @  19.2.2008,  08:53 Найти цитируемый пост)
Как бы поступили вы на моем месте?

Цитата(chaos @  19.2.2008,  08:53 Найти цитируемый пост)
или есть еще какие способы?

посоветуй ему перейти на дельфи, если не согласиться подсыпь мышьяка, или вали из этой конторы, если у них такой гл програмист, у них нет будущего  smile 

Автор: chaos 19.2.2008, 09:45
Цитата(Lazin @ 19.2.2008,  11:26)
посоветуй ему перейти на дельфи, если не согласиться подсыпь мышьяка, или вали из этой конторы, если у них такой гл програмист, у них нет будущего  smile

и конторы валить не хочется smile кормют не плохо. на счет будущего, у нас тут не один гл прогаммист, в данном случае он гл в моем отделе

Автор: Lazin 19.2.2008, 09:51
тогда темную ему  smile 

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

Автор: chaos 19.2.2008, 09:55
Цитата(Lazin @ 19.2.2008,  11:51)
можно только подсидеть

что это значит? smile

Автор: Lazin 19.2.2008, 10:28
это значить занять его место, но для этого нужно наверное научиться писать так-же как он  smile 

Автор: ТоляМБА 19.2.2008, 11:33
Я х**ю с нашего флейма  smile 

Можно узнать кто перенёс эту тему сюда?

Автор: chaos 19.2.2008, 11:41
Цитата(ТоляМБА @ 19.2.2008,  13:33)
Можно узнать кто перенёс эту тему сюда?

ответ прост smile никто!
я тут ее изначально создал smile

Автор: ТоляМБА 19.2.2008, 11:51
Ну землячок, скажу прямо-таки не ожидал от тебя такого

Автор: chaos 19.2.2008, 11:57
Цитата(ТоляМБА @ 19.2.2008,  13:51)
Ну землячок, скажу прямо-таки не ожидал от тебя такого

дааа smile беда!

есть что сказать по существу? smile

Автор: ТоляМБА 19.2.2008, 12:00
Во флейме?  smile 

Автор: chaos 19.2.2008, 12:08
Цитата(ТоляМБА @ 19.2.2008,  14:00)
Во флейме?  smile

все таки думаешь надо было в С++ запостить? smile

Автор: ТоляМБА 19.2.2008, 12:12
Да нахрена? Давай уж раз мы тут встретились о встрече Уральских Винградовцев в Екатеринбурге договоримся - нафига тематический раздел будем засорять, флейм всё стерпит, я сцылку всем нашим кину?  smile 

Автор: chaos 19.2.2008, 12:24
может тему прокоментируют всякого рода руководители проектов и тп.
было бы очень интересно

Автор: Shaggie 19.2.2008, 12:36
Или появится сам гл программист и скажет "Нихт! Я есть не писать этот кот! Это патделка!"

Автор: Kangaroo 19.2.2008, 12:45
Цитата(chaos @  19.2.2008,  11:24 Найти цитируемый пост)
может тему прокоментируют всякого рода руководители проектов и тп.

Такие во флейм не заходят  smile 

Автор: ТоляМБА 19.2.2008, 12:52
Kangaroo, не согласен.

Автор: Mayk 19.2.2008, 12:54
Цитата(chaos @  19.2.2008,  16:08 Найти цитируемый пост)
все таки думаешь надо было в С++ запостить? smile 

Ignat постил уже схожую тему в общеси. Правда у него кажеццо был не гл программист. Хотя не помню, лень смотреть.
А вообще да, беда.

Цитата(Lazin @  19.2.2008,  13:26 Найти цитируемый пост)

у него ФГМ

Что-что-что головного мозга?

Цитата(chaos @  19.2.2008,  16:24 Найти цитируемый пост)
может тему прокоментируют всякого рода руководители проектов и тп.

Рук-ли здесь редки гости. Это скорее в УП.  Там можно прижать всяких chief39'ов к стенке и сказать "чего ты боишься больше всего? я этим буду пугать своего рук-ля".


Цитата(chaos @  19.2.2008,  12:53 Найти цитируемый пост)

если вы поддерживаете мою досаду, как решать подобные проблемы? жаловаться начальнику отдела на отвратительное программирование гл программиста или есть еще какие способы?

ыы! точно подходишь к руклю рукля  и гришь "главный рукль пишет вот такой код: <цитата на 50 строк>, я ту же задачу решаю в 10 раз экономичнее и быстрее: <цитата кода на 5 строк>. следовательно я заслуживаю 10 зарплат главного рукля, свободный график и молоко за вредность."  
smile

Добавлено через 1 минуту и 24 секунды
Цитата(Kangaroo @  19.2.2008,  16:45 Найти цитируемый пост)
Такие во флейм не заходят  smile  

Ты что, Wowa'у ни разу не видел во флейме?

Автор: Kangaroo 19.2.2008, 13:03
Цитата(chaos @  19.2.2008,  07:53 Найти цитируемый пост)
если вы поддерживаете мою досаду, как решать подобные проблемы? жаловаться начальнику отдела на отвратительное программирование гл программиста или есть еще какие способы?

А начальник отдела понимает что-нибудь в программировании? Если да - то почему бы просто не показать ему эти куски кода? А дальше сам будет делать выводы.


Цитата(ТоляМБА @  19.2.2008,  11:52 Найти цитируемый пост)
Kangaroo, не согласен. 

Ну тогда так: "их немного и заходят нечасто"


Цитата(Mayk @  19.2.2008,  11:54 Найти цитируемый пост)
Ты что, Wowa'у ни разу не видел во флейме?

Честно - нет  smile
Правда я только полгода на Винграде.

Автор: Mayk 19.2.2008, 13:08
Цитата(Kangaroo @  19.2.2008,  17:03 Найти цитируемый пост)
Правда я только полгода на Винграде. 

Шоша недавно рекламировал свой "zanud.net". 
А ещё тут ТоляМБА, только тсс, никто не должен знать что он руководит подразделением рамблера. Даже он сам, и даже сам рамблер.

Автор: ТоляМБА 19.2.2008, 13:10
Цитата(Mayk @  19.2.2008,  15:08 Найти цитируемый пост)
никто не должен знать что он руководит подразделением рамбера. Даже он сам
 smile 

Автор: THandle 19.2.2008, 13:17
Цитата(Kangaroo @  19.2.2008,  13:03 Найти цитируемый пост)
Правда я только полгода на Винграде. 

Цитата

Регистрация: 7.10.2006


Э... Как понимать?

Автор: ТоляМБА 19.2.2008, 13:21
Ну тут видел одного - регистрация в 2005 а постов всего 7 - редко заходил человек видать

Автор: Сумасшедший 19.2.2008, 13:21
Цитата(Kangaroo @  19.2.2008,  12:45 Найти цитируемый пост)
Такие во флейм не заходят

ага.. ню-ню.. у меня даж девушка как-то зашла.. бывшая=(

Автор: ТоляМБА 19.2.2008, 13:27
потому видать и бывшей стала?  smile 

Автор: Kangaroo 19.2.2008, 13:29
Цитата(Mayk @  19.2.2008,  12:08 Найти цитируемый пост)
Шоша недавно рекламировал свой "zanud.net". 

Спам не считается участием во флейме  smile 


Цитата(THandle @  19.2.2008,  12:17 Найти цитируемый пост)
Э... Как понимать? 

Зарегистрировался, за первый год пару раз зашел и пару постов написал  smile  Поэтом веду отсчет только с этой осени.


Автор: Shaggie 19.2.2008, 13:36
Цитата(ТоляМБА @  19.2.2008,  13:21 Найти цитируемый пост)
регистрация в 2005 а постов всего 7 - редко заходил человек видать

Ты бы ещё katrin вспомнил!

Автор: Kangaroo 19.2.2008, 13:38
Цитата(Shaggie @  19.2.2008,  12:36 Найти цитируемый пост)
Ты бы ещё katrin вспомнил! 

Хто така?  smile 

Автор: Mayk 19.2.2008, 13:47
Цитата(Kangaroo @  19.2.2008,  17:38 Найти цитируемый пост)
Хто така

Если мы будем произность её имя очень часто, то она восстанет в "А здесь смотрели" внизу экрана!

Катрина! СРУК! Шушпанчики! Явитесь!

Автор: Shaggie 19.2.2008, 14:03
Цитата(Mayk @  19.2.2008,  13:47 Найти цитируемый пост)
она восстанет в "А здесь смотрели"

Там щас про секс только...

Автор: Сумасшедший 19.2.2008, 14:32
ТоляМБА, угу smile 

Автор: Mayk 19.2.2008, 14:35
CHER то есть
СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК 

Обновляйся подлый низ страницы!

Добавлено через 25 секунд
не обновился :( smile
хотя  наверное ему просто нужно время

Автор: marykone 19.2.2008, 15:51

Цитата(Mayk @  19.2.2008,  13:08 Найти цитируемый пост)
Шоша недавно рекламировал свой "zanud.net". 


а Шоша это что грузинский псевдоним Wowa

Цитата(Kangaroo @  19.2.2008,  13:38 Найти цитируемый пост)
Хто така? 


баян второе имя 

поиск рулит со словами " привет катрин "

Автор: Mayk 19.2.2008, 19:28
Цитата(marykone @  19.2.2008,  19:51 Найти цитируемый пост)

а Шоша это что грузинский псевдоним Wowa

Историческая справка по "Шоше":
http://forum.vingra.ru/forum/topic-48549/unread-1/hl/%25D1%2588%25D0%25BE%25D1%2588%25D0%25B0/index.html
http://forum.vingrad.ru/index.php?showtopic=36634&view=findpost&p=379631


кстати о "тынц"'ах. был сегодня на sql.ru/forum где это слово заменяет фразу "пруфлинк давай" и подумал что надо было идти во всякие dba --- судя по счетичкам сообщений dba'шники вообще нифига не делают
а только флеймят.  smile  smile  smile 

Автор: Str!pe 20.2.2008, 00:41
Цитата(Mayk @  19.2.2008,  12:54 Найти цитируемый пост)
и молоко за вредность

Молоко за вредное производство дают, а у вас тут вреда физического пока нет. Вот вернется главный программист, посмотрит, и будет членовредительство, тогда и молоко просите.
Цитата(Сумасшедший @  19.2.2008,  13:21 Найти цитируемый пост)
ага.. ню-ню.. у меня даж девушка как-то зашла.. бывшая=(

Помню, помню...

Добавлено @ 00:43
Кстати, а меня бог миловал от Катрин, то ли дел много было, толи я появился поздно и не решился вливаться в тот поток чистого сознания... Но в общем, мимо прошел.

Автор: Samotnik 22.2.2008, 00:00
2 all
я В сях не шарю, расскажите, что  не то в коде  плз !!!!   smile 

Автор: Mayk 22.2.2008, 08:47
Цитата(Samotnik @  22.2.2008,  04:00 Найти цитируемый пост)

я В сях не шарю, расскажите, что  не то в коде  плз !!!


Код
template <class _I>
void UIntToStr(_I i, std::string &dst) 
{
    char buf[200];
    char result_buf[200];
    BOOL neg = FALSE;
    unsigned buf_ptr = 0;
    do 
    {
        buf[buf_ptr++] = (i%10)+int('0');
        i /=10;
    } while (i);
    int ix = 0;
    while ( buf_ptr-- > 0)
    {
        result_buf[ix] = buf[buf_ptr];
        ix++;
    }
    result_buf[ix] = '\0';
    dst = result_buf;
}

template <class _I>
void IntToStr(_I i, std::string &dst) 
{
    char buf[200];
    char result_buf[200];
    BOOL neg = FALSE;
    if ( i < 0 ) 
    {
        neg = TRUE;
        i = -i;
    }
    unsigned buf_ptr = 0;
    do 
    {
        buf[buf_ptr++] = (i%10)+int('0');
        i /=10;
    } while (i);
    if ( neg ) 
    {
        buf[buf_ptr++] = '-';
    }
    int ix = 0;
    while ( buf_ptr-- > 0)
    {
        result_buf[ix] = buf[buf_ptr];
        ix++;
    }
    result_buf[ix] = '\0';
    dst = result_buf;
}


это нужно писать короче, пример так:

Код

template<typename I>
void IntToStr(_I i, std::string &dst) { // 
std::ostringstream out;
out << i;
dst = i.str();
}

а с 
Цитата(chaos @  19.2.2008,  12:53 Найти цитируемый пост)
boost

вообще вот так:
Код

template<typename I>
void IntToStr(_I i, std::string &dst) { 
 dst  = boost::lexical_cast<std::string>(i);
}

(иными словами при использовании буста inttostr вообще нафиг не нужен)

Автор: ТоляМБА 22.2.2008, 10:05
Mayk, винраром подрабатываешь?  smile 

Автор: Poseidon 22.2.2008, 10:20
Цитата(Mayk @  22.2.2008,  08:47 Найти цитируемый пост)
это нужно писать короче, пример так:
Хренасе!!! И это главный программист такие поэмы пишет? Тип делает вид что работает, что вот смотрите сколько строк кода накатал. Я в шоке. Он победил бы в конкурсе на самый большой код по выводу сообщение "Hello world"

Автор: marykone 22.2.2008, 13:10
Цитата(Poseidon @  22.2.2008,  10:20 Найти цитируемый пост)
Хренасе!!! И это главный программист такие поэмы пишет? Тип делает вид что работает, что вот смотрите сколько строк кода накатал. Я в шоке. Он победил бы в конкурсе на самый большой код по выводу сообщение "Hello world"


да меня тоже возмутило !!! 

Автор: Mayk 22.2.2008, 13:45
Цитата(marykone @  22.2.2008,  17:10 Найти цитируемый пост)


да меня тоже возмутило !!!  

КАК ОН ПОСМЕЛ НАПИСАТЬ ТАКОЕ! Праведный гнев хворумовцов не знает границ.  

Автор: marykone 22.2.2008, 15:49
Цитата(Mayk @  22.2.2008,  13:45 Найти цитируемый пост)
КАК ОН ПОСМЕЛ НАПИСАТЬ ТАКОЕ!

давно мы на костре жгли еретиков ?

Автор: Mayk 22.2.2008, 16:03
Цитата(marykone @  22.2.2008,  19:49 Найти цитируемый пост)

давно мы на костре жгли еретиков ? 

Да вроде в эту среду парочку -- было холодно и надо было согрется. А что? Только не говорите что этого нельзя было делать. Это же ой какой конфуз.

Автор: Kangaroo 22.2.2008, 16:16
Цитата(Mayk @  19.2.2008,  13:35 Найти цитируемый пост)
СРУК СРУК СРУК СРУК ... СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК СРУК 


Решил посмотреть в гугле че такое СРУК. Первая ссылка выдала ЭТУ страницу  smile  smile  smile 

Так кто объяснит?  smile 

Автор: Mayk 22.2.2008, 16:39
Цитата(Kangaroo @  22.2.2008,  20:16 Найти цитируемый пост)

Решил посмотреть в гугле че такое СРУК. Первая ссылка выдала ЭТУ страницу  smile  smile  smile 

Так кто объяснит? 

Ой. Кажеццо я сломал гугл smile 
Цитата(ТоляМБА @  22.2.2008,  14:05 Найти цитируемый пост)
Mayk, винраром подрабатываешь?  smile  

Лишь бы не он мной smile

Автор: Shaggie 22.2.2008, 21:02
Kangaroo, посмотри теперь вторую страницу...

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)