Skocz do zawartości

System wymiany linków stałych

whiner

Przez whiner
w Wymiana linków

Rekomendowane odpowiedzi

  • Odpowiedzi 48
  • Dodano
  • Ostatniej odpowiedzi
xnn Newbie

xnn

Opublikowano
Opublikowano

jesli mam byc sczery? caly system jest do kitu.. napisany przez jakiegos [aaa tam...] ktory chyba nie wie co to oszczednosc zasobow, miales kiedys jakas wieksza liczbe odwiedzajacych na stronie? kod - nieczytelny, rozwiazania/algorytmy ( jak zwal tak zwal ) wrecz archaiczne...

caly system latwy do ominiecia.. jak dla mnie przykro to mowic ale.. totalne DNO... zapisalbym sie.. ale gdy zobaczylem cyberka.php to zwatpilem... przeciez ja tam moge nabic jakakolwiek liczbe punktow...

nastepnym razem przemysl skrypt 10x a pozniej go napisz

Nie Polecam

Odnośnik do komentarza
Udostępnij na innych stronach
whiner Newbie

whiner

Opublikowano
  • Autor
Opublikowano

xnn programista z Ciebie chyba marny , gdybys nie wiedział lepiej gdy skrypt jest pobierany ze strony uzytkownika, niz serwera zewnetrznego - co moze wydawac sie przyczyną takiej a nie innej długosci kodu . Kod jest zoptymalizowany jak najbardziej sie da . Jesli mozesz nabić dlaczego tego nie uczynisz?

Pozdrawiam

88x31_en.png - semrush z trialem

Odnośnik do komentarza
Udostępnij na innych stronach
xnn Newbie

xnn

Opublikowano
Opublikowano
Fujara -

xnn programista z Ciebie chyba marny , gdybys nie wiedział lepiej gdy skrypt jest pobierany ze strony uzytkownika, niz serwera zewnetrznego - co moze wydawac sie przyczyną takiej a nie innej długosci kodu . Kod jest zoptymalizowany jak najbardziej sie da . Jesli mozesz nabić dlaczego tego nie uczynisz?

Pozdrawiam

wyzwanie? prosze bardzo.. ciekawe czy zmiany beda widoczne w nastepnej odslonie *skryptu *...

jeden.. zbedne komentarze. po co to jest? //////////////////////////// ? ta linie parser tez parsuje... 1-0 dla mnie

$iduser="twoj_numer_uzytkownika";
traktujemy wartosc cyfrowa jak string? nie no okej.. jak kto lubi
$wszystko = " <a href=\"$1\" target=\"new\" title=\"$2\">$2</a>";
' ' sa szybsze... parser nie musi pierw szukac zmiennych a dopiero pozniej *podkladac* w stringu tak jak to ma miejsce przy ""
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////

/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////

///////////////////////////////////////////                            Cyberka link                            //////////////////////////////////////////

///////////////////////////////////////////    Contents copyright © 2005 by Cyberka.pl. All rights reserved.  //////////////////////////////////////////

/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////

/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////

kolejne zbedne linie dla parsera

$ipa=$_SERVER['HTTP_USER_AGENT'];

$aa=$_SERVER['SERVER_NAME'] . $_SERVER['REQUEST_URI'];

$hash=md5($aa);

$plik="cyberkalink/". $hash .".cyb";

$plik2="cyberkalink/index.cyb";

$aa=urlencode($aa);

$ipa=urlencode($ipa);

najczesciej zmienne wykorzystywane raz.. nie wiem jak sie to ma do szybkosci... ale mniej lini, parser szybciej parsuje.. lepiej juz to zapisac tak:

$ipa=urlencode( $_SERVER['HTTP_USER_AGENT'] ); ( pozbywamy sie zbednej lini w kodzie.. )

$hash=md5($aa);

$plik="cyberkalink/". $hash .".cyb";

lepiej zapisac $plik = 'cyberkalink/'. md5 ( $aa ) .'.cyb';

$przeg=0;

$przegladarka[] = trim("InternetExplorer");

$przegladarka[] .= trim("IE");

$przegladarka[] .= trim("Mozilla");

$przegladarka[] .= trim("FireFox");

$przegladarka[] .= trim("Opera");

$przegladarka[] .= trim("Lynx");

$przegladarka[] .= trim("Konqueror");

$przegladarka[] .= trim("Netscape");

$przegladarka[] .= trim("Midas"); 

for($pe=0;$pe<count($przegladarka);$pe++)

{

if(eregi($przegladarka[$pe], $ipa))

{

$przeg++; 

}

}

tutaj mialem wiekszy ubaw niz z komentarzami...

1) skladnia..

'$przegladarka[] = trim("InternetExplorer");

$przegladarka[] .= trim("IE"); '

hahaha.. normalnie mam dosyc.. obcinasz z poczatkowej i koncwej spacji string ktory takich nie ma i ktory jest wpisany *na stale*... zbedne zuzycie zasobow systemowych i nadzwyczaj glupie rozwiazanie...

do tego...

2) Skladnia.. Manual mowi:

array_push() traktuje zmienną tablica  jako stos i wstawia przekazane parametry na koniec podanej tablicy. Długość parametru tablica zwiększa się o liczbę przekazanych wartości. Ma to taki sam efekt jak kod:

<?php

$tablica[] = $wartosc;

?>

