Правила форума Гаранты форума
Размещение рекламы AMX-X компилятор

Здравствуйте, гость Вход | Регистрация

Наши новости:

14-дек
24-апр
10-апр
11-апр

> Полезные ссылки

2 страниц V   1 2

[Услуга] PHP + MySQL

Статус пользователя Puwok
сообщение 17.12.2018, 22:00
Сообщение #21


Стаж: 14 лет

Сообщений: 100
Благодарностей: 33
Полезность: < 0

offtop
UP thank_you2.gif


Уже ничем не владею, совсем обленился...
Перейти в начало страницы         Просмотр профиля    Отправить личное сообщение
   + Цитировать сообщение
Статус пользователя Puwok
сообщение 26.12.2018, 11:06
Сообщение #22


Стаж: 14 лет

Сообщений: 100
Благодарностей: 33
Полезность: < 0

УПТЬ


Уже ничем не владею, совсем обленился...
Перейти в начало страницы         Просмотр профиля    Отправить личное сообщение
   + Цитировать сообщение
umprex
сообщение 26.12.2018, 17:14
Сообщение #23


Иконка группы

Стаж: 16 лет
Город: Киев

Сообщений: 2909
Благодарностей: 4781
Полезность: 2197

Раз ТС так часто подымает тему, потому и я ему поднять её разок)
Мне просто стало интересно от куда клиент знает, что код написан качественно - я нигде в теме не увидел примера кода.
Было бы круто, если бы ТС всё таки выложил хотя бы маленький пример кода, если он будет "качественным" я сам оставлю тут отзыв о ТСе :)


Перейти в начало страницы         Просмотр профиля    Отправить личное сообщение
   + Цитировать сообщение
Статус пользователя Puwok
сообщение 26.12.2018, 19:03
Сообщение #24


Стаж: 14 лет

Сообщений: 100
Благодарностей: 33
Полезность: < 0

Цитата(umprex @ 26.12.2018, 18:14) *
Раз ТС так часто подымает тему, потому и я ему поднять её разок)
Мне просто стало интересно от куда клиент знает, что код написан качественно - я нигде в теме не увидел примера кода.
Было бы круто, если бы ТС всё таки выложил хотя бы маленький пример кода, если он будет "качественным" я сам оставлю тут отзыв о ТСе :)


Код

Код
<?php
if (!defined("IN_OS") )
     {header('HTTP/1.1 404 Not Found');die;}

$Time = new TimePlayers();
include(dirname(__FILE__) . "/geoip/geoip.php");
?>
<div class="col-12 col-lg-9">
    <div class="card flex-fill w-100">
        <div class="card-header" style="border-bottom-width: 10px;">
            <h5 class="card-title mb-0">Топ игроков</h5>
        </div>
        <div class="card-body">
            <div class="table-responsive">
                <table class="table table-borderless table-sm">
                    <tr style="background-color: #f5f5f5;">
                        <td width="2%">#</td>
                        <td><div class="text-center">Скилл</div></td>
                        <td>Ник</td>
                        <td>Убийств</td>
                        <td>Смертей</td>
                        <td>Хедшотов</td>
                        <td>В игре</td>
                        <td>K/D</td>
                        <td>Играл</td>
                    </tr>
                    <?php
                    $result = $db->query("SELECT COUNT(*) FROM `$t_stats` LIMIT 1");
                    $r = $result->fetch_row();
                    $numrows = $r[0];
                    $result_per_page = $player_per_page;
                    $is_home = 1;
                    $draw_pagination = 0;
                    include('pagination.php');
                    if($numrows >= $player_per_page) $draw_pagination = 1;
                        
                    if($cfg_order == 4) $short = '`connection_time`';else
                    if($cfg_order == 3) $short = '`skill`';else
                    if($cfg_order == 2) $short = 'round(`kills` + `hs`)';else
                    if($cfg_order == 1) $short = '`kills`';else
                        $short = 'round(`kills` - `deaths` - `tks`)';
                  
                    $sql     = "SELECT * FROM `$t_stats` ORDER BY $short DESC LIMIT $offset, $rowsperpage";
                    $result = $db->query($sql);
                    
                    $rank = 1 + $offset;
                    while($row = $result->fetch_assoc())
                    {
                        $last_join = strtotime($row['last_join']);
                        
                        $from     = ip_code($row['ip']);
                        $fName     = ip_name($row['ip']);
                        
                        $skill     = round($row['skill']);
                        $kd     = round($row['kills'] / ($row['deaths'] + 1), 2);
                    
                        echo '
                        <tr>
                            <td>' . $rank++ . '</td>
                            <td><center><span style="background:url(' . $website . 'images/rank/' . skill($skill) . '.png); background-repeat: no-repeat"> <span style="margin-left: 15px;margin-right: 1px;">' . $skill . '</span></span></center></td>
                            <td><img width="24px" src="' . $website . 'images/flags/' . $from . '-min.png" alt="' . $fName . '" /> <a href="' . $website . 'players/' . $row['id'] . '/">' . $row['name'] . '</a></td>
                            <td>' . $row['kills'] . '</td>
                            <td>' . $row['deaths'] . '</td>
                            <td>' . $row['hs'] . '</td>
                            <td>' . $Time->TimeOn($row['connection_time']) . '</td>
                            <td>' . $kd . '</td>
                            <td>' . showtime($last_join, 1) . '</td>
                        </tr>';
                    }
                    ?>
                </table>
            </div>
            <?php include(dirname(__FILE__) . "/pagination.php"); ?>
        </div>
    </div>
