Fix history bug

src/data_provider/card_stats.rs

@@ -73,15 +73,6 @@ pub fn update_stat_score(stat: &CardStatistics, connection: &Connection) {
     println!("Updated stat: {}", time.elapsed().as_millis());
 }
 
-pub fn delete_stat(stat: &CardStatistics, connection: &Connection) {
-    if !stat.id.is_valid() {
-        return;
-    }
-    connection
-        .execute("DELETE FROM card_stats WHERE id = ?1", (&stat.id,))
-        .unwrap();
-}
-
 pub fn delete_stats(stat: &[CardStatistics], connection: &Connection) {
     if stat.is_empty() {
         return;
@@ -93,7 +84,7 @@ pub fn delete_stats(stat: &[CardStatistics], connection: &Connection) {
         .join(", ");
     connection
         .execute(
-            format!("DELETE FROM card_stats WHERE id in {ids}").as_str(),
+            format!("DELETE FROM card_stats WHERE id in ({ids})").as_str(),
             (),
         )
         .unwrap();

src/data_provider/history.rs

@@ -39,7 +39,7 @@ fn parse_history_items(strings: Vec<String>) -> Vec<HistoryItem> {
                 continue;
             }
             items.push(HistoryItem {
-                timestamp: DateTime::from_timestamp(time.parse().unwrap_or_default(), 0).unwrap(),
+                timestamp: DateTime::from_timestamp(time.parse().unwrap_or_default(), 0).unwrap_or_default(),
                 word_id: id,
                 mode: match mode.parse::<u8>().unwrap_or_default() {
                     2 => WordOpenMode::Hard,

src/data_provider/web_api.rs

@@ -1,5 +1,6 @@
 use crate::dictionary::app_data_dir;
 use std::io;
+use reqwest::Error;
 use tokio::fs::{File, OpenOptions};
 use tokio::io::{AsyncReadExt, AsyncWriteExt};
 use zstd::{DEFAULT_COMPRESSION_LEVEL, Decoder, Encoder};
@@ -7,10 +8,10 @@ use zstd::{DEFAULT_COMPRESSION_LEVEL, Decoder, Encoder};
 // const API_URL: &str = "http://127.0.0.1:8089/";
 const API_URL: &str = "https://learning.micialware.ru/";
 
-pub async fn send_data(id: String) {
+pub async fn send_data(id: String) -> Result<(), ()> {
     let path = app_data_dir();
     let db_file = path.join("data.db");
-    let data = tokio::fs::read(db_file).await.unwrap();
+    let data = tokio::fs::read(db_file).await.map_err(|_| ())?;
     let data = compress(data);
 
     let api_url = API_URL.to_string();
@@ -21,12 +22,11 @@ pub async fn send_data(id: String) {
         .post(&id_url)
         .body(data)
         .send()
-        .await
-        .unwrap()
+        .await.map_err(|_| ())?
         .text()
-        .await
-        .unwrap();
+        .await.map_err(|_| ())?;
     set_local_version(new_version.parse::<u32>().unwrap()).await;
+    Ok(())
 }
 
 fn compress(data: Vec<u8>) -> Vec<u8> {
@@ -35,18 +35,16 @@ fn compress(data: Vec<u8>) -> Vec<u8> {
     encoder.finish().unwrap()
 }
 
-pub async fn load_data(id: String, temp: bool) {
+pub async fn load_data(id: String, temp: bool) -> Result<(), Error> {
     let api_url = API_URL.to_string();
     let id_url = format!("{api_url}download/{id}");
     let client = reqwest::Client::new();
     let data = client
         .get(&id_url)
         .send()
-        .await
-        .unwrap()
+        .await?
         .bytes()
-        .await
-        .unwrap()
+        .await?
         .to_vec();
     let data = decompress(data);
 
@@ -60,12 +58,11 @@ pub async fn load_data(id: String, temp: bool) {
     let version = client
         .get(&version_url)
         .send()
-        .await
-        .unwrap()
+        .await?
         .text()
-        .await
-        .unwrap();
-    set_local_version(version.parse::<u32>().unwrap()).await;
+        .await?;
+    set_local_version(version.parse::<u32>().unwrap_or_default()).await;
+    Ok(())
 }
 
 pub fn decompress(data: Vec<u8>) -> Vec<u8> {
@@ -78,7 +75,7 @@ pub fn decompress(data: Vec<u8>) -> Vec<u8> {
     decompressed
 }
 
-pub async fn first_sync() -> String {
+pub async fn first_sync() -> Result<String, ()> {
     let id_url = API_URL.to_string() + "generate";
 
     let client = reqwest::Client::new();
@@ -86,12 +83,12 @@ pub async fn first_sync() -> String {
         .get(&id_url)
         .send()
         .await
-        .unwrap()
+        .map_err(|_| ())?
         .text()
         .await
-        .unwrap();
+        .map_err(|_| ())?;
     println!("id: {}", id);
-    id
+    Ok(id)
 }
 
 pub async fn get_web_version(key: &str) -> Result<u32, reqwest::Error> {
@@ -112,7 +109,7 @@ pub async fn get_local_version() -> u32 {
             .read_to_string(&mut data)
             .await
             .unwrap();
-        return data.parse::<u32>().unwrap();
+        return data.parse::<u32>().unwrap_or_default();
     }
 
     let mut file = OpenOptions::new()

src/dictionary.rs

@@ -1,6 +1,6 @@
 use crate::data_provider::words::{delete_group, delete_word, update_group, update_word};
 use crate::dictionary::DictionaryMessage::*;
-use crate::dictionary_test::DictionaryQuizState;
+use crate::dictionary_quiz::DictionaryQuizState;
 use crate::import::ImportState;
 use crate::lang::{WordData, WordGroup};
 use crate::navigation::Page::{Import, Word};

src/dictionary_quiz.rs

@@ -1,6 +1,6 @@
 use crate::RootMessage;
 use crate::dictionary::split_with_coma;
-use crate::dictionary_test::DictionaryQuizMessage::*;
+use crate::dictionary_quiz::DictionaryQuizMessage::*;
 use crate::lang::WordData;
 use crate::navigation::Page::PreviousPage;
 use crate::navigation::*;

src/history.rs

@@ -1,9 +1,8 @@
-use crate::data_provider::history::{HistoryItem, get_history_of_set};
-use crate::lang::WordData;
+use crate::data_provider::history::{get_history_of_set, HistoryItem};
+use crate::lang::{WordData, INVALID_ID};
 use crate::navigation::{NavigatedPage, Page};
 use crate::styling::*;
 use crate::{AppState, RootMessage};
-use HistoryMessage::Back;
 use chrono::Local;
 use iced::alignment::Horizontal::Center;
 use iced::widget::space::horizontal;
@@ -11,6 +10,7 @@ use iced::widget::*;
 use iced::{Element, Fill, FillPortion, Task};
 use parking_lot::Mutex;
 use std::sync::Arc;
+use HistoryMessage::Back;
 
 #[derive(Clone)]
 pub struct HistoryState {
@@ -52,8 +52,19 @@ impl HistoryState {
         let history = get_history_of_set(id);
         let words = history
             .iter()
-            .filter_map(|item| state.dictionary.iter().find(|w| w.id == item.word_id))
-            .cloned()
+            .map(
+                |item| match state.dictionary.iter().find(|w| w.id == item.word_id) {
+                    None => WordData {
+                        id,
+                        key: "Удаленное слово".to_string(),
+                        value: "Удаленное слово".to_string(),
+                        tags: "Удаленное слово".to_string(),
+                        additional: Default::default(),
+                        group_id: INVALID_ID,
+                    },
+                    Some(word) => word.clone(),
+                },
+            )
             .collect();
 
         Self {
@@ -64,7 +75,7 @@ impl HistoryState {
 
     fn history_lines(&self) -> Column<'_, HistoryMessage> {
         let mut column = Column::new();
-        for (item, index) in self.list.iter().zip(0..self.list.len()) {
+        for (index, item) in self.list.iter().enumerate() {
             column = column.push(
                 row![
                     iced::widget::column![

src/import.rs

@@ -489,7 +489,7 @@ impl ImportState {
                         add_words(word, connection);
                         index += 1;
                         let mut progress = progress.lock();
-                        *progress = index as f32 / total_len;
+                        *progress = (index as f32 / total_len).min(1.0);
                     }
 
                     state.dictionary.append(&mut words_list);

src/lang.rs

@@ -1,10 +1,11 @@
+use crate::data_provider::card_stats::load_stats_of_deck;
+use crate::data_provider::card_stats::update_stat_score;
+use crate::data_provider::history::{push_note, HistoryItem};
 use crate::AppState;
-use crate::data_provider::card_stats::{delete_stat, load_stats_of_deck, update_stat_score};
-use crate::data_provider::history::{HistoryItem, push_note};
 use chrono::{DateTime, Utc};
 use parking_lot::Mutex;
-use rand::distr::Distribution;
 use rand::distr::weighted::WeightedIndex;
+use rand::distr::Distribution;
 use rand::prelude::SliceRandom;
 use rand::rng;
 use rand::rngs::ThreadRng;
@@ -12,8 +13,8 @@ use rayon::iter::IndexedParallelIterator;
 use rayon::iter::IntoParallelRefIterator;
 use rayon::iter::ParallelIterator;
 use rhai::{Engine, Scope};
-use rusqlite::ToSql;
 use rusqlite::types::{FromSql, FromSqlError, FromSqlResult, ToSqlOutput, ValueRef};
+use rusqlite::ToSql;
 use std::cmp::PartialEq;
 use std::collections::HashMap;
 use std::fmt::{Display, Formatter};

src/main.rs

@@ -1,7 +1,7 @@
 #![windows_subsystem = "windows"]
 mod data_provider;
 mod dictionary;
-mod dictionary_test;
+mod dictionary_quiz;
 mod history;
 pub mod import;
 mod lang;

src/navigation.rs

@@ -1,10 +1,8 @@
 use crate::data_provider::history::{get_history_of_set, history_dir};
 use crate::data_provider::sqlite::{create_db, default_connection};
-use crate::data_provider::web_api::{
-    get_local_version, get_web_version, load_data, set_local_version,
-};
+use crate::data_provider::web_api::{get_local_version, get_web_version, load_data, set_local_version};
 use crate::dictionary::{DictionaryMessage, DictionaryState, app_data_dir};
-use crate::dictionary_test::{DictionaryQuizMessage, DictionaryQuizState};
+use crate::dictionary_quiz::{DictionaryQuizMessage, DictionaryQuizState};
 use crate::history::{HistoryMessage, HistoryState};
 use crate::import::{ImportMessage, ImportState};
 use crate::lang::Id;
@@ -154,7 +152,7 @@ impl ScreenState {
         println!("{} {}", local, web);
         if web > local {
             println!("Web version is newer than local version");
-            load_data(string, true).await;
+            load_data(string, true).await?;
             set_local_version(web).await;
         } else {
             return Ok(false);

src/repetition.rs

@@ -9,8 +9,8 @@ use chrono::Local;
 use iced::alignment::Horizontal::Center;
 use iced::keyboard::key::Physical::Code;
 use iced::widget::text::Style;
-use iced::widget::{Column, button, column, container, row, rule, space, text, tooltip};
-use iced::{Element, Fill, Task, Theme, alignment, keyboard};
+use iced::widget::{button, column, container, row, rule, space, text, tooltip, Column};
+use iced::{alignment, keyboard, Element, Fill, Task, Theme};
 use parking_lot::Mutex;
 use rodio::MixerDeviceSink;
 use std::collections::HashSet;
@@ -319,7 +319,9 @@ pub enum RepetitionMessage {
 async fn play_sound(sink: Arc<MixerDeviceSink>, text: String) {
     if let Ok(data) = get_voice(text.as_str()).await {
         spawn_blocking(move || {
-            rodio::play(sink.mixer(), data).unwrap().sleep_until_end();
+            if let Ok(play) = rodio::play(sink.mixer(), data) {
+                play.sleep_until_end()
+            }
         })
         .await
         .unwrap();

src/sync.rs

@@ -4,7 +4,7 @@ use crate::navigation::Page::PreviousPage;
 use crate::navigation::{NavigatedPage, Page};
 use crate::styling::*;
 use crate::sync::SyncMessage::*;
-use crate::{AppState, RootMessage, fill_state};
+use crate::{fill_state, AppState, RootMessage};
 use iced::widget::button::danger;
 use iced::widget::container::rounded_box;
 use iced::widget::{button, column, container, progress_bar, row, space, text, toggler};
@@ -59,7 +59,12 @@ impl NavigatedPage<SyncMessage> for SyncState {
                     .map(|_val: String| RootMessage::Sync(KeyCopied));
             }
             InitSync => {
-                return Task::perform(first_sync(), |id| RootMessage::Sync(IdReceived(id)));
+                return Task::perform(first_sync(), |id| {
+                    match id {
+                        Ok(id) => RootMessage::Sync(IdReceived(id)),
+                        Err(_) => RootMessage::None
+                    }
+                });
             }
             GetKey => {
                 return iced::clipboard::read()
@@ -218,10 +223,10 @@ impl SyncState {
                 ]
             }
         }
-        .spacing(DEFAULT_SPACING)
-        .align_x(Center)
-        .width(Length::Shrink)
-        .into()
+            .spacing(DEFAULT_SPACING)
+            .align_x(Center)
+            .width(Length::Shrink)
+            .into()
     }
 
     /*    fn app_updater(&self) -> Element<'_, SyncMessage> {