我希望這段代碼隨機化字符元素而不是整數。但是這些字符元素來自句子串列,如下所示:
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
void yodificacio(char arr[], int index[], int n);
int main()
{
const char *arr[] = { "Hello how are you", "I'm good.", "Okay! See you later" };
int index[] = { 1, 2, 0 };
int n = sizeof(arr) / sizeof(arr[0]);
printf("%d", n);
printf("%s", arr[1]);
yodificacio(arr, index, n);
printf("Reordered array is: \n");
for (int i = 0; i < n; i )
printf("%s ", arr[i]);
return 0;
}
void yodificacio(char arr[], int index[], int n)
{
int temp[n];
// arr[i] should be present at index[i] index
for (int i = 0; i < n; i )
temp[index[i]] = arr[i];
// Copy temp[] to arr[]
for (int i = 0; i < n; i ) {
arr[i] = temp[i];
index[i] = i;
}
}
(我不想將索引更改為 1 -> 0,1,2-> 1,2,0。我也想要任何順序2,0,1等。)
請注意,索引變數用于我希望arr[]放置字符的順序。如果我將索引設定為{0,1,2},效果很好,但是當我更改順序時,ASCII 符號會出現在列印部分。所以我希望輸出是:I'm good. See you later. Hello how are you.
這是整數的原始代碼:( 這個作業非常好,從互聯網上提取)
#include <stdio.h>
void reorder(int arr[], int index[], int n);
int main()
{
int arr[] = { 50, 40, 70, 60, 90 };
int index[] = { 4, 3, 2, 1, 0 };
int n = sizeof(arr)/sizeof(arr[0]);
reorder(arr, index, n);
printf("Reordered array is: \n");
for (int i = 0; i < n; i )
printf("%d ", arr[i]);
return 0;
}
void reorder(int arr[], int index[], int n)
{
int temp[n];
// arr[i] should be present at index[i] index
for (int i = 0; i < n; i )
temp[index[i]] = arr[i];
// Copy temp[] to arr[]
for (int i = 0; i < n; i ) {
arr[i] = temp[i];
index[i] = i;
}
}
uj5u.com熱心網友回復:
這里的一個重要見解是原始代碼的相似程度以及對字串重新排序的修改。下面的視圖來自diff --side-by-side,您可以看到唯一的區別是引數的型別宣告arr以及副本temp。
關鍵是在 C 中,指標只是另一個資料值。字串是一個以 NUL 結尾的字符陣列;在處理字串的各種庫函式中,字串由指向其第一個字符的指標(即字符陣列的衰減值)表示,但對于這樣的目的(和其他目的),沒有什么特別之處指標。
void reorder(int arr[], int index[], int n) | void reorder(char* arr[], int index[], int n)
{ {
int temp[n]; | char* temp[n];
for (int i = 0; i < n; i ) for (int i = 0; i < n; i )
temp[index[i]] = arr[i]; temp[index[i]] = arr[i];
// Copy temp[] to arr[] // Copy temp[] to arr[]
for (int i = 0; i < n; i ) { for (int i = 0; i < n; i ) {
arr[i] = temp[i]; arr[i] = temp[i];
index[i] = i; index[i] = i;
} }
} }
一個復雜的地方是使用const,我從上面的代碼中洗掉了它。(所以你也需要在你的函式中更改為。const char *arr[])char *arr[]main()
const當然更準確。元素指向的各種字串arr都是不可變的(至少在這種情況下),并且reorder不會嘗試修改其中的任何一個。(它只是對指標重新排序;它從不查看指標指向的位置。)
const但是,C 對函式實參和函式形參的一致性提出了嚴格的要求;您不能呼叫期望const char*[]帶有 type 引數的函式,char*[]反之亦然。char*(可以使用型別的引數呼叫期望的函式const char*。問題來自第二級間接。)
因此,如果您需要同時處理不可變字串陣列和可變字串陣列,則需要reorder具有兩種不同引數型別的兩個版本,一個const不包含。那是太多的代碼重復,所以我選擇const在這種情況下洗掉。
uj5u.com熱心網友回復:
“我希望這段代碼隨機化字符元素而不是整數。但這些字符元素來自句子串列”
由此,不難認為您的意圖是打亂每個字串中的所有字符,但從代碼中的背景關系中,我得出的結論是,意圖是重新排序 3 個字串,而不是字串中的字符。(我在評論中詢問是否是這樣,但在發布此答案時沒有回答。如果這是不正確的結論,請發表評論)
如評論中所述,您的代碼接近作業,除了產生隨機化。這個答案將確定解決所有這些問題的步驟,以及一兩個更小的問題。
首先,打開警告后,應標記以下專案:
int main()不是原型。main 的最小原型是int main(void)- 傳遞給型別引數的不兼容指標型別
const char *[3]char * - 和相關: 隱式轉換失去整數精度:
int到char
這是因為原型
void yodificacio(char arr[], int index[], int n);
但arr定義為:
const char *arr[] = {"...","...","..."}
有幾種方法可以更改一種或兩種以使其兼容,出于您的目的,保留arr原樣的定義并將原型更改為:
void yodificacio(int n, char *arr[n], int index[n]);//note the change in order of parameter types.
- 下一個不兼容的指向整數轉換的指標分配給
intfromchar *
此警告出現在yodificacio()中,因為
int temp[n];
不支持以下賦值:
temp[index[i]] = arr[i];// type of temp is not the same as arr
建議改變溫度
char *temp[m];//changed to match type of arr
這會處理所有警告。剩下的唯一事情是字串包含在 arr 中的順序的隨機化。
這需要一種演算法來生成有限序列的隨機排列,在您的情況下需要 3!(==6) 排列來填充index[3]陣列。可以做到這一點的演算法在此處進行了足夠詳細的描述。此外,為了支持隨機化index,可能值得考慮使其類似于arr6 個地址指向所需的 6 個排列:
int *index[] = {{ 0,1,2},{1,2,0},{2,0,1},{ 1,0,2},{ 0,2,1},{ 2,1,0}};
隨后,更新原型以適應新的 index
void yodificacio(int n, char *arr[n], int *index[n]);
轉載請註明出處,本文鏈接:https://www.uj5u.com/qita/410331.html
標籤:
上一篇:查找C程式中的漏洞