widzisz gdzies $tablica[] .= $wartosc; bledne przypisanie, nie wiem jeszcze jak sie to ma do zasobow i czasu generacji skryptu.. czas bedzie sprawdzic ab :) wiem, wniki beda podobne.. ale jak sie mam czepiac to czepiam wszystkiego...

3) skladnia..

nie znasz rozwiazania $przegladarka = array ( 'InternetExplorer', 'IE', 'Mozilla', '...' );

4) eregi...

Manual Mowi:

Notatka:  preg_match(), which uses a Perl-compatible regular expression syntax, is often a faster alternative to ereg().

ale hahahahaha,... jaki jest sens uzywac wyrazen regularnych dla sprawdzenia czy *zwykly* ciag znakow istnieje.. w takim wypadku stosowac nalezy znacznie szybszy strpos()

5) petla

zle zoptymalizowana for($pe=0;$pe<count($przegladarka);$pe++) za kazdym wywolaniem petli wykonywana jest funkcja count($przegladarka)... powinno to wygladac

$c = count( $przegladarka );

for( $pe=0; $pe< $c; $pe++ )

jedna linijka dla parsera wiecej jednak korzysci sa nieporownywalnie wieksze

6) algorytm.. zauwazylem ze dalsza czesc kodu wykonywana jest tylko gdy $przeg == '0' ... wiec po jaka cholere po znalezionym ciagu petla nie jest killowana? na co komu ona pozniej? marnotrawienie zasobow

jedziemy dalej...

if ($f = fsockopen("$serwer", 80)) {

  fputs($f, "GET $strona HTTP/1.0\r\nHost: $serwer\r\n\r\n");

  while(!feof($f)) $calosc .= fgets($f, 1024);

  fclose($f);

}

1) " "

2) Nie wiem czemu wszyscy uparli sie na stosowanie fsockopen.. to takie wg mnie udawanie pro.. ja go jedynie uzywam symulujac przegladarke gdzie musze pierw pobrac token, zalogowac sie, pobrac informacje, poslac je dalej... Manual mowi:

If you just want to get the page content it's better to use fopen instead of fsockopen... I did a benchmark and it's faster... depends on how big is the content, getting it using fopen can be sometimes even 2x faster.

pozniej... preg_replace.. omowie to na tym przykladzie:

$calosc = preg_replace("/(\r?\n)/", "", $calosc);

$calosc=preg_replace('/(.*?)Bad Request(.*?)/', "", $calosc);

1.. 2 linie.. mozna to latwo w jednej zapisac.. a i dalej bedzie czytelnie

2. rozbrajajace jest to ze ludzie uzywaja znacznie wolniejszy preg_replacow do *zamieniania* zwyczajnych ciagow... tak jakby nie znali funkcji str_replace, znacznie szybszej oczywiscie.. tez takie udawanie pro...

pare lini zostalo pominietych.. powiedzmy ze nie znam lepszego odpowiednika lub je przeoczylem...

wiesz co? wielka radoche mialem udawadniajac Ci jak kiepski jestes i jak bledny w swoich przekonaniach... hehe trening przepony od 5... no.. to teraz tylko pl.pregierz poczytac i mozna wrocic do normalnych zajec... a.. tka przy okazji.. cxc.pl czasem nie prwoadze/nie prowadzili te same osoby co speed-server.net / svs.pl ? jesli tak to niezly ubaw mialem gdy wykupilem tam hosting;) chlopaki nie mogli cUrla dokompilowac ani domeny podpiac... jesli to nie Wy, zwracam honor..

Strach pomyslec jak wyglada aplikacja servera ;-)

Na koniec... Polecam poczytac jak wyglada globalny standard wygladu kodu ( czy jak ktos woli 'kodzenia' ) w php ) i do tego lekturke: https://forum.php.pl/index.php?showtopic=23810...

A Tarnow to takie piekne miasto...

Pozdrawiam,

czekajacy na riposte

Xnn

przepraszam.. nie moglem sie powstrzymac

Kod jest zoptymalizowany jak najbardziej sie da
Odnośnik do komentarza
Udostępnij na innych stronach
RamigoS Newbie

RamigoS

Opublikowano
Opublikowano

Jeziu :) ja się nie znam na kodzie i zaufałem temu systemowi, jak na razie nie widzę żadnej różnicy z systemem czy bez, ale chyba jestem zbyt krótko, czas pokaże, a nawet jak nie działa to przynajmniej mam narzędzie do sprawdzenia ile razy byłem zaindeksowany :)

A tak poważnie to jestem ciekaw odpowiedzi na post xnn, pierwszy raz jestem świadkiem "bitwy" programistów, może być ciekawie a i przy okazji może się czegoś nauczę :)

Pozdrawiam "fujare" i "xnn" i obu panom życzę powodzenia w dalszej pracy.

Wybierz się na Wirtualny Spacer

Zapraszam do mojego miasta Myślibórz

Odnośnik do komentarza
Udostępnij na innych stronach

Zarchiwizowany

Ten temat przebywa obecnie w archiwum. Dodawanie nowych odpowiedzi zostało zablokowane.

Tematy

  • Ostatnio przeglądający   0 użytkowników

    • Brak zarejestrowanych użytkowników przeglądających tę stronę.
×
×
  • Dodaj nową pozycję...

Powiadomienie o plikach cookie

Umieściliśmy na Twoim urządzeniu pliki cookie, aby pomóc Ci usprawnić przeglądanie strony. Możesz dostosować ustawienia plików cookie, w przeciwnym wypadku zakładamy, że wyrażasz na to zgodę. Warunki użytkowania Polityka prywatności