Главная страница
    Top.Mail.Ru    Яндекс.Метрика
Форум: "Прочее";
Текущий архив: 2008.06.29;
Скачать: [xml.tar.bz2];

Вниз

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

 
Vlad Oshin ©   (2008-05-14 18:24) [0]

Про стили написания кода: думал придираетесь
Одному то все понятно было, а тут вот, когда толпа..
Вот такое встретилось:
 Summa:=0;
 SummaNds:=0;
 chislo2:=16;
вот нахрен оно равно 16? Что это такое? Смотрим: chislo,chislo2,n:Integer;
никаких коментов нигде, нихрена..


 
Palladin ©   (2008-05-14 18:27) [1]

может день месяца? :)


 
БарЛог ©   (2008-05-14 18:30) [2]

16=4^2=2^4 :)


 
^-k2-^ ©   (2008-05-14 18:32) [3]

ставка НДС, это же очевидно :) поменяй на 18 :)


 
TIF ©   (2008-05-14 18:34) [4]

Замени на const ;)


 
Renegat   (2008-05-14 18:34) [5]

16d = 10h = 10000b. Хм. Там где-нить shl/shr/and/or есть?
А вообще интересное программное решение :)
Можно посмотреть код?


 
Romkin ©   (2008-05-14 18:40) [6]

Ведущего программера с канделябром не было :D


 
^-k2-^ ©   (2008-05-14 18:42) [7]

может как раз их ведущий и написал :)


 
Ega23 ©   (2008-05-14 18:43) [8]


>  chislo2:=16;


жесть.


 
@!!ex ©   (2008-05-14 18:43) [9]

> [6] Romkin ©   (14.05.08 18:40)

Как показала практика ведущими программистами вполне могут работать ребята не далеко ушедшие от простого батонокидательства.


 
Юрий Зотов ©   (2008-05-14 18:58) [10]

> @!!ex ©   (14.05.08 18:43) [9]

Но не очень долго. Либо подрастают, либо возвращаются в кодеры.


 
Поп Гапон   (2008-05-14 19:07) [11]


> Romkin ©   (14.05.08 18:40) [6]
>
> Ведущего программера с канделябром не было :D


Вот часто и ведущие так пишут, когда "думать некогда, тут прыгать надо". И сдать проект нужно было ещё месяц назад...


 
Юрий Зотов ©   (2008-05-14 19:21) [12]

> Поп Гапон   (14.05.08 19:07) [11]

Те, кто уже созрел для ведущего, хороший код и плохой код пишут за одинаковое время. Точнее, хороший они пишут не задумываясь, на автопилоте, а плохой не пишут вообще. Просто не умеют его писать.

Зато на отладку хорошего кода времени уходит меньше (по сравнению с плохим). В итоге получается, что хороший код экономит время, а вовсе не расходует лишнее. Проверено много-много раз.


 
Поп Гапон   (2008-05-14 19:33) [13]


> Юрий Зотов ©   (14.05.08 19:21) [12]
>
> > Поп Гапон   (14.05.08 19:07) [11]
>
> Те, кто уже созрел для ведущего, хороший код и плохой код
> пишут за одинаковое время. Точнее, хороший они пишут не
> задумываясь, на автопилоте, а плохой не пишут вообще. Просто
> не умеют его писать.


Часто бывает уже готовый "студенческий код" и нужно его заставить как-то работать(не переделывая). Глючит после этого оно по прежнему, но уже даже почти работает(можно впарить заказчику).

А студент стоит в три раза дешевле ведущего. А ведущий в режиме подтирания соплей из кода может прибрать за пятью студентами.

Хоть продукт будет говно, но он будет дешевле чем у конкурентов. Пример - Vista и MacOS.


 
Юрий Зотов ©   (2008-05-14 19:49) [14]

> Поп Гапон   (14.05.08 19:33) [13]

