Скрипт создания скриншотов

Скрипт создания скриншотов - Visavi.net https://visavi.net/ RSS - Visavi.net https://visavi.net/assets/img/images/logo_small.png RSS - Visavi.net https://visavi.net/ [email protected] (admin) [email protected] (admin) Sat, 11 Jan 2025 05:02:08 +0300 Теперь скрипт делает скрины не только главной страницы сайта. https://visavi.net/topics/35798/609523 Скрипт создания скриншотов Александр Tue, 18 Dec 2012 17:28:05 +0400 Сообщения https://visavi.net/topics/35798/609523 20. ...RESIDENTOV..., утер? ок. https://visavi.net/topics/35798/609496 Скрипт создания скриншотов Дмитрий Tue, 18 Dec 2012 15:01:19 +0400 Сообщения https://visavi.net/topics/35798/609496 19. dima.london, бери парня на работу. Утер <img src="https://visavi.net/uploads/stickers/smile.gif" alt="smile"> https://visavi.net/topics/35798/609417 Скрипт создания скриншотов Макс Mon, 17 Dec 2012 23:22:21 +0400 Сообщения https://visavi.net/topics/35798/609417 ок[b][/b] https://visavi.net/topics/35798/609415 Скрипт создания скриншотов Дмитрий Mon, 17 Dec 2012 23:09:13 +0400 Сообщения https://visavi.net/topics/35798/609415 А причём тут СИ? Это разные языки программирования и сравнивать их глупо. В PHP, например, нет такой жёсткой типизации как в Си. P.S. код выше сработает не аналогично. Например: <pre class="prettyprint">$s = FALSE; if(isset($s)) { echo 'Условие выполнилось'; }</pre> Условие выполнится. А если сделать так: <pre class="prettyprint">$s = FALSE; if($s) { echo 'Условие выполнилось'; }</pre> Условие никогда не выполнится, а так как функция screen() возвращает либо FALSE, либо текст (что соответствует TRUE) это разные записи и результат их выполнения будет отличен. https://visavi.net/topics/35798/609412 Скрипт создания скриншотов Александр Mon, 17 Dec 2012 22:46:23 +0400 Сообщения https://visavi.net/topics/35798/609412 if ($s = screen($url)) { сработает аналогичному if(isset($s)) { и книжки тут не при чем. Читайте мануалы по СИ. https://visavi.net/topics/35798/609411 Скрипт создания скриншотов Дмитрий Mon, 17 Dec 2012 22:35:09 +0400 Сообщения https://visavi.net/topics/35798/609411 Рассмотрев весь твой код я привёл его к нормальному виду: <pre class="prettyprint">function screen($url, $browser="ie9") { $urlstr = 'http://ipinfo.info/netrenderer/index.php?url='.urlencode($url).'&browser='.$browser; if(($content = file_get_contents($urlstr))) { if(preg_match('/image\.php\?imgid=(.+?)"/i', $content, $h) != 0) { return $h[1]; } } return false; }</pre> P.S. если уж делаешь присвоение в условии то его необходимо выделять в отдельные круглые скобки. Это я тебе про этот кусок кода: <pre class="prettyprint">if ($s = screen($url)) { echo '<img src="http://renderer.geotek.de/image.php?imgid=' . $s . '" alt="">'; }</pre> Условие должно быть таким: <pre class="prettyprint">if(($s = screen($url))) {</pre> P.P.S. можешь сжечь книжки по которым тебя учили, "старший". Вместо того чтоб языком чесать не по делу пошёл бы и подучил PHP. Надеюсь я снизел твою самооценку и спустил тебя с небес <img src="https://visavi.net/uploads/stickers/smile.gif" alt="smile"> P.P.P.S. никого не хотел обидеть, просто надоело уже... всем peace, пишите скрипты, для вас 10 мин строчил пост Okula ;) https://visavi.net/topics/35798/609402 Скрипт создания скриншотов Александр Mon, 17 Dec 2012 21:54:48 +0400 Сообщения https://visavi.net/topics/35798/609402 9. dima.london, мда, тебе как кодеру должно быть стыдно за свои слова <img src="https://visavi.net/uploads/stickers/smile.gif" alt="smile"> Начну с самого начала: <pre class="prettyprint">function screen($url="", $browser="ie9") { if (!empty($url)) {</pre> Первое: Делать проверку на пустоту переменной внутри тела функции, делая этот аргумент функции по умолчанию пустым, - это быдлокод. Все проверки должны быть осуществлены до того как данные попали в функцию, дабы исключить пустой результат. Ты же умышленно разрешаешь использование пустого результата в переменной $url, тем самым провацируя скрипт на проверку. Второе: для чего был задан второй аргумент функции $browser, если он нигде не используется? Я так понимаю он должен вставляться в адрес URL browser='.$browser Сочтём это за твою невнимательность. Идём далее: <pre class="prettyprint">$content=file_get_contents('http://ipinfo.info/netrenderer/index.php?url=' .$url .'&browser=ie7');</pre> Вот ещё кусок быдлокода <img src="https://visavi.net/uploads/stickers/smile.gif" alt="smile"> Вставляя адрес из переменной $url ты забываешь экранировать данные функцией urlencode(). Привет XSS уязвимости <img src="https://visavi.net/uploads/stickers/hello.gif" alt="hello"> : Третье: смотрим код дальше: <pre class="prettyprint">preg_match_all('/image.php\?imgid=(.*?)"/i', $content, $h);</pre> Символ "." (точка) является спецсимволом регулятного выражения (совпадает с любым символом, кроме символа перевода строки) который тоже необходимо эранировать. Использование функции preg_match_all(), для получения единичного совпадения по шаблону регулярного выражения, в данном месте не уместно. Достаточно функции preg_match() Ну а проверка ниже меня просто убила: Твой код: <pre class="prettyprint">if (isset($h) && isset($h[1][0])) {</pre> Если регулярка сработала и совпадение нашлось то все её параметры будут заполнены. Достаточно поместить функцию preg_match() в условие и она исключит все ненужные проверки. https://visavi.net/topics/35798/609401 Скрипт создания скриншотов Александр Mon, 17 Dec 2012 21:54:17 +0400 Сообщения https://visavi.net/topics/35798/609401 <a href="http://www.s-shot.ru/" target="_blank" rel="nofollow">http://www.s-shot.ru/</a> https://visavi.net/topics/35798/609380 Скрипт создания скриншотов TRAIL Mon, 17 Dec 2012 20:22:34 +0400 Сообщения https://visavi.net/topics/35798/609380 12. Sonikflame, ок, покупай то,что в 7 посте выложено бесплатно )) https://visavi.net/topics/35798/609374 Скрипт создания скриншотов Дмитрий Mon, 17 Dec 2012 19:51:09 +0400 Сообщения https://visavi.net/topics/35798/609374 за 10р возьму. https://visavi.net/topics/35798/609373 Скрипт создания скриншотов Sonikflame Mon, 17 Dec 2012 19:47:26 +0400 Сообщения https://visavi.net/topics/35798/609373 Хороший ребята,из-за 50 рублей такую бессмысленную тему устроили https://visavi.net/topics/35798/609372 Скрипт создания скриншотов Geor Mon, 17 Dec 2012 19:46:54 +0400 Сообщения https://visavi.net/topics/35798/609372 на венике за 30рублей продавали <img src="https://visavi.net/uploads/stickers/D.gif" alt="D"> https://visavi.net/topics/35798/609371 Скрипт создания скриншотов BenZzOReZ Mon, 17 Dec 2012 19:44:47 +0400 Сообщения https://visavi.net/topics/35798/609371 // а вдруг пользователь не указал урл if (!empty($url)) { // а вдруг контент не отобрался if ($content) { // а вдруг массив $h не выбрался, а если выбралсяь, то в нем значения [1][0] нет if (isset($h) && isset($h[1][0])) { Слушай, чему тебя старшие учат, а не тому, чему Попов в своих уроках проповедует. Я бы этому Попову компилятор бы подчистил, чтобы не учил людей гoвнокодить. https://visavi.net/topics/35798/609370 Скрипт создания скриншотов Дмитрий Mon, 17 Dec 2012 19:44:01 +0400 Сообщения https://visavi.net/topics/35798/609370 7. dima.london, значит у тебя кокашка код <img src="https://visavi.net/uploads/stickers/smile.gif" alt="smile"> Во-первых ошибка в регулярке, во-вторых лишних проверок дофига. Мой скрипт работает с API стороннего сервиса, да это так. Код для получении скриншота - одна строка. Всё остальное - это подготовка данных для стороннего API. Скриншоты сохраняются на том же сервере что и установлен скрипт. В отличии от твоего сервиса мой умеет делать полноразмерные скриншоты. P.S. цена и так смешная, менять её я не буду, если для кого-то это много - копите деньги со школьных завтраков <img src="https://visavi.net/uploads/stickers/smile.gif" alt="smile"> https://visavi.net/topics/35798/609350 Скрипт создания скриншотов Александр Mon, 17 Dec 2012 18:01:09 +0400 Сообщения https://visavi.net/topics/35798/609350