Мой первый сайт на PHP

Стоит ли так тримить?

PHP:

1. $sth->bindValue(':name', trim($text), PDO::PARAM_STR);

 

А смысл их биндить?

 

Сделал вот так, пойдёт ли?

PHP:

1. class DataBase {
2.     public static function connect ($host, $dbname, $user, $pass) {
3.         try {
4.             $db = new PDO('mysql:host='.$host.'; dbname='.$dbname.'; charset=utf8', $user, $pass);
5.         }
6.        
7.         catch (PDOException $e) {
8.             if(file_exists('error.txt')) {
9.                 if($countError = file_get_contents('error.txt', NULL, NULL, 19, 2)) {
10.                     $countErrorFile = fopen('countError.txt', 'w');
11.                     fwrite($countErrorFile, $countError);
12.                
13.                     $readCountError = file_get_contents('countError.txt', NULL, NULL, 0, 2);
14.            
15.                     $fp = fopen('error.txt', 'w');
16.                     fwrite($fp, 'Count bad requests ');
17.                     fwrite($fp, $readCountError+1);
18.                     fwrite($fp, ' . Error = ');
19.                     fwrite($fp, $e->getMessage());
20.                 }
21.                
22.                 else {
23.                     $countError = file_get_contents('error.txt', NULL, NULL, 19, 2);
24.                     $fp = fopen('error.txt', 'w');
25.                 }
26.             }
27.             else {
28.                 $fp = fopen('error.txt', 'w');
29.                 fwrite($fp, 'Count bad requests 1 ');
30.                 fwrite($fp, $e->getMessage());
31.                
32.             }
33.             die('Не удалось подключиться к базе данных');
34.         }
35.         return $db;
36.     }
37. }

В файле, будет написано Count bad requests 9 . Error = SQLSTATE[HY000] [1045] Access denied for user 'faq'@'localhost' (using password: YES)

Цифра 9 означает, что 9 пользователей, не смогли подключится к базе, только я сделал счётчик до 100, в принципе, мне этого хватает, чтобы принять меры.

 

с флагом a+, содержимое файла будет увеличиваться.
--- Добавлено ---
Я сократил код, теперь счётчик работает до бесконечности

PHP:

1. catch (PDOException $e) {
2.             if(!file_exists('countError.txt')) {
3.                 $countErrorFile = fopen('countError.txt', 'w');
4.                 fwrite($countErrorFile, 1);
5.                 $fp = fopen('error.txt', 'w');
6.                 fwrite($fp, 'Count bad requests ');
7.                 fwrite($fp, 1);
8.                 fwrite($fp, '. Error = ');
9.                 fwrite($fp, $e->getMessage());
10.             }
11.            
12.             else {
13.                 $readCountError = file_get_contents('countError.txt', NULL, NULL, 0);
14.                 $countErrorFile = fopen('countError.txt', 'w');
15.                 fwrite($countErrorFile, $readCountError + 1);
16.                
17.                 $fp = fopen('error.txt', 'w');
18.                 fwrite($fp, 'Count bad requests ');
19.                 fwrite($fp, $readCountError + 1);
20.                 fwrite($fp, '. Error = ');
21.                 fwrite($fp, $e->getMessage());
22.             }
23.        
24.             die('<p>Произошла ошибка</p>');
25.         }

 

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

 

10)Здесь и далее

PHP:

1. <a href="?interface-admin=1&del-id='.$admin['id'].'">Удалить</a>'

Подобные конструкции - это CSRF-уязвимость. Я могу прям вот тут в тело этого сообщения закинуть картинку, у которой src будет равен запросу определенному. Ты, просто открыв сообщение на этом форуме, просто вот увидев это сообщение, сам того не подозревая выполнишь то, что я прописал в ссылке. Браузер отправит запрос на сайт, и, если ты там залогинен с правами администратора, запрос отработает в штатном режиме, как если бы ты сам руками нажал на нужную кнопку. Как пример, с использованием цитируемого кода я могу твоими руками удалить чужую учетную запись.
[/QUOTE]

Я так и не понял, как эта картинка будет работать, если запрос работает по параметру del-id? А в картинке его не будет.

 

сейчас проверю

 

Создал отдельный файл в корне сайта, и в него поместил

HTML:

1. <img src="http://localhost/dashboard/netology/sait/?interface-admin=1&del-id=5">

Открыл и удаление не сработало

 

а не, сработало
--- Добавлено ---
и как тут быть?

 

Ну а как же быть, со статьями, там же такая же схема, например, пользователь хочет удалить статью, для интереса заходит в исходник и видит там ссылку del-question=2 и изменяет 2 на 50 и удаляется статья другого пользователя.