Torna indietro   Hardware Upgrade Forum > Software > Programmazione

OPPO Find X3 Pro: la prova su ''strada'' con Lamborghini
OPPO Find X3 Pro: la prova su ''strada'' con Lamborghini
Abbiamo avuto modo di provare il nuovo OPPO Find X3 Pro nella fantastica cornice del circuito di Modena e del Museo Lamborghini. Il nuovo top di gamma della casa cinese vanta un’innovativa fotocamera quadrupla, in grado di catturare scatti fotografici ad alta velocità, un SoC di ultima generazione della linea Qualcomm Snapdragon, tante funzionalità per la registrazione di video accattivanti, la capacità di memorizzare le foto nel formato a 10 bit e molto altro ancora
Leica SL2-S: qualità tedesca per foto e video
Leica SL2-S: qualità tedesca per foto e video
Leica SL2-S non è la sorella minore della SL2, ma una fotocamera con un'anima e caratteristiche tecniche profondamente diverse. L'utilizzo di un sensore BSI retroilluminato apre il campo a tutti i formati video fino al 4K 60p, ma offre anche indubbi benefici sul fronte fotografico, con un rapporto segnale/rumore trai i migliori sul mercato
Juice Booster 2 carica i veicoli elettrici con ogni presa della corrente
Juice Booster 2 carica i veicoli elettrici con ogni presa della corrente
Le necessità di ricarica delle batterie dei veicolo elettrici non sempre può essere risolta con una colonnina tradizionale. Giunge in aiuto Juice Booster 2, un caricatore portatile che attraverso una costruzione modulare può diventare compatibile con pressoché tutti i tipi di prese di corrente, sostituendo tanto il cavo per la colonnina come una wallbox nel garage
Tutti gli articoli Tutte le news

Vai al Forum
Rispondi
 
Strumenti
Old 10-06-2021, 11:18   #1
sabansa01
Junior Member
 
Iscritto dal: Jun 2021
Messaggi: 5
[C] Problema con cifrario di Cesare

Ciao, sono alle prime armi con la programmazione e sto avendo problemi con un programma che non riesco a far funzionare, mi ci sto sbattendo da giorni...
La traccia consiste nel costruire un programma che si rifa al cifrario di cesare. In particolare si prende in input una stringa del tipo 4<parola-altraparola. Stringa all'interno della quale il primo carattere è detto "chiave", il secondo "direzione", i trattini separano le parole e il punto indica la fine della frase. La "traduzione" avviene, per ogni lettera della parola, prendendo la lettera che si trova *chiave* posti indietro/avanti (dipende dalla direzione) dell'alfabeto. Se ad esempio la lettera da tradurre è z e dobbiamo andare di tre posti avanti, si ricomincerà dalla a e viceversa.
Più semplicemente, se l'input è "3>come-stai." , l'output sarà frph vwdl
Scusate se mi sono spiegato male.

Comunque il mio codice è questo:
Qui trovo la chiave
Codice:
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define MAX 100

