Als sender altijd een cijfer is moet je tinytext veranderen naat int

Origineel gepost door Ferdi Reichart

Als sender altijd een cijfer is moet je tinytext veranderen naat int

Ik ga het even proberen

Origineel gepost door Ferdi Reichart

Als sender altijd een cijfer is moet je tinytext veranderen naat int

Aangepast, maar hij blijft nog steeds mailen. Heb de waarde in de database aangepast naar 100 en 91, dus hij zou niets moeten mailen omdat $sender hoger dan 80 is.

Moet het geen

PHP Code:

if($row['sender'] < 80) 


zijn?

Ik denk dat $sender geen waarde heeft

Origineel gepost door Ferdi Reichart

Moet het geen

PHP Code:

if($row['sender'] < 80) 


zijn?

Ik denk dat $sender geen waarde heeft

Hij geeft wel een waarde als ik hem normaal echo.

Als je twee integer velden wilt vergelijken maar 1 ervan is geen integer dan kun je het casten.

if ( (int)trim($sender) > 80 ) { /* doe iets */ }
else { /* doe iets anders */ }

Origineel gepost door John Timmer

Als je twee integer velden wilt vergelijken maar 1 ervan is geen integer dan kun je het casten.

if ( (int)trim($sender) > 80 ) { /* doe iets */ }
else { /* doe iets anders */ }

Geprobeerd maar werkt ook niet :(

Dit is het hele script:

PHP Code:

<?php
// Connect to DB ####################################
require_once('db.php');

$sql = "SELECT description, sender FROM table WHERE sender < 80 LIMIT 1";
$result = $conn->query($sql);
// meer dan 1
if ($result->num_rows > 0) {
    // output data of each row
    while ($row = $result->fetch_assoc()) {
        $description = $description."".$row["description"] . "";
    $sender = $sender."".$row["sender"] . "";
    }
} else {
// als leeg is
    echo "Geen issue";
}
$conn->close();
?>

<?php
require_once 'lib/swift_required.php';

// Create the Transport ####################################
$transport = Swift_SmtpTransport::newInstance('XXX', 587);
$transport->setUsername('XXXX');
$transport->setPassword('XXX');


// Create the Mailer using your created Transport
$mailer = Swift_Mailer::newInstance($transport);

// Create the message
// Create the message
$message = Swift_Message::newInstance()

  // Give the message a subject
  ->setSubject('XXXX')

  // Set the From address with an associative array
  ->setFrom(array('XXXX' => 'XXXX'))

  // Set the To addresses with an associative array
  ->setTo(array('XXX'))

  // Give it a body
  ->setBody('Hier komt de code voor de email', 'text/html');
  
// Check if count lower than 80

if($sender <80) {
    ($mailer->send($message));
        echo "Er is een email gestuurd";
} else {
    echo "Er is geen email verzonden";
}

?>

Ik heb de inloggegevens geanonimiseerd middels XXX

Je plakt $description en $sender ervoor maar deze variabelen bestaan niet? Krijg je daar geen foutmeldingen van? Bovendien heb je in je tabel nog een rij waarbij sender de waarde 60 heeft, dus die mail wordt gewoon verstuurd. Dat in de eerste rij sender de waarde 91 of 100 heeft maakt niks uit.

Probleem is opgelost, ik moet eerst checken of er uberhaubt iets uit de DB kwam als de waarde lager dan 80 was. Maar er stonden alleen getallen hoger in dan 80.

Eerst check ik of de waarde uit de DB leeg is of niet, zoniet dan check ik of de waarde <80 is.

PHP Code:

<?php

ini_set( 'display_errors', 1 );
error_reporting( E_ALL );

if (empty($sender)) { 
    echo "Er is geen waarde";
} elseif ($sender <80){
    $mailer->send($message);
}

?>

Dank voor alle hulp!!

Je hebt alles op de verkeerde plek staan.

Als je maar 1 resultaat uit de DB haalt hoef je geen while loop te gebruiken.

PHP Code:

<?php




// Connect to DB ####################################
require_once('db.php');

$sql = "SELECT description, sender FROM table WHERE sender < 80 LIMIT 1";
$result = $conn->query($sql);
// meer dan 1
if ($result->num_rows > 0) {
    // output data of each row
    $row = $result->fetch_assoc();
        
    $description = $row['description'];
    $sender = $row['sender'];

    require_once 'lib/swift_required.php';

    // Create the Transport ####################################
    $transport = Swift_SmtpTransport::newInstance('XXX', 587);
    $transport->setUsername('XXXX');
    $transport->setPassword('XXX');


    // Create the Mailer using your created Transport
    $mailer = Swift_Mailer::newInstance($transport);

    // Create the message
    // Create the message
    $message = Swift_Message::newInstance()

    // Give the message a subject
    ->setSubject('XXXX')

    // Set the From address with an associative array
    ->setFrom(array('XXXX' => 'XXXX'))

    // Set the To addresses with an associative array
    ->setTo(array('XXX'))

    // Give it a body
    ->setBody('Hier komt de code voor de email', 'text/html');
  
    // Check if count lower than 80

    if($row['sender'] < 80) {
        $mailer->send($message);
            echo "Er is een email gestuurd";
    } else {
        echo "Er is geen email verzonden";
    }

    
} else {
    // als leeg is
    echo 'Niets gevonden in de DB';
}
$conn->close();
?>

Dankjewel Ferdi!! Enorm bedankt!

Om dan nog verder te optimaliseren...: ;-)

Waarom met sql alles selecteren met een waarde lager dan 80, en vervolgens in de code nog een keer controleren of iets lager is dan 80?
Alles is toch al lager dan 80 als het uit de database komt? Mijns inziens kan die if < 80 eruit toch?

Waarom eerst in de code resources gebruiken om een mailtje op te stellen en dán pas controleren of er wel een mail verstuurt moet worden. Beter eerst controleren of er überhaupt een mail gestuurd moet worden en dan pas de mail opstellen?