Проверка на отсутствие строки

[zIP]

Проверка на отсутствие строки

Есть выражение:

foreach($file as $string)
   {
if($dt==rtrim($string)) {

проверяем есть ли такая строка и если есть то выполняем действие. А как сделать обратное - если нет, то выполняем действие?

 

[Фанат]

else тебе не подходит?

 

[sage]

не буду =) но так тож можно было бы

 

[Фанат]

sage
не тормози

 

[sage]

Фанат
ты имеешь ввиду

if (условие) {
  // Действие1
} else {
  // Действие2
}

При таком раскладе Действие2 будет срабатывать на каждом проходе цикла, если условие ложно. А требуется, как я понял, однократное выполнение в случае ложности. Так что, [m]in_array[/m] =)

-~{}~ 12.04.06 17:11:

+ применение этой функции является более лаконичным решением, нежели прогон цикла

 

[zIP]

Автор оригинала: Фанат
else тебе не подходит?

Если делаю так то почти работает. Т.е. проверяет и если нет то записывает.

foreach($file as $string)

if($dt==rtrim($string)) {


return false;

}

else

$file=fopen("data.dat",'a');
fputs($file,$dt."\r\n");
fclose ($file);

Но разве это корректный код?

 

[Фанат]

объясни словами, что тебе надо сделать.
поскольку кодом ты объясняешь совершенно непонятно.
я и sage поняли тебя совершенно по-разному

-~{}~ 12.04.06 10:32:

подробно

 

[zIP]

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

Это функция.
Берем IP сравниваем с тем что есть в базе и если нет, то записываем.

$dt=getenv("REMOTE_ADDR");
function getip($dt) {
$file=file("data.dat");
foreach($file as $string)

if($dt==rtrim($string)) {
return false;
}

else

$file=fopen("data.dat",'a');
fputs($file,$dt."\r\n");
fclose ($file);

}


getip($dt);

$file=file("data.dat");
$s = count($file);
echo "$s";

 

[Positive]

после else { не пробовал ставить?

 

[McSimm]

Вот что у вас фактически получилось внутри цикла. Сами, я уверен, разберетесь что исправить.

if($dt==rtrim($string))
{
    return false;
}
else
{
    $file=fopen("data.dat",'a');
}
fputs($file,$dt."\r\n");

 

[Andreika]

Positive
>после else { не пробовал ставить?
а будет заметна разница?

 

[zIP]

Пробовал. Получаеться цикл. Т.е. записывает один тотже IP два раза если в базе 2 строки, если 4 то 4 раза. И так до бесконечности.

 

[McSimm]

Positive
Зачем ? Чтобы внутрь цикла вложить не только многократное открытие файла, но и запись ?

-~{}~ 12.04.06 10:51:

zIP
пройдитесь мысленно по вашему циклу для трех строк и вы сами все поймете. Обещаю

 

[zIP]

Автор оригинала: McSimm
Вот что у вас фактически получилось внутри цикла. Сами, я уверен, разберетесь что исправить.

if($dt==rtrim($string))
{
    return false;
}
else
{
    $file=fopen("data.dat",'a');
}
fputs($file,$dt."\r\n");

Спасибо. Разобрался с циклом.

function getip($dt) {
$file=file("data.dat");
foreach($file as $string)

if($dt==rtrim($string)) {
return false;
}

else 

$file=fopen("data.dat",'a');
{
fputs($file,$dt."\r\n");
fclose ($file);
}
}

Отредактировал. Так?

 

[Andreika]

zIP
неа, не разобрался... try again

 

[Фанат]

то есть, тебя правильно понял sage
тебе просто надо проверить, что в файле нет такой строки?

да, твой код неправильный.
ты запись делаешь внутри цикла.
а надо её вынести наружу.

кстати, вот это:

$file=file("data.dat");
$s = count($file);

лишнее.
догадаешься - почему? или подсказать?

 

[McSimm]

Спасибо. Разобрался с циклом.

еще хуже стало.

первый совет - всегда пишите { } для всех конструкций циклов и ветвлений, в данном случае для foreach и else.
совет два (еще раз) - пройдитесь мысленно для трех строк мысленного файла. Один раз когдав нем есть искомая строка, другой - когда нет.

 

[zIP]

Предположу что из этой функции, посколько там массив, можно получить количество строк этого массива, но вот как?

 

[Фанат]

погоди, разберись с циклом сначала.

 

[zIP]

Все. Я сдаюсь.
Тот код который я привел выше, как не странно, у меня работает. Проверяет- добавляет-не добавляет.

Все изменения вносимые мной абсолютно не работоспособные.

Если для foreach и else использую {} то цикл не закрываеться и продолжает писать.

Если только поставть вместо
return false;

return true;

?