fix bot

Author: MikhailLipanin

.github/workflows/lines-of-code.yml

@@ -2,6 +2,7 @@ package main
 
 import (
 	"context"
+	"flag"
 	"log"
 	"os/signal"
 	"syscall"
@@ -10,7 +11,23 @@ import (
 	"github.com/cool-dudes-alpha-dynamite-wolves/chto-tam-po-peresdacham/internal/parser"
 )
 
+var (
+
+	// хранить секреты в коде плохо, поэтому используем флаги
+	// "1953480583:AAEU7eBaZnCUt525oUkCMRCQxK1TJmaoVd4"
+	botToken = flag.String("tg.token", "", "token for telegram")
+
+	// это не секрет, но для простоты тоже выносим под конфиг
+	// "https://5872-95-165-1-28.eu.ngrok.io"
+	WebhookURL = flag.String("tg.webhook", "", "webhook addr for telegram")
+
+	// запуск выглядит так:
+	// go run bot.go -tg.token="1953480583:AAEU7eBaZnCUt525oUkCMRCQxK1TJmaoVd4" -tg.webhook="https://5872-95-165-1-28.eu.ngrok.io"
+)
+
 func main() {
+	flag.Parse()
+
 	ctx := context.Background()
 
 	// Инициализация парсера
@@ -19,15 +36,18 @@ func main() {
 		log.Fatal("failed to initialize parser", err)
 	}
 
-	// Парсинг данных с использованием кэша
-	data, err := parser.ParseOnce()
+	// Парсинг данных
+	data, err := parser.Parse()
 	if err != nil {
 		log.Fatal("failed to parse input data", err.Error())
 	}
 	log.Println("Parsing completed!")
 
 	// Инициализация и запуск бота
-	bot := bot.NewTgBot()
+	bot, err := bot.NewTgBot(*botToken)
+	if err != nil {
+		log.Fatal("failed to initialize bot", err)
+	}
 	ctx, stop := signal.NotifyContext(ctx, syscall.SIGTERM, syscall.SIGINT)
 	defer stop()
 

cmd/main.go

@@ -3,12 +3,11 @@ module github.com/cool-dudes-alpha-dynamite-wolves/chto-tam-po-peresdacham
 go 1.23.0
 
 require (
+	github.com/go-telegram-bot-api/telegram-bot-api/v5 v5.5.1
 	github.com/xuri/excelize/v2 v2.9.0
-	golang.org/x/sync v0.8.0
 )
 
 require (
-	github.com/go-telegram-bot-api/telegram-bot-api/v5 v5.5.1 // indirect
 	github.com/mohae/deepcopy v0.0.0-20170929034955-c48cc78d4826 // indirect
 	github.com/richardlehane/mscfb v1.0.4 // indirect
 	github.com/richardlehane/msoleps v1.0.4 // indirect

go.mod

@@ -25,8 +25,6 @@ golang.org/x/image v0.18.0 h1:jGzIakQa/ZXI1I0Fxvaa9W7yP25TqT6cHIHn+6CqvSQ=
 golang.org/x/image v0.18.0/go.mod h1:4yyo5vMFQjVjUcVk4jEQcU9MGy/rulF5WvUILseCM2E=
 golang.org/x/net v0.30.0 h1:AcW1SDZMkb8IpzCdQUaIq2sP4sZ4zw+55h6ynffypl4=
 golang.org/x/net v0.30.0/go.mod h1:2wGyMJ5iFasEhkwi13ChkO/t1ECNC4X4eBKkVFyYFlU=
-golang.org/x/sync v0.8.0 h1:3NFvSEYkUoMifnESzZl15y791HH1qU2xm6eCJU5ZPXQ=
-golang.org/x/sync v0.8.0/go.mod h1:Czt+wKu1gCyEFDUtn0jG5QVvpJ6rzVqr5aXyt9drQfk=
 golang.org/x/text v0.19.0 h1:kTxAhCbGbxhK0IwgSKiMO5awPoDQ0RpfiVYBfK860YM=
 golang.org/x/text v0.19.0/go.mod h1:BuEKDfySbSR4drPmRPG/7iBdf8hvFMuRexcpahXilzY=
 gopkg.in/yaml.v3 v3.0.1 h1:fxVm/GzAzEWqLHuvctI91KS9hhNmmWOoWu0XTYJS7CA=

go.sum

@@ -9,8 +9,9 @@ import (
 	"strings"
 	"time"
 
-	"github.com/cool-dudes-alpha-dynamite-wolves/chto-tam-po-peresdacham/internal"
 	telegram "github.com/go-telegram-bot-api/telegram-bot-api/v5"
+
+	"github.com/cool-dudes-alpha-dynamite-wolves/chto-tam-po-peresdacham/internal"
 )
 
 type TgBot struct {
@@ -19,26 +20,29 @@ type TgBot struct {
 	bot      *telegram.BotAPI
 	subjects []*internal.Subject // Данные из парсера
 }
+
 type Subject struct {
 	Name string    // Название предмета
 	Date time.Time // Дата пересдачи
 }
 
-func NewTgBot() *TgBot {
+func NewTgBot(token string) (*TgBot, error) {
 	errChan := make(chan error, 1)
-	botAPI, err := telegram.NewBotAPI(token)
+	bot, err := telegram.NewBotAPI(token)
 	if err != nil {
-		log.Fatalf("failed to initialize Telegram Bot API: %v", err)
+		return nil, fmt.Errorf("failed to initialize Telegram Bot API: %v", err)
 	}
 
+	// bot.Debug = true
+
 	return &TgBot{
 		server:  &http.Server{ReadHeaderTimeout: 1 * time.Second},
 		errChan: errChan,
-		bot:     botAPI,
-	}
+		bot:     bot,
+	}, nil
 }
 
-func (b *TgBot) Start(ctx context.Context, subjects []*internal.Subject) error {
+func (b *TgBot) Start(_ context.Context, subjects []*internal.Subject) error {
 	b.subjects = subjects
 	updateConfig := telegram.NewUpdate(0)
 	updateConfig.Timeout = 60

internal/bot/tg_bot.go

@@ -17,11 +17,6 @@ import (
 	"github.com/cool-dudes-alpha-dynamite-wolves/chto-tam-po-peresdacham/pkg"
 )
 
-var (
-	cachedSubjects []*internal.Subject
-	cacheMutex     sync.RWMutex
-)
-
 type ExcelParser struct {
 	pathToRetakes string
 }
@@ -41,30 +36,6 @@ func NewExcelParser(pathToRetakes string) (*ExcelParser, error) {
 	}, nil
 }
 
-func (p *ExcelParser) ParseOnce() ([]*internal.Subject, error) {
-	// Защита от конкурентного доступа
-	cacheMutex.RLock()
-	defer cacheMutex.RUnlock()
-	if cachedSubjects != nil {
-		cacheMutex.RUnlock()
-		return cachedSubjects, nil
-	}
-	cacheMutex.RUnlock()
-
-	// Если кэш пустой, выполняем парсинг
-	subjects, err := p.Parse()
-	if err != nil {
-		return nil, err
-	}
-
-	// Сохраняем данные в кэш
-	cacheMutex.Lock()
-	cachedSubjects = subjects
-	cacheMutex.Unlock()
-
-	return cachedSubjects, nil
-}
-
 func (p *ExcelParser) Parse() ([]*internal.Subject, error) {
 	var files []string
 	err := filepath.WalkDir(p.pathToRetakes, func(path string, d fs.DirEntry, _ error) error {