[PHP] Podstawienie funkcji - problem Opcje

[mrRosso]

Witam


Jak widać jestem nowy na tym forum, znam HTML'a a z PHP to tylko początek...
i co do php mam pewien problem, myślę, że Wam uda się go rozwikłać. (jest to na pewno bardzo proste ale w php jestem naprawdę zielony)

GG Network pokazało przykład bota na swojej stronie https://boty.gg.pl/przyklady/
więc postanowiłem zrobić sobie bota który jak nie będzie mnie na GG to pokaże o której jestem, aczkolwiek żebym mógł aktualizować info o tym kiedy będę potrzebna jest funkcja do zapisu do pliku. (z odczytem nie będę miał problemu:) ) Nie będę owijał w bawełnę i przejdę do rzeczy,

[PHP] pobierz, plaintext 
<?
require_once 'GGBotApi/PushConnection.php';
require_once('GGBotApi/MessageBuilder.php');
 
$info = 1; // info 1-on 0-off
$info_file = 'temat.txt'; // Sciezka do pliku info.txt
 
$M=new MessageBuilder();
$czas=date("G:i:s - m - j - Y");
 
 
 
switch ($HTTP_RAW_POST_DATA) {
 
    case "/ver": $M->addBBcode('Wersja Bota v1.0[br] Autor: Mateusz Pietrzak [15:01 2012-01-02]'); break;
    case "/czas": $M->addText('aktualny czas: ."$czas".''); break;
 
 
 
default: $M->addBBcode('Nie ma mnie aktualnie na GG. Będę... $przykład'); break;
  }
$M->reply();
[PHP] pobierz, plaintext 

i funkcja czas działa pięknie i bot tak samo. A chcę stworzyć komendę np. /info która będzie aktualizowała zapytanie

[PHP] pobierz, plaintext 
$info
[PHP] pobierz, plaintext 

Zrobiłem taką funkcję:

[PHP] pobierz, plaintext 
 
        $info = (implode(" ", $msg));
        if($fp = fopen($info_file, 'a+'));
        if(fwrite($fp, $info." "));
        if(fclose($fp));
[PHP] pobierz, plaintext 

ale nie wiem jak ją podstawić pod komendę 'case'. Nie mam na myśli abyście wykonali za mnie całą robotę bo nie w tym rzecz tylko żebyście mogli mnie jakoś nakierować

dzięki i pozdrawiam, Mateusz

czyli nikt mi nie pomoże...? ;|

Ten post edytował mrRosso 2.01.2012, 20:32:45

[shinuexx]

rozumiem że chcesz wpisywać komende w postaci

Kod

/info informacja o tym kiedy będziesz

gdzie potem $msg="informacja o tym kiedy będziesz";
Niestety do tego nie użyjesz switch(a bynajmniej nie bezpośrednio z $HTTP_RAW_POST_DATA), ponieważ on wybiera spośród możliwych wartości(jak w przypadku "/ver" i "/czas") a tutaj masz wartość zmienną. Możesz albo wcześniej na $HTTP_RAW_POST_DATA użyć explode() i wtedy switchem, albo od razu w if'ach to zrobić.

Ten post edytował shinuexx 2.01.2012, 22:24:55

[mrRosso]

czyli wyglądałoby to tak?

[PHP] pobierz, plaintext 
} else if($msg[0] == "/info") {
        unset($msg[0]);
        $info = (implode(" ", $msg));
        if($fp = fopen($info_file, 'a+'));
        if(fwrite($fp, $info." "));
        if(fclose($fp));
[PHP] pobierz, plaintext 

wstawiłem to wyżej i wygląda to tak:

[PHP] pobierz, plaintext 
<?
require_once 'GGBotApi/PushConnection.php';
require_once('GGBotApi/MessageBuilder.php');
 
 
$M=new MessageBuilder();
$czas=date("G:i:s - m - j - Y");
 
 
 
switch ($HTTP_RAW_POST_DATA) {
 
    case "/ver": $M->addBBcode('Wersja Bota v1.0[br] Autor: Mateusz Pietrzak [15:01 2012-01-02]'); break;
 
    else if($msg[0] == "/info" && $user_is_online == 1) 
        unset($msg[0]);
        $info = (implode(" ", $msg));
        if($fp = fopen($info_file, 'a+'));
        if(fwrite($fp, $info." "));
        if(fclose($fp));
 
 
 
 
default: $M->addBBcode('Nie ma mnie aktualnie na GG.'); break;
  }
$M->reply();
 
[PHP] pobierz, plaintext 

i po uruchomieniu skryptu oddaje błąd taki:

[PHP] pobierz, plaintext 
<br />
<b>Parse error</b>:  syntax error, unexpected T_ELSE in <b>/home/gaduczat/public_html/ggbot.php</b> on line <b>15</b><br />
[PHP] pobierz, plaintext 

Ten post edytował mrRosso 3.01.2012, 00:25:54

[nospor]

żeby używać ELSE, najpierw musi być IF. U ciebie nie ma IF

[shinuexx]

Czym jest zmienna $msg[0]? Dodaj komentarze do linijek, żeby można było wiedzieć jak ten kod interpretujesz.

[mrRosso]

zrobiłem to tak jak kazaliście z tą zmienną $msg i wyszło to tak:

[PHP] pobierz, plaintext 
<?
require_once('GGBotApi/PushConnection.php');
require_once('GGBotApi/MessageBuilder.php');
 
 
$M=new MessageBuilder();
$czas=date("G:i:s - m - j - Y");
 
 
 
switch ($HTTP_RAW_POST_DATA) {
$message = trim($HTTP_RAW_POST_DATA);
$msg = explode(" ", $message);
 
    case "/ver": $M->addBBcode('Wersja Bota v1.0[br] Autor: Mateusz Pietrzak [15:01 2012-01-02]'); break;
 
    else if($msg[0] == "/info" && $user_is_online == 1) 
        unset($msg[0]);
        $info = (implode(" ", $msg));
        if($fp = fopen($info_file, 'a+'));
        if(fwrite($fp, $info." "));
        if(fclose($fp));
 
 
 
 
default: $M->addBBcode('Nie ma mnie aktualnie na GG...'.$msg.""); break;
  }
$M->reply();
[PHP] pobierz, plaintext 

ale nie za bardzo rozumiem Ciebie @nospor co mam dokładnie z IF'ami zrobić

Ten post edytował mrRosso 3.01.2012, 20:31:22

[nospor]

.....
Uzywasz ELSE nie wiedzac co to robi i jak to robi. Tlumacze ci ze ELSE bez IF jest bez sensu. Jak nie kumasz, to zajrzyj łaskawie do manuala, bo takich podstaw ci nikt tlumaczyc nie bedzie....
http://pl.php.net/manual/en/control-structures.if.php

[gorden]

if, switch
przeczytaj, a potem

wsadziłeś niekompletny warunek w środek switcha. zrób to na warunku a nie na switchu

[mrRosso]

niby ok

[PHP] pobierz, plaintext 
<?
require_once('GGBotApi/PushConnection.php');
require_once('GGBotApi/MessageBuilder.php');
 
 
$M=new MessageBuilder();
$czas=date("G:i:s - m - j - Y");
 
 
 
switch ($HTTP_RAW_POST_DATA) {
$message = trim($HTTP_RAW_POST_DATA);
$msg = explode(" ", $message);
 
 
 
     if($msg[0] == "/info") 
        unset($msg[0]);
        $info = (implode(" ", $msg));
        if($fp = fopen($info_file, 'a+'));
        if(fwrite($fp, $info." "));
        if(fclose($fp));
 
 
 
 
default: $M->addBBcode('Nie ma mnie aktualnie na GG...'.$msg.""); break;
  }
$M->reply();
[PHP] pobierz, plaintext 

ale taki błąd teraz jest

[PHP] pobierz, plaintext 
 <br />
<b>Parse error</b>:  syntax error, unexpected T_VARIABLE, expecting T_CASE or T_DEFAULT or '}' in <b>/home/gaduczat/public_html/ggbot.php</b> on line <b>12</b><br />
[PHP] pobierz, plaintext 

boję się cokolwiek ruszać bo doszczętnie zepsuje...;/

[shinuexx]

z tego co pokazujesz to wnioskuje że nie masz zielonego pojęcia co piszesz. Dodaj komentarze na końcu linijek jak po kolei interpretujesz kod, to powiem ci co robisz źle.