@phpgeeks

Страница 42 из 8430
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
а кто гарантирует эту долю, или вы себе уже имя сделали достаточное для подобного рода набора людей?

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

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: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
Звиздец смешанного HTML, пхп и ни одной попытки модулировать весь процесс
особо доставило $postIndustry = isset($_POST['industry']) && $_POST['industry'] != 0 ? $_POST['industry'] : false; вроде на этом можно было бы остановиться $postIndustry = isset($_POST['industry']) && $_POST['industry'];

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
Чем гуглить к чему приведеться '0';)
Ну можно просто знать, как работает интерпретатор)

Я просто поправил)

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

Денис
05.05.2016
19:30:28
зачем брал если все так плохо?
Не увидел всего айсберга наверно)

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

Noi
05.05.2016
19:32:37
Максим,я тебе сочувствую

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

что хоть за сайт то?

сложная b2b система?

html верстка есть?

если там меньше 10 страниц верстки

то я впринципе готов помочь

Страница 42 из 8430