int caesar_key(char *input)
{
	int in, key;
	in = input[0];
	key = 48 - in;
	
	return key;
}
Qui trovo la direzione
Codice:
int caesar_direction(char *input)
{
	int direction;
	if(input[1] == '<')
	    direction = 0;
	if(input[1] == '>')
	    direction = 1;
	
	return direction;
}
Qui traduco l'input
Codice:
char *caesar_code(int key, int direction, char *input)
{
	char *out;
	int end, i = 2;
	
	do
	{
    	for(; input[i]!='-' && input[i]!='.'; i++)
		{
			if(direction==0)
			{
				if(input[i] >= ('a' + key))
			    	out[i-2] = input[i]-key;
				else
			    	out[i-2] = 'z' - ((key-(input[i]-'a'))-1);
			}
		
			else if(direction==1)
			{
				if(input[i] <= ('z' - key))
			    	out[i-2] = input[i] + key;
				else
			    	out[i-2] = 'a' + ((key-('z'- input[i]))+1);
			}
		
			else
			{
		    	printf("\nERRORE - caesar_direction non riconosciuta\n");
		    	return NULL;
			}
		}
	
		if (input[i] == '-')
		{
			out[i-2] == ' ';
			i++;
			end = 0;
		}
	
		else if(input[i] == '.')
		{
			return out;
	    	end = 1;
	    }
	    
	    else
	    {
	    	printf("\nERRORE - . o - non trovato\n");
	    	return NULL;
		}
	}while(end == 0);
}
E questo è il main
Codice:
int main()
{
	char *input, *output;
	int key, direction;
	input = (char*)malloc(sizeof(char[MAX]));
	output = (char*)malloc(sizeof(char[MAX]));
	
	printf("Scrivere la parola da tradurre: ");
	scanf("%s", input);
		
	key = caesar_key(input);
	direction = caesar_direction(input);
	output = caesar_code(key, direction, input[MAX]);
	
	printf("\nTraduzione:\t%s", output);
		
	return 0;
}
Il problema si pone già al momento dello scanf: premo invio dopo aver scritto l'input, rimane fermo 3 secondi senza scrivere niente, e si chiude il programma.

Ovviamente ho già trovato altri modi per scrivere il programma, però per me è un esercitazione e vorrei capire cosa ho sbagliato, dato che ciò che ho trovato è strutturato in modo totalmente diverso da come ho fatto io.

Grazie per l'attenzione

Ultima modifica di sabansa01 : 10-06-2021 alle 16:03.
sabansa01 è offline   Rispondi citando il messaggio o parte di esso
Old 10-06-2021, 12:43   #2
Kaya
Senior Member
 
Iscritto dal: Apr 2005
Messaggi: 2152
Ho provato a compilarlo al volo e ottengo:

error: invalid conversion from ‘char’ to ‘char*’ [-fpermissive]
output = caesar_code(key, direction, input[MAX]);
Kaya è offline   Rispondi citando il messaggio o parte di esso
Old 10-06-2021, 13:14   #3
sabansa01
Junior Member
 
Iscritto dal: Jun 2021
Messaggi: 5
A me non usciva niente... però ho provato a cambiare quella riga, e semplicemente ho tolto il [MAX]

Codice:
output = caesar_code(key, direction, input);
Adesso almeno il codice supera lo scanf, e se l'input è scritto correttamente(es. 5<ciao-mondo.) traduce bene solo la prima parola, poi non va avanti
sabansa01 è offline   Rispondi citando il messaggio o parte di esso
Old 10-06-2021, 14:58   #4
wingman87
Senior Member
 
Iscritto dal: Nov 2005
Messaggi: 2658
Nella funzione caesar_code non hai allocato spazio per la variabile out in cui scrivi l'output. Inoltre nel chiamante non è necessario allocare spazio per la variabile output.

Ultima modifica di wingman87 : 10-06-2021 alle 15:25.
wingman87 è offline   Rispondi citando il messaggio o parte di esso
Old 10-06-2021, 15:44   #5
sabansa01
Junior Member
 
Iscritto dal: Jun 2021
Messaggi: 5
Grazie delle risposte, ho fatto alcune modifiche (premetto col dire che se non alloco la memoria nel main il programma non funziona).

Ora i problemi sono questi:
1) Il codice funziona solo per la prima parola

2) con un input del tipo 3<ciao, alle lettere "c" e "a" dovrebbero corrispondere le lettere "z" e "x", invece al loro posto mi escono altri simboli e punteggiature, come se non prendesse in considerazione il dover "ritornare alla z", che invece ho incluso nel codice.

3) con "<" la direzione dovrebbe essere da destra verso sinistra e con ">" viceversa. Il programma, invece, fa il contrario. Il problema è facilmente risolvibile scambiando i codici negli if in caesar_code, ma comunque non capisco il perchè...

Codice:
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define MAX 100

int caesar_key(char *input)
{
	int in, key;
	in = input[0];
	key = 48 - in;
	
	return key;
}