</div>



Может он и не качественный на 2018 год (php + html вместе как многие гуру пишут), но всё работает без ошибок, пример из Web-AeS. Пишу без фреймворков.

Отредактировал: Puwok, - 26.12.2018, 19:05


Уже ничем не владею, совсем обленился...
Перейти в начало страницы         Просмотр профиля    Отправить личное сообщение
   + Цитировать сообщение
Статус пользователя onotole
сообщение 26.12.2018, 20:15
Сообщение #25


Иконка группы

Стаж: 13 лет

Сообщений: 1572
Благодарностей: 1119
Полезность: 1550

Меценат Меценат

Puwok, не 2018 точно. 2008 может)
Перейти в начало страницы         Просмотр профиля    Отправить личное сообщение
   + Цитировать сообщение
Статус пользователя angryjack
сообщение 27.12.2018, 10:27
Сообщение #26
Стаж: 11 лет

Сообщений: 390
Благодарностей: 229
Полезность: 837

Puwok,
1. Название переменных нужно указывать с маленькой буквы. ($Time)
Это также относится и к методам.
2. $db->query("SELECT COUNT(*) FROM `$t_stats` LIMIT 1"); Зачет тут лимит?
3. Это что за убожество? Про elseif не слышал? А про switch?

Код:

if($cfg_order == 4) $short = '`connection_time`';else
if($cfg_order == 3) $short = '`skill`';else
if($cfg_order == 2) $short = 'round(`kills` + `hs`)';else
if($cfg_order == 1) $short = '`kills`';else


4. Запросы к базе данных прямо из шаблона.

5. 2 раза инклудится pagination.php, зачем?

6. Нужно придерживаться одного стиля написания кода, а у тебя черт ногу сломит. Стандартами тут даже и не пахнет.

Столько недочетов только в одном куске кода, боюсь смотреть весь скрипт.
Лично по моему мнению, с такими знаниями и с таким кодом рано еще оказывать услуги.
Перейти в начало страницы         Просмотр профиля    Отправить личное сообщение
Поблагодарили 1 раз
   + Цитировать сообщение
Статус пользователя Puwok
сообщение 27.12.2018, 12:43
Сообщение #27


Стаж: 14 лет

Сообщений: 100
Благодарностей: 33
Полезность: < 0

angryjack, А ты лучше напиши и выложи в паблик. Дай угадаю, (накуй мне это нужно). Вот и весь ответ.

Какая разница напишу я if else или elseif, работать будет одинаково.

Отредактировал: Puwok, - 27.12.2018, 12:50


Уже ничем не владею, совсем обленился...
Перейти в начало страницы         Просмотр профиля    Отправить личное сообщение
   + Цитировать сообщение
Статус пользователя angryjack
сообщение 27.12.2018, 13:00
Сообщение #28
Стаж: 11 лет

Сообщений: 390
Благодарностей: 229
Полезность: 837

Цитата(Puwok @ 27.12.2018, 13:43) *
angryjack, А ты лучше напиши и выложи в паблик. Дай угадаю, (накуй мне это нужно). Вот и весь ответ.

Какая разница напишу я if else или elseif, работать будет одинаково.

Нет, то что ты пишешь и выкладываешь в общее пользование это очень похвально, но в данный момент мы обсуждаем качество кода и предоставление тобой услуг.

Разница в восприятии кода. Чем непонятней код, тем сложнее с ним в дальнейшем работать.
Перейти в начало страницы         Просмотр профиля    Отправить личное сообщение
   + Цитировать сообщение
Статус пользователя Puwok
сообщение 27.12.2018, 13:05
Сообщение #29


Стаж: 14 лет

Сообщений: 100
Благодарностей: 33
Полезность: < 0

angryjack,

Понял, принял. Но мне кажется там всё просто и понятно и любой сможет что то подредактировать.


Уже ничем не владею, совсем обленился...
Перейти в начало страницы         Просмотр профиля    Отправить личное сообщение
   + Цитировать сообщение
Статус пользователя onotole
сообщение 27.12.2018, 15:46
Сообщение #30


Иконка группы

Стаж: 13 лет

Сообщений: 1572
Благодарностей: 1119
Полезность: 1550

Меценат Меценат

Цитата(Puwok @ 27.12.2018, 14:05) *
angryjack,

Понял, принял. Но мне кажется там всё просто и понятно и любой сможет что то подредактировать.

Это тебе так только кажется. И когда ты к этому коду вернешься только через год, ты будешь думать "Что я употреблял, когда писал это" =)
Все через это проходят
Перейти в начало страницы         Просмотр профиля    Отправить личное сообщение
Поблагодарили 2 раз
   + Цитировать сообщение
2 страниц V   1 2
 
Ответить в данную темуНачать новую тему
 
0 пользователей и 1 гостей читают эту тему: