Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: PHP Praca (praktyki)
Forum PHP.pl > Inne > Hydepark
KsaR
24.09.2015, 18:57:48
To brzmi tak głupio że aż muszę się zapytać:
Slyszal ktos moze o praktykach ale "zdalnych"(przez neta)? tongue.gif
Interesują mnie wszelkie informacje o tym tongue.gif
memory
24.09.2015, 19:07:47
słyszał zdalne to takie przez internet, pracujesz na swoim komputrze, nie musisz przychodzić do biura. Coś jeszcze chcesz wiedzieć ? biggrin.gif

A tak poważnie praktyki zdalne to troche chory pomysł, nie wiem jak można czegoś konkretnego się nauczyć.
KsaR
24.09.2015, 19:24:54
Cytat(memory @ 24.09.2015, 20:07:47 ) *
słyszał zdalne to takie przez internet, pracujesz na swoim komputrze, nie musisz przychodzić do biura. Coś jeszcze chcesz wiedzieć ? biggrin.gif

A tak poważnie praktyki zdalne to troche chory pomysł, nie wiem jak można czegoś konkretnego się nauczyć.

Hm, cos typu.
Ktos zadaje mi zadanie, do zrobienia w dzien,
I podaje warunki.
A ja musze to wykonac, + wysylam kod a ta osoba mnie ocenia co mozna by lepiej itd biggrin.gif
--
Bo potrzebowalbym praktyk (darmowych) biggrin.gif
Tylko nie wiem jak szukac, gdzie szukac, i (czy) szukac biggrin.gif
mrc
24.09.2015, 19:39:46
Po prostu pisz. To są najlepsze praktyki. Z czasem zauważysz, że coś jest niewygodne, że można coś łatwiej albo po prostu inaczej. Możliwe, że nie szukasz praktyk, tylko mentora przez internet, który pomógłby Ci w dalszej nauce.
KsaR
24.09.2015, 19:44:29
Cytat(mrc @ 24.09.2015, 20:39:46 ) *
[...]Możliwe, że nie szukasz praktyk, tylko mentora przez internet, który pomógłby Ci w dalszej nauce.

No, tak tongue.gif
Pisac juz wiele napisalem.
Tak to od czasu do czasu pomagam na forach z nadzieja ze ktos sie doczepi ze cos zle, ale poki co tylko raz ktos sie mnie czepil o to ze sie czepiam, to lipa troche tongue.gif
Tak to nwm w czym sie dalej rozwijac.

Najwieksze problemy z calego php mam z,
Bitami. (Za malo praktyk) mimo ze cos tam juz probowalem,

Z FW. Zaden mi sie nie spodobal, a co wazniejsze nie zachecil do siebie.

Jak googluje to o optymalizacji i trickach ale nic nowego nie znajduje ;D.
Oczywiscie znam rozne narzedzia (ale) nie mam do ich dostepu, jak np. AB. Czy cos.

I kod ktory pisze jest dla mnie akurat, najlepszy. Pytanie jak dla innych itd. Stad tez te pytania.

Ps. Do komputera tez nie mam dostepu aaevil.gif
aniolekx
25.09.2015, 10:09:19
pokaz kod to ci powiemy wink.gif
hikaryu
25.09.2015, 10:18:47
Gdzie mieszkasz miszczu ? To może załatwię jakieś praktyki albo staż(płatny) jeżeli dałbyś radę dojeżdżać.
KsaR
28.09.2015, 03:55:32
Cytat(aniolekx @ 25.09.2015, 11:09:19 ) *
pokaz kod to ci powiemy wink.gif

Jestem w trakcie programowania małego bloga,
Ale nie wiem czy za mały by się nie okazał tongue.gif
Poki co jeszcze nawet nie ostylowany, nie przetestowany.
Ale jak zrobie to moge na GitHuba wrzucić.

Hikaryu:
Tak jak pisałem: zdalne, przez neta.
A dwa: "Grupa: Zarejestrowani
Postów: 243
Pomógł: 50
Dołączył: 15.07.2014
Skąd: Świecie nad wisłą"
tongue.gif