Увы... знакомая картина...
:-(

Причесать код за пятью все же сложно (еще ведь и своя работа есть, ее же никто не отменяет). За тремя - реально (более-менее). Причем нередко это причесывание сводится к полной переписке довольно больших кусков (вплоть до изменения иерархиии и функционала классов).

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

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

Жаль, что лишь редкие манагеры это понимают. "Скупой платит дважды" или "Мы недостаточно богаты, чтобы покупать дешевые вещи" - золотые поговорки. Они ведь и к найму работников тоже относятся.

Вспоминается стих Маршака про семь шапок из овцы. "Больше" не всегда значит "лучше".


 
Поп Гапон   (2008-05-14 20:21) [15]


> Юрий Зотов ©   (14.05.08 19:49) [14]
>
> > Поп Гапон   (14.05.08 19:33) [13]
>
> Увы... знакомая картина...
> :-(
>
> Жаль, что лишь редкие манагеры это понимают. "Скупой платит
> дважды" или "Мы недостаточно богаты, чтобы покупать дешевые
> вещи"
- золотые поговорки. Они ведь и к найму работников
> тоже относятся.


Здесь вы не правы. Студент стоит допустим 1 единицу денег, ведущий 3 единицы.

Заказчику при определённых ухищрениях можно представить студента как девелопера ~ ведущему.

Студент может писать код со скоростью той же что и ведущий(пусть и будет этот код особо глюкавым). Если ведущий только "подтирает"(работа у него в фиксе багов) то он может справится за 5 студентами.

Итого стоимость хорошего продукта 3*6 = 18 денег.

Стоимость плохого 1*5 + 3 = 8 денег.(тоесть почти в два раза дешевле).

И пусть это будет калл, но "ано палюбому дешевле чем у конкурентов".

Ну ещё сюда нужно приплюсовать зарплату впаривателя(менеджера продажника) в две деньги. Итого плохой продукт стоит 10 денег.

Ну а переделки за дополнительные деньги.

А вот для заказчиков пока особо долго доходит, что совокупная стоимость владения плохим продуктом обойдётся в три раза дороже, хоть заплатил он с самого начала дешевле.

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


 
antonn ©   (2008-05-14 20:31) [16]


> никаких коментов нигде, нихрена..

а ниже по коду?


 
Сергей М. ©   (2008-05-14 20:34) [17]

Удалено модератором


 
Loginov Dmitry ©   (2008-05-14 22:49) [18]

> вот нахрен оно равно 16? Что это такое? Смотрим: chislo,
> chislo2,n:Integer;


Какая разница, что это? Главное, чтобы работало!

Это что, самое плохое, что удалось откапать? :)


 
ketmar ©   (2008-05-14 22:53) [19]

>[18] Loginov Dmitry © (2008-05-14 22:49:00)
откАпать? чего капаем? накапайте мне 1024 капли.

---
Understanding is not required. Only obedience.


 
Eraser ©   (2008-05-14 22:56) [20]

> [9] @!!ex ©   (14.05.08 18:43)

ведущий не ведущий, а РП почти все не в зуб ногой. не с проста в России нету адобе, корелов и т.п. микрософтов.


 
Германн ©   (2008-05-15 01:15) [21]


> ketmar ©   (14.05.08 22:53) [19]
>
> >[18] Loginov Dmitry © (2008-05-14 22:49:00)
> откАпать? чего капаем? накапайте мне 1024 капли.
>

Сразу видно отличие "программиста" от персонажа "художественного" романа.


 
antonn ©   (2008-05-15 01:24) [22]


> не с проста в России нету адобе, корелов и т.п. микрософтов.

есть Компасы, 1Cы...


 
wicked ©   (2008-05-15 01:31) [23]

> Поп Гапон   (14.05.08 20:21) [15]
одна поправочка - продажник нужен всегда - хороший это продукт или плохой... просто навыки у него должны быть в разных случаях слегка разные

и в том, что этот момент многие упускают, и есть большая беда "хороших программеров", которые сами пытаются продать свой продукт


 
ketmar ©   (2008-05-15 02:09) [24]

>[23] wicked © (2008-05-15 01:31:00)
«впариватель» это называется.

---
Understanding is not required. Only obedience.


 
wicked ©   (2008-05-15 02:11) [25]

> ketmar ©   (15.05.08 02:09) [24]
как хошь называй, хоть трутнем
он должен быть в команде, если команда хочет самостоятельно успешно что-то продать


 
Palladin ©   (2008-05-15 02:33) [26]


>Поп Гапон (14.05.08 20:21) [15]

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


 
ketmar ©   (2008-05-15 02:37) [27]

>[25] wicked © (2008-05-15 02:11:00)
если команда хочет самостоятельно, то она впаривателей на порог не пустит. со впаривателем — это как раз когда продавать не хочет, только писать.

---
Understanding is not required. Only obedience.


 
VirEx ©   (2008-05-15 09:34) [28]

Я бы удивился если бы было записано chislo16:=16;


 
Vlad Oshin ©   (2008-05-15 09:45) [29]

нашел что это было
 chislo2:=16; // vlad 14.05.2008 это оказывается место в шаблоне Excel, куда выводить.
итого:
Const
NumStrFromTOP=16;
наверное, так будет

PS
комментария ниодного вообще на весь модуль

.FieldByName("Izdelie").AsString;
.FieldByName("SAP").Asstring;
.FieldByName("Brand").AsString;
.FieldByName("Specification").AsString;
.FieldByName("PcCount").AsString;
- такое хоть как-то понятно


 
Vlad Oshin ©   (2008-05-15 09:45) [30]

Удалено модератором
Примечание: Чини прокси


 
Vlad Oshin ©   (2008-05-15 09:45) [31]

Удалено модератором


 
TUser ©   (2008-05-15 10:11) [32]

Радуйся, что не китайцы тот код писАли, а то было бы так

整数整數2

или в крайнем случае

zhěngshùèr

http://slovari.h15.ru/chi-1.php


 
KSergey ©   (2008-05-15 10:20) [33]

Удалено модератором
Примечание: Выражения выбираем


 
KSergey ©   (2008-05-15 10:21) [34]

> KSergey ©   (15.05.08 10:20) [33]
> Удалено модератором
> Примечание: Выражения выбираем

А вот это зря. Я не знаю ка ксказать это иначе. Извините. Тем более, что никого лично здесь это не касалось. Просто типичные термины, я других не знаю. Еще раз извините.


 
Rouse_ ©   (2008-05-15 11:03) [35]


> Студент может писать код со скоростью той же что и ведущий

Угу... пишу со скоростью миллион строк в час... такая фигня получается. Нафих-нафих. Плавали - знаем.


 
SPeller   (2008-05-15 11:08) [36]

одного грамотного именования переменных/методов/классов/таблиц/полей достаточно чтобы разобраться в основах чужого алгоритма, даже без комментариев. Себя в свое время к этому приучил, поэтому в свои старые исходники вникаю быстро )


 
Style ©   (2008-05-15 11:14) [37]

