Отметить опеределённые чекбоксы в зависимости от наличия данных в файле

@ljubomir · Регистрация: 27.04.2012

Студворк — интернет-сервис помощи студентам

Пишу и думаю, можно так делать, или это говнокод?

В общем смысл в том, что у меня есть скрипт, который отправляет данные для добавления книги в базу. Естественно, там есть куча полей, которые надо заполнять.

В ходе загрузки файла я получаю из него некую информацию. В соответствии с полученными данными мне нужно сделать выбранным соответствующий чекбокс, расположенный на этой странице. Суть в том, что чекбокс там должен стоять в любом случае, но если из файла были подтянуты нужные данные, он должен становится отмеченным.

И вот я сделал следующим образом:

на странице в

JavaScript
1
$( document ).ready

я пишу такое:

PHP
1
2
3
4
        <?php 
            echo "$('#" . $genre . "').prop('checked', true);\n";   
        
        ?>

Естественно, выше есть пхп код, который эту инфу выдариет и пихает в переменную $genre.

Так вот и вопрос: Это говнокод или нормально все??

Если что, вот полный код страницы:

PHP/HTML
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
<?php
require("/home/" . $_SERVER[HTTP_HOST] ."/www/upl.php");
 
$link_book = uploadd ("img_book", 1024*3*1024, "/www/uploads/books/");
if ($link_book != "1" and $link_book != "2")
{
$xml = simplexml_load_file($link_book);
 
 
  $ti = "title-info";
  $fn = "first-name";
  $ln = "last-name";
  $bt = "book-title";
  $sl = "src-lang";
    $dt =  "date";
 
foreach ($xml->description as $description) {
    
$fio = trim($description->$ti->author->$fn . " " . $description->$ti->author->$ln); 
$genre = trim($description->$ti->genre);
$title = trim($description->$ti->$bt); 
$desc = trim($description->$ti->annotation->p);
$year = trim($description->$ti->$dt); // ???
$trans = trim($description->$ti->translator);  
$seriesname = trim($description->$ti->sequence['name']);
$seriesnumber = trim($description->$ti->sequence['number']);
 
if (trim($description->$ti->$sl) == "ru") {
    $countr = "1";  
} else if (trim($description->$ti->$sl) == "" ){
    $countr = "";
} else {
    $countr = "2";
}
 
}
 
 
//print_r($xml->binary->attributes['0']);
///////////echo $xml->binary; 
 
echo "<script language=\"JavaScript\">
fromfile = true;
</script>\n";
 
//$genreslist = array('sf','de','th','pr','lo','ad','ch','po','dr','an','sc','co','re','no','de','re','hu','ho'); 
 
$genre = substr($genre, 0, 2);
 
 
 
// Дернуть яваскрипт на описке серий и авторов, доделать с ними все. 
}
 
?>
 
<html>
 
<head>
  <title></title>
  <script type="text/javascript" src="../jquery.js"></script>
  <link rel="stylesheet" href="../stylelib.css" type="text/css" />
</head>
 
<body>
 
<form name="" action="" method="post" enctype="multipart/form-data">
<input type="file" name="img_book">
<input type="submit" value="КНИГГГГ!">
</form><br /><br /><br /><br />
 
  
<form action="add-bookP.php" method="post" enctype="multipart/form-data">
 
 
<!--//=====ПОИСК И ВЫБОР АВТОРОВ=================================================//-->
 
Автор книги: <input id="find_author" type="text" value="<? echo $fio ?>" onkeyup="findauthor();"><br />
 
<select id="authorlist" size = "10" ></select>
 
<select id="authorsbook" name="id_author[]" size = "10" multiple="multiple"></select>
 
<input type="button" value=">>" onclick="plusauthor()">
<input type="button" value="<<" onclick="minusauthor()">
 
 
<input type="button" value="Добавить автора" onclick="showmodal('modalaut');">
 
<div id="modalaut" class="modalaut">
    <input id="fioauthor" type="text" value=""><br />
    <textarea id="descauthor" rows=10 cols=25  value=""></textarea><br />
    <input type="button" value="Добавить" onclick="addauthor()"><br>
</div>
 
<div id="mask" class="mask" onclick="hidemodal(activmodal)"> </div> 
<input type="button" value="!!!!!!!!!!!!!!!!!!!!" onclick="qq()"><br>
<p>==========================================</p>
 
<script language="JavaScript">
 
function qq()
{
 
    $("#serieslist :last").attr("selected", "selected");
 
}
 
function plusauthor()
{
 
    $("#authorsbook").append( $('<option value="' + $("#authorlist :selected").val() + '">' + $("#authorlist :selected").text() + '</option>'));
 
}
 
 
 
 
function minusauthor()
{
$("#authorsbook :selected").remove();
}
 
