
Maxim
05.05.2016
11:29:11
И изза этого проекта деньги теряю(((

Roman
05.05.2016
11:29:26
воспринимай как вызов скиллам и опыту

Noi
05.05.2016
11:29:41
тогда не интересно
там где не платят,мне не интересно

Google

Noi
05.05.2016
11:29:47
)

Maxim
05.05.2016
11:29:48
Та какой тут вызов скилам - тупой быдлокод

Roman
05.05.2016
11:30:17

Noi
05.05.2016
11:30:21
а
)
ну да
раз обещнулся
надо делать
)
с этим я согласен

Maxim
05.05.2016
11:32:03
Просто на поржать бросаю
И вот весь сайт строится на ОЧЕНЬ плохожих кусках кода
похожих*

Google

Roman
05.05.2016
12:04:13
напрямую потоком пишут
ниче потом преобразуют если надо будет часто переписывать
умиляет отсутствие комментов

Vladimir
05.05.2016
12:05:38
Глобалы, echo html кода
сделайте меня ращвидеть это

Maxim
05.05.2016
12:21:33
Глобалы и эхо ХТМЛ - это проблемы модха(цмс, на которой сайт)
А так да код более чем убог
80% переменных не соответствуют названию их смысл
Но, честно, этот код пишется абы писаться. В многих случаях есть вообще лишние переменные
90% фильтрации происходит на сервере, вместо фильтрованной выборки из базы

Alex
05.05.2016
12:52:10
Ребят, а если я скажу вам, что у меня есть шикарная не занятая ниша для разработки веб-сервиса, в котором реально нуждаются десятки тысяч людей. Кто из вас будет готов поработать за долю в компании? Это не очередной тревел-стартап, какое-то непонятное *уйпойми что для непонятно кого. Абсолютно прозрачная вещь для огромного пласта потребителей. Решим их проблемы — разбогаетеем.
Уже есть небольшая команда. Проект в стадии прототипа)

Roman
05.05.2016
13:04:36
а кто гарантирует эту долю, или вы себе уже имя сделали достаточное для подобного рода набора людей?

stgalkin
05.05.2016
13:05:09

Roman
05.05.2016
13:05:36
я просто давно в инете кручусь :)

Alex
05.05.2016
13:06:47

Roman
05.05.2016
13:06:59
либо ищите бизнес ангелов, покровителей инвесторов, либо как то публично заверяйте людей с юридическим подтверждением если у вас там все по белому храсиво и крустально

Oleg
05.05.2016
13:59:25
умиляет отсутствие комментов
я работаю в проекте где одного только кода на 2 гига - и ни единого коммента нет - переменные и функции нужно грамотно называть имхо

? Snyp
05.05.2016
14:20:31
Покажите пример таких переменных? В названии переменных.
Я например, комменты пишу на определенных участках кода.

Google

Oleg
05.05.2016
14:24:13
например
new CustomerPaydayFilter(Db) { PayDateFrom = searchparams.customerpaydatefrom, PayDateTo = searchparams.customerpaydateto },
new PaymentScheduleMaturityDateFilter(Db) { MaturityDate = searchparams.paymentschedulematuritydate },
new PaymentSchedulePaymentDateFilter(Db) { PaymentDate = searchparams.paymentschedulepaymentdate }

? Snyp
05.05.2016
14:32:35
Т.е. сокращять переменные не стоит?
Сколько раз видел - говорят длинные названия переменных только путают.
Например
Resdb (это resultdatabase)

Oleg
05.05.2016
14:36:18
естественно есть свои неудобства, и в сайте визитке смысла в подробных названиях нет - а когда у тебя переменная названа $a и откомментирована как //день рождения пользователя, и вызывается в 100 классах - то не надо так делать
ну с датабазами тут сам бог велел писать db
есть общепринятые сокращения например DAO и т.д...

? Snyp
05.05.2016
14:46:38
Спасибо за справку. Очень помогли)

Roman
05.05.2016
15:25:40

Oleg
05.05.2016
15:52:41

Roman
05.05.2016
15:57:05
не говорю что это сплошной минус, но лучше бы дополнять шаги все же комментированием, чтобы было удобно ориентироваться по проекту

Oleg
05.05.2016
16:00:04

? Snyp
05.05.2016
16:31:03
Покажите пожалуйста пример лапша-кода).

Oleg
05.05.2016
16:44:17

