PHP.SU

Программирование на PHP, MySQL и другие веб-технологии

PHP.SU Портал

Главная

Помощь

Поиск

Поиск Яндекс Вакансии

Пользователи

Здравствуйте, Гость

( Вход · Регистрация · Правила форума )

Забыли пароль?

Нужен совет бывалых

Форумы портала PHP.SU » PHP » Программирование на PHP (Модераторы: valenok, OrmaJever, Саныч)

Страниц (1): [1]

Описание: как правильно написать код

Поиск в теме | Версия для печати

haligali	Отправлено: 31 Марта, 2011 - 14:17:44
Новичок Покинул форум Сообщений всего: 11 Дата рег-ции: Март 2011 Помог: 0 раз(а)	Всем форумчанам доброго времени суток. Я в пыхе новичок, учусь сам, потому прошу не пинать сильно. В общем попытался тут устроиться на работу стажером. Мне было выслано тестовое задание. Суть в следующем: Цитата: ЗАДАЧА: Требуется разработать веб-сайт со следующей функциональностью: 1 На главной странице должна быть форма для ввода данных. Форма должна иметь следующие элементы: "Укажите размерность массива" select список должен состоять из элементов 3,4,5,6,7,8,9,10 Отсортировать по убыванию checkbox OK submit 2 Пользователь заполняет форму и нажимает кнопку "OK". Пользователь переходит на страницу результатов, где представлено: Матрица1 - матрица случайных чисел от 0 до 100. Матрица имеет размерность NxN, где N размерность массива, которую пользователь ввел на главной странице. Матрица2 - отсортированная по порядку матрица1. Матрица сортируется по убыванию, если пользователь поставил галку на главной странице в элементе "Отсортировать по убыванию". Каждая следующая строка матрицы, должна размещаться от левого края дальше предыдущей строки на 30 пикселей. Для решения этой задачи необходимо использовать: PHP, HTML, CSS В общем с задачей я справился. Протестировал код на трех системах WIN XP, WIN 7, Ubuntu 10.10. В качестве сервера Apache 2.2. php двух версий 5.2 и 5.3. Браузеры: Firefox 4, Opera 11, IE 6, IE 9, Chrome. Все везде работает отлично. НО! У работодателя мой код работать не захотел. Во-первых, как мне написали, на странице результатов у них браузер выдавал код php в окно браузера, но не весь, а начиная с первого спец-символа "\n". Убрал спец символы, написали что результат тот же! Я себе всю голову сломал, в итоге залил свои сэмплы хостеру на 1Gb.ru. У хостера все работает.Вот ссылка Во-вторых, пожаловались на не читаемость кода. В связи с этим хотелось бы узнать мнения бывалых. В чем могла быть ошбка? Почему у работодателя не работает мой код? Ну и по поводу не читабельного кода. Как его сделать более читабельным, не нарушая при этом никаких требований ? Заранее благодарен за любой ответ. Очень сильно нужна Ваша помощь дабы не совершать подобных ошибок впредь. Собственно вот сам код: index.html (Отобразить) CODE (html): скопировать код в буфер обмена <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd"> <html> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <title>Матрица случайных чисел</title> </head> <body> <h1>Выберите размерность матрицы</h1> <form action="action.php" method="post" name="raz"> <p>Размерность матрицы: <select name="slt" size="1"> <option value="3">3</option> <option value="4">4</option> <option value="5">5</option> <option value="6">6</option> <option value="7">7</option> <option value="8">8</option> <option value="9">9</option> <option value="10">10</option> </select> </p> <p> <input name="sort" type="checkbox" value="0">Отсортировать по убыванию</p> <input type="submit" name="sub" value="OK"> </form> </body> </html> action.php (Отобразить) CODE (htmlphp): скопировать код в буфер обмена <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd"> <html> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <title>Вывод результата</title> <link rel="stylesheet" href="style.css" type="text/css"> <!--[if IE 6]> <style type="text/css"> .stc{text-align:center; margin-left:0px; display:inline; border:1px solid black; width:40px;} </style> <![endif]--> </head> <body> <?php $items = $_POST["slt"] * $_POST["slt"]; // Переменная для общего колличества элементов массива $count = 1; // Счетчик итераций // Заполняем массив случайными числами от 1 до 100 for ($i = 1; $i <= $items; $i++) { $arr[$i] = rand(0, 100); } // Функция для отрисовки матрицы function myMatrix($a) { $items = $_POST["slt"] * $_POST["slt"]; $count1 = 1; foreach ($a as $item) { if (array_search($count1, range(1, $items, $_POST["slt"])) !== FALSE) echo "<div id='pos" . round($count1 / $_POST["slt"]) . "'>"; echo "<div class='stc'>$item</div>"; if (array_search($count1, range($_POST["slt"], $items, $_POST["slt"])) !== FALSE) echo "</div>"; $count1++; } } // Отрисовка матрицы в цикле foreach ($arr as $item) { if (array_search($count, range(1, $items, $_POST["slt"])) !== FALSE) echo "<div>"; echo "<div class='stc'>$item</div>"; if (array_search($count, range($_POST["slt"], $items, $_POST["slt"])) !== FALSE) echo "</div>"; $count++; } echo "<br>"; // Проверка, было ли отмечено поле "Отсортировать по убыванию." if (isset($_POST["sort"])) { rsort($arr); myMatrix($arr); } else { sort($arr); myMatrix($arr); } ?> </body> </html> Ну и собственно style.css (Отобразить) CODE (htmlphp): скопировать код в буфер обмена @charset "utf-8"; body{margin:0px;padding:0px;} div{margin:0px;padding:0px;} #pos1{margin-left:30px;} #pos2{margin-left:60px;} #pos3{margin-left:90px;} #pos4{margin-left:120px;} #pos5{margin-left:150px;} #pos6{margin-left:180px;} #pos7{margin-left:210px;} #pos8{margin-left:240px;} #pos9{margin-left:270px;} .stc{text-align:center; margin-left:0px; display:inline-block; border:1px solid black; width:40px;}

