PHP.SU

Программирование на PHP, MySQL и другие веб-технологии

PHP.SU Портал

Главная

Помощь

Поиск

Поиск Яндекс Вакансии

Пользователи

Здравствуйте, Гость

( Вход · Регистрация · Правила форума )

Забыли пароль?

Нормально, или нет?

Форумы портала PHP.SU » PHP » Программирование на PHP » Вопросы новичков (Модераторы: OrmaJever, Саныч, Строитель)

Страниц (1): [1]

Без описания

Поиск в теме | Версия для печати

event	Отправлено: 12 Февраля, 2012 - 20:24:20
Частый посетитель Покинул форум Сообщений всего: 542 Дата рег-ции: Май 2011 Помог: 0 раз(а)	Привет, есть такой кусочек кода: PHP: скопировать код в буфер обмена <tr> <td class="rowhead" style="line-height:25px;"><?=$tracker_lang['gender_person']?> - <font color=red>*</font></td> <td><select name="gender"> <?PHP if($row['gender'] == 1) {$selgend = " selected=\"selected\"";} elseif ($row['gender']==2){$selgend2 = " selected=\"selected\"";} ?> <option value="0"><?=$tracker_lang['gender_choose']?></option> <option <?=$selgend ?> value="1"><?=$tracker_lang['gender_male']?></option> <option <?=$selgend2 ?> value="2"><?=$tracker_lang['gender_female']?></option> </tr> Нормально ли я сделал выбор selected из базы?

OrmaJever	Отправлено: 12 Февраля, 2012 - 20:41:49
Активный участник Покинул форум Сообщений всего: 7540 Дата рег-ции: Янв. 2010 Откуда: Чернигов Помог: 299 раз(а)	Ну условие можно переписать, и в форме использовать одну переменую $selgend PHP: скопировать код в буфер обмена <?PHP $selgend = ($row['gender'] == 1) ? ' selected="selected"' : ($row['gender']==2 ? ' selected="selected"' : ''); ?> или так PHP: скопировать код в буфер обмена <?PHP if($row['gender'] == 1) $selgend = " selected=\"selected\""; elseif ($row['gender'] == 2) $selgend = " selected=\"selected\""; ?> ----- Если вы хотя бы 3-4 раза не решите всё выкинуть и начать заново - вы явно что-то делаете не так.

event	Отправлено: 12 Февраля, 2012 - 20:54:43
Частый посетитель Покинул форум Сообщений всего: 542 Дата рег-ции: Май 2011 Помог: 0 раз(а)	Спасибо, а тут нормально? PHP: скопировать код в буфер обмена $date = explode('-', $row['birthday']); $a = $date[0]; $b = $date[1]; $c = $date[2]; $year .= "<select name=year><option value=\"0000\">".$tracker_lang['my_year']."</option>\n"; $i = "1920"; while ($i <= (date('Y',time()))) { $year .= "<option value=\"" .$i. "\""; if ($a == $i) $year .= " selected=\"selected\""; $year .= ">".$i."</option>\n"; $i++; } $year .= "</select>\n"; $birthmonths = array( "01" => $tracker_lang['my_months_january'], "02" => $tracker_lang['my_months_february'], "03" => $tracker_lang['my_months_march'], "04" => $tracker_lang['my_months_april'], "05" => $tracker_lang['my_months_may'], "06" => $tracker_lang['my_months_june'], "07" => $tracker_lang['my_months_jule'], "08" => $tracker_lang['my_months_august'], "09" => $tracker_lang['my_months_september'], "10" => $tracker_lang['my_months_october'], "11" => $tracker_lang['my_months_november'], "12" => $tracker_lang['my_months_december'], ); $month = "<select name=\"month\"><option value=\"00\">".$tracker_lang['my_month']."</option>\n"; foreach ($birthmonths as $month_no => $show_month) { $month .= "<option value=\"" .$month_no. "\""; if ($b == $month_no) $month .= " selected=\"selected\""; $month .= ">".$show_month."</option>\n"; } $month .= "</select>\n"; $day .= "<select name=day><option value=\"00\">".$tracker_lang['my_day']."</option>\n"; $i = 1; while ($i <= 31) { if ($i < 10) { $day .= "<option value=\"0" .$i. "\""; if ($c== $i) $day .= " selected=\"selected\""; $day .= ">".$i."</option>\n"; } else { $day .= "<option value=\"" .$i. "\""; if ($c== $i) $day .= " selected=\"selected\""; $day .= ">".$i."</option>\n"; } $i++; } $day .="</select>\n"; ?>

