Главная страница
Top.Mail.Ru    Яндекс.Метрика
Текущий архив: 2008.06.29;
Скачать: CL | DM;

Вниз

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

 
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;
Скачать: CL | DM;

Наверх




Память: 0.57 MB
Время: 0.024 c
15-1210925436
Ega23
2008-05-16 12:10
2008.06.29
Года полтора-два назад тема была


15-1211081879
Mem
2008-05-18 07:37
2008.06.29
Как освободить память?


15-1210843260
Пробегал2...
2008-05-15 13:21
2008.06.29
Странно сохраняются cookie


2-1212169529
VovKul
2008-05-30 21:45
2008.06.29
Как пользоватся консольными приложениями


11-1190364115
grim
2007-09-21 12:41
2008.06.29
по поводу создания форм