function addauthor()
{
 
    $.ajax({
        type: "POST",
        url: "add-authorP.php",
        data: "fio=" + $("#fioauthor").val() + "&description=" + $("#descauthor").val(),
        success: function(data){
 
            alert(data);
 
        }
 
    });
 
}
 
function findauthor()
{
 
    $.ajax({
        type: "POST",
        url: "../search/author-listP.php",
 
        data: "fio=" + $("#find_author").val(),
        success: function(data){
            $("#authorlist").empty();
            dat = $.parseJSON(data);
 
            for (var key in dat)
            {
 
                $("#authorlist").append( $('<option value="' + key + '">' + dat[key] + '</option>'));
                if (fromfile){
                    
                    $("#authorlist :last").attr("selected", "selected");
                    
                }
            }
 
        }
 
    });
 
}
 
 
 
 
</script>
 
<br />
 
<!--//=====ПОИСК И ВЫБОР АВТОРОВ=============================================//-->
 
<!--//=======================================================================//-->
 
 
<!--//=====ПОИСК И ВЫБОР СЕРИЙ=============================================//-->
 
Серия книги: <input id="find_series" type="text" value="<? echo $seriesname ?>" onkeyup="findseries();"><br />
 
<select id="serieslist"  name="id_series" size = "10" onchange="showbooks()"></select>
 
<select id="booksserieslist" size = "10" ></select>
 
 
<input type="button" value="Добавить серию" onclick="showmodal('modalser');">
 
<div id="modalser" class="modalser" >
 
    <input id="name-series" type="text" value=""><br />
    <input type="button" value="Send" onclick="addseries();">
 
</div>
 
<div id="mask" class="mask" onclick="hidemodal(activmodal)"> </div>
 
 
 
<p>==========================================</p>
<script language="JavaScript">
 
function findseries ()
{
 
    $.ajax({
        type: "POST",
        url: "../search/series-listP.php",
        data: "ser=" + $("#find_series").val(),
        success: function(data){
            $("#serieslist").empty();
            dat = $.parseJSON(data);
 
            for (var key in dat)
            {
 
                $("#serieslist").append( $('<option value="' + key + '">' + dat[key] + '</option>'));
                if (fromfile){
                    
                    $("#serieslist :last").attr("selected", "selected");
                    showbooks();
                }   
            }
 
        }
 
    });
 
}
 
function showbooks ()
{
 
 
 
$.ajax({
        type: "POST",
        url: "../search/series-booksP.php",
        data: "ser=" + $("#serieslist :selected").val(),
        success: function(data){
            $("#booksserieslist").empty();
            dat = $.parseJSON(data);
 
            for (var i = 0; i < dat.length; i++)
            {
 
                $("#booksserieslist").append( $('<option value="' + i + '">' + dat[i] + '</option>'));
            }
 
 
 
        }
 
    });
 
}
 
 
 
 
function addseries ()
{
 
$.ajax({
        type: "POST",
        url: "add-seriesP.php",
        data: "ser=" + $("#name-series").val(),
        success: function(data){
            $("#booksserieslist").empty();
            $("#serieslist").empty();
 
            $("#serieslist").append( $('<option value="' + data + '">' + $("#name-series").val() + '</option>'));
 
            $("#serieslist :last").attr("selected", "selected");
 
 
 
        }
 
    });
 
}
 
 
</script>
 
 
 
<!--//=====ПОИСК И ВЫБОР СЕРИЙ=============================================//-->
 
<!--//=====================================================================//-->
 
<!--//=====ИНФОРМАЦИЯ О КНИГЕ==============================================//-->
 
Название книги:    <input name="name_book" type="text" value="<? echo $title ?>"><br />
Описание:   <textarea name="description_book" rows=10 cols=25  value=""><? echo $desc ?></textarea><br />
Год выпуска:  <input name="year_book" type="text" value="<? echo $year  ?>"><br />
Страна выпуска:<br />
 
    <input type="radio" id="cb1" name="country_book" value="1"> Русская<br />
   <input type="radio" id="cb2" name="country_book" value="2"> Зарубежная<br />
Переводчик:   <input name="trans_book" type="text" value="<? echo $trans ?>"><br />
Номер книги в серии:    <input name="series_number" type="text" value="<? echo $seriesnumber ?>"><br /><br />
Обложка книги:  <input type="file" name="img_book"><br />
 
