Как исправить ошибку в коде?

Question

[Люба]

Подскажите пожалуйста как поправить код ? У меня переменная $ost выводит сколько осталось дней до окончания срока обявления но она не правильно работает.
Задумка такая 30 дней зеленый цвет, 20 дней оранжевый, 10 красный.
Сам код такой:

?>
	<div class='progressBlock'>
    <div class='percent'></div>
</div>


<style>
.progressBlock {
    width: 100px;
    height: 10px;
    border-radius: 3px;
    border: 1px solid black;
    position: relative;
}
 .percent{
    position: absolute;
    <?
	   
        //$ost = 30; //Осталось 10 дней это для примера так работает
		$total = 30; //Всего 30 дней
		$p = $ost * 100 / $total; // Получим 33%
		echo "width: $p%;\n";
		$color = "lime";
        if ($ost >= 10 and $ost < 20) $color = "yellow";
        if ($ost < 10) $color = "red";
        echo "background-color: $color;";
      
    ?>
    height: 10px;
}
</style>
 <?

Comments

[Lander]

Вроде всё работает
Проверяйте верстку.

Answer 1

[Sanovskiy]

switch (true){
    case $ost>=10 && $ost <20:
        $color = "yellow";
        break;
    case $ost<10:
        $color = "red";
        break;
    default:
        $color = 'green';
        break;
}

Как вариант вот так

Comments

[Люба]

тоже не верно показывает у меня правильно работает только вот эта конструкция

if($ost>0){
              echo '<br><progress class="time" max="30" value="'.$ost.'"></progress>';
              }

но она мне не подходит хотела сделать как на авито

[Sanovskiy]

Люба, проверяйте что у вас в $ost

[Люба]

Sanovskiy, как проверить ? вот например этот код работает правильно все показывает а приведенный выше показывает не правильно вот весь код целиком посмотрите если не сложно что не так my-files.ru/4ch650 код вставлен строка 312-343

if($ost>0){
              echo '<br><progress class="time" max="30" value="'.$ost.'"></progress>';
              }

[Sanovskiy]

Люба, используйте песочницу

[Люба]

Sanovskiy, если бы я понимала что там смотреть sandbox.onlinephpfunctions.com/code/05c0937ab99934...

[Sanovskiy]

<b>Notice</b>:  Undefined variable: ost in <b>[...][...]</b> on line <b>19</b><br />
width: 0%;
<br />

Переменная $ost не определена.

[Люба]

Sanovskiy, в чем может быть дело ? ведь сам $ost работает и выводи правильно в виде текста ?

if($ost>0)
  echo '<font color=red>Осталось '.$ost.' дн.</font>';

[Sanovskiy]

Я бы это делал не через генерацию css, а через прописывание соответствующего класса в самом элементе

[Люба]

Sanovskiy, можете подробнее написать как ? Хорошо бы вообще scc вынести в таблицу стилей

[Sanovskiy]

Люба, я посмотрел в этот код. Знаете, я сейчас рефакторю очень старый код. Года эдак 2008-2009.
Он местами ещё на PHP4. Так вот в нем мне было проще разбираться чем в вашем скрипте. понимаю, что вы ещё учитесь, но учитесь вы по совершенно неправильным материалам.
Основные недостатки вашего кода

1. винегрет из PHP/HTML
Есть множество способов отделить представление от бизнес-логики. Т.е. сначала вы прогоняете всю логику с выборками из БД, собираете, например, массив с данными, необходимыми для сборки страницы, и передаёте его в сборщик страницы. Самый простой сборщик просто напросто подключает шаблон phtml

public function render(string $template = null)
    {
        if (!$this->beforeRender()){
            return false;
        }

        if (is_null($template)) {
            $template = $this->layout;
        }

        if (!$this->templateExist($template)) {
            return 'Can\'t render not existing view ' . $template;
        }

        ob_start();
        include $this->getTemplateFilename($template);
        $html = $this->afterRender(ob_get_contents());
        ob_end_clean();

        if ($template != $this->layout && !is_null($this->layout) && $this->templateExist($this->layout)){
            $this->data['LayoutContent'] = $html;
            $html = $this->render($this->layout);
        }
        header('Content-Type: text/html; charset=UTF-8');
        return $html;
    }

Обратите внимание, что это не просто функция, а метод класса, если соберетесь использовать его логику.

2. Использование php_mysql/php_mysqli вместо PDO
На сегодняшний день расширение php_mysql/php_mysqli морально устарело и лично я рекомендую переходить на объектную модель взаимодействия с БД.

$m=mysql_query("SELECT * FROM MyTable WHERE foo='".$_REQUEST['foo']."' AND bar='".$_REQUEST['bar']."'");
$mm=mysql_fetch_array($m);

Код гораздо более уязвим, нежели вот такой

$stmt = $dbcon->prepare('SELECT * FROM MyTable WHERE foo=:foo AND bar=:bar');
$stmt->execute([
	'foo' => $_REQUEST['foo'],
	'bar' => $_REQUEST['bar']
]);
if ($stmt->rowCount()>0){
	$allRecords = $stmt->fetchAll(\PDO::FETCH_ASSOC);
}

3. echo
На самом деле вам не нужен echo. PHP сам по себе является шаблонизатором. Для этого у него есть королкий тег <?= ?>
А также весьма удобная запись для циклов и условий
Сравните

echo '<table>';
if (is_array($myvar) && count($myvar) > 0) {
    foreach ($myvar as $values) {
        if (is_array($values) and count($values) > 0) {
            echo '<tr>';
            foreach ($values as $value){
                echo '<td>'.$value.'</td>';
            }
            echo '</tr>';
        }
    }
}
echo '</table>';

и

<table>
<? if (is_array($myvar) && count($myvar) > 0): ?>
    <? foreach ($myvar as $values): ?>
        <? if (is_array($values) and count($values) > 0) : ?>
            <tr>
                <? foreach ($values as $value): ?>
                    <td><?= $value ?></td>
                <? endforeach; ?>
            </tr>
        <? endif; ?>
    <? endforeach; ?>
<? endif; ?>
</table>

Попробуйте реструктуризировать ваш код. Для этого не надо зразу гнаться за шаблонизаторами и MVC.
Просто вынесите всю сборку данных в начало вашего скрипта. А HTML с подстановкой данных изолируйте в конце.
И многие ошибки сразу станут заметнее.

Answer 2

[Дмитрий]

Добрый день.
Вставлю свои пять копеек)))

$ost = 9;
switch (true){
    case $ost >= 10 && $ost < 20:
        $color = "yellow";
        break;
    case $ost < 10:
        $color = "red";
        break;
    case $ost > 20:
        $color = 'green';
        break;
}
?>
<div style="color:<?= $color ?>">test</div>

Default в данном случае, я думаю, лишний. Хотя должно работать и с default, так что проверяйте содержимое переменной $ost. И вообще, существует ли переменная $ost? Где Вы её инициализируете?

$ost = 45;
$p = $ost * 100 / $total;
// далее остальной код

Comments

[Sanovskiy]

Как вариант можно добавить и default с серым цветом. Так сразу станет понятно, что в $ost что-то не то.

[Дмитрий]

Sanovskiy, тоже вариант. Думаю, что даже нужно добавить. Хотя... Пусть у автора вопроса голова болит)))

[Sanovskiy]

slo_nik, лично я всегда добавляю default/ Это как минимум убирает необходимость в одной проверке.