grefon	Отправлено: 31 Марта, 2011 - 14:37:50
Частый посетитель Покинул форум Сообщений всего: 823 Дата рег-ции: Нояб. 2010 Помог: 32 раз(а)	Как по мне так у работодателя руки растут из того места, от куда он ср...т (это о не работающем коде), а глаза находятся в том месте где он сс...т (это о не читаемости кода). Если бы мне стажер принес такое выполненное задание, то у меня к нему претензий по этим двум поводам точно бы не было. (Отредактировано автором: 31 Марта, 2011 - 14:40:40) ----- ____________________________________________________________________ Ну как то так, наверное. http://grefon[dot]com

haligali	Отправлено: 31 Марта, 2011 - 14:55:32
Новичок Покинул форум Сообщений всего: 11 Дата рег-ции: Март 2011 Помог: 0 раз(а)	grefon Спасибо. Мне этот работодатель, кстати, судя по переписке. тоже показался не адекватным.

AdMeen	Отправлено: 31 Марта, 2011 - 16:15:51
Посетитель Покинул форум Сообщений всего: 370 Дата рег-ции: Сент. 2009 Откуда: Воронеж Помог: 1 раз(а)	haligali пишет: grefon Спасибо. Мне этот работодатель, кстати, судя по переписке. тоже показался не адекватным. по переписке?