OrmaJever	Отправлено: 12 Февраля, 2012 - 21:17:54
Активный участник Покинул форум Сообщений всего: 7540 Дата рег-ции: Янв. 2010 Откуда: Чернигов Помог: 299 раз(а)	Ну первое что сразу кидается в глаза это экранизация кавычек, зачем брать текст в двойные кавычки если в них нету переменых? Что бы потом экранизировать такие кавычки? PHP: скопировать код в буфер обмена $year .= '<select name=year><option value="0000">'.$tracker_lang['my_year']."</option>\n"; В двойные кавычки нужно брать только строки со спец символами (\r, \n, \t и т.д). Но в некоторых случаях с двойными кавычками лутше код читать PHP: скопировать код в буфер обмена $day .= "<option value=\"" .$i. "\""; // если уж есть переменая посреди текста то можно её в кавычки убрать $day .= "<option value=\"$i\""; // а ещё лучше и экранизацию убрать $day .= "<option value='$i'"; // заменив html кавычки на одинарные вобще для html'a вроде бы нету разницы одинарные или двойные кавычки, поэтому иногда можно их комбинировать для читабельности кода. Особо в цыклы и условия не всмартивался. ----- Если вы хотя бы 3-4 раза не решите всё выкинуть и начать заново - вы явно что-то делаете не так.

moderated.weblink.20130603	Отправлено: 13 Февраля, 2012 - 12:50:24
Новичок Покинул форум Сообщений всего: 26 Дата рег-ции: Февр. 2012 Помог: 0 раз(а)	А можно сделать и так, по проще! $day .= "<option value=".$i; строку $a = $date[0]; $b = $date[1]; $c = $date[2]; я бы убрал она вообще не нужна я бы наверняка сделал по другому, но разбираться не хочу. а отформатировал бы я вот так - PHP: скопировать код в буфер обмена $date = explode('-', $row['birthday']); $year .= "<select name=year><option value='0000'>".$tracker_lang['my_year']."</option>\n"; $i = "1920"; // зачем это? while ($i <= (date('Y',time()))) { $year .= "<option value=".$i; if ($date[0] == $i) $year .= ' selected="selected"'; $year .= ">".$i."</option>\n"; $i++; } $year .= "</select>\n"; $birthmonths = array( "01" => $tracker_lang['my_months_january'], "02" => $tracker_lang['my_months_february'], "03" => $tracker_lang['my_months_march'], "04" => $tracker_lang['my_months_april'], "05" => $tracker_lang['my_months_may'], "06" => $tracker_lang['my_months_june'], "07" => $tracker_lang['my_months_jule'], "08" => $tracker_lang['my_months_august'], "09" => $tracker_lang['my_months_september'], "10" => $tracker_lang['my_months_october'], "11" => $tracker_lang['my_months_november'], "12" => $tracker_lang['my_months_december'],); $month = "<select name='month'><option value='00'>".$tracker_lang['my_month']."</option>\n"; foreach ($birthmonths as $month_no => $show_month) { $month .= "<option value='".$month_no."'"; if ($date[1] == $month_no) $month .= " selected='selected'"; $month .= ">".$show_month."</option>\n"; } $month .= "</select>\n"; $day .= "<select name=day><option value='00'>".$tracker_lang['my_day']."</option>\n"; $i = 1; while ($i <= 31) { if ($i < 10) { $day .= "<option value='0".$i."'"; if ($date[2] == $i) $day .= " selected='selected'"; $day .= ">".$i."</option>\n"; } else { $day .= "<option value='".$i."'"; if ($date[2] == $i) $day .= " selected='selected'"; $day .= ">".$i."</option>\n"; } $i++; } $day .="</select>\n"; (Отредактировано автором: 13 Февраля, 2012 - 13:15:21)

Поиск в теме | Версия для печати

Страниц (1): [1]

Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)

« Вопросы новичков »

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