int caesar_direction(char *input)
{
	int direction;
	if(input[1] == '<')
	    direction = 0;
	else if(input[1] == '>')
	    direction = 1;
	
	return direction;
}

char *caesar_code(int key, int direction, char *input)
{
	char *out;
	out = (char*)malloc(sizeof(char[MAX]));
	int end, i = 2;
	
	do
	{
    	for(; input[i]!='-' && input[i]!='.'; i++)
		{
			if(direction==0)
			{
				if(input[i] >= ('a' + key))
			    	out[i-2] = input[i]-key;
				else
			    	out[i-2] = 'z' - ((key-(input[i]-'a'))-1);
			}
		
			else if(direction==1)
			{
				if(input[i] <= ('z' - key))
			    	out[i-2] = input[i] + key;
				else
			    	out[i-2] = 'a' + ((key-('z'- input[i]))+1);
			}
		
			else
			{
		    	printf("\nERRORE - caesar_direction non riconosciuta\n");
		    	return NULL;
			}
		}
	
		if (input[i] == '-')
		{
			out[i-2] == ' ';
			i++;
			end = 0;
		}
	
		else if(input[i] == '.')
		{
			return out;
	    	end = 1;
	    }
	    
	    else
	    {
	    	printf("\nERRORE - . o - non trovato\n");
	    	return NULL;
		}
	}while(end == 0);
}
	
int main()
{
	char *input, *output;
	int key, direction;
	input = (char*)malloc(sizeof(char[MAX]));
	output = (char*)malloc(sizeof(char[MAX]));
	
	printf("Scrivere la parola da tradurre: ");
	scanf("%s", input);
		
	key = caesar_key(input);
	direction = caesar_direction(input);
	output = caesar_code(key, direction, input);
	
	printf("\nTraduzione:\t%s", output);
		
	return 0;
}
sabansa01 è offline   Rispondi citando il messaggio o parte di esso
Old 10-06-2021, 15:56   #6
sabansa01
Junior Member
 
Iscritto dal: Jun 2021
Messaggi: 5
Ho risolto gli ultimi due problemi! Avevo sbagliato nella prima funzione, al posto di 48- in andava in-48

Ora rimane solo un problema che non capisco: la seconda parola, se inserita, non viene tradotta.

Codice:
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define MAX 100

int caesar_key(char *input)
{
	int in, key;
	in = input[0];
	key = in - 48;
	
	return key;
}

int caesar_direction(char *input)
{
	int direction;
	if(input[1] == '<')
	    direction = 0;
	else if(input[1] == '>')
	    direction = 1;
	
	return direction;
}

char *caesar_code(int key, int direction, char *input)
{
	char *out;
	out = (char*)malloc(sizeof(char[MAX]));
	int end, i = 2;
	
	do
	{
    	for(; input[i]!='-' && input[i]!='.'; i++)
		{
			if(direction==0)
			{
				if(input[i] >= ('a' + key))
			    	out[i-2] = input[i]-key;
				else
			    	out[i-2] = 'z' - ((key-(input[i]-'a'))-1);
			}
		
			else if(direction==1)
			{
				if(input[i] <= ('z' - key))
			    	out[i-2] = input[i] + key;
				else
			    	out[i-2] = 'a' + ((key-('z'- input[i]))+1);
			}
		
			else
			{
		    	printf("\nERRORE - caesar_direction non riconosciuta\n");
		    	return NULL;
			}
		}
	
		if (input[i] == '-')
		{
			out[i-2] == ' ';
			i++;
			end = 0;
		}
	
		else if(input[i] == '.')
		{
			return out;
	    	end = 1;
	    }
	    
	    else
	    {
	    	printf("\nERRORE - . o - non trovato\n");
	    	return NULL;
		}
	}while(end == 0);
}
	