Maxim
05.05.2016
18:35:35
Звиздец смешанного HTML, пхп и ни одной попытки модулировать весь процесс

Noi
05.05.2016
18:37:12
@id111 условия все теже yii2 и mongodb)

Oleg
05.05.2016
18:52:03

Maxim
05.05.2016
19:02:50
Много разных вариантов значений - поведение такой конструкции, которую вы предложили сложно спрогнозировать(

Oleg
05.05.2016
19:06:49
судя по тому что ниже $postIndustry используется как Boolean - то видимо она может быть либо 0 либо 1 - ну можно конечно перестраховаться и сделать
$postIndustry = isset($_POST['industry']) && $_POST['industry'] != 0;
но смысла особого нет так как в изначальном коде $postIndustry = $_POST['industry'];
и ниже if($postIndustry

Google

Maxim
05.05.2016
19:07:23
Она используется как стринг
Буленом она становится только если она пустая
Либо равна 0
if ($postIndustry && !in_array($postIndustry,$industriesIds)) {
$isOk = false;
}
Здесь industriesIds - массив интов(стрингов), смотря к чему приведется
Так что и postIndustry должна быть интом(стрингом)

Oleg
05.05.2016
19:10:58
тогда $postIndustry = isset($_POST['industry'])
? $_POST['industry'] : false;

Maxim
05.05.2016
19:12:46
При $_POST['industry'] = 0 значение $postIndustry тоже станет 0(по-идее должен кастонутся в фолс), но я решил не рисковать:)

Oleg
05.05.2016
19:14:13
так а если 5 например?
if(5 && !in_array....

Admin
ERROR: S client not available

Oleg
05.05.2016
19:15:35
как бы некрасиво это

Maxim
05.05.2016
19:16:04
Знаю.
Но вводить еще один флаг не хочу.
Все что не емпти и не фолс кастуется как тру, так что имеем полное право.
Зачем нам еще динамическая типизация, если не для того, чтобы творить такую хрень;-)

Oleg
05.05.2016
19:18:43
тогда $postIndustry = $_POST['industry']; будет достаточно

Maxim
05.05.2016
19:19:16
Может прийти '0' и вот оно в фолс врядле кастонется.

Oleg
05.05.2016
19:19:58
почему врятли?

Maxim
05.05.2016
19:20:26
Потому что это строка
Не нулевой длины

Oleg
05.05.2016
19:22:00
$industry = $projectObj->getTVValue('industries'); сюда добавь

Google

Maxim
05.05.2016
19:23:12
Это не то. Эта сущность не есть отображением $postIndustry

Oleg
05.05.2016
19:23:50
в смысле сюда $industriesIds

Денис
05.05.2016
19:23:52
Приведется

Oleg
05.05.2016
19:24:57
и этот кусок вроде так должен быть
if(!$postIndustry || in_array($postIndustry,$industriesIds) ||
!$postService || $services == $postService ||
!$postYear || $year == $postYear ||
!$postPerson || $person == $postPerson){
$resultHtml .= getNewsItem($title, $date, $industry, $href);
}

Maxim
05.05.2016
19:26:50
Привиделся к false
Честно я не помню вот всех этих нюансов что к чему приводится неявно;-) когда у тебя дедлайны просроченны на неделю. При этом не простые дедлайны, а дедлайны дедлайнов, когда тебе за проект не платят, когда ты, когда обещал его сделать, думал, что там вообще сам ни строчки ПХП не напишешь, то гораздо быстрее сбыдлокодить;-)
Чем гуглить к чему приведеться '0';)

Денис
05.05.2016
19:27:50
Я просто поправил)

Maxim
05.05.2016
19:28:07
Ну теперь буду знать

Денис
05.05.2016
19:28:16

Noi
05.05.2016
19:29:52

Денис
05.05.2016
19:30:28

Maxim
05.05.2016
19:31:49
Увольнялся с роботы, предупредив за пару дней, ну и там на меня рассчитывали этот проект. Пообещал его сделать(дабы получить остатки ЗП), думая, что работы на 2 дня. Я никогда так не ошибался......

Noi
05.05.2016
19:32:37
Максим,я тебе сочувствую
но как сказал один умный человек в этой группе,зато ты наберешься скилла)
что хоть за сайт то?
сложная b2b система?
html верстка есть?
если там меньше 10 страниц верстки
то я впринципе готов помочь