Zuldek	Отправлено: 31 Марта, 2011 - 17:42:38
Постоянный участник Покинул форум Сообщений всего: 2122 Дата рег-ции: Июнь 2010 Помог: 50 раз(а)	0. Взять предоплату. 1. Спрашивать версию оси сервера, мускула, пхп у работодателя. 2. Ставить себе такую же связку. 3. Проверить скрипты на ней. 3. Спрашивать что работодатель пониманет под читабельным кодом, предоставив 2 возможных варианта ответа: стандарты Zend и стандарты Pear(http://pear.php.net/manual/ru/standards.php ). 4. Поправить под стандарты ваш код. Судя по всему там не так много. С такими работодателями только так. И ещё после сдачи потом брать с них за работу почасовую оплату, чтобы знали меру требовательности. (Отредактировано автором: 31 Марта, 2011 - 17:44:47)

haligali	Отправлено: 31 Марта, 2011 - 18:19:37
Новичок Покинул форум Сообщений всего: 11 Дата рег-ции: Март 2011 Помог: 0 раз(а)	Спасибо всем. Zuldek Действительно. Впредь так и буду поступать.А ссылку в закладки.

JustUserR	Отправлено: 31 Марта, 2011 - 21:08:40
Активный участник Покинул форум Сообщений всего: 8715 Дата рег-ции: Июнь 2009 Помог: 17 раз(а)	haligali пишет: Убрал спец символы, написали что результат тот же! Я себе всю голову сломал, в итоге залил свои сэмплы хостеру на 1Gb.ru В качестве возможной причины осуществления некорректного функционирования предполагаемого PHP-приложения на целевого web-сервере, допустимо рассмотрение используемых локальных региональных параметров, на основании которых производится проведение процесса переведения специфицированного определения PHP-приложения в повышенном базовом элементом уровне трактования, для целевого ассоцированного исполняемого потока ----- Сделать можно все что угодно - нужно только старание, терпение и хороший поисковик Безлимитный web-хостинг от 15 рублей за 40 МБ дискового пространства - http://ihost[dot]oks71[dot]ru/

haligali	Отправлено: 01 Апреля, 2011 - 12:45:26
Новичок Покинул форум Сообщений всего: 11 Дата рег-ции: Март 2011 Помог: 0 раз(а)	JustUserR Жесть !!! А можно более понятней? Для нуба так сказать. (Отредактировано автором: 01 Апреля, 2011 - 12:48:01)

emslt	Отправлено: 01 Апреля, 2011 - 13:29:03
Новичок Покинул форум Сообщений всего: 27 Дата рег-ции: Февр. 2011 Помог: 0 раз(а)	haligali пишет: JustUserR Жесть !!! А можно более понятней? Для нуба так сказать. Да он просто постебался.

haligali	Отправлено: 01 Апреля, 2011 - 13:32:59
Новичок Покинул форум Сообщений всего: 11 Дата рег-ции: Март 2011 Помог: 0 раз(а)	emslt Почему-то сразу так и подумал. Да-а, все таки оригинальный юмор у программистов

emslt	Отправлено: 01 Апреля, 2011 - 13:38:08
Новичок Покинул форум Сообщений всего: 27 Дата рег-ции: Февр. 2011 Помог: 0 раз(а)	haligali пишет: emslt Почему-то сразу так и подумал. Да-а, все таки оригинальный юмор у программистов Ну ничего, мы тоже так когда то будем шутить. Надо задать вопрос работодателю какой версии у него PHP. Может у него PHP 4 или даже PHP 6. PHP 6 глючный пока что. Если чувствуешь что работник - немного ПСИХ - даже и не пытайся к нему устроится. Не только не попрактикуешся, но и нервы испортишь.

Мелкий	Отправлено: 01 Апреля, 2011 - 14:16:11
Активный участник Покинул форум Сообщений всего: 11926 Дата рег-ции: Июль 2009 Откуда: Россия, Санкт-Петербург Помог: 618 раз(а)	emslt пишет: Да он просто постебался. Постебался - это если бы заговорил языком, понятным неподготовленному читателю Так-то постоянно изъясняется. ----- PostgreSQL DBA

haligali	Отправлено: 01 Апреля, 2011 - 14:27:12
Новичок Покинул форум Сообщений всего: 11 Дата рег-ции: Март 2011 Помог: 0 раз(а)	emslt Цитата: Может у него PHP 4 или даже PHP 6. PHP 6 глючный пока что. Насколько я знаю php 6 еще не вышел и возможно не выйдет никогда. (выйдет сразу версия 7) Потому как версия 5.3 это и есть php 6 только без полной поддержки юникода. Именно из за юникода разработчики столкнулись с неразрешимыми противоречиями. На данный момент над разработкой новой версии работают совсем другие люди, а прошлая команда разбежалась. Потому скорее всего весь движок php будет переписан с нуля и версия скорее всего будет 7. Хотя может оставят и цифру 6. Только вот когда это будет неизвестно. Ну а представленный мной код был протестирован на двух версиях 5.2 и 5.3 и в принципе должен корректно работать и в четвертой версии. (хотя надо доку покурить, может какая-то из использованных функций появилась только в пятой ветке)

emslt	Отправлено: 01 Апреля, 2011 - 15:03:06
Новичок Покинул форум Сообщений всего: 27 Дата рег-ции: Февр. 2011 Помог: 0 раз(а)	Ну это полнейшая фигня есть работодатель будет просить писать код на PHP 4. Если все работает на PHP 5.2 - 5.3. Значит все гуд. Но надо ещё покопаться в php.ini авось что то у дома включено чего нет у работодателя.

Поиск в теме | Версия для печати

Страниц (1): [1]

Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)

« Программирование на PHP »

Все гости форума могут просматривать этот раздел.
Только зарегистрированные пользователи могут создавать новые темы в этом разделе.
Только зарегистрированные пользователи могут отвечать на сообщения в этом разделе.