--
Dobra jednak bloga nie wrzuce na GH bo bym musial troche pozmieniac, ale np.
install.php bloga taki, jeszcze nie testowalem:
[PHP] pobierz, plaintext 
  1. <!DOCTYPE html>
  2. <html lang="pl">
  3. <head>
  4.  <meta charset="UTF-8"/>
  5.  <title>Instalacja bloga KsaR.</title>
  6.  <style>.scc{color:green}.err{color:red}</style>
  7. </head>
  8. <body>
  9. <?php
  10. function tabExs($pdo, $table)
  11. {
  12.  try
  13.  {
  14.   $r=$pdo->query("SELECT 1 FROM `$table` LIMIT 1");
  15.  }
  16.  catch (PDOException $e) // PDOE. Bo domyslnie zmienione w $pdo.
  17.  {
  18.   return 0;
  19.  }
  20.  return $r!==false;
  21. }
  22.  
  23. $install='';
  24.  
  25. if (file_exists('sql/blog_users.sql') && !tabExs($pdo, 'blog_posts'))
  26. {
  27.  $pdo->query(file_get_contents('sql/blog_users.sql'));
  28.  unlink('sql/blog_users.sql');
  29.  $install.='<span class="scc">Dodano tabelę, `blog_users`</span><br/>';
  30. }
  31.  
  32. if (file_exists('sql/blog_categories.sql') && !tabExs($pdo, 'blog_categories'))
  33. {
  34.  $pdo->query(file_get_contents('sql/blog_categories.sql'));
  35.  unlink('sql/blog_categories.sql');
  36.  $install.='<span class="scc">Dodano tabelę, `blog_categories`</span><br/>';
  37. }
  38.  
  39. if (file_exists('sql/blog_tags.sql') && !tabExs($pdo, 'blog_tags'))
  40. {
  41.  $pdo->query(file_get_contents('sql/blog_tags.sql'));
  42.  unlink('sql/blog_tags.sql');
  43.  $install.='<span class="scc">Dodano tabelę, `blog_tags`</span><br/>';
  44. }
  45.  
  46. if (file_exists('sql/blog_posts.sql') && !tabExs($pdo, 'blog_posts'))
  47. {
  48.  $pdo->query(file_get_contents('sql/blog_posts.sql'));
  49.  unlink('sql/blog_posts.sql');
  50.  $install.='<span class="scc">Dodano tabelę, `blog_posts`</span><br/>';
  51. }
  52.  
  53. if (isset($install[0]))
  54. {
  55.  echo $install,'Odśwież teraz stronę.';
  56. }
  57. else
  58. {
  59.  $form='<form method="post" action="#"><input type="text" name="adm" required/><input type="submit" value="Dodaj admina"/></form>';
  60.  if (isset($_POST['adm']))
  61.  {
  62.   $pdo->prepare('UPDATE `blog_users` SET `staff`=1 WHERE `email`=?')->execute([$_POST['adm']]);
  63.   if ($pdo->rowCount())
  64.   {
  65.    echo '<br/><span class="scc">Dodano admina, aktualny plik został skasowany ze względów bespoieczeństwa..</span>';
  66.    unlink(__FILE__);
  67.   }
  68.   else
  69.   {
  70.    echo $form;
  71.   }
  72.  }
  73.  else
  74.  {
  75.   echo $form;
  76.  }
  77. }
  78. ?>
  79. </body>
  80. </html>
[PHP] pobierz, plaintext

Taki byle jaki bo na 2 requesty bedzie (dodanie bazy, dodanie admina - zarejestrowanego).
+ jeszcze trzeba by PDO zaincludować.

