За последние 24 часа нас посетили 22764 программиста и 1217 роботов. Сейчас ищут 797 программистов ...

Слишком ли говнокод?

Тема в разделе "PHP для новичков", создана пользователем MrSullex, 24 май 2016.

  1. MrSullex

    MrSullex Активный пользователь

    С нами с:
    15 сен 2015
    Сообщения:
    122
    Симпатии:
    1
    PHP:
    1. $favicon = "SELECT * FROM settings WHERE id='1'"; $getfavicon = mysql_query($favicon);
    2. $head = "SELECT * FROM settings WHERE id='2'"; $gethead = mysql_query($head);
    3. $background = "SELECT * FROM settings WHERE id='3'"; $getbackground = mysql_query($background);
    4. $counts = "SELECT * FROM settings WHERE id='4'"; $getcounts = mysql_query($counts);
    HTML:
    1. <form action="" class="form" method="POST" target="_self">
    2.          
    3.             <div class="form_row">
    4.             <label>Путь к иконке:</label>
    5.             <input type="text" class="form_input" name="favicon" value="<?php while($row = mysql_fetch_array($getfavicon)) { echo $row['value']; } ?>" />
    6.             </div>
    7.            
    8.             <div class="form_row">
    9.             <label>Путь к шапке:</label>
    10.             <input type="text" class="form_input" name="head" value="<?php while($row = mysql_fetch_array($gethead)) { echo $row['value']; } ?>" />
    11.             </div>
    12.          
    13.             <div class="form_row">
    14.             <label>Путь к фону:</label>
    15.             <input type="text" class="form_input" name="background" value="<?php while($row = mysql_fetch_array($getbackground)) { echo $row['value']; } ?>" />
    16.             </div>
    17.          
    18.             <div class="form_row">
    19.             <label>Счетчики/баннеры:</label>
    20.             <textarea class="form_textarea" name="counts"><?php while($row = mysql_fetch_array($getcounts)) { echo $row['value']; } ?></textarea>
    21.             </div>
    22.             <div class="form_row">
    23.             <input type="submit" class="form_submit" name="save" value="Сохранить" />
    24.             </div>
    25.             <div class="clear"></div>
    26.         </form>
     
    #1 MrSullex, 24 май 2016
    Последнее редактирование модератором: 24 май 2016
  2. Deonis

    Deonis Старожил

    С нами с:
    15 фев 2013
    Сообщения:
    1.521
    Симпатии:
    504
    Как бы это сказать? Ну, в пятерку "чемпионов" может войти.
     
  3. Хыиуду

    Хыиуду Активный пользователь

    С нами с:
    3 июн 2014
    Сообщения:
    618
    Симпатии:
    5
    А почему не вытащить все четыре строки одним запросом и не запихнуть в ассоциативный массив по id?
     
  4. MrSullex

    MrSullex Активный пользователь

    С нами с:
    15 сен 2015
    Сообщения:
    122
    Симпатии:
    1
    потому что я тупой и не понимаю как это сделать. объясните пожалуйста :D
     
  5. Хыиуду

    Хыиуду Активный пользователь

    С нами с:
    3 июн 2014
    Сообщения:
    618
    Симпатии:
    5
    SELECT * FROM settings WHERE id in (1,2,3,4)
     
  6. igordata

    igordata Суперстар
    Команда форума Модератор

    С нами с:
    18 мар 2010
    Сообщения:
    32.410
    Симпатии:
    1.768
    это не говнокод, это индусский код
     
  7. SamyRed

    SamyRed Старожил

    С нами с:
    23 июл 2015
    Сообщения:
    1.196
    Симпатии:
    111
    Адрес:
    Украина
    Даже я бы до такого не додумался :D
     
  8. [vs]

    [vs] Суперстар
    Команда форума Модератор

    С нами с:
    27 сен 2007
    Сообщения:
    10.553
    Симпатии:
    631
    Код (Text):
    1. echo mysql_fetch_array($getbackground)['value'];
     
  9. igordata

    igordata Суперстар
    Команда форума Модератор

    С нами с:
    18 мар 2010
    Сообщения:
    32.410
    Симпатии:
    1.768
  10. NoMonster

    NoMonster Новичок

    С нами с:
    7 май 2016
    Сообщения:
    38
    Симпатии:
    1
    И ты показал шаблон Html не увидел ни о php а лишь запрос о чем речь говнокода?