• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Problem z zadaniem

VPS Starter Arubacloud
0 głosów
137 wizyt
pytanie zadane 20 kwietnia 2017 w C i C++ przez niezalogowany

Witam, mam zadanie: LINK

Zrobiłem je tak:

#include <iostream>

using namespace std;

string szyfr(string k, int n)
{
	for(int i=0; i<k.length(); i++)
	{
		char c = k[i];
		c - n;
		if(c<65)
		{
			c+=26;
		}
		k[i] = c;
	}
	return k;
}

int main()
{
	int n;
	char c;
	cin>>n>>c;
	string q;
	cin>>q;
	string a;
	string naja;
	int naj = 0;
	for(int i=0; i<26; i++)
	{
		a = q;
		a = szyfr(a, i);
		int q = 0;
		cout<<a;
		for(int j=0; j<n; j++)
		{
			if(a[j]==c)
				q++;
		}
		if(q>naj)
		{
			naj = q;
			naja = a;
		}
	}
	cout<<naja;
}

Nie działa, nic nie wypisuje.

Pozdrawiam

1 odpowiedź

0 głosów
odpowiedź 20 kwietnia 2017 przez BlackMoon Obywatel (1,730 p.)
wybrane 21 kwietnia 2017
 
Najlepsza

Poeksperymentowałem troszeczkę z Twoim kodem i sporo czasu spędziłem szukając błędu, a był on naprawdę banalny jak się okazuje wink. Jest on dowodem na to, że jeden pojedynczy znak może zrujnować nam całe godziny pracy cheeky.


string szyfr(string k, int n)
{
	for (int i = 0; i<k.length(); i++)
	{
		char c = k[i];
		c -= n; //Ktoś tu zapomniał z powrotem przypisać wartośc do zmiennej c ;) (ERROR: c-n;)
		if (c<65)
		{
			c += 26;
		}
		k[i] = c;
	}
	return k;
}

Po poprawieniu tej linijki kodu wszystko świetnie śmiga. Swoją drogą, ciekawy sposób na rozwiązanie problemu.

Pozdrawiam i miłego kodowania smiley

komentarz 20 kwietnia 2017 przez niezalogowany

Dzięki smiley. Przygotowuję się do olimpiady w przyszłym roku (gimnazjalnej). W tym roku, zawaliłem II etap. Jak możesz zajrzyj tutaj: LINK

To mój post, który też jest dla mnie bardzo ważny. 

Pozdrawiam i jeszcze raz dziękuję za pomoc smiley.

komentarz 20 kwietnia 2017 przez mokrowski Mędrzec (155,460 p.)
No to @Hacper czas na "obrycie" <algorithm>. W Twoim kodzie większość błędów nie będzie nawet możliwa do popełniania jeśli użyjesz sprawdzonych algorytmów dostępnych w bibliotece standardowej i sensownych kontenerów.
komentarz 21 kwietnia 2017 przez niezalogowany
Dzięki, za poradę. Do tej pory używałem tylko sort() z algorithm
komentarz 21 kwietnia 2017 przez mokrowski Mędrzec (155,460 p.)

Np. najczęściej występujący znak w zaszyfrowanym tekście zwrócisz tak:

#include <limits>
#include <utility>
#include <unordered_map>

char mostFreqChar(const std::string& message) {
    std::unordered_map<char, size_t> charMap;
    auto maxPair = std::make_pair('A', std::numeric_limits<size_t>::min());
    for(const auto& ch: message) {
        auto val = ++charMap[ch];
        if(val > maxPair.second) {
            maxPair = std::make_pair(ch, val);
        }
    }
    return maxPair.first;
}

To naprawdę będzie szybkie bo:

1. Nie przekazujesz kopii string'a co zrobiłeś w swoim kodzie.

2. Wymaga 1 przejścia przez string.

3. Dostęp do litery jest w czasie stałym a u Ciebie jest 26 * string.length()

Z kolei deszyfrowanie zrobić możesz tak:

std::string& decrypt(std::string& message, char origChar, char cryptChar) {
    auto diff = cryptChar - origChar;
    for(auto& ch: message) {
        ch -= diff;
        // Tu sprawdzenie litery czy nie wychodzi poza zakres ['A', 'Z'] i korekta.. 
    }

    return message;
}

Znów masz 1 przejście i działanie na oryginalnym string'u aby oszczędzić pamięć. Takie "śmieszne przyjmij i zwróć" jest dla wywołania w main():

// mostChar to z przykładu 'A'
std::cout << decrypt(message, mostChar, mostFreqChar(message)) << std::endl;

Jak chcesz, poskładaj to sobie :-) 

Podobne pytania

0 głosów
0 odpowiedzi 320 wizyt
pytanie zadane 29 września 2021 w C i C++ przez diedassel Użytkownik (570 p.)
0 głosów
1 odpowiedź 422 wizyt
pytanie zadane 20 marca 2018 w C i C++ przez kuba1245677 Obywatel (1,080 p.)
0 głosów
0 odpowiedzi 1,210 wizyt
pytanie zadane 1 grudnia 2017 w C i C++ przez Krystek102 Bywalec (2,440 p.)

92,452 zapytań

141,262 odpowiedzi

319,080 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...