Hmmm, lub tez taki inny (ktory koloruje literki, oraz podaje bbcode) robiony na szybko bo tylko do jednorazowego użytku.
Takze spacje, nowe linie itd tez bbcoduje biggrin.gif, oraz nie sprawdza jaki poprzedni kolor... ale serio do 1x uzytku..
[PHP] pobierz, plaintext 
  1. <!DOCTYPE html>
  2. <html>
  3. <head>
  4.  <meta charset="UTF-8"/>
  5.  <title>KsaR - color mixer.</title>
  6.  <style scoped>body,html{background:#3C3D3C;color:white}</style>
  7. </head>
  8. <body>
  9. <form action="#mix" method="POST"/>
  10. <input type="text" name="t" required/><br/>
  11. <select name="c">
  12.  <option value="1">Niebieskie</option>
  13.  <option value="2">Zielone</option>
  14.  <option value="3">Wszystkie razem</option>
  15. </select>
  16. <input type="submit" value="Mixuj !"/>
  17. </form>
  18. <?php
  19. if (isset($_POST['t'],$_POST['c']))
  20. {
  21.  $new='';
  22.  $newBB='';
  23.  switch ($_POST['c'])
  24.  {
  25.   case '1': $colors=['62A2E3','3280CF','196CBF','197CE0','1381F0','1F6CB8','4286C9']; break;
  26.   case '2': $colors=['35B53B','18AD20','04BA0D','1ACC23','2ADE33','09D913','27C22F']; break;
  27.   default: $colors=['35B53B','18AD20','04BA0D','1ACC23','2ADE33','09D913','27C22F','62A2E3','3280CF','196CBF','197CE0','1381F0','1F6CB8','4286C9']; # wszystkie
  28.  }
  29.  shuffle($colors);
  30.  $colorsCt=count($colors)-1;
  31.  preg_match_all('#(.)#u',$_POST['t'],$str);
  32.  $str=$str[0];
  33.  $i=-1;
  34.  do
  35.  {
  36.   ++$i;
  37.   $curr=$colors[mt_rand(0,$colorsCt)];
  38.   $new.='<span style="color:#'.$curr.'">'.$str[$i].'</span>';
  39.   $newBB.=''.$str[$i].'';
  40.  }
  41.  while (isset($str[$i]));
  42.  echo '<br/><br/><span style="color:gray">Rezultat:</span> ',$new,'<br/><span style="color:gray">BBcode:</span> ',$newBB;
  43. }
  44. ?>
  45. </body>
  46. </html>
[PHP] pobierz, plaintext

Tu mysle ze lepiej by bylo na checkboxach ale tak jw. Pisalem.

Więc jak cos więcej to lepiej zapytać, bo ja zazwyczaj robie krotkie.. I nie ma wtedy co oceniac.
A jak dlugie to zwyczajnie nie daje bo mi zalezy na tym ze duzo czasu poswiecilem tongue.gif
Pyton_000
28.09.2015, 07:16:07
Przepraszam że to powiem, ale to jest straszne. To nawet proceduralnie nie jest a przynajmniej wygląda beznadziejnie.
KsaR
28.09.2015, 07:29:59
Cytat(Pyton_000 @ 28.09.2015, 08:16:07 ) *
Przepraszam że to powiem, ale to jest straszne. To nawet proceduralnie nie jest a przynajmniej wygląda beznadziejnie.

Hee, ja wlasnie tego nie widze, co innego gdyby inaczej formatowal kod jak co niektorzy,
A co bys zmienil? Zeby nie bylo strasznie?
--
Instalacja to robilem z 2-5min.
Mix kolorow do 20min, na tablecie.
Czyli na szybko po mojemu tongue.gif
--
Aha, ps.
Normalnie to tak nie mam HTML-a wmieszanego.
Do podstron.
Tu tak jest zeby przegladarka wrazie czego nie zwariowala tongue.gif
kapslokk
28.09.2015, 07:42:16
To po co właściwie wrzucasz kod zrobiony w 5 minut?
Co do samego kodu:
Tak na dobrą sprawę to bym zaczął od jakiegoś rozdzielenia htmla od php... 


[PHP] pobierz, plaintext 
  1. if (file_exists('sql/blog_users.sql') && !tabExs($pdo, 'blog_posts'))
  2. {
  3.  $pdo->query(file_get_contents('sql/blog_users.sql'));
  4.  unlink('sql/blog_users.sql');
  5.  $install.='<span class="scc">Dodano tabelę, `blog_users`</span><br/>';
  6. }
  7.  
  8. if (file_exists('sql/blog_categories.sql') && !tabExs($pdo, 'blog_categories'))
  9. {
  10.  $pdo->query(file_get_contents('sql/blog_categories.sql'));
  11.  unlink('sql/blog_categories.sql');
  12.  $install.='<span class="scc">Dodano tabelę, `blog_categories`</span><br/>';
  13. }
  14.  
  15. if (file_exists('sql/blog_tags.sql') && !tabExs($pdo, 'blog_tags'))
  16. {
  17.  $pdo->query(file_get_contents('sql/blog_tags.sql'));
  18.  unlink('sql/blog_tags.sql');
  19.  $install.='<span class="scc">Dodano tabelę, `blog_tags`</span><br/>';
  20. }
  21.  
  22. if (file_exists('sql/blog_posts.sql') && !tabExs($pdo, 'blog_posts'))
  23. {
  24.  $pdo->query(file_get_contents('sql/blog_posts.sql'));
  25.  unlink('sql/blog_posts.sql');
  26.  $install.='<span class="scc">Dodano tabelę, `blog_posts`</span><br/>';
  27. }
[PHP] pobierz, plaintext


A teraz pomyśl, że chcesz zmienić klasę w tym spanie. I robisz to w x miejscach... A wystarczyło zrobić funkcję smile.gif

Ogólnie to wrzuć coś, co nie było robione na szybko, bo tym to aż wstyd się chwalić biggrin.gif
KsaR
28.09.2015, 07:57:32
Kapslokk, co do HTML-a to zedytowalem chwile wczesniej post zanim odpisales tongue.gif,
Ee ja nie mam takich problemow. ;D w edytorze moge zmieniac dowolny ciag na inny.
I sie niczym tu nie chwale, sami prosiliscie kod to jest ogolny zarys tongue.gif.
[PHP] pobierz, plaintext 
  1. # (...)
  2. $install='';
  3.  
  4. function tableExists($patch='', $table='')
  5. { global $pdo; # ach to global biggrin.gif, nie chce mi sie na szybko teraz przerabiac aaevil.gif
  6.  if (file_exists($patch) && !tabExs($pdo, $table))
  7.  {
  8.   $pdo->query(file_get_contents($patch));
  9.   unlink($patch);
  10.   return '<span class="scc">Dodano tabelę, `'.$table.'`</span><br/>';
  11.  }
  12.  return '';
  13. }
  14.  
  15. $install.=tableExists('sql/blog_posts.sql', 'blog_posts');
  16. $install.=tableExists('sql/blog_categories.sql', 'blog_categories');
  17. $install.=tableExists('sql/blog_tags.sql', 'blog_tags');
  18. $install.=tableExists('sql/blog_users.sql', 'blog_users');
  19.  
  20. if (isset($install[0]))
  21. # (...)
[PHP] pobierz, plaintext

Czy o cos takiego chodzilo? tongue.gif
kapslokk
28.09.2015, 08:00:57
Tak.. o coś takiego chodziło. Ja wiem, że możesz zmienić dowolny ciąg na inny, ale jeżeli oprócz zmiany ciągu znaków będziesz musiał przerobić coś więcej, to sie okaże, że musisz robić to ręcznie. I jeśli Ci dojdzie x tabel, to będziesz musiał się nieźle narobić ;P A poza tym kod staje się czytelniejszy. Za miesiąc przyjdziesz i stwierdzisz, że tak naprawdę nie wiesz co robisz w tym miejscu, a jeśli sobie sensownie nazwiesz funkcję to łatwo się domyślisz smile.gif

Staraj się nazywać funkcje tak jak działają. Nazwa tableExists sugeruje, że ona sprawdza czy tabela istnieje. Nic nie mówi o tym, że zostanie utworzona jeśli nie istnieje smile.gif
KsaR
28.09.2015, 08:04:14
No tak, tongue.gif nie pomyslalem o DRY w tamtym wypadku. < dziękuje za uswiadomienie, zobacze inne kody czy tak nie odwalilem biggrin.gif.

--

Mialem problem zeby nazwac teraz na szybko funkcje biggrin.gif
Pyton_000
28.09.2015, 08:06:20
lepiej, ale uważasz że nazwa funkcji jest adekwatna do działania?
KsaR
28.09.2015, 08:08:21
Cytat(Pyton_000 @ 28.09.2015, 09:06:20 ) *
lepiej, ale uważasz że nazwa funkcji jest adekwatna do działania?

Nie w pelni, ale na chwile obecna nie wiem jak nazwac (jak wyzej).

tblNonExsts (tableNonExists)
kapslokk
28.09.2015, 08:09:30 
[PHP] pobierz, plaintext 
  1.  $i=-1;
  2.  do
  3.  {
  4.   ++$i;
  5.   $curr=$colors[mt_rand(0,$colorsCt)];
  6.   $new.='<span style="color:#'.$curr.'">'.$str[$i].'</span>';
  7.   $newBB.=''.$str[$i].'';
  8.  }
  9.  while (isset($str[$i]));
[PHP] pobierz, plaintext


To jest fajny fragment. biggrin.gif
Wiesz, że najpierw wykonają się instrujcje z pętli, a później sprawdzisz czy w ogóle $str[$i] istnieje ?biggrin.gif

@edit

Zamiast tblNonExsts zrób tak jak napisałeś w nawiasie. Po cholere to skracać?
KsaR
28.09.2015, 08:12:42
Cytat(kapslokk @ 28.09.2015, 09:09:30 ) *
[PHP] pobierz, plaintext 
  1.  $i=-1;
  2.  do
  3.  {
  4.   ++$i;
  5.   $curr=$colors[mt_rand(0,$colorsCt)];
  6.   $new.='<span style="color:#'.$curr.'">'.$str[$i].'</span>';
  7.   $newBB.=''.$str[$i].'';
  8.  }
  9.  while (isset($str[$i]));
[PHP] pobierz, plaintext


To jest fajny fragment. biggrin.gif
Wiesz, że najpierw wykonają się instrujcje z pętli, a później sprawdzisz czy w ogóle $str[$i] istnieje ?biggrin.gif

Naprostuj, co tu złego? Jako ze jest w ifie:
[PHP] pobierz, plaintext 
  1. if (isset($_POST['t'],$_POST['c']))
  2. {
[PHP] pobierz, plaintext


questionmark.giftongue.gif
Czyli po tym ifie zakladam ze jednak jest cos, w tym wypadku 1 znak:pp
--
@edit - ok
Pyton_000
28.09.2015, 08:16:44
Cytat
# ach to global biggrin.gif, nie chce mi sie na szybko teraz przerabiac aaevil.gif

Wiesz, źle to świadczy o Tobie.

A zamiast tableExists -> executeSchema
kapslokk
28.09.2015, 08:18:14
Tak na dobrą sprawę to nie wiem o co chodzi w ogóle w tej pętli do...while(). Tzn do czego ona służy, ale tak czy inaczej, czemu nie foreach ? A źle jest to, że jak Ci sie skonczy tablica $str to zrobi jeszcze jedno kółko.
KsaR
28.09.2015, 08:18:32
Cytat(Pyton_000 @ 28.09.2015, 09:16:44 ) *
Wiesz, źle to świadczy o Tobie.

A zamiast tableExists -> executeSchema

Ten global dodalem przy edycji dopiero, bo sie pytalem czy o "cos takiego" chodzi, i nagle zauwazylem ze $pdo nie lokalne, to nie chcialem na szybko przerabiac co zaznaczylem w komentarzu smile.gif
--
kapslokk, to chyba nie ogarnelem dzialania do...while hm?

to nie dziala tak ze:
do{} sie zawsze wykona z ta roznica ze od drugiego obiegu sprawdza warunek (tzn po wykonaniu?)

A robi to ze podmienia wpisany tekst na losowe kolorowanie liter.
Na poczatku bylo cos typu $str[$x].. Że by sie nie dalo nawet foreach. (Foreach po stringu), ale niestety problem z UNICODE i musialem na szybko cos wykombinowac ponownie. Zeby polskie znaki mi obsluzylo tongue.gif i tak kolejne kombinacje.
kapslokk
28.09.2015, 08:27:00
Do while działa tak, że sprawdza warunek po wykonaniu pętli. W Twoim przypadku ustawiasz $i = -1; Na początku masz inkrementację i to powoduje błąd. $i na starcie powinno być równe 0, na końcu pętli inkrementacja i warunek działał by poprawnie. A tak, to warunek jest spełniony, następuje inkrementacja i lecimy po elemencie tablicy który nie istnieje.
Odpal sobie takie 2 kody:
1. 
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. $arr = array(
  4.     0=>0,
  5.     1=>1,
  6.     2=>2,
  7.     3=>3,
  8. );
  9. $i = -1;
  10. do{
  11.     $i++;
  12.     var_dump($arr[$i]);
  13. }while(isset($arr[$i]));
[PHP] pobierz, plaintext


2.
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. $arr = array(
  4.     0=>0,
  5.     1=>1,
  6.     2=>2,
  7.     3=>3,
  8. );
  9. $i = 0;
  10. do{
  11.     var_dump($arr[$i]);
  12.     $i++;
  13. }while(isset($arr[$i]));
[PHP] pobierz, plaintext
KsaR
28.09.2015, 09:46:37
Tyle rzeczy sie nauczyc w chwile tongue.gif, dzięki.
Na szczescie rzadko uzywam do-while.
Najczesciej while/foreach.
Z petlami mialem najwiekszy problem gdy zaczynalem nauke "dawno" temu tongue.gif

Co do bardziej przemyslanego:
[PHP] pobierz, plaintext 
  1. <?php
  2. ob_start('ob_gzhandler')?:ob_start();
  3.  
  4. define('IS_DEBUG', true); // DEBUG MODE.
  5. define('ok', true); // for included files.
  6. define('RP', $_SERVER['DOCUMENT_ROOT']); // root path.
  7.  
  8. if (IS_DEBUG)
  9. {
  10.  ini_set('display_errors',true);
  11.  ini_set('display_startup_errors',true);
  12.  ini_set('track_errors',true);
  13. }
  14. else ini_set('display_errors',false);
  15. error_reporting(-1);
  16.  
  17. session_set_cookie_params(86400,'/',$_SERVER['HTTP_HOST'],false,true); #-1:ssl; -0: httpOnly.
  18. session_save_path(__DIR__.'/sess/'); # change dir of sess; defend for shared hosting.
  19. session_start();
  20.  
  21. # functions include #
  22. include RP.'/php/functions.php';
  23.  
  24. register_shutdown_function('footer');
  25.  
  26. if (!isset($_GET['p'])) $_GET['p']='home';
  27. switch ($_GET['p'])
  28. {
  29.  case 'signin': $page=['title'=>'Logowanie','name'=>'signin']; break;
  30.  case 'signup': $page=['title'=>'Rejestracja','name'=>'signup']; break;
  31.  case 'addpost': $page=['title'=>'Dodaj post','name'=>'addpost']; break;
  32.  case 'addcategory': $page=['title'=>'Dodaj kategorię','name'=>'addcategory']; break;
  33.  case 'signout': $_SESSION=[]; session_destroy(); $page=['title'=>'Cytaty online','name'=>'home']; break;
  34.  default: $page=['title'=>'Blog - KsaR','name'=>'home'];
  35. }
  36.  
  37. $server=sprintf('//%s/~%s/',$_SERVER['SERVER_ADDR'],explode('/',$_SERVER['SCRIPT_FILENAME'])[2]);
  38.  
  39. echo '<!DOCTYPE html SYSTEM "about:legacy-compat">
  40. <html lang="pl" dir="ltr">
  41. <head>
  42.  <meta charset="UTF-8"/>
  43.  <title>',$page['title'],'.</title>
  44.  <meta name="viewport" content="width=device-width,maximum-scale=2.5,initial-scale=1.0"/>
  45.  <link rel="stylesheet" href="',$server,'css/style.min.css?v=0"/>
  46.  <meta name="MobileOptimized" content="true"/>
  47.  <meta name="HandheldFriendly" content="true"/>
  48.  <meta name="author" content="KsaR"/>
  49.  <meta name="keywords" content="ksar,blog,php,polski"/>
  50. </head>
  51. <body>
  52. <header>';
  53.  
  54.  
  55. // @author: KsaR.
  56. ## session security ##
  57. // before stolen session
  58. if (isset($_SESSION['owner']) && $_SESSION['owner']!==$_SERVER['HTTP_USER_AGENT'])
  59. {
  60.  exit('<div style="color:red"><b>Wykryto i udaremniono próbę przejęcia sesji użytkownika.</b></div>');
  61. }
  62. else
  63. {
  64.  $_SESSION['owner']=$_SERVER['HTTP_USER_AGENT'];
  65. }
  66. ##-session security-##
  67.  
  68. $logged=isset($_SESSION['uid']);
  69.  
  70. try
  71. {
  72.  $config=
  73.  [
  74.   'db_host'=>'127.0.0.1',
  75.   'db_name'=>'baza',
  76.   'db_user'=>'uzytkownik',
  77.   'db_pass'=>'haslo'
  78.  ];
  79.  $pdo=new PDO(sprintf('mysql:host=%s;dbname=%s;charset=utf8;encoding=utf8',$config['db_host'],$config['db_name']),$config['db_user'],$config['db_pass'],[PDO::ATTR_ERRMODE=>PDO::ERRMODE_EXCEPTION,PDO::ATTR_PERSISTENT=>true,PDO::MYSQL_ATTR_INIT_COMMAND=>'SET time_zone=\''.date('P').'\'']);
  80. }
  81. catch(PDOException $e)
  82. {
  83.  exit(sprintf('<div class="error">Wystąpił problem podczas połączenia z bazą danych.<br/>%s</div>',$e->getMessage()));
  84. }
  85. if ($logged)
  86. {
  87.  $user=$pdo->prepare('SELECT `staff` FROM `users` WHERE `userId`=? LIMIT 1');
  88.  $user->execute([$_SESSION['userId']]);
  89.  $user=$user->fetch(PDO::FETCH_ASSOC);
  90. }
  91.  
  92. echo '<nav id="menu">
  93. <ul>
  94.  <li><a href="/">Strona główna</a></li>',($logged?($user['staff']?(
  95. '<li><a href="/addpost/">Dodaj post</a></li>
  96.  <li><a href="/addcategory/">Dodaj kategorie</a></li>':'')).
  97. '<li><a href="/signout/">Wyloguj</a></li>'):(
  98. '<li><a href="/signin/">Logowanie</a></li>
  99.  <li><a href="/signup/">Rejestracja</a></li>'),
  100. '</ul>
  101. </nav>
  102. </header>
  103. <main>';
  104.  
  105. ob_start('ob_clear');
  106.  include sprintf('%s/pages_include/%s.php',RP,$page['name']);
  107. ob_end_flush();
  108. ob_end_flush();
[PHP] pobierz, plaintext

Ps.
Ten HTML w PHP tez mi sie nie podoba (to co w echo), jednak cos wolniej ladowala mi sie strona gdy byl html poza php, jakby go kompresja nie dosiegala ale nie mam jak to potwierdzic.
Pyton_000
28.09.2015, 09:52:11 
[PHP] pobierz, plaintext 
  1. exit(sprintf('<div class="error">Wystąpił problem podczas połączenia z bazą danych.<br/>%s</div>',$e->getMessage()));
[PHP] pobierz, plaintext

Wrzuć sobie jakieś błędne dane i zobacz co Ci wywali na ekran...
kapslokk
28.09.2015, 09:54:52
Kumpel z pracy Cię wyśmiał za to:
[PHP] pobierz, plaintext 
  1. define('ok', true); // for included files.
[PHP] pobierz, plaintext


Poza tym coś takiego: 
[PHP] pobierz, plaintext 
  1. $pdo=new PDO(sprintf('mysql:host=%s;dbname=%s;charset=utf8;encoding=utf8',$config['db_host'],$config['db_name']),$config['db_user'],$config['db_pass'],[PDO::ATTR_ERRMODE=>PDO::ERRMODE_EXCEPTION,PDO::ATTR_PERSISTENT=>true,PDO::MYSQL_ATTR_INIT_COMMAND=>'SET time_zone=\''.date('P').'\'']);
[PHP] pobierz, plaintext

To imo tylko zaciemnianie kodu.


[PHP] pobierz, plaintext 
  1. if (!isset($_GET['p'])) $_GET['p']='home';
[PHP] pobierz, plaintext

Serio musisz to przypisywać prosto do GET'a ?
KsaR
28.09.2015, 10:11:18
Co do define chodzi o 'ok'?
Co do zaciemniania, wedlug mnie wlasnie czytelniej tongue.gif bo i tak w tablicy dane.
Co do przypisania do get,
Bylo na poczatku tak:

if (...)
{
tu switch
}
else $page=[...];
Jednak zrobilem jak wyzej bo bylo podwujnie w else i default to samo tongue.gif

--
Pyton_000 - nie mam jak teraz sprawdzic, hostingodawca wylaczyl cos dostep do error_loga nie wiem czemu aaevil.gif musze cos pokombinowac, bo blank page -,- < nie testowalem po zmianach, tak samo jak widac tam "cytaty online" biggrin.gif heh
kapslokk
28.09.2015, 10:19:51
Tak, chodzi o ok... Po co to?biggrin.gif

Jeżeli uważasz, że ta linijka jest czytelna, no to cóż biggrin.gif musisz z tym żyć haha.gif
Co do geta, to juz lepiej sobie przepisz to do innej zmiennej i tam coś zmieniaj, a nie bezposrednio w gecie.
KsaR
28.09.2015, 10:22:49
Cytat(kapslokk @ 28.09.2015, 11:19:51 ) *
Tak, chodzi o ok... Po co to?biggrin.gif

W includowanych plikach robie:

if (!defined('ok')) exit;

Zeby ktos nie wszedl w plik bezposrednio (includowany).
kapslokk
28.09.2015, 10:24:31
Cytat(KsaR @ 28.09.2015, 11:22:49 ) *
W includowanych plikach robie:

if (!defined('ok')) exit;

Zeby ktos nie wszedl w plik bezposrednio (includowany).


To żeś ładnie nazwał stałą. Od razu wiedziałem o co chodzi.
KsaR
28.09.2015, 11:18:28 
[PHP] pobierz, plaintext 
  1. <?php
  2. ob_start('ob_gzhandler')?:ob_start();
  3.  
  4. define('IS_DEBUG', true); // DEBUG MODE.
  5. define('included', true); // for included files.
  6. define('RP', $_SERVER['DOCUMENT_ROOT']); // root patch.
  7.  
  8. if (IS_DEBUG)
  9. {
  10.  ini_set('log_errors',true);
  11.  ini_set('error_log','errors.log.txt');
  12.  ini_set('display_errors',true);
  13.  ini_set('display_startup_errors',true);
  14.  ini_set('track_errors',true);
  15. }
  16. else ini_set('display_errors',false);
  17. error_reporting(-1);
  18.  
  19. session_set_cookie_params(86400,'/',$_SERVER['HTTP_HOST'],false,true); #-1:ssl; -0: httpOnly.
  20. session_save_path(__DIR__.'/sess/'); # change dir of sess; defend for shared hosting.
  21. session_start();
  22.  
  23. # functions include #
  24. include RP.'/php/functions.php';
  25.  
  26. register_shutdown_function('footer');
  27.  
  28. $page=['title'=>'Blog - KsaR','name'=>'home'];
  29. switch ((isset($_GET['p']) ? $_GET['p'] : ''))
  30. {
  31.  case 'signin': $page=['title'=>'Logowanie','name'=>'signin']; break;
  32.  case 'signup': $page=['title'=>'Rejestracja','name'=>'signup']; break;
  33.  case 'addpost': $page=['title'=>'Dodaj post','name'=>'addpost']; break;
  34.  case 'addcategory': $page=['title'=>'Dodaj kategorię','name'=>'addcategory']; break;
  35.  case 'signout': $_SESSION=[]; session_destroy(); break;
  36. }
  37.  
  38. $server=sprintf('//%s/~%s/',$_SERVER['SERVER_ADDR'],explode('/',$_SERVER['SCRIPT_FILENAME'])[2]);
  39. ?>
  40. <!DOCTYPE html SYSTEM "about:legacy-compat">
  41. <html lang="pl" dir="ltr">
  42. <head>
  43.  <meta charset="UTF-8"/>
  44.  <title><?=$page['title']?></title>
  45.  <meta name="viewport" content="width=device-width,maximum-scale=2.5,initial-scale=1.0"/>
  46.  <link rel="stylesheet" href="<?=$server?>'css/style.min.css?v=0"/>
  47.  <meta name="MobileOptimized" content="true"/>
  48.  <meta name="HandheldFriendly" content="true"/>
  49.  <meta name="author" content="KsaR"/>
  50.  <meta name="keywords" content="ksar,blog,php,polski"/>
  51. </head>
  52. <body>
  53. <?php
  54. // @author: KsaR.
  55. ## session security ##
  56. // before stolen session
  57. if (isset($_SESSION['owner']) && $_SESSION['owner']!==$_SERVER['HTTP_USER_AGENT'])
  58. {
  59.  exit('<div class="err"><b>Wykryto i udaremniono próbę przejęcia sesji użytkownika.</b></div>');
  60. }
  61. else
  62. {
  63.  $_SESSION['owner']=$_SERVER['HTTP_USER_AGENT'];
  64. }
  65. ##-session security-##
  66.  
  67. $logged=isset($_SESSION['uid']);
  68.  
  69. try
  70. {
  71.  $config=
  72.  [
  73.   'db_host'=>'127.0.0.1',
  74.   'db_name'=>'baza',
  75.   'db_user'=>'uzytkownik',
  76.   'db_pass'=>'haslo'
  77.  ];
  78.  $pdo=new PDO('mysql:host='.$config['db_host'].';dbname='.$config['db_name'].';charset=utf8;encoding=utf8',$config['db_user'],$config['db_pass'],
  79.  [
  80.   PDO::ATTR_ERRMODE=>PDO::ERRMODE_EXCEPTION,
  81.   PDO::ATTR_PERSISTENT=>true,
  82.   PDO::MYSQL_ATTR_INIT_COMMAND=>'SET time_zone=\''.date('P').'\''
  83.  ]);
  84. }
  85. catch(PDOException $e)
  86. {
  87.  exit(sprintf('<div class="err">Wystąpił problem podczas połączenia z bazą danych.<br/>%s</div>',$e->getMessage()));
  88. }
  89.  
  90. if ($logged)
  91. {
  92.  $user=$pdo->prepare('SELECT `staff` FROM `users` WHERE `userId`=? LIMIT 1');
  93.  $user->execute([$_SESSION['userId']]);
  94.  $user=$user->fetch(PDO::FETCH_ASSOC);
  95. }
  96. ?>
  97. <header>
  98. <nav id="menu">
  99. <ul>
  100.  <li><a href="/">Strona główna</a></li>
  101.  <?php
  102.  if ($logged)
  103.  {
  104.   if ($user['staff']):
  105.  ?>
  106.   <li><a href="/addpost/">Dodaj post</a></li>
  107.   <li><a href="/addcategory/">Dodaj kategorie</a></li>
  108.  <?php
  109.   endif;
  110.  ?>
  111.   <li><a href="/signout/">Wyloguj</a></li>
  112.  <?php
  113.  }
  114.  ?>
  115.  <li><a href="/signin/">Logowanie</a></li>
  116.  <li><a href="/signup/">Rejestracja</a></li>
  117. </ul>
  118. </nav>
  119. </header>
  120. <main>
  121. <?php
  122. ob_start('ob_clear');
  123.  include sprintf('%s/pages_include/%s.php',RP,$page['name']);
  124. ob_end_flush();
  125. ob_end_flush();
[PHP] pobierz, plaintext

Ok zmiany i juz nie blank page,

Pyton_000:
Cytat
Wystąpił problem podczas połączenia z bazą danych.<br/>SQLSTATE[HY000] [1045] Access denied for user 'uzytkownik'@'localhost' (using password: YES)


Juz chyba wszystko poprawilem, jeszcze coś źle? tongue.gif
kapslokk
28.09.2015, 11:29:05 
[PHP] pobierz, plaintext 
  1. $page=['title'=>'Blog - KsaR','name'=>'home'];
  2. switch ((isset($_GET['p']) ? $_GET['p'] : ''))
  3. {
  4.  case 'signin': $page=['title'=>'Logowanie','name'=>'signin']; break;
  5.  case 'signup': $page=['title'=>'Rejestracja','name'=>'signup']; break;
  6.  case 'addpost': $page=['title'=>'Dodaj post','name'=>'addpost']; break;
  7.  case 'addcategory': $page=['title'=>'Dodaj kategorię','name'=>'addcategory']; break;
  8.  case 'signout': $_SESSION=[]; session_destroy(); break;
  9. }
[PHP] pobierz, plaintext


ten switch jest głupi, bo i tak będzie sprawdzał warunki nawet gdy nie będzie $_GET['p'];
Wg. mnie lepiej by było:
[PHP] pobierz, plaintext 
  1. if(isset($_GET['p']){
  2. switch ($_GET['p']))
  3. {
  4.  case 'signin': $page=['title'=>'Logowanie','name'=>'signin']; break;
  5.  case 'signup': $page=['title'=>'Rejestracja','name'=>'signup']; break;
  6.  case 'addpost': $page=['title'=>'Dodaj post','name'=>'addpost']; break;
  7.  case 'addcategory': $page=['title'=>'Dodaj kategorię','name'=>'addcategory']; break;
  8.  case 'signout': $_SESSION=[]; session_destroy(); break;
  9. }
  10. }
  11. if(!isset($page){
  12. $page=['title'=>'Blog - KsaR','name'=>'home'];
  13. }
[PHP] pobierz, plaintext

Ale to już ktoś mądrzejszy by się musiał wypowiedzieć.

A co do samego kodu, jeżeli chcesz rozwijać ten 'blog' to przydały by się jakieś obiekty i system szablonów.
Pyton_000
28.09.2015, 11:31:14
Wiedz że jeśli serwer się wykrzaczy lub inny timeout to Exception z POD wywali Ci PLAIN password. Dlatego nie rób czegoś takiego jak $e->getMessage().
KsaR
28.09.2015, 11:33:13 
[PHP] pobierz, plaintext 
  1. $page=['title'=>'Blog - KsaR','name'=>'home'];
  2. if (isset($_GET['p']))
  3. {
  4.  switch ($_GET['p'])
  5.  {
  6.   case 'signin': $page=['title'=>'Logowanie','name'=>'signin']; break;
  7.   case 'signup': $page=['title'=>'Rejestracja','name'=>'signup']; break;
  8.   case 'addpost': $page=['title'=>'Dodaj post','name'=>'addpost']; break;
  9.   case 'addcategory': $page=['title'=>'Dodaj kategorię','name'=>'addcategory']; break;
  10.   case 'signout': $_SESSION=[]; session_destroy(); break;
  11.  }
  12. }
[PHP] pobierz, plaintext


Cytat(Pyton_000 @ 28.09.2015, 12:31:14 ) *
Wiedz że jeśli serwer się wykrzaczy lub inny timeout to Exception z POD wywali Ci PLAIN password. Dlatego nie rób czegoś takiego jak $e->getMessage().

A co zamiast robić? Zeby blad otrzymac?
Pyton_000
28.09.2015, 11:34:22
Przecież masz tam tylko połączenie z bazą, więc zwykły komunikat Ci wystarczy.
KsaR
28.09.2015, 11:35:57
Cytat(Pyton_000 @ 28.09.2015, 12:34:22 ) *
Przecież masz tam tylko połączenie z bazą, więc zwykły komunikat Ci wystarczy.

A no, tak biggrin.gif

[PHP] pobierz, plaintext 
  1. catch(PDOException $e)
  2. {
  3.  exit('<div class="err">Wystąpił problem podczas połączenia z bazą danych.</div>');
  4. }
[PHP] pobierz, plaintext


Poprawione.
com
28.09.2015, 13:15:44
pytałeś o praktyki zdalne, słyszałem IBM takie robił jakiś czas temu tongue.gif
Ale lepiej zrobić coś w tradycyjny sposób, bo tam nie wiele sie nauczysz.
memory
28.09.2015, 13:16:22
widzisz KsaR dlatego praktyki powinieneś odbyć stacjonarnie
Wazniak96
28.09.2015, 14:53:05
Ajj KsaR.. Z tego co pamiętam cały czas zaciemniałeś kod przez optymalizację i przez to co Ci się podoba.

Niestety ten zawód nie działa tak jak i cały świat. Musisz się układać pod standardy, a nie własne upodobania by pracować w tym zawodzie.

Dzisiaj właśnie było na wykładach wspomniane o tym słowo. Oczywiście zazwyczaj stylistyka kodu ma pierwszeństwo przed optymalizacją wink.gif
Pyton_000
28.09.2015, 15:16:15
Cytat(Wazniak96 @ 28.09.2015, 15:53:05 ) *
Oczywiście zazwyczaj stylistyka kodu ma pierwszeństwo przed optymalizacją wink.gif

Powodzenia w zawodzie...
mrc
28.09.2015, 15:32:30
Co ma stylistyka kodu do optymalizacji? Klamrowania, wcięcia, nawiasowanie - co to ma wspólnego z optymalizacją?

Jak ktoś nauczy się ładnie pisać na początku, to zawsze będzie pisał ładny kod, przed i po optymalizacji.
Wazniak96
28.09.2015, 18:36:11
Chodziło mi o to, że np. mieszanie kodu PHP z HTML. wink.gif
pyro
28.09.2015, 19:39:22
Cytat(Wazniak96 @ 28.09.2015, 19:36:11 ) *
Chodziło mi o to, że np. mieszanie kodu PHP z HTML. wink.gif


To nie jest stylistyka kodu... Mylisz pojęcia
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.