>>417027

Ты читал мануал по move_uploaded_file и по загрузке файлов? Я подозреваю, что нет, так как иначе ты бы не пытался скопировать код. Потому иди и прочитай мануал сначала:

http://php.net/manual/ru/function.move-uploaded-file.php
http://php.net/manual/ru/features.file-upload.post-method.php

Копировать FILES нет никакого смысла, так как там не хранится файл, а только его имя и расположение.

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

>>417030

У тебя MySQL пишет

> 5 warnings

Логично прочитать их с помощью SHOW WARNINGS;

> Алсо, как символы табуляции ставить не пойму?

По идее клаившей Tab но в редакторе может быть стоит опция заменять таб на несколько пробелов. Я думаю. в твоем случае дело не в этом.

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

Я бы советовал другой разделитель так как таб на вид очень трудно отличить от пробела и ты получишь кучу возни с ним.

В твоем случае виноват может быть невидимый символ \r который перемещает курсор в начало строки (и затирает то, что было до этого напечатано). Чтобы от него избавиться выбери Line Endings -> Unix вместо Windows. В Windows конец строки помечается 2 символами\r\n.

Аноны, я написал большую задачу на разработку сайта для сдачи тестов c условным названием «TestHub». Задача действительно большая, месяца на 2 если не больше, хотя я старался все что можно, выкинуть. Но зато она даст вам хорошие знания того фреймворка, на котором вы будете ее делать.

Вот, держите: https://gist.github.com/codedokode/8733007

Может кто-то хочет сделать? Если вы делали файлообменник, то это может быть ваша вторая задача.

Еще, напомню, тут был один анон, который хотел дальше что-то изучать: >>415362

> Привет, ОП, привет падаваны. Я давно здесь не появлялся в общем-то с полгода как устроился на работу. До этого прошел учебник опа, запилил файлообменник , арканоид и бложик. Пока хикковал было время учиться и процесс обучения шел довольно быстро, но работая с 10 до 19 (хуёвый график) ничего не успеваю теперь и чувствую что не развиваюсь, ну а если и развиваюсь то очень медленно. Но со следующей недели иду в двухнедельный отпуск и хочу посвятить всё это время какому-нибудь проекту и выучить при этом что-нибудь годное и востребованное на рынке труда (yii? ) Потому как мне остопиздел этот битрикс и клепание второсортных сайтиков и дебагинг этой параши. Тащемта зачем пишу, я уже пробовал пилить штуки в одинокого но никогда не работал над чем-то в команде. Мне было бы интересно с другим аноном попробовать запилить что-нибудь. Я вот думаю, мы могли бы с кем-нибудь объединиться, Оп, бы нам придумал задачку (или мы бы сами придумали). Заодно потренировались бы использовать коллаборативный аспект гита и гитхаба (все эти бранчи мёрджи и пул реквесты) а не просто гит адд гит коммит гит пуш, как мы умеем уже. Что думаете, есть тут такие кому интересно?

Вот, что я могу предложить от себя:

- сделать клон Youtube: можно загружать, смотреть видео, писать комменты. Придется повозиться с конвертацией видео в нужный формат, подбором настроек кодировкания, ffmpeg/mencoder, gearman. Делать разумеется на фреймворке Yii1, 2 или Symfony 2
- изучить тестирование и что-нибудь потестировать:

https://gist.github.com/codedokode/a455bde7d0748c0a351a
http://habrahabr.ru/company/yandex/blog/245869/ (пост от яндекса про тестирование и еще что-то)

