Module5 task1 #13

Glorferiel39 · 2025-01-13T17:38:21Z

No description provided.

sergeivl · 2025-01-13T18:24:49Z

functions/db.php

+* @param array $config Настройки подключения
+* @return mysqli|bool Возваращемый тип данных
+*/
+function dbConnect(array $config):mysqli|bool


Передать в нее $config['db']
и тогда внутри можно будет использовать ключи
$config['host'], $config['user'], ...

sergeivl · 2025-01-13T18:25:26Z

functions/db.php

+        echo "Подключения к базе данных установлено";
+    }
+
+mysqli_set_charset($con, "utf8");


4 пробела отступ

sergeivl · 2025-01-13T18:25:36Z

functions/db.php

+    }
+
+mysqli_set_charset($con, "utf8");
+return $con;


аналогично

sergeivl · 2025-01-13T18:25:59Z

functions/db.php

+return $con;
+}
+//  Запрос на получение новых лотов
+function getNewLotsFromDb(mysqli $con)


phpdoc и прописать тип возвращаемого значения в сигнатуре

sergeivl · 2025-01-13T18:26:24Z

functions/db.php

+
+    if (!$result) {
+        $error = mysqli_error($con);
+        print("SQL Error: $error");


sergeivl · 2025-01-13T18:27:09Z

functions/db.php

+}
+
+//  Запрос на получения всех категорий
+function getAllCategoriesFromDb(mysqli $con)


phpdoc и типа возвращаемого

sergeivl · 2025-01-13T18:27:40Z

functions/functions.php

@@ -0,0 +1,39 @@
+<?php
+/**


перенести описание к той функции, для которой оно написано

sergeivl · 2025-01-13T18:27:52Z

functions/functions.php

+
+
+/**
+ *Форматирует время лота


пробелы после *

sergeivl · 2025-01-13T18:29:00Z

helpers.php

-function is_date_valid(string $date) : bool {
-    $format_to_check = 'Y-m-d';
-    $dateTimeObj = date_create_from_format($format_to_check, $date);
+function isDateValid(string $date) : bool


создать файл
./funtions/validators.php
перенести туда функцию и подключить его через require

sergeivl · 2025-01-13T18:29:48Z

helpers.php

@@ -29,7 +30,8 @@ function is_date_valid(string $date) : bool {
 *
 * @return mysqli_stmt Подготовленное выражение
 */
-function db_get_prepare_stmt($link, $sql, $data = []) {
+function dbGetPrepareStmt(mysqli $link, $sql, $data = []):mysqli_stmt


пробел после :

перенести в fuctions/db.php

sergeivl · 2025-01-13T18:30:59Z

helpers.php

@@ -96,7 +98,7 @@ function db_get_prepare_stmt($link, $sql, $data = []) {
 *
 * @return string Рассчитанная форма множественнго числа
 */
-function get_noun_plural_form (int $number, string $one, string $two, string $many): string
+function getNounPluralForm (int $number, string $one, string $two, string $many): string


перенести в functions/template.php

sergeivl · 2025-01-13T18:31:09Z

helpers.php

@@ -126,7 +128,8 @@ function get_noun_plural_form (int $number, string $one, string $two, string $ma
 * @param array $data Ассоциативный массив с данными для шаблона
 * @return string Итоговый HTML
 */
-function include_template($name, array $data = []) {
+function includeTemplate($name, array $data = [])


перенести в functions/template.php

sergeivl · 2025-01-13T18:32:18Z

functions/functions.php

+ * @param int|float $price
+ * @return string
+ */
+function formatAmount(int|float $price): string


перенести в functions/template.php

sergeivl · 2025-01-13T18:32:42Z

functions/functions.php

+ *@param string $data
+ *@return array
+ */
+function remainingTime(string $date): array


можно тоже в перенести в functions/template.php

sergeivl

code review

Glorferiel39 added 6 commits December 23, 2024 21:26

Добавлена новая функция

10b9eb2

Добавлена схема по созданию db

e995a20

Добавлена схема для создания бд

f132aab

Внесены правки по созданию базы данных

1d4f563

Добавлены запросы к sql

4593227

Подкючение к бд и написание запросов

5c880df

sergeivl reviewed Jan 13, 2025

View reviewed changes

functions/db.php Outdated

}

mysqli_set_charset($con, "utf8");

return $con;

Copy link

sergeivl Jan 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

аналогично

sergeivl reviewed Jan 13, 2025

View reviewed changes

functions/db.php

if (!$result) {

$error = mysqli_error($con);

print("SQL Error: $error");

Copy link

sergeivl Jan 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

exit();

sergeivl reviewed Jan 13, 2025

View reviewed changes

functions/functions.php

/**

*Форматирует время лота

Copy link

sergeivl Jan 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

пробелы после *

sergeivl reviewed Jan 13, 2025

View reviewed changes

Glorferiel39 added 2 commits January 14, 2025 18:02

Добавлена схема по созданию бд

1b691d3

Внесены правки по формированию папок с функциями

15c35d6

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Module5 task1 #13

Module5 task1 #13

Glorferiel39 commented Jan 13, 2025

sergeivl Jan 13, 2025

sergeivl Jan 13, 2025

sergeivl Jan 13, 2025

sergeivl Jan 13, 2025

sergeivl Jan 13, 2025

sergeivl Jan 13, 2025

sergeivl Jan 13, 2025

sergeivl Jan 13, 2025

sergeivl Jan 13, 2025

sergeivl Jan 13, 2025

sergeivl Jan 13, 2025

sergeivl Jan 13, 2025

sergeivl Jan 13, 2025

sergeivl Jan 13, 2025

sergeivl Jan 13, 2025

sergeivl left a comment

Module5 task1 #13

Are you sure you want to change the base?

Module5 task1 #13

Conversation

Glorferiel39 commented Jan 13, 2025

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sergeivl left a comment

Choose a reason for hiding this comment