Skip to content

Commit

Permalink
[Test]Aggiunto Plugin FAQ
Browse files Browse the repository at this point in the history
  • Loading branch information
Desiderato committed May 15, 2015
1 parent 1b13301 commit d8540d0
Showing 1 changed file with 34 additions and 0 deletions.
34 changes: 34 additions & 0 deletions plugin/Faq.pm
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
package Polebot::Plugin::Faq;
use strict;
use warnings;

use base 'Polebot::Plugin::Base';

my $filename = '/home/perl/faq'; # Inserire il path dove sono presenti le voci

This comment has been minimized.

Copy link
@polettix

polettix May 15, 2015

Questo tipo di configurazioni va inserito all'interno del file di configurazione, in una sezione specifica per questo plugin. Puoi vedere il plugin Authen per un esempio (c'è anche una sezione nel file di esempio)

open my $fh, "<", $filename or die "Can't open $filename:#!";

This comment has been minimized.

Copy link
@polettix

polettix May 15, 2015

Credo che qui sia $! anziché #!. Lascerei anche uno spazio dopo i due punti.

my @faq = <$fh>;
close $fh;

sub description { 'Prova di un FAQ Vediamo se va' }

This comment has been minimized.

Copy link
@polettix

polettix May 15, 2015

Se deve essere un plugin utile probabilmente la definizione non dovrebbe essere così


sub public {

my $self = shift;
my ($who, $where, $msg) = @_;
return unless $self->is_for_me($msg);

if ($msg =~ /\!faq\s+(\S+)\s*\z/mxs) {
# if ( /\!faq\s+(\S+)\s*\z/mxs) {

This comment has been minimized.

Copy link
@polettix

polettix May 15, 2015

Dovremmo cercare di avere codice pulito.

my $faq_ask = quotemeta($1);

This comment has been minimized.

Copy link
@polettix

polettix May 15, 2015

Personalmente indento a 3 caratteri utilizzando solo spazi. Dovremmo adottare una policy consistente per tutti i file nel progetto


foreach my $faq (@faq) {

# $self->say($where->[0], "$who: $faq") if ($faq =~ /$faq_ask/);
$self->say($where->[0], "$faq") if ($faq =~ /$faq_ask/);

This comment has been minimized.

Copy link
@polettix

polettix May 15, 2015

A meno che non si parli di oggetti, "$faq" è lo stesso di $faq ma con un po' più di elaborazione

}
}

return 1;
}

1;

1 comment on commit d8540d0

@polettix
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mi sembra interessante, non sono entrato nella "business logic". Probabilmente dovresti anche supportare una versione in cui i messaggi vengono inviati in privato - magari non voglio far sapere che sto cercando una cosa.

Please sign in to comment.