- написать плагин чтобы тестовый фреймворк codeception поддерживал Slim (требуется изучить codeception)
- может изучить что-нибудь из серии backbone/knockout/angular и поучиться писать SPA (Single page Application) на яваскрипт. Но для этого надо хорошо знать сам яваскрипт.
- может ты не очень хорошо знаешь HTML/CSS и хочешь на них задания порешать?
- изучить яваскриптовый Audio API и написать генератор всяких звуков (если ты в этой теме хоть немного разбираешься)
- сделать борду (верстка: https://github.com/codedokode/board-markup ) на фреймворке, там еще с оптмизацией наверно придется повозиться
- огромное задание на TestHub (выше) которое ты вряд ли успеешь сделать
- сложное задание: есть такая популярная ORM Doctrine 2, она довольно сложно устроена. Если ты в ней создаешь десятки тысяч сущностей и пытаешься их сохранить в БД, она может тупить минутами (по моим расчетам, это слишком много) и ест память. Кто бы поковырялся в ней профайлером и оптимизировал этот процесс? И заодно снизить потребление памяти. Но учти, что доктрину разрабатывает крупная компания, и если бы все было так просто они бы сами давно все оптимизировали.
- есть такой пакетный менеджер как композер. Он в ходе обновления/установки пакетов ест память как не в себя и долго думает. кто бы это оптимизировал? Учти, что сам разработчик знает об этой проблеме и пишет что он пока не придумал что делать, так как алгоритм сложный.

Насчет участия в open source, что я тебе советовал. Наверно имеет смысл улучшать те продукты, которые ты сам используешь. Если тебе в них ничего не хочется улучшить или исправить, то ничего пока делать и не надо.

Ну и еще, ты бы мог сделать (и может выложить в open source) что-то для своей работы. Может, тебе приходится какие-то рутинные правки делать в CSS и это можно автоматизировать. Может можно автоматизировать работу с картинками. Может, можно написать бота который будет обходить страницы сайта и искать ошибки. В общем, это тоже вариант, уменьшить объем рутины.

>>417076

Переменные названы ужасно. Вот как я угадаю, что ты обозначаешь буква i, x, y? Надо называть переменные понятно, вроде level, total, и тд.

> if ($n <= $y) {

> $i -= 2;

> }

не очень понятно, зачем это. Нельзя ли как-то поменять начальные значения переменных, чтобы того же добиться?

Так, считает вроде верно.

>>417079

Ой, как все запутанно. Не, это нехорошее решение по моему. И плохо соответствует принципам ООП.

> try/catch

Ты по моему не пониамешь как используются исключения. try и throw не должны быть на одном уровне. Вот у меня есть урок про исключения в php, почитай: https://gist.github.com/codedokode/65d43ca5ac95c762bc1a

> e.msg = "неверный размер";

Это неправльно вообще. И делать console.log из конструктора незачем — непойманные исключения и так пишутся в консоль

> if (size == "big") {

Надо использовать константы, а не строки

> this.calorificValue = 40;

> this.cost = 100;

Незачем делать «цену» и «число калорий» свойством гамбургера так как их всегда можно рассчитать зная тип и начинку с помощью метода. Раз так. то хранить их незачем. Ведь в твоем варианте если поменять тип гамбургера, цена не пересчитается, а с методом - пересчитается.

> this.filling.push(....

> this.countPrice( );

А зачем вызывать countPrice? Пусть ее пользователь вызывает когда хочет узнать цену.

Нет никакой защиты от добавления одной добавки несколько раз.

>>417318

>>И плохо соответствует принципам ООП

> Почему?

Потому, что цена/калории в данном случае не являютя свойством объекта, а вычисляются из имеющихся. И потому что исключения неправльно используются. Если исправить, то может быть начнет соответствовать.

>>Надо использовать константы, а не строки

> Почему?

Потому, что константы делают код понятнее, а магические строки/числа — запутаннее.

Hamburger.SIZE_BIG — понятно что речь идет о размере гамбургера. Понятно, какие размеры в принципе бывают (можно посмотреть на список констант).

'big' — менее понятно, что это. Может размер, а может название какое-нибудь. Само по себе оно ничего не говорит. Более того, в одном месте оно может означать размер гамбургера, а в другом например что-то еще. В случае констант — это будет 2 разных константы.

Вот еще объяснение на английском: http://stackoverflow.com/a/4351110

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

> Ну я вызвал просто чтобы показать что оно работает.

Это хорошо, но ты его скопипастил раз 8. Чтобы показать что оно работает, можно написать его где-то отдельно, а не в самом классе.

> В условии задачи об этом речи и не идет.

Ну как же?

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

Естественно, попытка добавить 2 одинаковых добавки это тоже непраивльное испльзование класса. Да и сам подумай, какой код лучше — который говорит об ошибке или который молча делает неправильно действие?

>>417302

Черных рамок на картинке нет

У тебя ширина текста жестко зафиксирована и равна 230px а он должен тянуться и занимать свободное место.

Вот примеры резиновых лейаутов:

http://blog.html.it/layoutgala/
http://alistapart.com/article/holygrail
http://chikuyonok.ru/2009/06/float-columns/
http://chikuyonok.ru/2010/01/liquid-site-markup/

> ul {

> list-style-type: none;

Этим ты удаляешь точки у всех списков на странице, включая например списки в тексте. А надо удалять только у меню.

> font-family: "Trebuchet MS";

Надо писать в конце списка шрифтов стандартный шрифт: http://htmlbook.ru/css/font-family

> font-size: 20px;

Когда задаешь font-size надо переопределять line-height

Меню выглядит не как на картинке: не тот цвет, нет подчеркивания, пункты прижаты слишком тесно.

Заголовок должен быть нежирным шрифтом

>>417310

> function indexOf(

> return undefined;

Обычно такая функция возвращает -1 так как undefined легко спутать с 0 при сравнени через ==

> function Hamburger(type, stuff) {

Не проверяется правильность stuff

> if (this.ADDS_PROPS[additiveName] === undefined) {

Исплоьзуй оператор in

> if (indexOf(additiveName, this.adds) === undefined) {

Если такая добавка уже есть то лучше выкинуть исключение

> } catch (e) {

try|catch по моему лишний так как ошибки и так по умолчанию идут в консоль

Так, в остальном верно.

>>417321

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

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

Также, можно использовать таблицу (не в html, а в css display: table) растянутую на 100%.

Также, можно попробовать абсолютное позиционирование.

2 последних способа поддерживают прокрутку если контент больше высоты экрана.

> и ещё чтобы размер его уменьшался в зависимости от размера экрана. т.е на большом экране он больше, на маленьком -- меньше.

В 99% это смотрится глупо, имей в виду. Большой экран покупают чтобы на него помещалось больше информации.

>>417324

> $corner = number_format(360 / $len, 2);

number_format предназаначен для преобразования числа в строку. Для округления есть round, floor, ceil

В строках 15-25 какие-то лишние циклы по моему. Что мешает сразу строку разбить на буквы?

> почему не заполняется полностью круг

Потому что ты зачем-то все пробелы вырезал, а длину строки брал с пробелами

> foreach ($v as $val) {

Это можно заменить на implode

Насчет переменных, если плохо знаешь английский просто испоьзуй slovari.yandex.ru или google translate.

Круг не круглый, так как надо домножать не на 2, а на чуть больший коэффициент.

>>417325

Если у тебя элемент шире чем полэкрана появится линейка прокрутки. Надо позиционировать не левым краем по центру, а правым, чтобы он уезжал за левый край и не появлялась прокрутка.

>>417333

4 уровня отступов в учебной задаче???? надо упростить.

------------

Если у тебя есть код

if (условие) {
50 строк
} else {
2 строки
}

То его надо попробовать перевернуть, чтобы было

if (не выполняется условие) {
2 строки
} else {
50 строк
}

А еще лучше

if (не выполняется условие) {
2 строки;
return;
}

50 строк;

-----------

> if (variable === undefined) {

Чтобы проверить наличие элемента в объекте есть оператор in. Что за костыли? И почему у нас в массиве не может храниться undefined? Конечно, может, от этого он псевдомассивом быть не перестанет.

> arrayLike = false;

Тут незачем продолжать цикл

> if (toString.call

А где определение toString?

В остальном, верно.

>>417335

Зачем яваскрипт тащить? На CSS делается

Неприавльно проверять размеры экрана так как размер окна браузера с ними не совпадает.

>>417367

Потому что я пока не структурировал, некогда.

Аноны, я никого не пропустил? Напомните о себе тогда.

>>417580

> for ($k = 0; $k <= $len 4; $k++) {

> $screen[$i][$k] = ' ';

Этот цикл можно заменить на array_fill. Второй цикл, кстати, тоже.

> ?>

> <?php

Не очень понятно зачем они тут

В остальном, хорошо.

>>417752

А в задаче все понятно? Вайрфреймы страниц посмотрел?

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

Начать наверно стоит с модели. Спроектируй таблицы для хранения теста и вопросов, добавь SQL код как миграцию, сделай на их основе модели в Yii (там есть генератор, если что, но может тебе лучше их руками писать). После этого попробуй сделать страницу создания теста, пока без регистрации, пользователей и всего прочего. Для создания страницы нужен контроллер и HTML-шаблон.

В качестве шаблонизатора крайне рекомендую twig.

Устанавливать все стоит через композер — ты с ним работал?

Ну и как можно скорее все на гитхаб заливай, чтобы можно было проверить.

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

Если что задавай вопросы.

Гайд, который дал анон ниже, вполне хорош, но на английском, ты как с ним?

>>417762

А почему доки не на официальном сайте? А так, спасибо за ссылки, особенно на перевод.

>>417796

В шаблонах обычно находится только HTML код с редкими вставками переменных или вызовов функций:

<h1><?= html($title) ?></h1>

<p>Имя: <?= html($name) ?></p>

Потому проблемы о которой ты говоришь, нет. PHP код с логикой не должен быть в шаблоне, а должен быть в php-файле, примерно так:

index.php
--------

(логика на php)
require 'templates/index.phtml';

templates/index.phtml
-------------

(html-шаблон для отображанения данных)

То есть в одном файле логика в другом шаблон для отображения. По моему, удобно. Если тебе кажется что нет, покажи код, посмотрим.

>>417805

Я тебе еще советую решить 3 задачи по моей ссылке. Для начинающего они полезны. А потом, если тебе этого будет мало, позанимаемся оптимизиацией запросов на таблицах из миллионов строк (на маленьких неинтересно).

>>417809

Кстати чтобы посмотреть структуру таблицы иногда удобнее писать SHOW CREATE TABLE students\G

>>417800

Ты правильно делаешь, что задаешь вопросы. Я ведь мог и напутать что-то.

>> попытка добавить 2 одинаковых добавки это тоже непраивльное испльзование класса.

> Почему?

Потому что по условиям задачи

> Дополнительно, гамбургер можно посыпать приправой (+ 15 тугриков, 0 калорий) и полить майонезом (+ 20 тугриков, + 5 калорий).

Но 2 раза полить майонезом или приправой нельзя. Каждую добавку можно добавить только один раз. И при попытке добавить ее во второй раз логично указать программисту что он делает ошибку, а не игнорировать ее. Ведь может быть программист опечатался или что-то перепутал и он рад будет сразу получить информацию что что-то не так, чем обнаружить ошибку позже.

Вообще, лучше на любое подозрительное действие выдавать ошибку, а не игнорировать. Это называется принцип fail fast: http://habrahabr.ru/post/218325/

Это делает твой код более надежным.

Я замечу, что многие старые функции в php (а также всякие сишные API, например Windows API) работают по принципу Forgive, то есть прощают ошибки, выдавая предупреждение и продолжая работать. На мой взгляд, это плохо и способствует написанию менее качественного кода.

>>417797

Ох, надо подумать. Backbone — фреймворк, предоставляющий роутер и модели для написания SPA, правда он не предаставляет шаблонизатор, и надо подключать сторонний, вроде mustache или knockout.

У меня была мысль сделать задачку на SPA, но она не до конца оформилась. Идея сделать какое-то приложение с таблицами и формами, которыми можно добавлять данные, и чтобы все это могло синхронизироваться с сервером (тут сразу же возникают задачи загрузки/выгрузки данных, инкрементальных изменений, блокировок, конфликтов), а также работать в офлайне. Может блокнот для заметок? Приложение для рисования вайрфреймов? Или приложение для оформления страховки на машину? Приложение для офлайн-тестирования для сайта TestHub? Я к чему-то такому склоняюсь, тут много интересных вещей которые можно изучить. Может, на этой неделе соберусь с силами и напишу.

Такие задачки, кстати, придумывать не так просто, надо рыться в документации, искать разные плагины и библиотеки, прикидывать, сложно ли это будет сделать, и т.д. Но она конечно нужна тем, кто хочет хорошо разбираться в фронтенде.

Анон день добрый или не день.
Заканчиваю универ, скоро предстоит преддипломная практика, а в последствии и написание диплома. Всем известная штука.
Немного о себе. С самого рождения нет смысла рассказывать. Начну с того момента, когда первый раз сделал "привет мир". Первый раз "привет мир" было сделано на первом курсе, когда по программе изучали C++. По началу маленькие победы и большие проблемы, из-за свой туполобости, меня подстеригали. Но данные проблемы не останавливали, а наоборот мотивировали, что-то делать. Спустя какое-то время, примерно один семестр, заметил, что не все ребята тянут программу и решил помогать советами и делом. Под конец семестра на горизонте появились парни с заочного отделения, которым было трудно осваивать данное ремесло и решено было им помочь - начал делать их лабораторные работы за какое-то мелкого денежное довольстие. Спустя пару месяцев мне написал хороший знакомый, девушке которого необходимо было сделать лабораторные работы по программированию. Как в последствии оказалось лабораторные работы предстояло делать на PHP. Немного подумав, решил попробовать. Спустя пару дней лабораторные работы были выполнены. Так был получен первый опыт в PHP. Вспомнив детские забавы с некрасивой версткой в возрасте четырнадцати лет, захотел попробовать сделать хоть какой-нибудь сайтик. Далее методом проб и ошибок учился. Летом после третьего курса нашел заказчика и продал ему сайт визитку. Потом мелкий фриланс, работа в которе. Мэтт Зандстра.
На сегодняшний день реальный опыт работы в вэбе больше, чем полтора года. Последних месяца два использую Laravel. Классная документация, очень доставляет работа с ним). Хочу развиваться дальше. Дальше вижу перспективу в Symfony. И параллельно со всем этим углублять знания английского языка.
По специальности совсем не программист, а инженер по телекоммуникациям.
"Такой инженер помимо инженерии должен и программировать" (С) Хороший преподаватель с кафедры.
Как ты мог понять из моего маленького рассказа - еще на втором курсе мой выбор пал на программирование, а не на инженерию. Хотя в рамках свой специальности, умею что-то делать.
Суть моего вопроса сводится к следующему. Необходима годная тема диплома. Планирую написать веб-сервис. Например банкинг (работа с какой-нибудь отечественной платежной системой) или что-то в духе booking.com. Так же вариант написать большую систему для колл-центра, по сбору информации о входящих вызовах и распределении вызовов между операторами. Как-то так. С меня девочки)

>>417918

Миграции — это способ версионировать (то есть поместить в репозиторий вместе с кодом) изменения в БД. Представь, ты разрабатыаешь сайт и хочешь добавить колонку в таблицу. Как ты поместишь информацию об этом в репозиторий, передаешь ее своим воображаемым коллегам, чтобы они обновили свои базы, и как обновишь боевую БД? Очевидно, миграциями (плохие дети не знают про миграции и передают информацию на словах: «посоны, я там колонку создал, возьмите мою базу и у себя так же добавьте»).

Миграция — это скрипт, вносящий изменения в структуру базы данных.

В Yii есть поддержка миграций.

http://stuff.cebe.cc/yii2docs/guide-db-migrations.html (англ)
http://yiiframework.ru/doc/guide/ru/database.migration (русский, из Yii 1)

>>417921

> почему

>> size != Hamburger.prototype.BIG_TYPE

> при size = "small"

Так и должно быть

> Еще думаю надо методы countPrice() и countCalorificValue() объединить в один.

напиши как выглядит вывод цены в консоль с объединенными и раздельными методами и сравни.

Писать код лучше не в HTML создавая смесь языков, а в вкладке Javascript.

> if (size == Hamburger.prototype.BIG_TYPE) {

Что за пустые ифы?
if (size != ... && size != ...) { ...

> Hamburger.prototype = {

> BIG_TYPE: "big",

Как к такой константе обращаться извне класса? Не лушче ли писать Ham.XX = yy ?

> new Hamburger("small","cheese");

Ты не используешь константы тут, а надо бы использовать.

> switch (fill) {

> case "cheese":

Не удобнее будет сделать словарь вида (тип -> стоимость и калории) ?

У тебя нет проверок на то, что добавляется. Любую добавку можно добавить только один раз.

> } catch (e) {

> console.log(e);

Можешь объяснить зачем try/catch? Что изменится если их убрать?

>>418033

Расстояние между input и кнопкой не равно 10px. Там есть ссылка полезная: http://css-live.ru/articles/zagadochnye-otstupy-mezhdu-inlajn-blokami.html

ПРо box-sizing знаешь? Это не потому, что оно тут нужно, а просто спрашиваю.

>>418038

Удаление должно делаться методом POST то ест формой с кнопкой.

> Что бы строчки id, login например были ссылками/кнопками и при нажатии на них у меня таблица сортировалась по этим самым id или username?

Ну так поставь туда ссылку и сделай чтобы она передавала имя поля для сортировки в query string

> удаление пользователя по его юзернейму

Лучше по id

>>418040

> Дальше вижу перспективу в Symfony.

Правильно видишь

> Необходима годная тема диплома.

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

> Например банкинг (работа с какой-нибудь отечественной платежной системой)

А платежная система тебе даст достиуп к своим данным?

> в духе booking.com.

А как организации будешь заманивать?

> Так же вариант написать большую систему для колл-центра, по сбору информации о входящих вызовах

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

>>418041

Прочитай пожалуйста замечания >>418195\t

>>418044

Норм для начала.

>>418047

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

>>418056

Сделай на заголовках колонок ссылки вида

index.php?sort=+name
index.php?sort=-date

и т.д.

И если честно у меняи ощущение что ты плохо знаешь php, может тебе стоит задачку какую-нибудь решить? У нас есть похожая: https://gist.github.com/codedokode/d7e7f11449fc3bcb24b4

>>418060

Где изврат? Этой технологии много лет, например Апач так список файлов в каталоге позволяет сортирвать. Ты странные вещи пишешь.

>>418066

Зачем яваскрипт если можно без него, быстрее и проще?

>>418180

Ты по моему не знаешь php. PHP работает на сервере и в нем есть команда для вывода текста. Начни с учебника: https://php.net/manual/ru/language.basic-syntax.php

При этом doc.write вообще использовать не стоит, так как он во многих случаях не работает и от него только проблемы.

>>418196

В моем учебнике все решил? Хорошо, давай помучаем регулярки.

Задача 1: дан кусок HTML состоящий из комментариев, тегов с атрибутами, текста. Выведи названия всех встречающихся в нем тегов (ох-ох, HTML нельзя парсить регулярками, говорили они). Вот пример:

<b><!-- <i> это комментарий --><div class="1"><x это битый тег и он не считается

выведет

b
div

Усложнение: все что находится внутри тегов script, style, textarea считается текстом и как теги не рассматривается.

>>418347

> тесты к ней - 48 строк

Опубликуй код где-нибудь, я гляну (и может кто из анонов заинтересуется). Алсо, напиши как ты запускаешь тесты.

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

Код решения лучше постить на jsbin, jsfiddle, codepen или аналогичный сервис где он выполняется.

Так, твой код выглядит верным.

>>418206

> "small" != "big" возвращает false, почему так и должно быть?

А, значит я не так тебя понял. Вот я сейчас проверил: возвращает true http://jsbin.com/selureqota/2/edit?js,console

Если ты считаешь что что-то в яваскрипте работает не так, сделай пример и выложи, чтобы я мог его проверить.

>> Как к такой константе обращаться извне класса? Не лушче ли писать Ham.XX = yy ?

> Зачем обращаться к ней вне класса?

Потому что константы пишутся в первую очередь для «пользователей» класса, то есть тех кто твой класс будет использовать. Ты им как бы говоришь: вот вам класс, вот сюда надо передать тип гамбургера, а вот тут определены возможные типы, которые поддерживаются. Для этого константы и нужны (когда есть один из N вариантов).

Это общепринятый подход.

Есть константа Math.PI которая хранит догадайся сам какое число. Есть константы для типов узлов в DOM (дереве объектов которые представляют старницу в браузере) вроде Node.ELEMENT_NODE, Node.TEXT_NODE и другие. Например document.nodeType == Node.DOCUMENT_NODE, а document.body.nodeType == Node.ELEMENT_NODE. Есть константы для обозначения разных типов CSS-правил: https://developer.mozilla.org/en-US/docs/Web/API/CSSRule#Type_constants

Вот еще константы для названий клавиш: https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent.keyCode#Constants_for_keyCode_value

В то же время такие константы как BIG_PRICE не обязательно должны быть доступны снаружи, так как их все равно никак нельзя там использовать.

> И по-моему это не очень,

Это общепринятый способ. Он больше информации несет, то есть по «Haburger.SIZE_SMALL» понятно что речь о маленьком размере гамбургера, а не чем-то еще.

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

Тут ты не прав. Это все должно быть в коде, а не в документации, потому что:

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

Гораздо эффективнее писать «самодокументирующий» код, то есть такой, при взгляде на который все становится очевидно без документации. То есть код написан по общепринятым принципам, без изобретения велосипедов, имена функций, переменных объясняют их назначение, используются константы, и т.д. Конечно, документация все равно остается нужна, но можно разобраться и без нее.

То же касается и проверки параметров. Лучше пусть функция при неправильном значении выкинет исключение чем это будет описано только в документации.

>> У тебя нет проверок на то, что добавляется. Любую добавку можно добавить только один раз.

> Как уже обсуждалось, это спорный вопрос. Но добавлю, ок.

Ты можешь сделать свои ограничения, например не больше 5 добавок всего — лишь бы они были.

>>Можешь объяснить зачем try/catch? Что изменится если их убрать?

>Да ничего

Ну так тогда наверно незачем их и писать?

По твоему коду тут http://jsbin.com/wocugidiqu/1/edit?html,console — надо добавить ограничение на добавки и начинки. У меня в задаче начинку можно выбрать только одну из трех. И с константами разобраться. В остальном, норм.

Вообще, я смотрю, что идея про константы не нравится почти всем, кто решает эту задачу, и все пытаются там использовать просто строки. Почему, интересно.

>>418241

Она выводит лишнее. В примере тут она должна вывести только b и div, а выводит еще и i, который является частью комментария: http://ideone.com/qDgiWw

Надо улучшить код.

И да, HTML действительно не стоит разбирать регулярками, надо использовать DOM парсер. Но ради изучения регулярок можно отойти от этого правила.

>>418243

Код выглядит верно. Насчет ideone, это у них баг. Установи PHP себе, как описано тут: https://gist.github.com/codedokode/7054af4a03865c4cc863 и запускай в консоли (гайд по консоли: https://gist.github.com/codedokode/10539568 )

Если ты используешь IDE (Eclipse, Netbeans, PhpStorm) то можно указать путь к PHP в настройках и запускать программы одной кнопкой из IDE.

>>418266

Если тебе предъявляют ошибку с сарказмом, то это человек невопитанный или невежливый. Ничего плохого в том, чтобы сделать ошибку нет — любой может ошибиться. Я тоже делаю ошибки в коде (но бугурта не испытываю). Важно оперативно ее исправить, а в идеале наладить тестирование − чтобы программисты тестировали свои изменения, а тестеры - сам продукт перед релизом, чтобы ошибки отлавливались как можно раньше. Увы, автоматизированным тестированием во многих компаниях не занимаются.

>>418276

Но это же человек виноват что он невоспитанный или асоциальный, а не компания.

>>418284

Мне об ошибках пишут в скайп (либо я сам их вижу в логах ошибок и исправляю).

>>418300

У тебя долг в минус уходит по моему.

>>418304

Что-то идеон сломался. Вот, я перепостил решение (fork + run): http://ideone.com/RMxEy6

Считает неверно. Там не 60262 получается, а около 61270. Потому что анон, который только что выплатил 5000 не может заплатить 262 и должен подождать месяц, за который набегут проценты.

Проверь также программу на других суммах: кредит = 1000, выплата = 2030, кредит = 4000, выплата должна быть около 6123.

Мне кажется, лучше алгоритм сделать так:

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

> Хочу полную сумму узнать, а что-то не придумаю, как её задать.

Сделай переменную и в нее добавляй каждую выплату. По моему, ты это уже сделал.

>>418305

> Проблема заключается в том, что когда я вызываю этот метод из другого метода, то метод преобразования в латиницу начинает работать не правильно, а если я вызову этот метод где-то по среди кода, то все работает нормально.

Такого конечно быть не должно. Функция не должна зависеть от внешних условий. Давай-ка код посмотрим.

> class Database {

> public function newName($originalname) {

> $replace = Database::newName($name);

Так (используя имя класса и двоеточие) можно вызывать только статические методы (статический = не привязанный к конкретному объекту). Обычные методы вызываются только на объекте, то есть

> $db = new Database( );

> $replace = $db->newName($name);

PHP должен был вывести сообщение об ошибке. Если он не выводит - у тебя отключен вывод ошибок и надо его включить (я могу написать как).

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

У тебя есть еще одна ошибка. Ты исплоьзуешь регулярки без флага u для работы с русскими буквами. Они работать не будут, по причинам описанным тут: https://gist.github.com/codedokode/ff99e357e9860ea169b8 Из-за этого замена идет некорректно.

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

> $ruExplode = explode(' ' , $ru);

Я подумал, это плохая идея так как при такой записи легко допустить ошибку. Посмотри на свой код, и ответь, какой букве соответствует русская «П»? Чтобы это определить, тебе надо вручную отсчитывать пробелы и это долго и легко что-то перепутать. Такого быть не должно, код должен быть понятен, я думаю лучше записать пары в виде массива array("Ш" => "Sch"). Сам массив можно нагуглить.

> $originalname = preg_replace($ruExplode, $engExplode, $originalname);

Ой-ой-ой, как все сложно. А чем тебя не устраивает strtr которую мы изучали в разделе учебника про программу для зашифровки текста? Алсо ты забыл флаг u.

> $originalname = explode('.', $originalname);

> $extension = array(end($originalname));

> $originalname = array_diff($originalname, $extension);

Можно сделать проще, используй array_pop который вынимает последний элемент (расширение) из массива. А лучше, используй стандартную функцию pathinfo (UPD: в комментах пишут она из-за бага вырезает нелатинские символы из имени файла. Но из расширения не вырезает, я проверил).

Ну и конечно «определить расширение файла» — это отдельная задача и она должна быть вынесена в отдельную функцию.

>>418322

> https://github.com/sqghub/TestHub/blob/master/config/web.php#L31

Я тут обращу внимание что это какой-то секретный токен для подписи кук, и разумеется, он не должен быть известен всем жалеющим. Например, если ты разворачиваешь сайт, то для него надо этот токен заменить на секретный, и для этого должен быть предусмотрен способ. Я вижу, ты предусмотрел файл db.local.php, а вот чтобы токен (или любой другой параметр) переопредедить, не предусмотрел.

Делать с этим пока ничего не надо, но все равно, ты должен такие вещи понимать. Кроме токена, может потребоваться переопределять какие-то настройки вроде API ключей, имен серверов, чтобы они были разные на боевом и тестовом сервере например.

Еще, включи строгий режим в MySQL чтобы она показывала больше ошибок. Для этого надо после соединения сделать SET sql_mode = 'STRICT_ALL_TABLES': http://dev.mysql.com/doc/refman/5.1/en/sql-mode.html#sql-mode-setting

Насчет миграций, я вижу ты down не делал, хорошо, это обычно не требуется. Но в других фреймвокрках, вроде Ruby On Rails, миграции сделаны интереснее. Там ты не пишешь SQL код, а команды вроде таких:

create_table :products do |t|
t.string :name
t.text :description
t.timestamps null: false
end

За счет использования такого описания RoR умеет автоматически делать обратную операцию, например удалить таблицу или колонку. Ну это так, для расширения кругозора.

> $command = $this->db->createCommand("DROP TABLE IF EXISTS `questions`");

Этого в миграции быть не должно. Система миграций никогда не выполнит ее повторно.

Ну и кстати можно было сразу писать execute, не создавая переменную.

> `name` TEXT NOT NULL,

Хватит и VARCHAR, он до 65000 символов вроде поддерживает (но размер одной строки таблицы не может быть больше 65000 байт так что юникодных символов туда влезет меньше, инфа: http://dev.mysql.com/doc/refman/5.6/en/column-count-limit.html ).

> `author` VARCHAR(255),

Автора пока не надо делать, если пользователей нет. А если есть то нужно использовать внешние ключи тут (не знаком с ними? попроси ссылку и задачку).

> `text` TEXT,

Название плохое. Что такое «текст»? Предисловие? Содержимое? Послесловие? Правила?

> `time`

> `date`

Название плохое. Время чего? Обычно называют created, added, edited, deleted и тд. Если это ограничение то timeLimit.

Также, ты можешь (то есть требуется так делать) добавлять комментарии когда смысл не очень ясен:

rabbitPerTree INT (10) COMMENT 'Максимальное число зайцев, сидящих под одним деревом'

Эти комментарии сохраняются в БД и отображаются в разных программах для работы с ней.

> `questions` INT(11) UNSIGNED NOT NULL,

> `points` INT(11) UNSIGNED NOT NULL,

Не очень понятен смысл этих полей если их значение легко считается. Оптимизация? Тогда почему нет комментария? Я думаю, все же это ошибка, а не оптимизация. Ведь у каждого вопроса есть свое число баллов.

> Question.php

> QuestionMapper.php

Ты знаешь что в Yii есть базовый класс для моделей с ActiveRecord? Или ты по каким-то причинам не хочешь его использовать? http://www.yiiframework.com/doc-2.0/guide-db-active-record.html

ActiveRecord — это когда модель сама себя сохраняет и достает из базы. Вот моя паста про мапперы: https://gist.github.com/codedokode/c4cbc4d7dc8e45ea074a

Если нет веских причин против AR, стоит использовать ее. Если есть, стоит исплоьзовать Doctrine 2 а не изобретать велосипед и свои мапперы, ибо когда ты дойдешь до реализации связей, ты замучаешься (в файлообменнике связей вроде не было особо и проблемы тоже).

Отдельная тема — что делать с разными классами вопросов? Как хранить в БД наследующиеся сущности? Для этого есть 3 паттерна и их надо знать (это могут спросить на собеседовании, к слову):

http://martinfowler.com/eaaCatalog/singleTableInheritance.html
http://martinfowler.com/eaaCatalog/classTableInheritance.html
http://martinfowler.com/eaaCatalog/concreteTableInheritance.html
http://martinfowler.com/eaaCatalog/inheritanceMappers.html

(русский перевод если что есть на http://design-pattern.ru/ )

В Yii реально реализовать STI с полем type, один из анонов это ранее делал, насчет других подходов не знаю, но предупрежу что искать вопросы в 3 таблицах сложнее чем в одной.

> public function getAllTests()

Если у тебя есть сущность Test то логично возвращать массив этих Test, а не «глупый» массив с сырыми данными. Но если ты решишь использовать ActiveRecord. Также, по идее в эти Test должны быть вбиты вопросы. Если ты захочешь это написать, ты скоро увидишь что приходится писать кучу кода для маппинга между сущностями и таблицами. Гораздо умнее в такой ситуации не изобретать велосипед, а использовать библиотеку-маппер вроде Doctrine 2 (или activeRecord).

> https://github.com/sqghub/TestHub/tree/master/tests

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

> TestHub/views/test/createTest.php

Почему twig не используешь, а на PHP пишешь?

> {!{questionNumber}!}

Не очень удачный выбор, печатать тяжело. Я думаю, можно либо шаблон вынести в отдельных файл (чтобы не было конфликта с twig), либо использовать 2 знака процента например или [[ .

https://github.com/sqghub/TestHub/blob/master/views/test/createTest.php#L41 слишком длинная строка

Ну и я бы советовал использовать тут jQuery, если ты с ним знаком. Ну или сделать свои функции, например, var div = create('div', { id: 'xyz' }) вместо ручной возни с DOM методами.

Насчет формы, почитай что есть в Yii для этого: http://www.yiiframework.com/doc-2.0/guide-input-forms.html

>>418365

> Нужно любой из них добавить в локальное хранилище, перезагрузить страницу, и отобразить в DOM (пикрилейтед).

Не очень понимаю, зачем перезагружать страницу

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

Зря. У тебя пока что очень слабые знания JS, а задачи подняли бы его до хорошего. И ведь это видно по коду.

> for (i=0; i<articles.length; i++) {

> if (localStorage.getItem(articles[a]) == articles[a])

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

> <a href = "#"

Для этого надо использовать кнопку или спан. Ссылка должна куда-то вести, а # это не ссылка. Хотя много где так пишут, но это неправильно.

> //Удаляем пустые элементы из массива

Их проще было в него не добавлять

> deleteFromDOM(newDiv.id = "article_1")

Так писать неправильно. Зачем тут менять id у одного из элементов? Из-за замыканий он меняется не у того элемента о котором ты думаешь.

Плюс, ты зачем-то скопипастил большой кусок кода 4 раза. А если надо еще 4 товара добавить, ты еще 4 куска скопипастишь?

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

Для портфолио лучше найди готовый код или плагин.

>>418370

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

>>418377

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

>>418380

В дампе в формате SQL. Сделать дамп в консоли можно командой mysqldump, а в других программах обычно есть кнопка или опция.

>>418374

>Насчет миграций, я вижу ты down не делал, хорошо, это обычно не требуется. Но в других фреймвокрках, вроде Ruby On Rails, миграции сделаны интереснее. Там ты не пишешь SQL код, а команды вроде таких:

В yii2 тоже есть что-то подобное, но я не захотел этим пользоваться.

>Этого в миграции быть не должно. Система миграций никогда не выполнит ее повторно.

Это я пока что для себя оставил. Я вангую, что буду переделывать много чего в ближайшее время и в наглую дропать таблицу с миграциями.

>Автора пока не надо делать, если пользователей нет.

Это я оставил как просто имя автора.

>Не очень понятен смысл этих полей если их значение легко считается.

Баллы там вообще не нужны, я их хотел в Question добавить, но, видимо, засыпая уже совсем куда попало писал. С числом вопросов - все же считать?

>Ты знаешь что в Yii есть базовый класс для моделей с ActiveRecord? Или ты по каким-то причинам не хочешь его использовать?

Мне как-то мапперы больше понравились, чем AR. Но в doctrine 2 я лезть пока что побаиваюсь. Попробую смочь в ar.

>Отдельная тема — что делать с разными классами вопросов?

Ну я изначально планировал общие поля, которые в базовом классе есть, хранить полями таблицы, а специфичное для класса, в том числе и сам класс, хранить в json строке. Другой вариант, о котором я думал, это создать таблицу для общих данных и по таблице на каждый специфичный класс. В первой же хранить и имя класса, чтобы знать в какие таблицы ломиться. Я как-то забыл, что все придумано до меня. Почитаю ссылки чуть позже.

>Если у тебя есть сущность Test то логично возвращать массив этих Test

Я пока там обозначил методы, чтобы хоть как-то представлять, что там будет, над тем, что оно там возвращает, я еще не думал даже.

>Насчет формы, почитай что есть в Yii для этого

Я еще не читал особо, но думал, что формы там для работы с полученными данными, а не для вывода. Почитаю еще.

>Изменения в БД наверно надо будет сделать новой миграцией, либо исправить старую.

Пока что буду править старую.

>>418378

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

Я там просто пропускаю сначала через render от Yii, а затем результат отправляю в Twig. Не знаю, насколько это адекватное решение.