<input type="checkbox" id="sf" name="1" value="sf">Фантастика<Br>
<input type="checkbox" id="de" name="2" value="de">Детектив<Br>
<input type="checkbox" id="th" name="3" value="th">Триллер<Br>
<input type="checkbox" id="pr" name="4" value="pr">Проза<Br>
<input type="checkbox" id="lo" name="5" value="lo">Мыльная опера<Br>
<input type="checkbox" id="ad" name="6" value="ad">Приключения<Br>
<input type="checkbox" id="ch" name="7" value="ch">Детская литература<Br>
<input type="checkbox" id="po" name="8" value="po">Поэзия<Br>
<input type="checkbox" id="dr" name="9" value="dr">Драматургия<Br>
<input type="checkbox" id="an" name="10" value="an">Старинная литература<Br>
<input type="checkbox" id="sc" name="11" value="sc">Научная литература<Br>
<input type="checkbox" id="co" name="12" value="co">Компьтерная литература<Br>
<input type="checkbox" id="re" name="13" value="re">Справочная литература<Br>
<input type="checkbox" id="no" name="14" value="no">Искусство и Дизайн<Br>
<input type="checkbox" id="de" name="15" value="de">Документальная литература<Br>
<input type="checkbox" id="re" name="16" value="re">Религионая литература<Br>
<input type="checkbox" id="hu" name="17" value="hu">Юмор<Br>
<input type="checkbox" id="ho" name="18" value="ho">Домоводство<Br>
 
 
Файл книги:    <input type="file" name="link_book"><br />
<br />
    <input type="submit" value="Добавить" onclick="selall()">
    <script>
        function selall()
        {
            $('#authorsbook option').each(function(){
                this.selected=true;
            });
        }
        
    function showmodal(id)
    {
        activmodal = id;
        $("#mask").fadeIn("slow");
        $("#" + id).fadeIn("slow");
    }
 
    function hidemodal(id)
    {
        $("#mask").fadeOut("slow");
        $("#" + id).fadeOut("slow");
    }
    
    $( document ).ready(function() {
        if ($.trim($("#find_author").val()) != "") {
            findauthor();           
        }
        if ($.trim($("#find_series").val()) != "") {
            findseries();
        }
        
  <?php if ($countr != "") {
            echo "$(\"#cb" . $countr . "\").attr(\"checked\", true);\n";
        }?>
    
        <?php 
            echo "$('#" . $genre . "').prop('checked', true);\n";   
        
        ?>
        
    }); 
    
    </script>
 
</form>
 
 
<!--//=====ИНФОРМАЦИЯ О КНИГЕ==============================================//-->
 
 
 
 
 
 
 
</body>
 
</html>

(код из цитаты внизу, а по загрузке файла вверху).

@sash · 05.06.2015, 22:53

Я по говнокоду спец
Засовываем пхп переменную в жейквери

JavaScript
1
2
3
$(document).ready(function () {
$('#" + <?=$genre?>+ "').prop('checked', true);
});

Не вникал в твой код, но пробуй, а вдруг

@pav1uxa · 05.06.2015, 22:55

тоже самое

Сообщение от ljubomir

echo "<script language=\"JavaScript\">
fromfile = true;
</script>\n";

можно же просто

PHP/HTML
1
2
3
4
5
6
7
..
?>
<script>
    fromfile = true;
</script>
<?php
...

Лучше стараться отделять html-код от php.

Да и не мешало бы табуляцией пользоваться. И непонятно зачем столько пустых строк...

@sash · 05.06.2015, 22:59

Или так

JavaScript
1
2
3
$(document).ready(function () {
$('#" + <? echo $genre; ?>+ "').prop('checked', true);
});

@ljubomir · 05.06.2015, 22:59 **[ТС]**

да, так лучше. Спасибо.

Я больше о самом принципе, нормально ли совать яваскрипт в пхп?

Добавлено через 32 секунды
Точняк, пасиб.

@pav1uxa · 05.06.2015, 23:01

Сообщение от sash

Или так

Не по теме:

первый вариант был лучше

@ljubomir · 05.06.2015, 23:02 **[ТС]**

Код в редакторе нормально выглядит. Вплане с тауляцией, отступами. Это он чета тут склеился... Пустые строки то так, пока пишу. Чтоб искать проще было. Уберу потом.

@sash · 05.06.2015, 23:16

А не, стойстой, не получится, там с кавычками неправильно

JavaScript
1
2
3
$(document).ready(function () {
$('#' + <?=$genre?>).prop('checked', true);
});

Или так

JavaScript
1
2
3
$(document).ready(function () {
$('#' + <? echo $genre; ?>).prop('checked', true);
});

Ну если нужно и без этого никак то даже очень нужно

@Eva Rosalene · 05.06.2015, 23:16

Сделайте в PHP отдельную функцию по выводу чекбокса и используйте в строках 320-337. Пусть функция в зависимости от переданных данных выводит или отмеченный, или не отмеченный чекбокс.

@ljubomir · 05.06.2015, 23:31 **[ТС]**