int main()
{
	char *input, *output;
	int key, direction;
	input = (char*)malloc(sizeof(char[MAX]));
	output = (char*)malloc(sizeof(char[MAX]));
	
	printf("Scrivere la parola da tradurre: ");
	scanf("%s", input);
		
	key = caesar_key(input);
	direction = caesar_direction(input);
	output = caesar_code(key, direction, input);
	
	printf("\nChiave: %d", key);
	printf("\nDirezione: %d ", direction);
	if(direction == 0)
	    printf("(dx-sx)");
	else if(direction == 1)
	    printf("(sx-dx)");
	printf("\nTraduzione: %s", output);
		
	return 0;
}

Ultima modifica di sabansa01 : 10-06-2021 alle 16:03.
sabansa01 è offline   Rispondi citando il messaggio o parte di esso
Old 10-06-2021, 16:18   #7
wingman87
Senior Member
 
Iscritto dal: Nov 2005
Messaggi: 2658
Il problema è in questo if:
Codice:
		if (input[i] == '-')
		{
			out[i-2] == ' ';
			i++;
			end = 0;
		}
La riga in rosso ha un = di troppo, così non avviene l'assegnazione.
La ragione per cui non vedi la seconda parola è che probabilmente out è inizializzato tutto a zero e lo zero è il terminatore di stringa.
Da quello che ricordo però malloc non dovrebbe inizializzare tutto a zero (per quello c'è calloc). Forse è una caratteristica del compilatore, non saprei...

Volevo aggiungere che se consideri il fatto che malloc in teoria non inizializza il vettore a zero, nel caso in cui input[i] == '.' dovresti anche scrivere in out[i-2] il valore zero, altrimenti out conterrà una stringa priva di terminatore.
wingman87 è offline   Rispondi citando il messaggio o parte di esso
Old 10-06-2021, 16:25   #8
sabansa01
Junior Member
 
Iscritto dal: Jun 2021
Messaggi: 5
Grazie mille... Un errore di distrazione che non avevo notato, ora il programma funziona benissimo. Fra l'altro, come avevi detto, senza out[i-2]=0 la frase terminava con un simbolo, invece aggiungendolo la stringa si conclude come dovrebbe. Grazie ancora!

Il codice finale

Codice:
/************ CIFRARIO DI CESARE - INPUT DEL TIPO *key**direction(< o >)**parola1*-*parola2*. ************/

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define MAX 100

int caesar_key(char *input)
{
	int in, key;
	in = input[0];
	key = in - 48;
	
	return key;
}

int caesar_direction(char *input)
{
	int direction;
	if(input[1] == '<')
	    direction = 0;
	else if(input[1] == '>')
	    direction = 1;
	
	return direction;
}

char *caesar_code(int key, int direction, char *input)
{
	char *out;
	out = (char*)malloc(sizeof(char[MAX]));
	int end, i = 2;
	
	do
	{
    	for(; input[i]!='-' && input[i]!='.'; i++)
		{
			if(direction==0)
			{
				if(input[i] >= ('a' + key))
			    	out[i-2] = input[i]-key;
				else
			    	out[i-2] = 'z' - ((key-(input[i]-'a'))-1);
			}
		
			else if(direction==1)
			{
				if(input[i] <= ('z' - key))
			    	out[i-2] = input[i] + key;
				else
			    	out[i-2] = 'a' + ((key-('z'- input[i]))-1);
			}
		
			else
			{
		    	printf("\nERRORE - caesar_direction non riconosciuta\n");
		    	return NULL;
			}
		}
	
		if (input[i] == '-')
		{
			out[i-2] = ' ';
			i++;
			end = 0;
		}
	
		else if(input[i] == '.')
		{
			out[i-2] = 0;
			return out;
	    }
	    
	}while(end == 0);
}
	
int main()
{
	char *input, *output;
	int key, direction;
	input = (char*)malloc(sizeof(char[MAX]));
	output = (char*)malloc(sizeof(char[MAX]));
	
	printf("Scrivere la parola da tradurre: ");
	scanf("%s", input);
		
	key = caesar_key(input);
	direction = caesar_direction(input);
	output = caesar_code(key, direction, input);
	
	printf("\nChiave: %d", key);
	printf("\nDirezione: %d ", direction);
	if(direction == 0)
	    printf("(dx-sx)");
	else if(direction == 1)
	    printf("(sx-dx)");
	printf("\nTraduzione: %s", output);
		
	return 0;
}

Ultima modifica di sabansa01 : 10-06-2021 alle 16:30.
sabansa01 è offline   Rispondi citando il messaggio o parte di esso
Old 10-06-2021, 16:47   #9
wingman87
Senior Member
 
Iscritto dal: Nov 2005
Messaggi: 2658
Bene, però scusa se insisto ma nel main non devi allocare spazio per la variabile output, puoi rimuovere questa riga:
Codice:
output = (char*)malloc(sizeof(char[MAX]));
Insisto perché è un memory leak ed è bene notarli anche se in questo caso è irrilevante perché il programma termina subito. Se questo codice fosse parte di un progetto più grande e venisse eseguito tante volte sarebbe tutto spazio sprecato e mai deallocato.
wingman87 è offline   Rispondi citando il messaggio o parte di esso
Old 10-06-2021, 16:51   #10
misterx
Senior Member
 
Iscritto dal: Apr 2001
Città: Milano
Messaggi: 3518
mi hai battuto sul tempo, ci stavo lavorando su anch'io
misterx è offline   Rispondi citando il messaggio o parte di esso
 Rispondi


OPPO Find X3 Pro: la prova su ''strada'' con Lamborghini OPPO Find X3 Pro: la prova su ''strada'' con Lam...
Leica SL2-S: qualità tedesca per foto e video Leica SL2-S: qualità tedesca per foto e v...
Juice Booster 2 carica i veicoli elettrici con ogni presa della corrente Juice Booster 2 carica i veicoli elettrici con o...
GeForce RTX 3070 Ti, recensione: Founders Edition vs Inno3D X3 OC GeForce RTX 3070 Ti, recensione: Founders Editio...
EnGenius ECS1528P, uno switch PoE a 24 porte ideale per la videosorveglianza EnGenius ECS1528P, uno switch PoE a 24 porte ide...
IBM contro GlobalFoundries: vuole 2,5 mi...
E3 2021: Xbox e Bethesda intrattengono, ...
Elon Musk gioca Cyberpunk 2077 su una Te...
Beats presenta i nuovi Studio Buds: gli ...
OnePlus Nord CE 5G: un po' anonimo e spa...
Entro il 2022 più smartphone 5G c...
Google annuncia Google Workspace per tut...
The Sandbox: come creare videogiochi e g...
Presentato il nuovo obiettivo Laowa 7.5m...
NASA SLS: lo stadio principale è stato p...
ASUS Zenfone 8, Unieuro regala la WOOW B...
OPPO Enco Free2 e Enco Buds sono realt&a...
Qualcomm pronta a investire in ARM se l'...
Stranger of Paradise Final Fantasy Origi...
SSD con memoria NAND PLC (5 bit per cell...
K-Lite Codec Pack Update
K-Lite Mega Codec Pack
K-Lite Codec Pack Full
K-Lite Codec Pack Standard
K-Lite Codec Pack Basic
CrystalDiskInfo
Dropbox
Chromium
LibreOffice 7.1.4
Advanced SystemCare Ultimate
PeaZip
Google Chrome Portable
Driver NVIDIA GeForce 466.77 WHQL
ZoneAlarm Antivirus + Firewall
VLC Media Player
Tutti gli articoli Tutte le news Tutti i download

Strumenti

Regole
Non Puoi aprire nuove discussioni
Non Puoi rispondere ai messaggi
Non Puoi allegare file
Non Puoi modificare i tuoi messaggi

Il codice vB è On
Le Faccine sono On
Il codice [IMG] è On
Il codice HTML è Off
Vai al Forum


Tutti gli orari sono GMT +1. Ora sono le: 05:04.


Powered by vBulletin® Version 3.6.4
Copyright ©2000 - 2021, Jelsoft Enterprises Ltd.
Served by www3v