Ну обычно когда не знаешь что такое chislo2 проще поискать поиском в файлах проекта и посмотреть, зачем же оно действительно нужно. Я не думаю что это такая проблема. Гораздо хуже когда переменная называется, "правильно" - NumStrFromTOP, но предназначена она совсем для другого )
так что в данном коде думаю ничего страшного нет, меня больше смущает отсутствие пробелов между переменной и ее значением, это нечитабельно!


 
Игорь Шевченко ©   (2008-05-15 11:19) [38]

SPeller   (15.05.08 11:08) [36]


> одного грамотного именования переменных/методов/классов/таблиц/полей
> достаточно чтобы разобраться в основах чужого алгоритма,
>  даже без комментариев


недостаточно


 
Style ©   (2008-05-15 11:23) [39]


> одного грамотного именования переменных/методов/классов/таблиц/полей
> достаточно чтобы разобраться в основах чужого алгоритма,
> даже без комментариев


чтобы разобраться в основах чужого алгоритма - даже комментариев не достаточно )))


 
Romkin ©   (2008-05-15 11:34) [40]

Глубокое заблуждение, что начинающий пишет с той же скоростью, что и опытный. Сколько раз уже измеряли, скорость иногда на порядок отличается.
А если работа ведущего убирать мусор за подчиненными - то и ведущего быстро не станет, и студенты ничему не научатся.
Есть, конечно, такие фирмы, с жуткой текучкой кадров, по этому признаку их и распознать можно. Но сейчас они уходят: написать-то продукт таким стилем можно, а вот сопровождение сделает его золотым.



Страницы: 1 2 3 вся ветка

Форум: "Прочее";
Текущий архив: 2008.06.29;
Скачать: [xml.tar.bz2];

Наверх




Память: 0.56 MB
Время: 0.04 c
2-1212096870
deras
2008-05-30 01:34
2008.06.29
Функция в MSSQL2000


15-1210793838
No_Dead
2008-05-14 23:37
2008.06.29
Перекрестные ссылки на сайте


8-1182504859
@!!ex_
2007-06-22 13:34
2008.06.29
Зацикливание в fmod.


15-1210745245
pasha_golub
2008-05-14 10:07
2008.06.29
Нарезалка DVD CD


2-1212375734
snake-as
2008-06-02 07:02
2008.06.29
MediaPlayer





Afrikaans Albanian Arabic Armenian Azerbaijani Basque Belarusian Bulgarian Catalan Chinese (Simplified) Chinese (Traditional) Croatian Czech Danish Dutch English Estonian Filipino Finnish French
Galician Georgian German Greek Haitian Creole Hebrew Hindi Hungarian Icelandic Indonesian Irish Italian Japanese Korean Latvian Lithuanian Macedonian Malay Maltese Norwegian
Persian Polish Portuguese Romanian Russian Serbian Slovak Slovenian Spanish Swahili Swedish Thai Turkish Ukrainian Urdu Vietnamese Welsh Yiddish Bengali Bosnian
Cebuano Esperanto Gujarati Hausa Hmong Igbo Javanese Kannada Khmer Lao Latin Maori Marathi Mongolian Nepali Punjabi Somali Tamil Telugu Yoruba
Zulu
Английский Французский Немецкий Итальянский Португальский Русский Испанский