FraidZZ, я думал так сделать, но чекбоксов там дочерта, причется нагружать пхп массивом, где будут прописаны нужные параметры чекбоксов. + проверку условия при каждом выводе, то это или не то, надо отмечать или нет. Это все равно адекватнее будет чем строчка в

JavaScript
1
$(document).ready

?

Я самоучка просто, опыта реальной разработки нет.Плохо представляю, что оптимальнее...

@Eva Rosalene · 05.06.2015, 23:36

ljubomir, тут огромная разница. Ваше решение - клиент-сайд, фронт-энд. Предложенное мной - сервер-сайд, бэк-энд.

Оптимальнее был бы выбор бэк-энда, конечно. Только оптимизированного. К сожалению, код показать не могу, сам я не PHP-кодер.

@ljubomir · 05.06.2015, 23:50 **[ТС]**

FraidZZ, хмм, спасибо, попробую

Новые блоги и статьи Все статьи Все блоги /
Загрузка PNG-файла с альфа-каналом с помощью библиотеки SDL3_image на Android 8Observer8 27.01.2026 Содержание блога SDL3_image - это библиотека для загрузки и работы с изображениями. Эта пошаговая инструкция покажет, как загрузить и вывести на экран смартфона картинку с альфа-каналом, то есть с. . .	влияние грибов на сукцессию anaschu 26.01.2026 Бифуркационные изменения массы гриба происходят тогда, когда мы уменьшаем массу компоста в 10 раз, а скорость прироста биомассы уменьшаем в три раза. Скорость прироста биомассы может уменьшаться за. . .	Воспроизведение звукового файла с помощью SDL3_mixer при касании экрана Android 8Observer8 26.01.2026 Содержание блога SDL3_mixer - это библиотека я для воспроизведения аудио. В отличие от инструкции по добавлению текста код по проигрыванию звука уже содержится в шаблоне примера. Нужно только. . .	Установка Android SDK, NDK, JDK, CMake и т.д. 8Observer8 25.01.2026 Содержание блога Перейдите по ссылке: https:/ / developer. android. com/ studio и в самом низу страницы кликните по архиву "commandlinetools-win-xxxxxx_latest. zip" Извлеките архив и вы увидите. . .
Вывод текста со шрифтом TTF на Android с помощью библиотеки SDL3_ttf 8Observer8 25.01.2026 Содержание блога Если у вас не установлены Android SDK, NDK, JDK, и т. д. то сделайте это по следующей инструкции: Установка Android SDK, NDK, JDK, CMake и т. д. Сборка примера Скачайте. . .	Использование SDL3-callbacks вместо функции main() на Android, Desktop и WebAssembly 8Observer8 24.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .	моя боль iceja 24.01.2026 Выложила интерполяцию кубическими сплайнами www. iceja. net REST сервисы временно не работают, только через Web. Написала за 56 рабочих часов этот сайт с нуля. При помощи perplexity. ai PRO , при. . .	Модель сукцессии микоризы anaschu 24.01.2026 Решили писать научную статью с неким РОманом

@ljubomir 20 / 20 / 12 Регистрация: 27.04.2012 Сообщений: 145
	05.06.2015, 22:59 [ТС]
	да, так лучше. Спасибо. Я больше о самом принципе, нормально ли совать яваскрипт в пхп? Добавлено через 32 секунды Точняк, пасиб. 0

@ljubomir 20 / 20 / 12 Регистрация: 27.04.2012 Сообщений: 145
	05.06.2015, 23:02 [ТС]
	Код в редакторе нормально выглядит. Вплане с тауляцией, отступами. Это он чета тут склеился... Пустые строки то так, пока пишу. Чтоб искать проще было. Уберу потом. 0

@Eva Rosalene Модератор 5241 / 2115 / 416 Регистрация: 06.01.2013 Сообщений: 4,846
	05.06.2015, 23:16
	Сделайте в PHP отдельную функцию по выводу чекбокса и используйте в строках 320-337. Пусть функция в зависимости от переданных данных выводит или отмеченный, или не отмеченный чекбокс. 0

@Eva Rosalene Модератор 5241 / 2115 / 416 Регистрация: 06.01.2013 Сообщений: 4,846
	05.06.2015, 23:36
	ljubomir, тут огромная разница. Ваше решение - клиент-сайд, фронт-энд. Предложенное мной - сервер-сайд, бэк-энд. Оптимальнее был бы выбор бэк-энда, конечно. Только оптимизированного. К сожалению, код показать не могу, сам я не PHP-кодер. 0

@ljubomir 20 / 20 / 12 Регистрация: 27.04.2012 Сообщений: 145
	05.06.2015, 23:50 [ТС]
	FraidZZ, хмм, спасибо, попробую 0